Lesson 05 #5
Lesson 05 #5
Conversation
| public class Course { | ||
| private String name; | ||
| private String teacher; | ||
| private ArrayList<Student> student = new ArrayList<>(); |
There was a problem hiding this comment.
списки всегда именуем во множественном числе
| private String name; | ||
| private String teacher; | ||
| private ArrayList<Student> student = new ArrayList<>(); | ||
| private ArrayList<Student> attendanceJournal = new ArrayList<>(); |
There was a problem hiding this comment.
Посещаемость можно реализовать как отдельный журнал где будет указан (id, course_id, student_id, attendance_date, status(enums))
Также можно реализовать и бальную систему через журнал где будет указан (id, course_id, student_id, assignment, point, date)
Курс сам по себе обособленная сущность, которая не должна содержать студентов и посещаемость.
| } | ||
|
|
||
| public void addStudent(Student s) { | ||
| for (Student student1 : student) { |
There was a problem hiding this comment.
Проверка дубликатов это хорошо, и как ее можно улучшить? Чтобы не перебирать весь список, можно использовать Set вместо List или же Map
| public void showResults() { | ||
| System.out.println("\nQuiz results for " + "\"" + title + "\""); | ||
| int maxScore = 0; | ||
| for (Team team : teams) { |
There was a problem hiding this comment.
два одинаковых цикла не хорошо, лучше реализовать всю логику в одном цикле
There was a problem hiding this comment.
Понял, спасибо за комментарии
Completed Lesson 05