Skip to content

homework/26.12/terentev

Михаил Терентьев requested to merge homework/26.12/Terentev into main

Не мерджи сразу. Я реализовал хендлеры для файлового меню, обновил немного файловый сервис, но остается послевкусие, что не совсем архитектурно все хорошо. Я бы вынес проверку паролей и отдельный сервис, да и реализовать бы уже хэширование паролей, а не password.equals(newPassword) 😄

А, ну и почему нельзя мерджить - там перегрузка handle нормально не работает в FileMenu. Мне ведь по идее надо каждый раз юзера передавать в каждый хендлер, чтобы была типа валидация. И крч если добавить в интерфейс меню перегрузку с User еще, то надо же как-то и первый handle реализовать).

И сама идея передачи юзера в каждый хендлер мне не нравится, хочется услышать, как это сделать лучше.

Кароче нужен хороший код ревью на занятии

Edited by Михаил Терентьев

Merge request reports