заготовки для урока по ФП

[saitonakamura]

No description provided.

[nickovchinnikov]

Точно нужны такие большие закомментированные куски?

[saitonakamura]

Да, там кроется логика для второй валидации которую они делают самостоятельно

[nickovchinnikov]

поставь тогда пометку плиз, чтобы не забыть зачем, будет удобно

[saitonakamura]

сделал

[nickovchinnikov]

Потенциальный баг Boolean(0) === false
Все таки стоит сделать более явно, можно с помощью ramda

   import { is, isNil, isEmpty } from 'ramda'
   //...
    const value = this.values[key];
    if (is(Boolean, value)) {
       this.errors[key] = undefined
       return
    }
    if (isEmpty(value) || isNil(value)) {
       this.errors[key] = new Error(`${key} is required`);
    }

[saitonakamura]

Ну как вот эта проверка это и делает typeof value !== "number" && typeof value !== "boolean"

[nickovchinnikov]

Boolean(0) === false явно не обрабатывается
ну и мне пришлось потратить много времени, чтобы понять все кейсы обрабатываемые этой строчкой и додебажить в голове до бага. Являюсь сторонником явных подходов, они и читаются лучше и ошибиться сложнее

[nickovchinnikov]

В том числе я за явные преобразования типов, !!string и прочее очень смущают

[saitonakamura]

переписал на явный switch

[nickovchinnikov]

Прошу обратить внимание на комменты