Skip to content

Conversation

@davidbilalov
Copy link
Collaborator

пока что не решена проблема с кодировкой. (в работе)

Есть проблемы с кирилицей
Copy link
Contributor

@hodkonem hodkonem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Давид, спасибо за работу! 👍
Код интерфейса DataProvider и логика в целом правильные.
Есть пару мелких моментов, чтобы всё соответствовало архитектуре:

метод load() должен иметь сигнатуру throws IOException (контракт интерфейса);

в FileDataProvider стоит обрабатывать null от парсера до валидации;

в RandomDataProvider лучше использовать единый ThreadLocalRandom и задать корректные диапазоны годов/мощностей;

при следующем коммите добавляй только свои классы (без ConsoleDataProvider, App, build.gradle и т.п.).

Общая структура хорошая — после этих доработок можно будет мёржить. 💪

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants