task2909. Рефакторинг 8

Author: SkyrocketStan

3.JavaMultithreading/src/com/javarush/task/task29/task2909/human/Human.java

@@ -5,7 +5,7 @@
 import java.util.List;
 
 public class Human implements Alive{
-    public static int nextId = 0;
+    private static int nextId = 0;
     private int id;
     protected int age;
     protected String name;
@@ -77,10 +77,6 @@ public int getId() {
         return id;
     }
 
-    public void setId(int id) {
-        this.id = id;
-    }
-
     public void printSize() {
         System.out.println("Рост: " + size[0] + " Вес: " + size[1]);
     }

3.JavaMultithreading/src/com/javarush/task/task29/task2909/human/Student.java

@@ -24,6 +24,14 @@ public void live() {
     public void learn() {
     }
 
+//    public void incAverageGradeBy01() {
+//        averageGrade += 0.1;
+//    }
+//
+//    public void incAverageGradeBy02() {
+//        averageGrade += 0.2;
+//    }
+
     public void incAverageGrade(double delta){
         averageGrade += delta;
     }

3.JavaMultithreading/src/com/javarush/task/task29/task2909/human/StudentsDataBase.java

@@ -4,36 +4,37 @@
 import java.util.List;
 
 public class StudentsDataBase {
-    public static List<Student> students = new ArrayList<>();
-
-
-    public static void addInfoAboutStudent(Student student) {
-        students.add(student);
-        printInfoAboutStudent(student);
+  public static List<Student> students = new ArrayList<>();
+
+  public static void addInfoAboutStudent(Student student) {
+    students.add(student);
+    printInfoAboutStudent(student);
+  }
+
+  public static void printInfoAboutStudent(Student student) {
+    System.out.println("Имя: " + student.name + " Возраст: " + student.getAge());
+  }
+
+  public static void removeStudent(int index) {
+    try{
+      students.remove(index);
+    } catch (Exception e) {
+      // Do nothing
     }
 
-    public static void printInfoAboutStudent(Student student) {
-        System.out.println("Имя: " + student.name + " Возраст: " + student.getAge());
-    }
+  }
 
-    public static void removeStudent(int index) throws IndexOutOfBoundsException {
-        students.remove(index);
-    }
+  public static void findDimaOrSasha() {
+    for (int i = 0; i < students.size(); i++) {
+      if (students.get(i).getName().equals("Dima")) {
+        System.out.println("Студент Dima есть в базе.");
+        break;
+      }
 
-    public static void findDimaOrSasha() {
-        boolean found = false;
-        for (int i = 0; i < students.size(); i++) {
-            if (!found) {
-                if (students.get(i).getName().equals("Dima")) {
-                    System.out.println("Студент Dima есть в базе.");
-                    found = true;
-                }
-
-                if (students.get(i).getName().equals("Sasha")) {
-                    System.out.println("Студент Sasha есть в базе.");
-                    found = true;
-                }
-            }
-        }
+      if (students.get(i).getName().equals("Sasha")) {
+        System.out.println("Студент Sasha есть в базе.");
+        break;
+      }
     }
-}
+  }
+}