NR
чиво блять?
Size: a a a
2021 July 09
Э
Ну дай свитч, я перепишу
NR
switch (eventType) {
case `FETCH`:
yield doFetchProcess(action);
break;
case `LOAD_FROM_STORE`:
yield put(setState({}));
break;
case `CREATE_NEW`:
yield doNewProcess();
break;
}DE
Охуелд ))
DE
Можно ретурны и без брейков
NR
нельзя
NR
там дальше код
NR
yield можешь опустить, будем считать что там обычный вызов
NR
это конечно можно сделать через valueOf или геттеры, но нахуя использовать обьект вместо свитч и так извращаться - не понятно
Э
const map = {
FETCH: () => doFetchProcess(action),
LOAD_FROM_STORE: () => put(setState({})),
CREATE_NEW: () => doNewProcess(),
}
(map[eventType] ?? (() => doDefault()))()Э
лямбды в отдельные функции, потому как кол-во строчек кода может расти
NR
хуйня, переделай на свитч (с) любой тимлид на код-ревью
Э
switch break break break break
Э
case case case case
Э
не надоело писать?
NR
и? в чем проблема?
Э
раздувает код, это хуйня
NR
это вообще не проблема
T
названия переменных тоже раздувает код
Э
и второе. Представь, что тебе нужно из свитча достать результат и положить в переменную