Open dmittrey opened 2 years ago
https://github.com/Matveezy/ProgrammingLab5/blob/583b4b12b89ec0d62ec3ed82d125fa80bbc24a91/src/main/java/file/DragonFieldsReader.java#L16-L18
Я правильно понимаю, что метод, считывающий поля, занимается вместо этого созданием объекта, а не перепоручает это фабрике?
В итоге мы имеем бэкдор и объекты импортируемые с файла обкастыливаются в угоду уникальных id.
Перестать смешивать ответственность и убрать создания в фабричный метод.
https://github.com/Matveezy/ProgrammingLab5/blob/583b4b12b89ec0d62ec3ed82d125fa80bbc24a91/src/main/java/file/DragonFieldsReader.java#L16-L18
Я правильно понимаю, что метод, считывающий поля, занимается вместо этого созданием объекта, а не перепоручает это фабрике?
В итоге мы имеем бэкдор и объекты импортируемые с файла обкастыливаются в угоду уникальных id.
Перестать смешивать ответственность и убрать создания в фабричный метод.