初学者java，帮我修复我的程序吗？

Question

我正在尝试制作一个大学平均绩点计算器。我删掉了 20 个 if 语句，只说明每个字母的等级。我为再次关注这个问题的人修复了我的第一个程序。该程序现在可以运行，但无论我在 gpa 中输入什么字母，它返回的都是 2.0 。如果有人发现任何问题，我们将非常感激......再次。谢谢

import java.util.Scanner;

public class universityGPA {
    public static void main(String args[]){

        int classes = 4;
        int units[] = {3, 2, 4, 4};
        double[] grade = new double[4];
        double[] value= new double[4];
        int counter = 0;
        double total = 0;
        double gpa;
        String letter;


        while(classes > counter){
            Scanner gradeObject = new Scanner(System.in);
             letter = gradeObject.next();

            if(letter.equalsIgnoreCase("A+") || letter.equalsIgnoreCase("A")){
                grade[counter] = 4;
            }
            if(letter.equalsIgnoreCase("F")){
                grade[counter] = 0;
            }

            value[counter] = grade[counter] * units[counter];
            counter++;
        }

        for(int i = 0; i < classes; i++  ){
            total += value[i];
        }

        gpa = total/classes;
        System.out.println("You gpa is " +gpa);
    }
}

Answer 1

您忘记初始化等级。 NullPointerException 告诉您 grade 为 null。第一次尝试在语句 grade[counter] = 4; 中使用 grade 时，会引发异常。使用 new 分配所需的空间。

Answer 2

成绩的初始化可以静态也可以动态完成：

double []grade = new double[4];

或者

double []grade = new double[classes];

对 value 也执行相同的操作。

Answer 3

以下是清理代码的一些建议：

• 尝试与格式更加一致。确保所有内容都正确缩进，并且在行的开头或结尾处没有残留空格（第 18 行）。
• 您应该在尽可能靠近第一个使用变量的地方声明变量。这不仅使您的代码更具可读性，还最大限度地减少了范围。例如，在第 18 行，您初始化了 letter，但它绝不会在 while 语句的范围之外使用。您应该在此处声明该变量以及初始值设定项（String letter =gradeObject.next()）。
• 不鼓励以 type name[] 形式声明数组。建议使用type[] name 形式。
• 尝试将您的程序分成不同的部分。例如，对于这个程序，您可以清楚地看到涉及几个步骤。也就是说，您首先必须获取一些输入，然后解析它，然后计算返回值。这些部分可以分解为单独的方法来清理代码并促进重用。虽然对于这样一个简单的程序来说，它似乎不会产生很多好处，但一旦你开始解决更大的问题，这个组织将是绝对强制性的。

Answer 4

NullPointerException 意味着您正在尝试访问不存在的内容。

由于您的 Grade[] 为空，因此在第 21 行通过 Grade[counter] 访问它实际上意味着您正在访问尚未创建的内容。

您需要初始化该数组，因此它实际上有一个实例。