В требованиях есть упоминание паттернов. Возможно ожидали использование мидлварей для авторизации, валидации запроса и тд. Сейчас ендпоинт обработки суммы не проходит по единственной ответственности.
Может быть, хотя я как раз подумал после митинга с тех менеджером, что они больше про алгоритмы и скорость кодирования, поэтому интересно, есть ли другие варианты решения самой задачи. Про ответственность не понял. Имеется ввиду лишняя проверка авторизации?
Я про то что метод ендпоинта и проверяет авторизацию и валидирует запрос и собственно считает сумму. А так залезть в голову авторам тестового достаточно сложно - по итогу причина отказа может быть совсем в другом =)
Поставь себя на место занятого ревьювера, к которому валится куча решений от кандидатов. Программа не структурирована, все в куче. Методы не имеют описаний. Вывод: да ну его нафиг, лучше посмотрю другого.
У меня после митинга с тех. менеджером сложилось ощущение, что они больше про алгоритмы и скорость кодирования, чем про придирки к оформлению, поэтому на такой просто задаче не стал на этом фокусироваться и выкатил решение за час, правда в одном из коммитов упомянул chatGPT, с помошью которого сгенерил часть хэндлеров, чтоб не тратить время - тоже могло повлиять )
В если json перебрать как массив символов?
нет.. после рабочей недели, рабочие выходные на второй работе)
в json числа недолжны быть в кавычках, видимо поиск чисел в строке лишнее
В требованиях есть упоминание паттернов.
Возможно ожидали использование мидлварей для авторизации, валидации запроса и тд.
Сейчас ендпоинт обработки суммы не проходит по единственной ответственности.
Может быть, хотя я как раз подумал после митинга с тех менеджером, что они больше про алгоритмы и скорость кодирования, поэтому интересно, есть ли другие варианты решения самой задачи. Про ответственность не понял. Имеется ввиду лишняя проверка авторизации?
Я про то что метод ендпоинта и проверяет авторизацию и валидирует запрос и собственно считает сумму.
А так залезть в голову авторам тестового достаточно сложно - по итогу причина отказа может быть совсем в другом =)
@@mikhailbobkov6230 понял, спасибо. В общем да, по паттернам проблемы есть )
Поставь себя на место занятого ревьювера, к которому валится куча решений от кандидатов. Программа не структурирована, все в куче. Методы не имеют описаний. Вывод: да ну его нафиг, лучше посмотрю другого.
У меня после митинга с тех. менеджером сложилось ощущение, что они больше про алгоритмы и скорость кодирования, чем про придирки к оформлению, поэтому на такой просто задаче не стал на этом фокусироваться и выкатил решение за час, правда в одном из коммитов упомянул chatGPT, с помошью которого сгенерил часть хэндлеров, чтоб не тратить время - тоже могло повлиять )