0

我在解决我从课堂上的getAverageScore()方法中得到的 ArithmeticException 时遇到了麻烦。Student我正在尝试编写一个读取以下文本文件的程序scores.txt：

34     c081 c082 c083 c084
S2023  99   75   85   62
S2025  -1   92   67   52
S1909  100  83   45   -1

c081等c数字是学校的课程代码，s2023等s数字是学号。中间的数字代表他们的分数，-1 也意味着他们没有被录取。

我希望能够编写一个程序，在该程序中，我可以将输出作为班级中平均水平最高的学生，以及该平均水平和他们的学生人数。我希望能够确保 -1 值不包含在该班级的平均值中，因此如果学生有其中之一，则他们的平均值仅针对他们实际上过的班级计算。

这是Student课程：

import java.util.ArrayList;

public class Student {

    private String studentNumber;
    private ArrayList<Integer> scores = new ArrayList<Integer>();

    public Student(String studentText) {
        String[] parts = studentText.split(studentText, ' ');
        this.studentNumber = parts[0];

        for (int i = 1; i < parts.length - 1; i++) {
            scores.add(Integer.parseInt(parts[i + 1]));
        }
    }

    public String getStudentNumber() {
        return this.studentNumber;
    }

    public float getAverageScore() {
        int sum = 0;
        int numberOfCoursesNotTaken = 0;

        for (int i = 0; i <this.scores.size(); i++) {
            if (i != -1) {
            sum += this.scores.get(i);
            System.out.println(sum);
            } else {
                numberOfCoursesNotTaken++ ;
                System.out.println(numberOfCoursesNotTaken);
            }
        }
        return sum / (this.scores.size() - numberOfCoursesNotTaken);
    }


}

我从以下类调用此方法：

import java.io.File; 
import java.io.FileNotFoundException;
import java.util.ArrayList;
import java.util.Scanner;

public class MySchool {
    public static void main(String[] args) {
        int numberOfStudents;
        ArrayList<Student> allStudents = new ArrayList<Student>() ;

        try {

            File scoresFile = new File("Scores.txt");
            Scanner scoresFileReader = new Scanner(scoresFile);


            String headerRow = scoresFileReader.nextLine();

            char c = headerRow.charAt(0);
            numberOfStudents = Character.getNumericValue(c);  


            for (int studentI = 0; studentI < numberOfStudents; studentI++) {
                String studentText = scoresFileReader.nextLine();
                System.out.println(studentText);
                Student student = new Student(studentText);
                allStudents.add(student);
            }

            scoresFileReader.close();

        } catch (FileNotFoundException e) {
            System.out.println("An error occurred");
            e.printStackTrace();
        }

        float highestAverage = 0;
        String highestScoringStudentNumber = null;


        for (Student student : allStudents) {
            if (student.getAverageScore() > highestAverage) {
                highestAverage = student.getAverageScore();
                highestScoringStudentNumber = student.getStudentNumber();
            }

        }
        System.out.println("Highest scoring student: " + highestScoringStudentNumber);
    }
}

getAverageScore()但是， in的返回行Student给出了以下错误：Exception in thread "main" java.lang.ArithmeticException: / by zero at Student.getAverageScore(Student.java:34) at MySchool.main(MySchool.java:42)

我以为我已经使用numberOfCoursesNotTaken变量解决了这个问题？有人可以帮我吗？

4

2 回答 2

0

Student class -> getAverageScore ：在 for 循环中，您正在检查 i != -1，而不是 if scores.get(i) != -1。

于 2020-06-15T11:11:04.820 回答
0

我真的建议在发现无效值后立即删除它们。您可以通过以下方式替换您的Student构造函数：

public Student(String studentText) {
    String[] parts = studentText.split(" "); // previously (studentText, ' ')
    this.studentNumber = parts[0];

    for (int i = 1; i < parts.length - 1; i++) {
        int score = Integer.parseInt(parts[i]); // NOT i+1
        if (score >=0) this.scores.add(score);
    }
}

当您使用它时，您可以大大简化平均分计算（现在没有 -1）：

public float averageScore() {
   if (scores.isEmpty()) {
      return 0; // avoids divide-by-zero
   } else {
      int total = 0;
      for (int score : scores) total += score;
      return total / (float) scores.size();
   }
}
于 2020-06-15T11:26:45.507 回答