Как можно зарефакторить код?

Question

[Rebuslol]

function func(s, a, b) {
        if (s.match(/^$/)) {
            return -1;
        }
        let i = s.length - 1;
        let aIndex = -1;
        let bIndex = -1;
        while ((aIndex == -1) && (bIndex == -1) && (i > 0)) {
            if (s.substring(i, i + 1) == a) {
                aIndex = i;
            }
            if (s.substring(i, i + 1) == b) {
                bIndex = i;
            }
            i = i - 1;
        }
        if (aIndex != -1) {
            if (bIndex == -1) {
                return aIndex;
            }
            else {
                return Math.max(aIndex, bIndex);
            }
        }
        if (bIndex != -1) {
            return bIndex;
        }
        else {
            return -1;
        }
    }

Comments

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

собеседование проходите на Тостере?
Что сами сделали, в чём застряли?

[Rebuslol]

Сергей Соколов, для интервью,да) да если честно мысли только о тернарном выражении есть и все. Учусь только

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

Rebuslol, например проверять строку на пустоту регуляркой некруто, можно заменить s.match(/^$/) на проверку длины. Длина и дальше ещё раз используется.
Ну и вообще понять, что там делается и, возможно, эффективнее переписать.

[Rebuslol]

Сергей Соколов, а на счет тернарного выражение? или тут это не нужно?

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

Rebuslol, вроде тут знание тернарных не главное. Посмотрите, как сделан возврат результата. Все условия может, можно сократить до 1 строки?

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

От повторяющегося кода избавиться неплохо бы.

[Rebuslol]

Сергей Соколов, не очень понял,о чем именно речь,но спасибо) буду разбираться

Answer 1

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

Вроде бы, сокращается до 2 строк. Но вы проверьте, напишите побольше тестов. Проверьте все, включая невероятные, значения входных параметров и их отсутствие )

spoiler

function func(s, a, b) {
  const subject = s.substr(1);
  return Math.max(subject.lastIndexOf(a[0]), subject.lastIndexOf(b[0]));

}

Comments

[Rebuslol]

спасибо,проверю)