whats wrong with this code

i am trying to get this program to ask the user to answer whether or not the class is pass fail or for a grade. prior to asking what the weight of the grade is


#include <stdio.h>
char grade (int exam1, int exam2, float exam1_weight);
char type (int P, int G);
main ()
{
int ans, mid_term, final, P, G, pass, letter;
float weight;
char letter_grade,c_type;
do {
printf("\n\nCompute another grade (1 = Yes, 0 =No)? ");
scanf("%d", &ans);
printf("\n Is this class Pass or Fail or For a Grade?");
scanf("%c\n", &pass,&letter);
if (ans == 1) {
printf( "\nEnter mid_term, final, weight: ");
scanf("%d%d%f", &mid_term, &final, &weight );
letter_grade = grade(mid_term, final, weight);
printf("Grade is: %c\n", letter_grade);
}
if (pass == P, letter == G) {
c_type = type(P, G);
printf("Class Type is: %c\n", c_type);
}
}
while (ans == 1);

}

char grade (int exam1, int exam2, float exam1_weight)
{
float average;

average = exam1_weight * exam1 + (1.0 - exam1_weight) * exam2;
if (average > 7.0)
return 'P';
else
return 'F';
}

> scanf("%c\n", &pass,&letter);

Why are you scanning for ONE value (%c) and passing two arguments to scan into?

Furthermore, why are you checking the value of "ans" after you do the second scanf? Shouldn't you actually check to see if they want to do a test and then if they do, ask for the second prompt?

Quzah.

1/ scanf("%c\n", &pass,&letter);

not only are you scanning for one value and passing two arguments, but the arguments are of type int and you are scanning for a char.

2/ if (pass == P, letter == G)

Haven't seen this before should it be?

if (pass == P && letter == G)

3/ char type (int P, int G);

Where is the function definition and what does it do?

4/ average = exam1_weight * exam1 + (1.0 - exam1_weight) * exam2;

You might have to cast int to float, ie

average = exam1_weight * (float)exam1 + (1.0 -
exam1_weight) * (float)exam2;

using code tags would be nice