Thread: I want to recieve comments!

Code:

main()

And main() is of what return type? int. Say as such, it never hurts:

Code:

int main()

I personally use a tab as my indent, if you use spaces, I would recommend more than 1.

On if() statements that don't require { }'s, I tend to either write:

Code:

if(blah) blah;

Or, if I move it to the next line,

Code:

if(blah)
   blah;

... then I indent it, as above. This avoids some confusion with what the if statement actually deals with.

How to make it repeat? You could stick it in a while loop:

Code:

int main()
{
  int grade;
  
  while(1)
  {
    printf("Please enter your grade:\n");
    ...
  }
  return 0;
}

But that will go on forever, so you'll want to provide a way out. (Entering something like -1 to quit, etc.)

Hope that helps!

I recommend you get this book: The C Programming Language by Brian W. Kernighan and Dennis M. Ritchie.

Code:

main()

Should be:

Code:

int main()

Here is a simple loop construct to be executed 5 times:

Code:

int counter;
for(counter=0; counter<5; counter++)
{
    \* to do. *\
}

edit
Darn it, beaten again!
/edit

puts() does everything for your program that printf() does in a much easier way. printf was made to validate format strings, which your program doesn't use. puts writes a string to the stdout and appends a newline character.

Before you loop your program, I would feel better if you got rid of scanf entirely.Use fgets() to read in a string from the user, and then you can use the function strtol in <stdlib.h> to turn it into a number.
Here's an example:

Code:

char userin[40];
char *end = NULL;
puts("Enter your grade");
fgets(userin, sizeof userin, stdin);
long grade = strtol(userin, &end, 10);

more about this in the FAQ

Now as for looping the program, after you output the grade, ask the user if he's like to exit. Use his answer as a loop condition.