0

谁能看到为什么我的toString()方法不起作用？它编译得很好，但是当我运行它时出现运行时错误：

这是我的代码：

import java.util.*;

public class ExamMarks {

    // fields
    private String moduleName;
    private int moduleCode;
    private int year;
    private int[] numbers;

    // methods
    public static int highestMark(int[] numbers) {
        int highest = numbers[numbers.length - 1];
        int i = numbers[0];
        for (i = 0; i < numbers.length - 1; i++) {
            if (numbers[i] > numbers[i + 1]) {
                highest = numbers[i];
            }
        }
        return highest;
    }

    public static double averageMark(int[] numbers) {
        int sum = 0;
        double average = 0.000;
        for (int i = 0; i < numbers.length; i++) {
            sum = numbers[i] + sum;
            average = sum / numbers.length;
        }
        return average;
    }

    // Constructor
    public ExamMarks() {
        moduleName = "Object Oriented Programming";
        moduleCode = 5015;
        year = 2013;
    }

    // toString
    @Override
    public String toString() {
        String result = ("The title of this module is : " + moduleName
                + "The code number of this module is: " + moduleCode
                + "The year of the Exam is: " + year + "Array average = "
                + averageMark(numbers) + "Highest grade = " + highestMark(numbers));
        return result;
    }

    // Tester
    public static void main(String[] args) {
        ExamMarks grade = new ExamMarks();
        Scanner percentages = new Scanner(System.in);
        int[] numbers = new int[20];
        double x = averageMark(numbers);

        System.out.println("Please enter 20 marks");

        for (int i = 0; i < numbers.length; i++) {
            numbers[i] = percentages.nextInt();

        }

        System.out.println(grade);
    }
}

这是我得到的错误

java.lang.NullPointerException at ExamMarks.averageMark(ExamMarks.java:26) 
    at ExamMarks.toString(ExamMarks.java:42) 
    at java.lang.String.valueOf(Unknown Source) 
    at java.io.PrintStream.println(Unknown Source) 
    at ExamMarks.main(ExamMarks.java:65) 
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
    at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) 
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) 
    at java.lang.reflect.Method.invoke(Unknown Source) 
    at edu.rice.cs.drjava.model.compiler.JavacCompiler.runCommand(JavacCompiler.java:27‌​2)
4

6 回答 6

3

NullPointerException 在这里。私有 int [] 数字；

numbers 在您的实例中为空。

当您调用grade.toString() 时，该方法调用

 public static double averageMark(int [] numbers){
    int sum = 0;  
    double average = 0.000;
    for (int i = 0; i < numbers.length; i++){   // !!!! numbers is null
于 2013-02-04T14:36:03.777 回答
0

你从来没有设置类的private int [] numbers;字段ExamMarks。

numbersintoString()方法属于ExamMarks类的实例，而numbersinmain(...)只是一个局部变量。

于 2013-02-04T14:36:18.047 回答
0

未设置“moduleName”、“moduleCode”和“year”字段。尝试在调用 System.out.println(grade); 之前设置它们

于 2013-02-04T14:34:01.573 回答
0

你需要让你的toString()函数对异常有弹性。

在最基本的层面上，您可以在整个实现周围添加一个tryand或制作函数并处理空指针异常。catchaverageMark(numbers)highestMark(numbers)

于 2013-02-04T14:34:27.923 回答
0

你打过电话了

averageMark(numbers);

直接地。因此，您用作参数的数字很好。

但是 toString() 方法也调用了这个方法，但是带有一个未初始化的类字段“数字”。

您也必须将“数字”设为静态并为其设置一些值。

于 2013-02-04T14:37:20.190 回答
0

在与 + 运算符连接之前，您应该检查您的成员是否为 != null。

如果您使用 eclipse 使用 rightclick -> source -> generate toString() ...它会toString在您的类中生成一个简单的方法，您只需要更改文本即可。

效果一般很好，而且很容易使用。

于 2013-02-04T14:38:16.760 回答