Как переписать без else и else if?

Question

[dmitry2000]

if (ids.length > 0 && !item.name) {
                        state.push(false);
                    } else if (ids.length > 0) {
                        const filterName = ids.find((name) => name === item.name);

                        if (filterName) {
                            this.isFilters = false;
                            state.push(filterName != null);
                        } else {
                            state.push(false);
                        }
                    }

                    if (types.length > 0) {
                        if (!item.surname) {
                            state.push(false);
                        } else {
                            const filter = types.find((type) => type === item.surname);

                            if (filter) {
                                this.isFilters = false;
                                state.push(filter != null);
                            } else {
                                state.push(false);
                            }
                        }
                    }

Как бы я не переписывал, мои попытки заканчиваются неудачно. Что-то упускаю. Как можно переписать без else и else if?

Comments

[Антон Литвиненко]

зачем?

[WbICHA]

Как можно переписать без else и else if?

Зачем?

[dmitry2000]

Антон Литвиненко, не очень

[rPman]

Это вопрос академический или какую то красоту в коде желаешь навести?

Первое условие от else избавиться легко, там в обоих условиях есть общий ids.length>0 вот и вынеси его наружу в свой if а оставшееся условие !item.name внутрь него.

Второе условие от else избавиться не так красиво, заменив else на условие с not, каждый if перенести на один уровень вверх и каждый добавлять все условия из исходных т.е.
if (types.length > 0&&!item.surname)...
if (types.length > 0&&item.surname)...

[WbICHA]

dmitry2000, ты бы хотя бы комменты к той статье прочёл.

[WbICHA]

dmitry2000, а вообще можно так написать:

if (ids.length > 0) {
    const hasName = ids.includes(item.name);

    if (hasName) {
      this.isFilters = false;
    }

    state.push(hasName)
  }

Или выносить в отдельную функцию / метод.

Answer 1

[Rsa97]

Вынести код в отдельный метод и использовать ранний возврат.

Answer 2

[chelnokov_a]

Что-то такое...

if (ids.length > 0) {
        const filterName = ids.find((name) => name === item.name);

        if (!item.name) {
            state.push(false);

            return;
        }

        if (filterName) {
            this.isFilters = false;
            state.push(filterName != null);

            return;
        }

        state.push(false);
    }
    
    if (types.length > 0) {
        const filter = types.find((type) => type === item.surname);

        if (!item.surname) {
            state.push(false);

            return;
        }

        if (filter) {
            this.isFilters = false;
            state.push(filter != null);

            return;
        }

        state.push(false);
    }