Доработана функциональность по техническому заданию спринта #1
Доработана функциональность по техническому заданию спринта #1Max-Browckin wants to merge 8 commits intomainfrom
Conversation
avfyodorov
left a comment
avfyodorov
left a comment
There was a problem hiding this comment.
Добрый день, Максим!
Очень хорошая работа, серьёзных замечаний нет, есть уточнения, но и они на Ваше усмотрение.
|
|
||
| Subtask getSubtaskByID(int id); | ||
|
|
||
| ArrayList<Task> getTasks(); |
There was a problem hiding this comment.
ArrayList
Вообще в качестве типов, возвращаемых из методов, не стоит использовать типы конкретных реализаций, лучше использовать более высокие абстракции, List<..>, Map<..> или даже Collection<..> Детали реализации лучше скрывать :)
Необязательно, на Ваше усмотрение.
|
|
||
| public class InMemoryHistoryManager implements HistoryManager { | ||
| private final List<Task> history = new ArrayList<>(); | ||
| private static class CustomLinkedList { |
There was a problem hiding this comment.
private static class CustomLinkedList {
Это класс можно было и не создавать, а его методы реализовать как приватные методы менеджера истории.
Необязательно, на Ваше усмотрение.
avfyodorov
left a comment
avfyodorov
left a comment
There was a problem hiding this comment.
Добрый день, Максим!
Замечаний нет.
Работа принята.
Есть просьба-оставлять хоть какую-то реакцию на мои комментарии. Сделано/ Не сделано, потому что не согласен/ Не заметил. 🙂
2 participants
No description provided.