Чем заменить однотипные else if?

Question

[Иван Спасов]

Есть код:

const path = url.pathname; // "/shop/goods/{guid}/{guid}/write"
if (path.startsWith("/shop/goods")) { 
  ...
} else if (path.startsWith("/shop/union")) {
  ...
} else if (path.startsWith("/docs/tech")) {
  ...
} else if (path.startsWith("/docs/goods")) {
... еще пару таких условий

Как это можно переиграть и заменить, не нравится громоздкие else if else if

Изначально делал так:

const parts = url.pathname.split("/");
const routePrefix = `${parts[0]}/${parts[1]}`;

switch(routePrefix) {
  case "/shop/goods": 
    ...
  break;
  тут остальные
}

Но показалось излишне сплитить строку и собирать префикс, но так конечно выглядит и читается лучше.

Comments

[WbICHA]

Свитч?

[Иван Спасов]

WbICHA,

Изначально делал так:

const parts = url.pathname.split("/");
const routePrefix = `${parts[0]}/${parts[1]}`;

switch(routePrefix) {
  case "/shop/goods": 
    ...
  break;
  тут остальные
}

так ?
или хаки в стиле:

switch (true) {
  case path.startsWith("/shop/goods"): 
    ...
  break;
  ...
}

[Иван Спасов]

WbICHA, в общем, решил пока регуляркой просто брать слайс первых 2 параметров, и пихать строку в switch
Если подскажут более изящное решение, буду очень рад :)

[formasters777]

берите, пробуйте, открывайте профайлер,
смотрите на скорость выполнения кода,
свитч и if else это одно и тоже,
или меняйте алгоритм программы, делайте меньше операций и т.п., уменьшайте срок выполнения.

[Сергей Соколов]

а что там в действиях в каждом из вариантов — всё равно же придётся описывать?
Если там функции, то можно было бы в объект-словарь засунуть:

const routeActions = {
  '/shop/goods': () => { do_something(); },
  '/shop/union': () => { do_something_else(); },
};

Дальше смотреть, есть ли ключ – выполнять, или по дефолту 404.

[ProjectSoft]

Иван Спасов,

const path = url.pathname; //   "/shop/goods/{guid}/{guid}/write"
let str = path.split("/").splice(0, 3).join("/"); //   "/shop/goods"
switch(str) {
    case "/shop/goods":
        console.log("your code")
        break;
    /**
     * Code cases
     **/
    default:
        break;
}

[shurshur]

Это ж типичный роутер. Например, можно сделать словарь регулярка-обработчик и по регулярке в цикле выбирать функцию обработки конкретного запроса.

[WbICHA]

Иван Спасов, если брать первые 2 слова из строки, то что ты будешь делать, если позже тебе понадобиться брать в конкретном случае первые 3/5/10/100 слов? Или всего одно? Тебе всё переписывать придётся.
Поэтому решение через свитч(тру), имхо, самое надёжное удобное.

[Василий Банников]

Префиксное дерево?

[D3E]

Есть тернарные операторы тоже очень удобные.

Answer 1

[Alex Shatunov]

Как вариант создать массив строк с из путей. Потом его перебрать а внутри массива всего одно условие if по которому изменить какой-то флажек.