djpont / middle.messenger.praktikum.yandex Goto Github PK

Подробный README, сразу понятно что и как.

• ссылка кнопки Нетлифая открывает не детали проекта или его деплой а картинку кнопки.
• "компоненты, кроме View, настедуются"

Понравился дизайн, особенно использование стилей 98.css

• аж ностальгия пробрала :)
• окошки не масштабируются. Для чата было бы логично чтоб оно занимало всю или большую часть рабочей области window
• как то не привычно что в разрабатываемом проекте в консоли пусто
• нет никакой визуализации валидации полей по focuslost, хотя в рамках дизайна это и не нужно, тем более что сообщение об ошибке содержит детальные комментарии
• был странный баг валидации, когда появилось несколько сообщений об одной и той же ошибке. Повторить не смог. Вероятно связано с await.
• не создавал клон чтобы "покрутить" проект у себя поэтому могу только субъективно отметить что роутер вероятно работает и не верно и грузит ресурсы при каждой смене роута

Лично мне очень понравился код стайл.

• особенно что касается плотности кода и напихивания всего в один файлик. Лично мне так читать даже удобнее. Но мы же учимся оформлять "командные" проекты. Поэтому можно воспользоваться возможностью и потренироваться выносить модули в отдельные папки со своими index файлами, в которых систематизируется структура модуля и его компонент. По типу как у тебя это сделано в components, но там index содержит непосредственный код компонента.
• Понравился класс валидации. Рад что ты тоже посчитал логичным реализовать универсальный подход. И тут, к примеру, те же regexpsRules можно было вынести в отдельный модуль для удобства редактирования в отрыве от рабочей логики.
• Api в виде одного файлика это клёво,- всё в одном месте. Правда нет BaseAPI, UserAPI и т.д., которые наследуются друг от друга и передают end point пути, при неизменном основном пути. Тогда не пришлось бы создавать внутренние методы для каждого запроса. Нам это кратко давали в теории. Я, к примеру, немного развил концепцию и ВЕСЬ api у меня занял 30 строк кода против твоих 500.
• Store работает хорошо, логично организована работа с ним. Видно что ты хорошо разобрался в теме хранения данных.
• Может я был не внимателен но не обнаружил HOC, призванный объединять события изменений в store с ререндерами компонент. А это была основная тема 3-го спринта.
• Не нашёл isEqual, cloneDeep и пр. вспомогательные функции, которые мы писали, по сути для HOC чтобы придать ему логики чтоб избежать лишних перерисовок, к примеру.
• Немного смутил подход замесить api запросы, их обработку и операции со store в одном модуле. Было бы логичнее это деструктурировать чтобы повысить надёжность кода. Сейчас стоит где то сменить подход и придётся переписывать сразу всё, вместо модуля который отвечает за сегмент логики.

В целом:

• Креативный дизайн
• Разнообразный и логичный подход к написанию кода
• Понятная структура
• Обещанный "Франкенштейн" не обнаружен.

Добавил минимальную версию node v14.21.2 в .nvmrc