Как сделать рефакторинг кода?

Question

[Иван]

Хочу научиться упрощать и улучшать код, но не понимаю с чего начать. Я не прошу переписать этот код, а дать советы какую информацию почитать, на что обратить внимания. Для примера взял такой код, который надо улучшить

function​func(​s,​ ​a​, b​​) {
i​f​(s​.match(​​/^​$​/​)) { ​return​​-1​;​
}
v​ar​i ​=​s.l​ength​-​1​​; v​ar​aIndex =​​ -​1​​; v​ar​bIndex =​​ -​1​​;
w​hile​((aIndex =​=​-​​1​) ​&&​(bIndex =​=​​-​1​) ​&&​(i >​​​0​)) { i​f​(s​.substring(​i, i +​​1​) =​=​a) {
aIndex =​​i; }
i​f​(s​.substring(​i, i +​​1​) =​=​b) { bIndex =​​i;
}
i ​=​i -​​​1;​
}
i​f​(aIndex !​=​​-1​​) { i​f​(bIndex =​=​-​​1​) {
r​eturn​aIndex; }
e​lse​{
r​eturn​M​ath.m​ax​(aIndex, bIndex);
} }
i​f​(bIndex !​=​​-1​​) { r​eturn​bIndex;
} e​lse​{
r​eturn​-​​1;
}
}

Comments

[Алексей Ярков]

Для начала отформатируй ))

[Иван]

Алексей Ярков, с телефона пишу, на данный момент не могу. Видимо скопировал некорректно

[Егор Живагин]

Вопрос публике, а часто вы while юзаете? Я вот понял, что за 3 года ни разу не писал эту конструкцию

[Absolute138]

Егор Живагин, я использую))
while позволяет немного скорщать код в простых случаях.
Запиши все нечетные элементы кодом, короче, чем этот:

var arr=[1,2,3,4,5,6,7,8], i=arr.length;
while(i--){
  if(arr[i]%2) console.log(arr[i]);
}

:)

[Егор Живагин]

Absolute138, да пожалуйста :)

var arr=[1,2,3,4,5,6,7,8];
arr.forEach((elem) => {
  if (elem%2) console.log(elem);
});

[Absolute138]

Егор Живагин, Вместо forEach можно и map здесь поставить, раз на то пошло))
Речь именно о циклах, а не о методах, использующих циклы в своей реализации.

[Егор Живагин]

Absolute138, окей, я согласен
Но map здесь всё таки не стоит ставить)

[Absolute138]

Егор Живагин, да, согласен, map создан для других целей. Но работать будет и с map :)

Answer 1

[Егор Живагин]

Почитать про принципы KISS , Dry . И про строгое/не строгое сравнение в JS.
Конкретно для данного кода - этого хватит, я думаю.

А вообще, для рефакторинга было бы неплохо понимать сложность базовых алгоритмов - ссылка , ещё ссылка

Ну, и хорошо знать язык. Например, не засорять глобальную область видимости var-ами в 2019 году :) Честно говоря, кажется можно вспомнить про метод indexOf и сократить эту функцию в разы