Giters
AlexeiA
/
middle.messenger.praktikum.yandex
Самостоятельный проект
Geek Repo:
Geek Repo
Github PK Tool:
Github PK Tool
Stargazers:
0
Watchers:
1
Issues:
30
Forks:
0
AlexeiA/middle.messenger.praktikum.yandex Issues
URL бекенда сообщений лучше перенести в `.env` к URL основного бекенда.
Closed
2 years ago
У вас в проекте достаточно много `ts-ignore`.
Closed
2 years ago
Вы выложили свой проект на Heroku?
Closed
2 years ago
А почему вы здесь делаете `npm i express`, а не `npm i`?
Closed
2 years ago
Вижу, что вы зафиксировали версии всех зависимостей.
Closed
2 years ago
А я что-то не очень понял, вы делаете импорт картинки, но никуда ее не вставляете.
Closed
2 years ago
Лучше задавать фиксированные версии зависимостям.
Closed
2 years ago
В чате, после обновления страницы, сообщения пропадают. Хорошо бы, чтобы они оставались.
Closed
2 years ago
Когда меняешь данные в профиле, они меняются. Но подставляется в поле ввода паролей какой-то другой пароль, а нужно оставлять пароль пользователя.
Closed
2 years ago
У вас не проходят автотесты.
Closed
2 years ago
`JSON.parse` может упасть с ошибкой, при парсинге данных. Лучше обернуть блок в `try/catch`
Closed
2 years ago
Зачем это? Метод сам должен вызваться из родительского класса.
Closed
2 years ago
В больших продакшен проектах правильнее всего тесты запускать в CI.
Updated
2 years ago
Чтобы постоянно не писать `JSON.stringify`. Можно работу с веб сокетом обернуть в отдельный класс по аналогии с api. И там создать свойство метод send, который оборачивал бы объекты в `JSON.stringify`
Closed
2 years ago
А смысл от пустого интерфейса? (layout.ts)
Closed
2 years ago
Почистить HTTPTransport.ts
Closed
2 years ago
Comments count
2
Предлагаю избавиться от проверки `method === 'GET'` в общей функции request и перенести эту логику в тело метода get, т.к. она не нужна для всех остальных методов
Closed
2 years ago
Чтобы везде не дублировать {credentials: true} набор базовых опций можно передавать при инициализации httpTransport, базовый урл тоже можно задать при инициализации
Closed
2 years ago
А смысл от этой функции, которая выполняет такую простую операцию?
Closed
2 years ago
Хорошая практика писать 1 класс в 1 файле. Route и Router лучше разбить на 2 файла)
Closed
2 years ago
Лучше всегда употреблять оператор строго равенства, вместо сравнения с приведением типов. Результат строгого равенства легче предугадать
Closed
2 years ago
Регулярное выражение нужно записать в константу и дать ей понятное название, о том, что она делает. Чтобы проще было читать код.
Closed
2 years ago
Здесь нужно воспользоваться стрелочными функциями, чтобы избавиться от self (Block.ts)
Closed
2 years ago
Приватные методы нужно писать с модификатором доступа `private`, иначе по умолчанию все методы `public`
Closed
2 years ago
Comments count
1
Для времени есть специальный тег `time` (chat-summary)
Closed
2 years ago
Очень длинная строка, лучше каждое новое поле в объекте на новой строке писать, так будет проще читать код
Closed
2 years ago
Мутировать значение по ссылке, не очень хорошая практика. Лучше создать новый объект с новыми данными. Статья на эту тему https://habr.com/ru/company/ruvds/blog/346998/
Updated
2 years ago
Если здесь h2, то почему тег p? (error.ts)
Closed
2 years ago
Comments count
1
Лучше не обращаться к нативному тегу, а задать для него отдельный класс.
Updated
2 years ago
props лучше типизировать через [generics](https://www.typescriptlang.org/docs/handbook/2/generics.html), так будет возможность точно передать в базовый блок, тип пропсов наследников.
Updated
2 years ago