vladprizma / deeplay-camp-2024

Проект команды реверси в рамках deeplay camp 2024.
0 stars 3 forks source link

Не подходящее место для логики #13

Closed vladprizma closed 2 months ago

vladprizma commented 2 months ago

DTO должен только переносить информацию. Выглядит как то, что можно вынести в условный Utils.

https://github.com/vladprizma/deeplay-camp-2024/blob/208b843688af1a759158d4ef23dc06b42a65cbbe/model/src/main/java/dto/BoardDTO.java#L16C9-L25C10

DarlingInSteam commented 2 months ago

DTO может в себе иметь преобразовывающий метод. Обычно это toDto либо перегрузка toString, в этом случае, как я вижу, преобразование в нотацию доски. Но да, можно и вынести в utilities. Но вроде бы не будет ошибкой сохранить этот метод в DTO, если в нем исключительно логика преобразования.