Size: a a a

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

2020 December 11

PW

Pan Włodzimierz in Node.js — русскоговорящее сообщество
Pavel Shakhov (pongo)
можешь еще container.get('app')!;

(воскл знак)
это будет значить что?
источник

ДМ

Данил Мальков... in Node.js — русскоговорящее сообщество
Короче, спасибо. Переписал код в другой папке начал работать
источник

S

Susa in Node.js — русскоговорящее сообщество
Как правильнее
const { email } = req.body;
authService.createUser(email);
или сразу  authService.createUser(req.body);

При первом случае нужно есть бизнес логика на контролере.
Во втором случае мы сразу передаем req и на cервисе полность пишем логику
Кто знает?
источник

PS

Pavel Shakhov (pongo... in Node.js — русскоговорящее сообщество
Pan Włodzimierz
это будет значить что?
метод get у map возвращает T | undefined. через восклицательный знак мы скажем компилятору тс, что уверены что там нет undefined.
источник

GS

Grigorii K. Shartsev in Node.js — русскоговорящее сообщество
Можно ещё было просто const app: Application указать
источник

PW

Pan Włodzimierz in Node.js — русскоговорящее сообщество
Grigorii K. Shartsev
Можно ещё было просто const app: Application указать
так не работало(
источник

PW

Pan Włodzimierz in Node.js — русскоговорящее сообщество
источник

PS

Pavel Shakhov (pongo... in Node.js — русскоговорящее сообщество
Susa
Как правильнее
const { email } = req.body;
authService.createUser(email);
или сразу  authService.createUser(req.body);

При первом случае нужно есть бизнес логика на контролере.
Во втором случае мы сразу передаем req и на cервисе полность пишем логику
Кто знает?
я думаю правильно будет в контроллере получить email, убедиться что это строка и передать ее в createUser, который уже проверит действительно ли это email
источник

S

Susa in Node.js — русскоговорящее сообщество
Pavel Shakhov (pongo)
я думаю правильно будет в контроллере получить email, убедиться что это строка и передать ее в createUser, который уже проверит действительно ли это email
Еще есть проблема password в он не хешированный, я передаю не хешированый пароль между контролером и сервисом
источник

ДМ

Данил Мальков... in Node.js — русскоговорящее сообщество
Хмм, а можно ли подключить sql код к node JS, как отдельный файл к коду?
источник

PS

Pavel Shakhov (pongo... in Node.js — русскоговорящее сообщество
Данил Мальков
Хмм, а можно ли подключить sql код к node JS, как отдельный файл к коду?
через fs.readFile
источник

V

Van Der Graaf Genera... in Node.js — русскоговорящее сообщество
Susa
Как правильнее
const { email } = req.body;
authService.createUser(email);
или сразу  authService.createUser(req.body);

При первом случае нужно есть бизнес логика на контролере.
Во втором случае мы сразу передаем req и на cервисе полность пишем логику
Кто знает?
А я бы сделал так. Создал в папке API  файл а-ля createUser, из него эксопортировал эту функцию. И потом в контроллере бы ее вызывал, передавая туда мало
источник

ДМ

Данил Мальков... in Node.js — русскоговорящее сообщество
Пасибки)
источник

S

Susa in Node.js — русскоговорящее сообщество
Van Der Graaf Generator
А я бы сделал так. Создал в папке API  файл а-ля createUser, из него эксопортировал эту функцию. И потом в контроллере бы ее вызывал, передавая туда мало
можно где-то этот участок кода посмотреть в голову ничегоне приходит
источник

PW

Pan Włodzimierz in Node.js — русскоговорящее сообщество
Grigorii K. Shartsev
Если DI не позволяет указать дженериками, то хотя бы через as Application указаоть
в группе то тс сказали, что вариант через as App.. плохой))
источник

PS

Pavel Shakhov (pongo... in Node.js — русскоговорящее сообщество
Pan Włodzimierz
в группе то тс сказали, что вариант через as App.. плохой))
так тебе он и не нужен. тебе нужно проверить, что get вернул не undefined.
источник

S

Susa in Node.js — русскоговорящее сообщество
Pavel Shakhov (pongo)
так тебе он и не нужен. тебе нужно проверить, что get вернул не undefined.
Спасибо🙏 спас реально
источник

PW

Pan Włodzimierz in Node.js — русскоговорящее сообщество
Pavel Shakhov (pongo)
так тебе он и не нужен. тебе нужно проверить, что get вернул не undefined.
источник

PW

Pan Włodzimierz in Node.js — русскоговорящее сообщество
я честно говоря, думал ts увидит проверку и не подчеркнет
источник

PS

Pavel Shakhov (pongo... in Node.js — русскоговорящее сообщество
const app = container.get('app');
if (app == null) {
 throw new Error()
}
источник