Реализация роутинга, подключение клиентского слойя API и интеграция с бэкендом

[maximste]

Реализовано

• клиентский слой API и интеграция с бэкендом;
• создание групп (в том числе добавление аватара группы);
• создание приватных сообщений;
• просмотр и редактирование профиля;
• добавление аватара профиля;
• авторизация
• создание нового профиля

Важные замечания.

1. Функционал отправки сообщений не реализован, поскольку программа курса была изменена. Будет реализовано в 4 спринте.
2. Выкладка сделана на github pages (https://maximste.github.io/GilgaChat), так как Netlify нестабильно работает без ВПН.

[aromanovnik]

Здравствуйте, Максим! 👋 (Нужно развернуть общий комментарий ↓)

Работа выполнена очень хорошо:

Добавлен корректно работающий роутинг, реализовано HTTP API для чатов, авторизации и пользователей. Код отлично типизирован и структурирован. Все выглядит чисто и понятно. Отличная работа!

Тем не менее, есть несколько моментов, которые стоит доработать. 🔧 Подробнее в комментариях к коду.

Что нужно исправить:

• При выборе детального чата отображаются моковые данные. По заданию необходимо загружать реальные сообщения, которые были отправлены ранее, а не использовать заглушки.
• По заданию должна быть реализована возможность добавления и удаления пользователя из чата. Сейчас методы для этого есть, но они не используются.
• По заданию должна быть реализована работа с WebSocket для получения и отправки сообщений чатов в реальном времени.

Что можно улучшить:

• Все методы (get, put, post, delete) одинаковы, отличается только тип метода запроса. Можно использовать фабричный метод, который будет возвращать соответствующий функционал в зависимости от типа запроса.
• Для удобства и снижения вероятности ошибок при работе с HTTP-статусами стоит вынести их в enum.

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

Желаю успехов в доработке проекта! 

[aromanovnik]

Можно лучше: Все методы (get, put, post, delete) одинаковы, отличается только тип метода запроса. Можно использовать фабричный метод, который будет возвращать соответствующий функционал в зависимости от типа запроса. Это улучшит структуру кода, повысит его повторное использование и упростит изменения в будущем, поскольку все изменения будут централизованы в одном месте.

type HTTPMethod = <R = unknown>(url: string, options?: Partial<RequestOptions<QueryParams>>) => Promise<R>;


export class HTTPTransport {
  // Фабричный метод для создания HTTP-методов
  private createMethod(method: METHODS): HTTPMethod {
    return (url, options = {}) => this.request(url, { ...options, method });
  }

  // Методы HTTP
  protected readonly get = this.createMethod(METHODS.GET);

  protected readonly put = this.createMethod(METHODS.PUT);

  protected readonly post = this.createMethod(METHODS.POST);

  protected readonly delete = this.createMethod(METHODS.DELETE);
  
  // Реализация метода request
  private request<R>(
    url: string,
    options: RequestOptions<QueryParams>,
  ): Promise<R> {
    // Логика выполнения HTTP-запроса
  }
}

[maximste]

Добавил фабрику. Спасибо за идею

[aromanovnik]

Можно лучше: Для удобства и снижения вероятности ошибок при работе с HTTP-статусами стоит вынести их в enum. Это избавляет от “магических чисел” в коде, упрощает поддержку и помогает избежать опечаток. Пример:

export enum HttpStatus {
  Ok = 200,
  Created = 201,
  NoContent = 204,
  MultipleChoices = 300,
  BadRequest = 400,
  Unauthorized = 401,
  Forbidden = 403,
  NotFound = 404,
  Conflict = 409,
  InternalServerError = 500,
}

[maximste]

Доработал. Спасибо за рекомендацию, так почище и без магических чисел

[aromanovnik]

Надо исправить: При выборе детального чата отображаются моковые данные. По заданию необходимо загружать реальные сообщения, которые были отправлены ранее, а не использовать заглушки.

[maximste]

Получение и отправка сообщений будет реализована в четвертом спринте (как нам объяснили - была пересмотрена программа). Поэтому в этом спринте использовал моки

[aromanovnik]

Надо исправить: По заданию должна быть реализована возможность добавления и удаления пользователя из чата. Сейчас методы для этого есть, но они не используются.

[maximste]

Добавил. Как-то упустил этот момент

[aromanovnik]

Надо исправить: По заданию должна быть реализована работа с WebSocket для получения и отправки сообщений чатов в реальном времени.

[maximste]

Получение и отправка сообщений будет реализована в четвертом спринте (как нам объяснили - была пересмотрена программа). Поэтому в этом спринте использовал моки

[aromanovnik]

🎉 Поздравляю! Работа принята!

Все недочёты успешно исправлены, и результат получился просто отличным!

Вы проделали огромную работу, и это действительно впечатляет!
Желаю успехов в дальнейшей учёбе и новых достижений! 🚀