Ребят,Как посоветуете отрефакторить ? )

Question

Ребят,Как посоветуете отрефакторить ? )

if (isNumber(item)) {
numberBuffer.push(item)
}
else if (isOperator(item)) {
emptyNumberBuffer();
result.push(new Token(OPERATOR, item))
}
else if (isLeftParanthesis(item)) {
emptyNumberBuffer();
result.push(new Token(LEFT_PARENTHESIS, item))
}
else if (isRightParanthesis(item)) {
emptyNumberBuffer();
result.push(new Token(RIGHT_PARENTHESIS, item))
}
else {
throw new Error('введены невалидные данные')
}
}) по сути меняется только в new Token первый агрумент и условие

#javascript #programming #russian

0

20.02.2020

2 ответов

16 просмотров

rsmche

Или можно написать функцию, которая возвращает константы OPERATOR, LEFT_PARENTHESIS или RIGHT_PARENTHESIS в зависимости от item. Так ты избавишься от повторных result.push(new Token(, от проверок в кажом такой и похожей функции на isLeftParanthesis и от всех этих is else

0

20.02.2020

rsmche · Accepted Answer

rsmche

Добавить switch как минимум. Много if else это знак, что лучше использовать switch

0

20.02.2020

169 похожих чатов

Ребят,Как посоветуете отрефакторить ? )

2 ответов

Похожие вопросы