Faute de Segmentation core dumped...?

Quelques remarques sur ton code:

int checkMark(int x)
{
while(x<0 || x>100) //While x is a negative number:
{
printf("Invalid entry.Try again:");
scanf("%d",&x);
}
if(x>0 && x<=100)//If x is a positive number:
x=x;
return x;
}

devrait plutôt être:

#define MIN_GRADE 0
#define MAX_GRADE 100
int checkMark(int x)
{
while(x<MIN_GRADE || x>MAX_GRADE)
{
printf("Invalid entry.Try again:");
scanf("%d",&x);
}
return x;
}

Avis que j':

• retiré inutile commentaires

• retiré de la magie des chiffres (0 et 100)

• retiré inutile de test (si vous la sortie de la boucle while, alors x est dans {0, ..., 100}

• permis x être égale à 0

• enlevé un idempotent de cession (x=x ne pas faire quelque chose d'utile...)

Alors:

float getMark(void)
{
float mark=0,assignMark=0,midMark=0,examMark=0,finalMark=0;
char letterGrade;
printf("Enter assignment grade:");
scanf("%d",mark);

^ Comme mentionné par d'autres, vous devez utiliser %f et &mark dans ce scanf.

    checkMark(mark);

^ Ici, il convient de mark = checkMark(mark); si vous voulez le mark variable dans le présent code pour être mis à jour. C'est parce que la valeur de mark est passé à la fonction, pas une référence à la variable mark. Une autre solution est de passer de cette référence, mais je ne pense pas que vous êtes prêt pour cela encore 🙂

    assignMark=mark;
printf("\nEnter midterm mark:");
scanf("%f",&mark);
checkMark(mark);
midMark=mark;
printf("\nEnter exam mark:");
scanf("%f",&mark);
checkMark(mark);
examMark=mark;

^ Mêmes remarques pour toutes ces lignes

    finalMark=computeFinalGrade(assignMark,midMark,examMark,finalMark);

^ En passant finalMark comme un paramètre ici est complètement inutile. Il faut juste être finalMark = computeFinalGrade(assignMark, midMark, examMark);, vous ne pensez pas?

    letterGrade=computeLetterGrade(finalMark);
printf("The final grade for the course is %.1f, and the letter grade is %c.\n",finalMark,letterGrade);
return 0;
}

Comme mentionné plus tôt, vous n'avez pas besoin de passer quelques finalGrade de la valeur à computeFinalGrade, vous devriez être bien avec juste un:

float computeFinalGrade(float assign, float midterm, float finalExam)
{
return 0.2*assign + 0.3*midterm + 0.4*finalExam; //this could even be a macro
}

Enfin, dans computeLetterGrade, vous faites beaucoup de tests redondants, et votre flux de contrôle est gâché par cette affectation de variable, de plus, vous devez utiliser un char plutôt qu'un int. Essayez plutôt ceci:

char computeLetterGrade(float finalGrade)
{
if(finalGrade>=80) return 'A';
//Since we did not return, necessarily finalGrade < 80
if(finalGrade>=70) return 'B';
if(finalGrade>=60) return 'C';
if(finalGrade>=50) return 'D';
//All other ifs failed, therefore finalGrade < 50
return 'E';
}

Vous aurez probablement besoin de corriger certaines choses que je n'ai pas mentionné, mais au moins vous devriez être au plus près de vos objectifs avec certaines de ces corrections.

La Segmentation des défauts se produisent lorsque votre programme tente d'accéder à une zone de mémoire à l'extérieur de celle attribuée à votre programme. scanf() s'attend à une adresse de la variable. Dans votre code, vous avez passé la variable de la valeur à scanf() au lieu de son adresse, ainsi que l'a souligné cnicutar.

scanf("%d",mark);

La marque de la variable n'est pas initialisée et a des données aléatoires, qui est considérée comme un emplacement de la mémoire par le scanf et lorsqu'il tente d'écrire à cette adresse... erreur de Segmentation se produit en raison de l'accès limité de la mémoire. Aussi, la marque est variable de type float, et donc vous devriez utiliser %f spécificateur..

scanf("%f", &mark);