Conversation
Styopa1769
left a comment
Styopa1769
left a comment
There was a problem hiding this comment.
Все отлично, молодец, замечания в комментариях
| HomePet tame(StrayPet strayPet, String newOwnerName); | ||
| } | ||
|
|
||
| public class StrayPet { |
There was a problem hiding this comment.
Лучше так не делать
1 файл, 1 класс
если логично добавить тут же, то делать класс вложенным
но, в случае с функциональным интерфейсом, это точно не логично :)
| interface Taming { | ||
| HomePet tame(StrayPet strayPet, String newOwnerName); | ||
| } |
There was a problem hiding this comment.
наверное, я хотел что-то такое увидеть, но задача для этого не очень удачная, так что ок
| interface Taming { | |
| HomePet tame(StrayPet strayPet, String newOwnerName); | |
| } | |
| interface Taming<T, R> { | |
| R tame(T strayPet, String newOwnerName); | |
| } |
|
|
||
| Set<ComputerImpl> tree = new TreeSet<ComputerImpl>(); | ||
| for (int i = 0; i < 3; i++) { | ||
| tree.add((ComputerImpl)array[i]); |
There was a problem hiding this comment.
а вот так делать плохо
если тебе не нужен нигде именно Computer, то можно возвращать из testInit ComputerImpl
и проще было бы возвращать не массив, а лист
|
|
||
| @Override | ||
| public int compareTo(ComputerImpl o) { | ||
| if (!maker.equals(o.maker)) |
There was a problem hiding this comment.
можно было обойтись без equals, при помощи того же тернарника и compare
There was a problem hiding this comment.
и посмотри еще у компаратора метод thenComparing
2 participants
Вот 11ая работа
Файлы компьютера тут же (каждый из которых отправлен в отдельный файлик)
-Решила прописать сравнимость в ComputerImpl.
-Придумала специальный метод для теста 1ого таска (извините, файловый ввод-вывод не требовался, так что вот)
-Положила Taming в файл с бездомным питомцем, поскольку в задании не обговаривалось, и посчитала логичным ¯\(ツ)/¯
Папоротник.