Thread: Newbie college student :)

Try this you still need to modify a few things like taking the \n out of the string firstname and lastname and not use fflush to clear the stdin buffer

Code:

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

#define A 100
#define B 90
#define C 80
#define D 70
#define E 60
#define ZERO 0

int main()
{

	/*char ch;*/
	char firstname[80], lastname[80];
	/*char low=3;
	char high=12;*/
	int mark=0, n=0;

	int randNumber;

	srand(time(0));

	randNumber=rand() % 10 + 3;

	printf("The number of students in the class is %d.\n", randNumber);

	while (n != randNumber)
	{
		n++;
		printf ("Please enter your first name: \n");
		fgets(firstname, sizeof firstname, stdin);
		fflush(stdin);
		printf ("Please enter your last name: \n");
		fgets(lastname, sizeof lastname, stdin);
		fflush(stdin);
		printf ("Please enter your marks: \n");
		scanf ("%d", &mark);
		fflush(stdin);

		if (mark >= B && mark <= A)
			printf("%s %s %d Grade A\n", firstname, lastname, mark);
		else if (mark >= C && mark <= B)
			printf("%s %s %d Grade B\n", firstname, lastname, mark);
		else if (mark >= D && mark <= C)
			printf("%s %s %d Grade C\n", firstname, lastname, mark);
		else if (mark >= E)
			printf("%s %s %d Grade D\n", firstname, lastname, mark);
		else if (mark < E)
			printf("%s %s %d Grade F\n", firstname, lastname, mark);
		else
			printf("Marks are invalid./n");
	}
	
	return 0;
}

[edit]
Your logic was out too you use || instead of && its now changed
[/edit]

scanf() is not the function of choice for a many C programmers. It just opens a can of very stinky worms. If you must use it, remember that it returns a value, which corresponds to the amount of items correctly converted. This allows a small measure of error checking. For example:

Code:

#include <stdio.h>
#include <stdlib.h>

int main(void)
{
    int age;
    char fname[10];

    printf("Enter your first name: ");
    fflush(stdout);
    if (scanf("%s", fname) != 1) {
        puts("scanf() error!");
        exit(EXIT_FAILURE);
    }
    printf("Enter your age: ");
    fflush(stdout);
    if (scanf("%d", &age) != 1) {
        puts("scanf() error!");
        exit(EXIT_FAILURE);
    }
    printf("Your first name is %s, and your age %d\n", fname, age);
    return 0;
}

~/

Originally Posted by jat421

kermit,

so if "fname" is not equal to "1" it will exit?. So what does the vaule "1" stand for?. Sorry I am new to this, so trying to understand the code that you wrote .

Thanks

As Sake pointed out, the value 1 corresponds to the amount of arguments successfully converted by scanf().

Many functions return a value, which are free to be used, or ignored. A good example of an ignored return value would be the one returned from the printf() function - printf() returns the amount of characters printed, yet you will probably not see that value used very often. In the case of scanf(), as has been mentioned, we can choose to ignore or not ignore its return value, and instead use it to see if there was any problem with conversion.

Now consider how the return value from scanf() is obtained. When you call a function which returns a value, you can assign it to a variable like so:

Code:

int ret_val;  /* will hold whatever value scanf() returns */
ret_val = scanf("%s", fname);

To see if scanf() was successful, you would check the value stored in the variable ret_val against the number of arguments passed to scanf() (which in this case is 1):

Code:

int ret_val;
ret_val = scanf("%s", fname);
if(ret_val != 1) {
/* do some stuff */
}

Of course, writing code in the last method is awkward at best. Not much sense in declaring an extra variable if its not necessary.

Now then, if you had a call to scanf() that passed 2 arguments, you would check to see that the return value of scanf() equalled 2. Any number other than 2, indicates a problem. For example:

Code:

char fname[10];
char lname[10];

    printf("Enter your first and last name: ");
    fflush(stdout);
    if (scanf("%s%s", fname, lname) != 2) {
        puts("scanf() error!");
        exit(EXIT_FAILURE);
    }

Finally, in the examples I have given, if scanf() fails, for whatever reason, the program exits.

~/

Code:

char fname[10];
char lname[10];

    printf("Enter your first and last name: ");
    fflush(stdout);
    if (scanf("%s%s", fname, lname) != 2) {
        puts("scanf() error!");
        exit(EXIT_FAILURE);
    }

Never use scanf's %s directive without supplying width. Otherwise you've created something similar to gets.

You know its funny Dave, I was just thinking about people who know how to use scanf() better than me, and you came to mind. Of course using scanf() like that is a lovely way to over run the bounds of an array, and I should have mentioned it.