Conversation
DisterDE
left a comment
DisterDE
left a comment
There was a problem hiding this comment.
В целом хорошо, но нужны доработки.
| private String authorName; | ||
| private String authorNameTwo; | ||
| private String authorNameThree; | ||
| private String authorSurname; | ||
| private String authorFullName; |
There was a problem hiding this comment.
Такое количество имен, конечно, поражает. :)
Так же следует убрать из имен полей слово author, так как и так понятно, кому они принадлежат.
| } | ||
|
|
||
| public String getAuthorFullName() { | ||
| authorFullName = authorFullName.replace("null ", ""); |
There was a problem hiding this comment.
Чтобы этого не было, нужно собирать fullName через StringBuilder, проверяя каждое поле на null.
| public Book() { | ||
|
|
||
| } |
There was a problem hiding this comment.
Этот конструктор тут не нужен.
Книга не может быть создана без названия и автора.
| String author; | ||
| String nameOfBook; | ||
| int dateOfPublish; |
There was a problem hiding this comment.
Поля должны быть private, а в случае отсутствия сеттера, ещё и final
| String author; | ||
| String nameOfBook; | ||
| int dateOfPublish; |
| for (int i = 0; i < books.length; i++) { | ||
| if (books[i] == null) { | ||
| books[i] = book; | ||
| System.out.println(); |
| for (int i = 0; i < books.length; i++) { | ||
| books[i] = null; | ||
| } |
There was a problem hiding this comment.
Массив books при создании и так заполнился нуллами.
Этот цикл ничего не делает.
| } | ||
| } | ||
|
|
||
| public void AddNewBook(Book book) { |
There was a problem hiding this comment.
Методы в java принято называть с маленькой буквы.
| } | ||
| } | ||
|
|
||
| public void PrintAllBooks() { |
There was a problem hiding this comment.
Методы в java принято называть с маленькой буквы.
| } else { | ||
| return; | ||
| } |
There was a problem hiding this comment.
Можно поменять местами условие и это поможет избавиться от else блока.
if (books[i] == null) {
return;
}
Тут уже печатаем.
2 participants
No description provided.