Error_handler… what? - c

Question

I'm trying to learn some C and have chosen to use the book "SAMS Teach yourself C in 21 days". ( btw is there any good C book at all? This is my third! I always end up with bad or broken code that is supposed to work and hit a wall in my learning process when it doesn't! :-( )
Sadly I've run upon the code underneath that I just should type in and run. The typing went well but the running... well, not so well!
The compiler gives me an error about this line:
{
printf ("Printer busy or disconnected\n"); error_handler; }
when I try to run this code. And since I'm VERY MUCH a novice when it comes to C coding, I have NO IDEA at all what to do, when the editor returns an error messages like this:
"/usr/bin/make" -f nbproject/Makefile-Debug.mk QMAKE= SUBPROJECTS= .clean-conf
rm -f -r build/Debug
rm -f dist/Debug/GNU-MacOSX/type_and_run
CLEAN SUCCESSFUL (total time: 158ms)
"/usr/bin/make" -f nbproject/Makefile-Debug.mk QMAKE= SUBPROJECTS= .build-conf
"/usr/bin/make" -f nbproject/Makefile-Debug.mk dist/Debug/GNU-MacOSX/type_and_run
mkdir -p build/Debug/GNU-MacOSX
rm -f build/Debug/GNU-MacOSX/Type and run 1.o.d
gcc -c -g -MMD -MP -MF build/Debug/GNU-MacOSX/Type and run 1.o.d -o build/Debug
/GNU-MacOSX/Type\ and\ run\ 1.o Type\ and\ run\ 1.c
i686-apple-darwin10-gcc-4.2.1: and: No such file or directory
i686-apple-darwin10-gcc-4.2.1: run: No such file or directory
i686-apple-darwin10-gcc-4.2.1: 1.o.d: No such file or directory
Type and run 1.c: In function 'do_heading':
Type and run 1.c:54: error: 'error_handler' undeclared (first use in this function)
Type and run 1.c:54: error: (Each undeclared identifier is reported only once
Type and run 1.c:54: error: for each function it appears in.)
make[2]: *** [build/Debug/GNU-MacOSX/Type and run 1.o] Error 1
make[1]: *** [.build-conf] Error 2
make: *** [.build-impl] Error 2
BUILD FAILED (exit value 2, total time: 158ms)
All I can understand is that "error_handler" is undeclared and that it does not seem to be a library function. SIGH!
So said short, is there anything I can do to modify the code so it will work as intended and I can move on?
Or should I just throw the book over in the pile of useless books about C?
Here is the code in its full length:
void do_heading(char *filename);
int line = 0, page = 0;
int main(int argv, char *argc[])
{
char buffer[256];
FILE *fp;
if (argv < 2)
{
fprintf(stderr, "\nProper Usage is: " );
fprintf(stderr, "\n\nprint_it filename.ext\n");
return (1);
}
if ((fp = fopen(argc[1], "r")) == NULL)
{
fprintf(stderr, "Error opening file, %s!", argc[1]);
return (1);
}
page = 0;
line = 1;
do_heading(argc[1]);
while( fgets( buffer, 256, fp ) != NULL )
{
if( line % 55 == 0 )
do_heading(argc[1]);
fprintf( stdout , "%4d:\t%s", line++, buffer );
}
fprintf( stdout , "\f" );
fclose(fp);
return 0;
}
void do_heading(char *filename)
{
page++;
if (page > 1)
fprintf( stdout , "\f");
fprintf( stdout, "Page: %d, %s\n\n", page, filename);
if ((stdout = fopen("PRT:","w")) == NULL) {
printf ("Printer busy or disconnected\n"); error_handler; }
}

Answer 1

Try this book:
C Programming Language (2nd Edition) by Brian W. Kernighan, Dennis M. Ritchie.
You have argv and argc swapped.
if (argv 1)
Should be:
if (argc == 1)
You can simply remove the error_handler chunk as it seems to serve no purpose and is not declared in the code. What the book was likely trying to tell you was to fprintf to stderror perhaps?

Answer 2

So the compiler output might look intimidating, but, as a beginner, the bit you care about is usually in English. This example is no exception:
Type and run 1.c: In function 'do_heading':
Type and run 1.c:54: error: 'error_handler' undeclared (first use in this function)
Type and run 1.c:54: error: (Each undeclared identifier is reported only once
Type and run 1.c:54: error: for each function it appears in.)
It's telling you that error_handler is undeclared. It appears nowhere else in your program so there's no way you're able to use it here. Let's look at the line:
if ((stdout = fopen("PRT:","w")) == NULL) {
printf ("Printer busy or disconnected\n"); error_handler; }
You should know that semicolons are one way to indicate the end of a statement and that good practice is to put just one statement on a line. Break it up:
if ((stdout = fopen("PRT:","w")) == NULL) {
printf ("Printer busy or disconnected\n");
error_handler;
}
Now it's obvious: you're just saying error_handler;; it's not doing anything, it's not a function call (no parentheses), not a variable you're assigning to or reading from so it's clearly a huge fat error. Delete it and try again.
By the way, if the line appeared in the book exactly as you've put it here - with two semicolons on a line - then yes, you should throw it on the fire. Read Kernighan and Ritchie's "The C Programming Language" and do all the exercises in it - you will be amazed how quickly you go from knowing nothing to being ready to work on real programs.