Переключите с ОДНОМ случае или если еще - PullRequest
0 голосов
/ 06 ноября 2019

Я работаю над огромным проектом, и инспекции WebStorm всегда предлагают мне преобразовать конструкции переключателей с одним регистром или регистром + по умолчанию, как это

switch (fieldInfo.Type()) {
        case this.TYPE_NESTED:
            changedFieldInfo.addFieldFromPrototype(proto)
            break;

        default:
            changedFieldInfo._fieldInfo[this.FI_FIELDS] = [];
            break;
    }

, если еще такая конструкция

if (fieldInfo.Type() === this.TYPE_NESTED) {
        changedFieldInfo.addFieldFromPrototype(proto);
    } else {
        changedFieldInfo._fieldInfo[this.FI_FIELDS] = [];
    }

Мои коллеги говорят, что switch более читабелен, даже если в нем есть только одно предложение case, особенно при проверке перечислений, таких как TYPE, во фрагментах выше.

Что об этом думает сообщество js? Также предположим, что мы не собираемся добавлять второе предложение case и возвращаться к одному предложению очень часто, поэтому проблемы с обслуживанием незначительны.

И еще один вопрос , возможно ли преобразовать переключатель с разрывом внутри условия case, если без использования дополнительного кода, например,

    switch (fieldInfo.Type()) {
            case this.TYPE_NESTED:
                if(smthIsTrue)
                    break;
                changedFieldInfo.addFieldFromPrototype(proto)
                break;

            default:
                changedFieldInfo._fieldInfo[this.FI_FIELDS] = [];
                break;
        }
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...