Л
res.status(200).json можно вроде кстати сократить к res.json() просто, там 200 сам поставится
+
Size: a a a
2021 February 17
UA
сервер експресс?
да сервер express
N
здесь они не вызваются
UA
ну и с обработкой ошибок что то надо сделать, особенно в репозитории
что именно
N
что именно
у тебя ошибка просто в консоль будет выведена, а верхний слой ожидает получить созданного/найденного юзера из бд, а получит undefined если сработает catch
UA
+ исправлю
N
и соовественно где ошибка произшла, а главное в какое время в логах потом не найдешь
N
и кстати есть библиотеки готовы по типу jsonschema/joi которые могут валидировать запросы, писать самому такие небольшие regexpы можно но когда будет много входящих параметров замучаешся
UA
и соовественно где ошибка произшла, а главное в какое время в логах потом не найдешь
есть ссылки на статьи как обрабатывать ошибки или что надо с ними делать
N
есть ссылки на статьи как обрабатывать ошибки или что надо с ними делать
https://github.com/goldbergyoni/nodebestpractices#-21-use-async-await-or-promises-for-async-error-handling
я сам пока еще точно не понял как лучше всего, но пока следую этим принципам, используй на свой страх и риск
я сам пока еще точно не понял как лучше всего, но пока следую этим принципам, используй на свой страх и риск
K
https://github.com/goldbergyoni/nodebestpractices#-21-use-async-await-or-promises-for-async-error-handling
я сам пока еще точно не понял как лучше всего, но пока следую этим принципам, используй на свой страх и риск
я сам пока еще точно не понял как лучше всего, но пока следую этим принципам, используй на свой страх и риск
2.2 пункт вообще не согласен
N
2.2 пункт вообще не согласен
++++
N
по крайней мере хотя бы 1 класс можно и нужно создать
K
Подходит только для либ и экстернализации. Если ты свою бл не будешь куда-то эмбеддить, смысла нет
K
instanceof или switch(constructor) мощные инструменты и тупо не использовать
PS
у тебя сейчас контроллер занимается оркестрацией бизнес-логики (чем должен заниматься сервис (в рамках твоей архитектуры)).
т.е. контроллер должен вызывать userService.create(email, password), смотреть не возвращает ли тот ошибку, и на основании этого уже возвращать пользователю результат.
слово "должен" читай как "может". твой вариант сейчас вполне норм, переносить оркестрацию в сервис не обязательно.
т.е. контроллер должен вызывать userService.create(email, password), смотреть не возвращает ли тот ошибку, и на основании этого уже возвращать пользователю результат.
слово "должен" читай как "может". твой вариант сейчас вполне норм, переносить оркестрацию в сервис не обязательно.
K
у тебя сейчас контроллер занимается оркестрацией бизнес-логики (чем должен заниматься сервис (в рамках твоей архитектуры)).
т.е. контроллер должен вызывать userService.create(email, password), смотреть не возвращает ли тот ошибку, и на основании этого уже возвращать пользователю результат.
слово "должен" читай как "может". твой вариант сейчас вполне норм, переносить оркестрацию в сервис не обязательно.
т.е. контроллер должен вызывать userService.create(email, password), смотреть не возвращает ли тот ошибку, и на основании этого уже возвращать пользователю результат.
слово "должен" читай как "может". твой вариант сейчас вполне норм, переносить оркестрацию в сервис не обязательно.
+ прибавил бы что ты вообще не хендлишь ошибки и всё ловишь в кетч мидлваре экспресса
UA
https://github.com/goldbergyoni/nodebestpractices#-21-use-async-await-or-promises-for-async-error-handling
я сам пока еще точно не понял как лучше всего, но пока следую этим принципам, используй на свой страх и риск
я сам пока еще точно не понял как лучше всего, но пока следую этим принципам, используй на свой страх и риск
спасибо. а по разделению логики нет больших замечаний? потому что заметил когда делаешь sign in например то мне надо сделать запрос в бд что бы проверить есть ли такой user, а потом сделать еще один запрос что бы получить его password что бы сверить, а потом еще 1 запрос что бы получить _id для jwt sign.
PS
2.2 пункт вообще не согласен
у него там смешное объяснение.
а кто-то вообще выбрасывает СТРОКИ?
а кто-то вообще выбрасывает СТРОКИ?