Pull to refresh
0
0
Send message

А по какой причине было решено наплодить несколько combineValueN вместо одного, который принимает список value и этот же список посылает в обработчик?

А так в целом хорошая статья, спасибо, думаю смело можно идти в пабдев

Увидел много замечаний от товарищей по самому коду, так что об этом писать не буду. Хотелось бы обратить внимание автора статьи на одну огромную ошибку. Вы совмещаете бизнес-логику и UI. Придираться к тому, что само приложение в консоли, я не собирался, но логика приложения прочно связана с UI (выводом в консоль). Это находится даже не в одном классе, а в одном методе.

Если бы Вы вынесли логику приложения в отдельный компонент, то у Вас была бы возможность в будущем использовать написанный код и в приложении с полноценным UI.

Раз уж Вы находитесь на начальном этапе изучения программирования, я не прошу сразу вас вникать в архитектурные паттерны и учить MVC/MVVM и т. д. Но если получилось бы разделить консоль и логику, даже с таким уровнем написания кода это выглядело бы как действительно полноценное приложение.

Information

Rating
Does not participate
Registered
Activity