編集履歴

回答編集履歴

追記

2024/12/14 01:24

投稿

スコア13235

test CHANGED Viewed

@@ -4,6 +4,9 @@
 折角フィールドを private にしているのにメソッドでまんま参照が得られてしまっては public フィールドと変わりません。
 それと、 Scanner の nextInt が InputMismatchException を発した時のその後の Scanner の処理は大丈夫でしょうか?
+ついでに、同じ元から複数の Scanner を作るのは止めたほうが良いです。
+読み易さ的には、クラス名も Main とか Front のような何なのか役割が分からないとか、 StudentScore と StudentsScore みたいに 1 文字しか違わないパッと見間違えそうなのは止めたほうが良いと思います。
 まぁ兎にも角にも私好みにザーっと（過不足もあり）
 ```java

修正

2024/12/12 23:41

投稿

jimbe

スコア13235

test CHANGED Viewed

@@ -3,7 +3,7 @@
 一番の問題は冗長でも可読性でもなくカプセル化が出来ていないことに思います。
 折角フィールドを private にしているのにメソッドでまんま参照が得られてしまっては public フィールドと変わりません。
-それと、 Scanner の nextInt が InputMismatchException を発した時のその後の処理は大丈夫でしょうか?
+それと、 Scanner の nextInt が InputMismatchException を発した時のその後の Scanner の処理は大丈夫でしょうか?
 まぁ兎にも角にも私好みにザーっと（過不足もあり）
 ```java

追記

2024/12/12 23:28

投稿

jimbe

スコア13235

test CHANGED Viewed

@@ -2,6 +2,8 @@
 一番の問題は冗長でも可読性でもなくカプセル化が出来ていないことに思います。
 折角フィールドを private にしているのにメソッドでまんま参照が得られてしまっては public フィールドと変わりません。
+それと、 Scanner の nextInt が InputMismatchException を発した時のその後の処理は大丈夫でしょうか?
 まぁ兎にも角にも私好みにザーっと（過不足もあり）
 ```java

追記

2024/12/12 15:38

投稿

jimbe

スコア13235

test CHANGED Viewed

@@ -1,4 +1,7 @@
 年齢はどうなったんでしょうか。
+一番の問題は冗長でも可読性でもなくカプセル化が出来ていないことに思います。
+折角フィールドを private にしているのにメソッドでまんま参照が得られてしまっては public フィールドと変わりません。
 まぁ兎にも角にも私好みにザーっと（過不足もあり）
 ```java

コード追加

2024/12/12 14:56

投稿

jimbe

スコア13235

test CHANGED Viewed

@@ -1 +1,190 @@
 年齢はどうなったんでしょうか。
+まぁ兎にも角にも私好みにザーっと（過不足もあり）
+```java
+import java.util.*;
+import java.util.function.BiConsumer;
+public class StudentsScorer {
+  public static void main(String[] args) {
+    new StudentsScorer(new Scanner(System.in)).execute();
+  }
+  private static BiConsumer<String,Integer> SCORE_PRINTER =
+      (subject,score) -> System.out.println("  -" + subject + " : " + score);
+  private static BiConsumer<Integer,String> LIST_PRINTER =
+      (id,name) -> System.out.println("  - " + name + " : " + id);
+  private Scorer scorer = new Scorer();
+  private Scanner sc;
+  StudentsScorer(Scanner sc) {
+    this.sc = sc;
+  }
+  void execute() {
+    while(true) {
+      String ans = inputString(
+          "---------------------------------\n" +
+          "MENU\n" +
+          "  1: add student and their score\n" +
+          "  2: delete student info\n" +
+          "  3: find student based on id\n" +
+          "  4: list all student\n" +
+          "any: end");
+      switch (ans) {
+      case "1":
+        add();
+        break;
+      case "2":
+        delete();
+        break;
+      case "3":
+        find();
+        break;
+      case "4":
+        list();
+        break;
+      default:
+        return;
+      }
+    }
+  }
+  private void add() {
+    String name = inputString("Put name");
+    System.out.println("The name is " + name + ".");
+    Student student = new Student(name);
+    int id = scorer.regist(student);
+    System.out.println("Your ID is " + id + ".");
+    while (true) {
+      try {
+        String subject = inputString("what's subject?");
+        int score = inputInt("what's score?");
+        student.putScore(subject, score);
+      } catch (NumberFormatException e) {
+        System.out.println("** Invalid score **");
+      }
+      student.putScoresTo(SCORE_PRINTER);
+      String ans = inputString("continue? y/n");
+      if (!ans.equals("y")) break;
+    }
+    if (student.countScores() == 0) {
+      scorer.delete(id);
+    }
+  }
+  private void delete() {
+    try {
+      int id = inputInt("type ID");
+      Student student = scorer.find(id);
+      System.out.println("ID is " + id + ",");
+      System.out.println("this student is " + student.name + ".");
+      String ans = inputString("do you delete this info? y/n");
+      if (ans.equals("y")) {
+        scorer.delete(id);
+        System.out.println("Successfully deleted.");
+      } else {
+        System.out.println("you said no -> back to menu.");
+      }
+    } catch (NumberFormatException|NotFoundException e) {
+      System.out.println("** Invalid ID **");
+    }
+  }
+  private void find() {
+    try {
+      int id = inputInt("type ID");
+      Student student = scorer.find(id);
+      System.out.println("you are " + student.name + ".");
+      System.out.println("your ID is " + id + ".");
+      System.out.println("and your score :");
+      student.putScoresTo(SCORE_PRINTER);
+    } catch (NumberFormatException|NotFoundException e) {
+      System.out.println("** Invalid ID **");
+    }
+  }
+  private void list() {
+    System.out.println("this is the list :");
+    scorer.putListTo(LIST_PRINTER);
+  }
+  private String inputString(String prompt) {
+    System.out.print(prompt + ": ");
+    return sc.nextLine();
+  }
+  private int inputInt(String prompt) throws NumberFormatException {
+    System.out.print(prompt + ": ");
+    return Integer.parseInt(sc.nextLine());
+  }
+}
+class NotFoundException extends RuntimeException {
+  public NotFoundException(String message) {
+    super(message);
+  }
+}
+class Scorer {
+  private final Map<Integer,Student> studentMap = new HashMap<>();
+  private final Random random = new Random();
+  int regist(Student student) {
+    int id = generateId();
+    studentMap.put(id, student);
+    return id;
+  }
+  Student find(int id) throws NotFoundException {
+    if (!studentMap.containsKey(id)) throw new NotFoundException("id=" + id);
+    return studentMap.get(id);
+  }
+  void delete(int id) {
+    studentMap.remove(id);
+  }
+  void putListTo(BiConsumer<Integer,String> consumer) {
+    studentMap.entrySet().stream().forEach(e -> consumer.accept(e.getKey(), e.getValue().name));
+  }
+  private int generateId() {
+    while (true) {
+      int id = random.nextInt(10000);
+      if (!studentMap.containsKey(id)) return id;
+    }
+  }
+}
+class Student {
+  final String name;
+  private final Map<String,Integer> scoreMap = new HashMap<>();
+  Student(String name) {
+    this.name = name;
+  }
+  void putScore(String subject, int score) {
+    scoreMap.put(subject, score);
+  }
+  int countScores() {
+    return scoreMap.size();
+  }
+  void putScoresTo(BiConsumer<String,Integer> consumer) {
+    scoreMap.entrySet().stream().forEach(e -> consumer.accept(e.getKey(), e.getValue()));
+  }
+}
+```