search

lm-cyber / lab6_arhitip

0 stars 0 forks source link

Инкапсуляция #2

Closed AlphaBeth closed 2 years ago

AlphaBeth commented 2 years ago

https://github.com/lm-cyber/lab5_arhitip/blob/e462941d5f400acb59c21bff661a9d120af3f69d/lab-client/src/main/java/com/alan/lab/client/commands/AddIfMinCommand.java#L27-L29

Обединить логику сравнения и добавления в один метод в CollectionManager, из команды вызывать его.

AlphaBeth commented 2 years ago

https://github.com/lm-cyber/lab5_arhitip/blob/e462941d5f400acb59c21bff661a9d120af3f69d/lab-client/src/main/java/com/alan/lab/client/commands/FilterGreaterThanHeightCommand.java#L21-L23

Тут получение коллекции по условию стоит выделить в отдельный метод.

AlphaBeth commented 2 years ago

https://github.com/lm-cyber/lab5_arhitip/blob/e462941d5f400acb59c21bff661a9d120af3f69d/lab-client/src/main/java/com/alan/lab/client/commands/PrintDescendingCommand.java#L20

Тут - получение отсортированной коллекции.

AlphaBeth commented 2 years ago

Поис и удаление по id - это тоже должно быть методом в CollectionManager https://github.com/lm-cyber/lab5_arhitip/blob/e462941d5f400acb59c21bff661a9d120af3f69d/lab-client/src/main/java/com/alan/lab/client/commands/RemoveByIdCommand.java#L27-L30