回答編集履歴

説明追記

2020/01/21 10:21

投稿

スコア5406

test CHANGED Viewed

@@ -1,5 +1,7 @@
 直接の回答ではないですが、私もリファクタリングしてみました。
+オブジェクトには役割を持たせましょう。生徒役、先生役。先生が生徒に成績を伝える。平均点は生徒が計算する。
 何か参考になることがあれば幸いです。

コード変更

2020/01/21 10:21

投稿

スコア5406

test CHANGED Viewed

@@ -300,34 +300,36 @@
-    public boolean load(BufferedReader file) throws IOException {
+    public void load(BufferedReader file) throws IOException {
+        while (true) {
-        String number = file.readLine();
+            String number = file.readLine();
-        if (number == null) {
+            if (number == null) {
-            return false;
+                return;
+            }
+            Student student = new Student();
+            student.setNumber(number);
+            student.setName(file.readLine());
+            student.setEnglish(readInt(file));
+            student.setLiterature(readInt(file));
+            student.setMathematics(readInt(file));
+            file.readLine(); // drop out average
+            students.add(student);
         }
-        Student student = new Student();
-        student.setNumber(number);
-        student.setName(file.readLine());
-        student.setEnglish(readInt(file));
-        student.setLiterature(readInt(file));
-        student.setMathematics(readInt(file));
-        file.readLine(); // drop out average
-        students.add(student);
-        return true;
     }
@@ -338,7 +340,7 @@
         System.err.println(filename + "から読み込みます。");
-        while (load(file));
+        load(file);
         System.out.println(students.size() + "人のデータを読み込みました。");

コード修正

2020/01/21 10:17

投稿

スコア5406

test CHANGED Viewed

@@ -276,10 +276,6 @@
 ```ReadGradeFromFile.java
-//読み込み用クラス
 //package grade6;
 import java.io.*;
@@ -342,9 +338,7 @@
         System.err.println(filename + "から読み込みます。");
-        for (int n = 1; load(file); n++) {
+        while (load(file));
-        }
         System.out.println(students.size() + "人のデータを読み込みました。");