Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Problem with example 1.5.2 in K&R book on C

Tags:

c

kr-c

I'm teaching myself C with K&R and am stumped by one of the examples in the book. I compile the code exactly as it is written in the example but it does not do what the authors say it will. The program is supposed to count characters. The code given is as follows:

#include <stdio.h>

/* count characters in input; 1st version */
main()
{
    long nc;
    nc=0;
    while (getchar() != EOF)
     ++nc;
    printf("%ld\n", nc);
}

For it to compile I replace main() with int main(). But I assume that is not relevant to the question. The program compiles and runs fine. But it simply does not count characters as it was written to do. Am I missing something? Could something have changed in how modern compilers treat a code example such as this since the book was written? Any assistance the good folks on this message board might be able to offer would be greatly appreciated.

Best, Dan

like image 416
Dan Avatar asked Sep 05 '10 13:09

Dan


3 Answers

Although the other answers are technically correct, I feel that this example (1.5.2) and the following one (1.5.3) are pedagogically confusing. Just google "character counting 1.5.2" and you will find many others who got caught up by this example, just as the OP did. The reason it is so confusing is that there is no explanation in the text about how to generate the EOF character in interactive mode, AND the previous examples outputted the results as soon as "return" was entered. Thus, any beginner to C would assume that the program in 1.5.3 should do the same...

I would like to propose the following alternative code, which produces the expected result:

#include <stdio.h>
#define     EOL '\n'

main()
{
    long nc;
    int c;
    nc = 0;

    while ((c = getchar()) != EOF)
    {
        ++nc;
        if (c == EOL)
        {
            /* Print number of input characters (not including return character) */
            printf("%ld\n", nc-1); 
            nc = 0;
        }
    }
}

The only element of C not already explained in the text is the if statement, which is actually explained in the very next section (1.5.3). I hope this small alternative example will serve to help others who got caught up by the original example from the K&R book. A good "Exercise 1.7b" would be to examine the differences between the two versions and explain verify that they output the same results (after reading about CtrlD / CtrlZ from the other answers).

like image 133
robguinness Avatar answered Nov 15 '22 07:11

robguinness


The program only outputs the number of character after it read an "end of file". With interactive input, you can generate an "end of file" via ctrl+d (at least on *NIX, no idea about windows). Knowing this, the program works correctly here.

like image 34
Uli Schlachter Avatar answered Nov 15 '22 06:11

Uli Schlachter


Apart from the return value of main it looks OK.

Do you do the CtrlD (Unix) or CtrlZ (Windows) at the end of input if you are entering values from keyboard?

like image 5
AndersK Avatar answered Nov 15 '22 08:11

AndersK