Size: a a a

Node.js — русскоговорящее сообщество

2021 February 17

N

Nikita in Node.js — русскоговорящее сообщество
Uluk Abylbekov
спасибо. а по разделению логики нет больших замечаний? потому что заметил когда делаешь sign in например то мне надо сделать запрос в бд что бы проверить есть ли такой user,  а потом сделать еще один запрос что бы получить его password что бы сверить,  а потом еще 1 запрос что бы получить _id для jwt sign.
можно делать один запрос на получение юзера, если его нет то будет либо null либо error
источник

T

The Fallen Phoenix in Node.js — русскоговорящее сообщество
Nikita
можно делать один запрос на получение юзера, если его нет то будет либо null либо error
Если error, то не факт, что его нет.
источник

N

Nikita in Node.js — русскоговорящее сообщество
The Fallen Phoenix
Если error, то не факт, что его нет.
тем не менее продолжать операцию sign in нельзя
источник

UA

Uluk Abylbekov in Node.js — русскоговорящее сообщество
Pavel Shakhov (pongo)
а в чем тут проблема?
если бы все было в контроллерах то можно было бы после первого запроса сохранить документ в user и использовать его каждый раз не перезапрашивая все время
источник

K

Konstantin in Node.js — русскоговорящее сообщество
Nikita
а как вы отличаете ошибки в БЛ от ошибок самого "кода", а-ля фейл записать данные в бд?
Если проблемки с бд это чаще всего 500 по смыслу))
источник

PS

Pavel Shakhov (pongo... in Node.js — русскоговорящее сообщество
Uluk Abylbekov
если бы все было в контроллерах то можно было бы после первого запроса сохранить документ в user и использовать его каждый раз не перезапрашивая все время
так у тебя всё будет в одном методе сервиса
источник

T

The Fallen Phoenix in Node.js — русскоговорящее сообщество
Nikita
тем не менее продолжать операцию sign in нельзя
Зато если такая проверка (семантика отсутствие как ошибка)  будет и в sign up, то привет Security Issue
источник

N

Nikita in Node.js — русскоговорящее сообщество
The Fallen Phoenix
Зато если такая проверка (семантика отсутствие как ошибка)  будет и в sign up, то привет Security Issue
это другой вопрос))
источник

UA

Uluk Abylbekov in Node.js — русскоговорящее сообщество
Pavel Shakhov (pongo)
так у тебя всё будет в одном методе сервиса
а как тогда response кидать? у меня в sign in 5 вариантов не могу же их делать прямо в service
источник

N

Nikita in Node.js — русскоговорящее сообщество
Konstantin
Если проблемки с бд это чаще всего 500 по смыслу))
имею ввиду вот вывалися еррор, он пошел дальше к хендлеру ерроров, как определить что это просто юзер запросил запись в бд по неверному айдишнику и вернуть ему красивенький форматрованный еррор, или что это ошибка записи транзакции в бд и нужно кричать во все логи, юзеру вернуть Something went wrong (code 120) и отправить еще какое то уведомление в телегу разрабу? через instanceof тот же?
источник

UA

Uluk Abylbekov in Node.js — русскоговорящее сообщество
есть ли хорошие книги по архитектуре приложений или по той теме в которой у меня недостаточно знаний на node и express
источник

PS

Pavel Shakhov (pongo... in Node.js — русскоговорящее сообщество
Uluk Abylbekov
а как тогда response кидать? у меня в sign in 5 вариантов не могу же их делать прямо в service
сделать несколько кастомных ошибок под каждый случай, а в контроллере проверяй instanceof
источник

T

The Fallen Phoenix in Node.js — русскоговорящее сообщество
Nikita
это другой вопрос))
Ну видимо я что-то не понимаю о полезности логина в отсутствии возможности регистрации.
источник

UA

Uluk Abylbekov in Node.js — русскоговорящее сообщество
Pavel Shakhov (pongo)
сделать несколько кастомных ошибок под каждый случай, а в контроллере проверяй instanceof
ок
источник

PS

Pavel Shakhov (pongo... in Node.js — русскоговорящее сообщество
Uluk Abylbekov
есть ли хорошие книги по архитектуре приложений или по той теме в которой у меня недостаточно знаний на node и express
источник

N

Nikita in Node.js — русскоговорящее сообщество
The Fallen Phoenix
Ну видимо я что-то не понимаю о полезности логина в отсутствии возможности регистрации.
не верно прочитал предыдщуее сообщение

ну а так да верное замечание, лучше просто null если это репозиторий
источник

UA

Uluk Abylbekov in Node.js — русскоговорящее сообщество
спасибо
источник

UA

Uluk Abylbekov in Node.js — русскоговорящее сообщество
спасибо всем на сегодня.
источник

T

The Fallen Phoenix in Node.js — русскоговорящее сообщество
Nikita
не верно прочитал предыдщуее сообщение

ну а так да верное замечание, лучше просто null если это репозиторий
Отсутствие результатов запроса и отсутствие сущностей в результате запроса (пустой результат) это разные ситуации.
источник

N

Nikita in Node.js — русскоговорящее сообщество
The Fallen Phoenix
Отсутствие результатов запроса и отсутствие сущностей в результате запроса (пустой результат) это разные ситуации.
ничего не понял но очень инетресно
источник