Size: a a a

NestJS — русскоязычное сообщество

2020 October 12

М

Максим in NestJS — русскоязычное сообщество
Андрей
Продолжаю изучать nest и сегодня у меня возник такой вопрос)

предположим есть Users и Cities.
У каждого юзерах хранится cityId.

Когда прилетает запрос на обновление юзера, он может содержать поле cityId, в этом случае мы должны проверить существует ли город с таким id.

В сервисе city.service.ts есть метод checkIfExists, так вот, его лучше дергать в контроллере или же в сервисе юзера?
В сервисе
источник

JB

Joe Beretta in NestJS — русскоязычное сообщество
Андрей
Продолжаю изучать nest и сегодня у меня возник такой вопрос)

предположим есть Users и Cities.
У каждого юзерах хранится cityId.

Когда прилетает запрос на обновление юзера, он может содержать поле cityId, в этом случае мы должны проверить существует ли город с таким id.

В сервисе city.service.ts есть метод checkIfExists, так вот, его лучше дергать в контроллере или же в сервисе юзера?
Контроллер должен максимум передать данные в сервисы, которые выполняют бизнес-логику.

Я для себя сделал правило в виде "жирный сервис - худой контроллер/резолвел"
источник

М

Максим in NestJS — русскоязычное сообщество
Joe Beretta
Контроллер должен максимум передать данные в сервисы, которые выполняют бизнес-логику.

Я для себя сделал правило в виде "жирный сервис - худой контроллер/резолвел"
Плюсую этому господину
источник

М

Максим in NestJS — русскоязычное сообщество
Делай в сервисе di другого сервиса и пользуйся им
источник

GD

Goncharenko Dmitry in NestJS — русскоязычное сообщество
Привет. У кого-нить есть опыт интеграции AMO CRM?
источник

s

shaurgon in NestJS — русскоязычное сообщество
Привет всем.
Пытаюсь раскурить serverless для Nest, видел примеры под Azure у них в блоге, а никто не пытался сотворить подобное в Яндекс.Облаке? Мне эти ребятки как-то роднее что ли =)
источник

OM

Oleksandr Myshchyshy... in NestJS — русскоязычное сообщество
Ребят, а если у меня есть контроллер PUT  - update user. Обязательно мне передавать айди юзера, если я могу его вытащить с токена??
источник

F

Flex Airlines in NestJS — русскоязычное сообщество
если нацелен обновлять только залогиненного юзера, то в целом незачем
источник

F

Flex Airlines in NestJS — русскоязычное сообщество
но я бы советовал не сильно идти против системы и не нарушать каноны реста, потому что в будущем может аукнуться очень сильно
источник

🏡K

🏡 ILshat Khamitov in NestJS — русскоязычное сообщество
Oleksandr Myshchyshyn
Ребят, а если у меня есть контроллер PUT  - update user. Обязательно мне передавать айди юзера, если я могу его вытащить с токена??
если это обновление своего профиля, то только оттуда и брать нужно ид, иначе нужно еще проверки делать а вдруг ты не свой обновляешь а чужой профиль
источник

🏡K

🏡 ILshat Khamitov in NestJS — русскоязычное сообщество
все что с текущим юзером связанно, ид нужно всегда брать из токена
источник

🏡K

🏡 ILshat Khamitov in NestJS — русскоязычное сообщество
Flex Airlines
но я бы советовал не сильно идти против системы и не нарушать каноны реста, потому что в будущем может аукнуться очень сильно
нет такого, каждый лид свои каноны внедряет, спеки однозначной нету
источник

F

Flex Airlines in NestJS — русскоязычное сообщество
у меня есть горький опыт, когда я шел на поводу у запросов фронта про то, чтобы им было легче работать без лишних айдишек и всего-прочего, что потом было чревато долгими часами дебаггинга при необходимости хоть как-то расшириться или изменить кусочек логики
источник

🏡K

🏡 ILshat Khamitov in NestJS — русскоязычное сообщество
я написал причину почему только из токена брать нужно ид для текущего юзера
источник

🏡K

🏡 ILshat Khamitov in NestJS — русскоязычное сообщество
фронт тут не причем
источник

F

Flex Airlines in NestJS — русскоязычное сообщество
это пример)
источник

F

Flex Airlines in NestJS — русскоязычное сообщество
явно не про айдишку
источник

F

Flex Airlines in NestJS — русскоязычное сообщество
вдруг человеку надо админку писать?))
источник

GS

Grigorii K. Shartsev in NestJS — русскоязычное сообщество
Oleksandr Myshchyshyn
Ребят, а если у меня есть контроллер PUT  - update user. Обязательно мне передавать айди юзера, если я могу его вытащить с токена??
А если завтра появится админ, который может обновлять других юзеров?)
источник

F

Flex Airlines in NestJS — русскоязычное сообщество
именно!
источник