关于你的法典,还有几点意见:

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;
}

而应当:

#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;
}

通知一:

• 删除的无用评论

• (0和100)

• 取消了无用试验(如果你在休息期间离开,则X在{0、......、100})。

• 允许<代码>x

• removed an idempotent assignment (x=x doesn t do anything useful...)

然后:

float getMark(void)
{
    float mark=0,assignMark=0,midMark=0,examMark=0,finalMark=0;
    char letterGrade;

    printf("Enter assignment grade:");
    scanf("%d",mark);

^ 如其他发言者所述,您应使用<代码>%f和&mark,载于scanf。

    checkMark(mark);

^ Here, it should be mark = checkMark(mark); if you want the mark variable in this code to be updated. This is because the value of mark is passed to the function, not a reference to the variable mark. Another solution is to pass such a reference, but I don t think you re ready for this yet :)

    assignMark=mark;

    printf("
Enter midterm mark:");
    scanf("%f",&mark);

    checkMark(mark);
    midMark=mark;

    printf("
Enter exam mark:");
    scanf("%f",&mark);

    checkMark(mark);
    examMark=mark;

^ Same remarks for all these lines

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

^ 通过<条码>限值/代码>作为此处的参数是完全无用的。 更正应只对原文提出。 Mark,年中期市场,考试;,你不认为?

    letterGrade=computeLetterGrade(finalMark);

    printf("The final grade for the course is %.1f, and the letter grade is %c.
",finalMark,letterGrade);

    return 0;
}

As mentioned just earlier, you don t need to pass some finalGrade value to computeFinalGrade, you should be okay with just:

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

最后,在<代码>computeLetter Grade中,你正在做大量多余的测试,你的控制流动是用这一可变的任务加以衡量的,你应当使用<代码><>char/code>。 而不是<代码>int。 相反:

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 ;
}

You will probably need to fix some things I didn t mention, but at least you should be closer to your goals with some of these corrections.

If you are on linux, take a look at my answer here. It s about valgrind, a tool that helps debugging segfaults, memory leaks etc.

当贵方案试图进入分配给你的方案之外的记忆区时,会发生分裂。 <>代码>scanf(>预期该变量的地址。 在您的法典中,你将变数的数值转至scanf(),而不是像公诉人指出的那样。

scanf("%d",mark);

标记变量没有初步化,有些随机数据,用扫描仪作为记忆地点处理,在试图写到这个地址时。 由于你获得的记忆有限,因此出现分离错误。 此外,标记是浮动变量,因此,请使用<>%fspecifier。

scanf("%f", &mark);