Телеграмм чат группы nodeua страница 478

+++
Для опенсорса - топчик, такого очень не хватает. Для общего развития супер. Спасибо за share. Странно правда что сервисов нет, и контроллеры по-другому организовываю, это первое на что с телефона обратил внимание, но в целом это очень близко к реальному миру

UPD: понял, спасибо, изучу )

источник

23:08пожаловаться #8

SZ

Sasha Zmts in NodeUA - JavaScript and Node.js in Ukraine

;)

источник

23:09пожаловаться #9

SZ

Sasha Zmts in NodeUA - JavaScript and Node.js in Ukraine

Oleh

+++
Для опенсорса - топчик, такого очень не хватает. Для общего развития супер. Спасибо за share. Странно правда что сервисов нет, и контроллеры по-другому организовываю, это первое на что с телефона обратил внимание, но в целом это очень близко к реальному миру

UPD: понял, спасибо, изучу )

Там немного спорный подход юзается (ну мне так нравится в данный момент). Вместо сервисов там Экшены. Вот заметка про данный подход » https://gist.github.com/zmts/6ac57301e2e8e8e9e059e9c087732c05 Для прояснения философии

источник

23:13пожаловаться #10

SZ

Sasha Zmts in NodeUA - JavaScript and Node.js in Ukraine

Oleh

+++
Для опенсорса - топчик, такого очень не хватает. Для общего развития супер. Спасибо за share. Странно правда что сервисов нет, и контроллеры по-другому организовываю, это первое на что с телефона обратил внимание, но в целом это очень близко к реальному миру

UPD: понял, спасибо, изучу )

Будут вопросы, предложения пишите

источник

23:16пожаловаться #11

ОР

Олексій Румянцев... in NodeUA - JavaScript and Node.js in Ukraine

Sasha Zmts

Будут вопросы, предложения пишите

https://github.com/zmts/supra-api-nodejs/blob/42ab95944bb238f0f2bd001d6d1d4c506e2a7293/core/lib/RequestRule.js#L17


this.options = {
  required,
  allowed,
}

У тебя уже проверяется на то что реквайрд это булеан, а эллауд - это массив

источник

23:23пожаловаться #12

SZ

Sasha Zmts in NodeUA - JavaScript and Node.js in Ukraine

Олексій Румянцев

https://github.com/zmts/supra-api-nodejs/blob/42ab95944bb238f0f2bd001d6d1d4c506e2a7293/core/lib/RequestRule.js#L17


this.options = {
  required,
  allowed,
}

У тебя уже проверяется на то что реквайрд это булеан, а эллауд - это массив

Да. И что по этому поводу вы думаете? Уточните, не вижу подвоха

источник

23:29пожаловаться #13

ОР

Олексій Румянцев... in NodeUA - JavaScript and Node.js in Ukraine

Sasha Zmts

Да. И что по этому поводу вы думаете? Уточните, не вижу подвоха

Ну, я просто говорю, что можно было бы проще это записать)

источник

23:30пожаловаться #14

SZ

Sasha Zmts in NodeUA - JavaScript and Node.js in Ukraine

:) Точно :)

источник

23:31пожаловаться #15

2019 November 19

SZ

Sasha Zmts in NodeUA - JavaScript and Node.js in Ukraine

Alexander

https://github.com/zmts/supra-api-nodejs/blob/master/core/lib/assert/index.js#L58

Какой ужос.

https://github.com/zmts/supra-api-nodejs/blob/master/core/lib/Logger.js#L19

Каждому левелу по логгеру. 😱

Короч, @UserTrue, там ^^ можно посмотреть как писать не надо :).

Уговорил) Ужос убрал)

источник

21:18пожаловаться #16

A

Alexander in NodeUA - JavaScript and Node.js in Ukraine

Sasha Zmts

Уговорил) Ужос убрал)

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

if ((type === Number) && typeof value === 'number') return
if ((type === String) && typeof value === 'string') return

Дабы все эти проверки уместить в одну, можно сделать мапу для типа и значения тайпоф.

https://github.com/SkeLLLa/node-object-hash/blob/master/src/typeGuess.ts#L4
не совсем то, что вам нужно, но принцип такой же.

источник

21:29пожаловаться #17

SZ

Sasha Zmts in NodeUA - JavaScript and Node.js in Ukraine

Да куча ифов бесит)) Спасибо за подсказку :)

источник

21:31пожаловаться #18

A

Alexander in NodeUA - JavaScript and Node.js in Ukraine

Хотя... Возможно оно подойдет вам даже если просто скопипастить, только тогда надо будет тип передавать строкой.

источник

21:34пожаловаться #19

SZ

Sasha Zmts in NodeUA - JavaScript and Node.js in Ukraine

Вот я идумаю что тип у меня НЕ строкой. И строкой мне не охота это делать :)

источник

21:39пожаловаться #20