useruntitled / ts-etkons

Тестовое задание ЭтКонс на джуна

На какой грейд хочу попасть

На Junior

Какие задачи из тестового задания сделал

• Из Junior: сделал все.
• Из Middle: Сделал первую. Ещё я добавил контейнер редиса. В принципе мог бы сделать пятую (про кеш), но пожалел времени

Инструкция по установке

• Запустить в терминале git clone https://github.com/useruntitled/ts-etkons && cd ts-etkons
• Запустить в терминале cp .env.example .env
• Настроить .env
• Если занят 80 порт, то настроить ещё default.conf в .docker/local/nginx/conf.d. К сожалению, nginx не предоставляет возможность прокинуть переменные в темплейт. Можете ещё в контейнере попробовать через envsubst
• Запустить в терминале make init. Если у вас не установлен make (такое может быть), то просто скопируйте из Makefile команды и вставьте в терминал.

Связь со мной

Tg: @AccInTouch

фидбек

В коде чувствуется стремление писать грамотно с использованием ООП подхода, но местами логика сильно усложнена и позволяет допускать больше ошибок в реализации. Например абстрактный класс фильтров не обрабатывает ошибку, связанную с не существующим методом в классе фильтра. Не очень понятно, зачем response в юзерах и постах принимает ресурс, если возвращаемый ресурс сам по себе является полноценным json ответом. Лишняя обертка. Не реализованы даже примитивные макеты письма. routes/api.php выглядит громоздким и не читаемым. В политиках остался отладочный код, что в реальных условиях может оказаться проблемой. Не информативный README

Мои мысли про фидбек

Например абстрактный класс фильтров не обрабатывает ошибку, связанную с не существующим методом в классе фильтра.
Согласен
Не очень понятно, зачем response в юзерах и постах принимает ресурс, если возвращаемый ресурс сам по себе является полноценным json ответом. Лишняя обертка.
Не лишняя, есть разница
routes/api.php выглядит громоздким и не читаемым.
Не согласен, что это вообще стоит упоминать. Роуты можно сплитить по файлам, это знают все. Строк кода там удовлетворительно мало, чтобы "не усложнять" разделяя на файлы
Не реализованы даже примитивные макеты письма.
lol
В политиках остался отладочный код, что в реальных условиях может оказаться проблемой.
Согласен, забыл строчку стереть.
Не информативный README
lol