Thread: C 101 critique, please?

As a beginner's programming exercise I wrote this "letter grade calculator":

Code:

#include <stdio.h>
char	line[100];	/* line of data for input */
int	grade;		/* numeric grade */

int main()
{
    while (1) {
        printf("Enter the numeric grade\n");
        printf("  or 0 to stop:");
        fflush(stdout);
        
        fgets(line, sizeof(line), stdin);
        sscanf(line, "%d", &grade);
        
        if (grade == 0)
            break;
        else if
            (grade > 0)
                if
                (grade <= 60)
                    printf("F\n");
                else if
                (grade <= 70)
                    printf("D\n");
                else if
                (grade <= 80)
                    printf("C\n");
                else if
                (grade <= 90)
                    printf("B\n");
                else if
                (grade <= 100)
                    printf("A\n");
    }
    return (0);
}

which looks like this on the console:

Enter the numeric grade
or 0 to stop:66
D
Enter the numeric grade
or 0 to stop:77
C
Enter the numeric grade
or 0 to stop:33
F
Enter the numeric grade
or 0 to stop:88
B
Enter the numeric grade
or 0 to stop:100
A
Enter the numeric grade
or 0 to stop:81
B
Enter the numeric grade
or 0 to stop:0

Critiques, please! Is there a way to double up the comparison operators around the variable to test each range?

I don't know what curriculum topics you are trying to show here, but if global variables isn't one of them you should probably put your grade and line[] variables inside of main(). Also you could condense your test of grade from

Code:

if (grade == 0)
            break;
        else if
            (grade > 0)

to the single operation of

Code:

if (grade >0)
       {
     /* do test operations */
}else{
     break;
       }

Hopefully this will help you. Also, when you ask for critiques, try to not only give areas you think could be improved, but also give your ideas of how to actually change the code. This will give us more of an idea of what you're thinking of and how much you know . Most of us around here don't like it when you ask a question without trying to answer it yourself . People are more likely to respond when you do.

Re: Hammer

Are you aware that in America we don't have an 'E' letter grade?

I don't know why not. Just like you Brits, the way we do some things is a little weird to other people. I'll admit, after taking physics our system of measurement seems a little odd compared to metrics.

there is a tiny optimization you could make, since you are outputting \n everytime you could simply output that after the if-else if-else block. I see that more in C++ (cout << endl so maybe it's just a style thing for C only programmers.


useless information:
some schools in the US actually do use E instead of F, F is F simply to denote failure which I guess someone decided was too harsh, when we took some US standardized test 2 years ago (I forget, maybe the PSAT) and you had to bubble your average grade in some subjects the options were A,B,C,D,E/F.

Thanks to all for your helpful replies so far on my thread.

I suppose any single book on the subject would present C from the author's viewpoint and involve that person's particular strategy of presentation. Though I remember reading about global variables in passing, I hadn't realized that that's what I had constructed in this exercise, as I was merely following the author's example coding so far in the book.

For what it's worth this C language is unfolding for me nicely from the book and from your respective inputs. I intend to return the favors in due course.

Thanks, Sebastiani, for the tips and the compliment!

Code:

if(grade > 0 && grade <= 60)
    letter = 'F';
else if
    (grade <= 70)
        letter = 'D';

This won't work too well. Entering 0 or less will result in a grade D and will never exit the loop.

Re: Guest21

Good catch on that. I'd just like to point out that it would actually be an 'A' letter. Notice that all the letter grades above F have the same test as grade D.

Code:

if(grade > 0 && grade <= 60)
                    letter = 'F';
                else if
                (grade <= 70)
                    letter = 'D';
                else if
                (grade <= 80)
                    letter = 'C';
                else if
                (grade <= 90)
                    letter = 'B';
                else if
                (grade <= 100)
                    letter = 'A';
                else break;

Since a negative number (or zero) would be less than one it would follow all the letters up to A.

I don't know how I thought of that. I was thinking of switch statements. My bad