Error handling - how to get rid of multiple flags?

[Ed Davis]

In the following code fragment, I only want to print an error
message when an invalid symbol follows a valid symbol. My
'error_found' and 'error_this_case' handles that, but it seems a
little convoluted. Is there a clearer way?

Symbol get_sym(void);
Symbol sym;

void stmt_seq(void) {
int error_found = 0;

for (;;) {
int error_this_case = 0;
switch (sym) {
case ident_sym : ident_stmt(); break;
case while_sym : while_stmt(); break;
case if_sym: if_stmt(); break;

case endwhile_sym:
case else_sym:
case eof_sym:
return;

default:
if (!error_found)
error("expecting stmt");
error_this_case = 1;
get_sym();
}
if (error_this_case)
error_found = 1;
else
error_found = 0;
}
}

[Peter Pichler]

"Ed Davis" <ed*******@yahoo.com> wrote in message
news:42**************************@posting.google.c om...

In the following code fragment, I only want to print an error
message when an invalid symbol follows a valid symbol. My
'error_found' and 'error_this_case' handles that, but it seems a
little convoluted. Is there a clearer way?

Read your own specification. If I get it right, you only need to know
that the previous symbol was valid. One flag. Simple.

This is not a C question BTW, it is about algorithms. Hence OT here.

Peter

[Malcolm]

"Ed Davis" <ed*******@yahoo.com> wrote in message

In the following code fragment, I only want to print an error
message when an invalid symbol follows a valid symbol.

What you want to do is write this function.

void handlerr(char *message)
{
static int flag = 0;
if(!flag)
printf("%s\n", message);
flag = 1;
}

and pass all your errors through it.

[Richard Heathfield]

Malcolm wrote:

"Ed Davis" <ed*******@yahoo.com> wrote in message

In the following code fragment, I only want to print an error
message when an invalid symbol follows a valid symbol.

What you want to do is write this function.

void handlerr(char *message)
{
static int flag = 0;
if(!flag)
printf("%s\n", message);
flag = 1;
}

and pass all your errors through it.

You might want to make that parameter const, so that the caller feels a
little more confident about passing (the address of the first character in)
a string literal to the function.

--
Richard Heathfield : bi****@eton.powernet.co.uk
"Usenet is a strange place." - Dennis M Ritchie, 29 July 1999.
C FAQ: http://www.eskimo.com/~scs/C-faq/top.html
K&R answers, C books, etc: http://users.powernet.co.uk/eton