GPA方法计算不正确

Question

我的方法不断返回0。有人可以帮我找到为什么GPA无法正确计算吗？

public static double getGPA(String a)
{
    double value = 0;
    double sum = 0;
    for(int i = 0; i<a.length(); i++)
    {
        String grade = a.substring(i,i+1);
        if(grade == "A") value = 4;
        if(grade == "B") value = 3;
        if(grade == "C") value = 2;
        if(grade == "D") value = 1;
        if(grade == "F") value = 0;
        sum += value;
    }
    return sum/(a.length()+1);
}

Answer 1

你应该使用.equals（），而不是==。

你也有一些较小的错误：

1. 您需要除以字符串长度加上一个。
2. 您没有明智地处理无效输入。错误或忽略。

这是一张更正后的副本：

public static double getGPA(String a)
{
    long total = 0;
    long count = 0;
    for(int i = 0; i<a.length(); i++)
    {
        String grade = a.substring(i,i+1);
        count++;
        if(grade.equals("A")) total += 4;
        else if(grade.equals("B")) total += 3;
        else if(grade.equals("C")) total += 2;
        else if(grade.equals("D")) total += 1;
        else if(grade.equals("F")) total += 0;
        else count--;  /* Ignore invalid characters, e.g. whitespace. */
    }
    return total / (double) count;
}