Написал калькулятор без eval(). Насколько живуч этот подход?

Question

[Федор]

Задался вопросом написать калькулятор обыкновенный. Реультат потыкать можно тут: https://fedorpereverzev.github.io/Calculator-witho... Самым простым решением было использовать eval(), но потом почитал, что это не лучшая идея. В итоге написал функцию, которая принимает на вход строку, разбивает ее на массив и по знаку производит вычисления

const antiEval = (str) => {

        let arr = str.split(' ');

        switch (arr[1]) {
            case '+':
                return Math.round((+arr[0] + +arr[2]) * 100) / 100;
                break;
            case '-':
                return Math.round((+arr[0] - +arr[2]) * 100) / 100;
                break;
            case '/':

                return Math.round((+arr[0] / +arr[2]) * 100) / 100;
                break;
            case '*':
                return Math.round((+arr[0] * +arr[2]) * 100) / 100;
                break;
        };
    };

Все ли в порядке с таким решением или это кривой велосипед?
Если что весь проект здесь: https://github.com/FedorPereverzev/Calculator-with...

Comments

[Bhudh]

Вам самому-то глаза не режет обилие строк, отличающихся одним знаком?
Можно определить объект с функциями действий и одну функцию их вызова:

const actions = {
    '+' : ( a, b ) => a + b,
    '-' : ( a, b ) => a - b,
    '*' : ( a, b ) => a * b,
    '/' : ( a, b ) => a / b,
    '^' : ( a, b ) => a ** b,
};

function act ( fn, a, b, precision ) {
    precision = precision || 100;
    return Math.round( fn(+a, +b) * precision) / precision;
}

const antiEval = (str) => {

    let arr = str.split(' ');

    return act( actions[ arr[1] ], arr[0], arr[2] );

};

Answer 1

[Kalombyr]

По-моему лучше строить дерево...
Строку разбирать лучше в "пошаговом" режиме, т.е. смотреть на каждый символ и строить дерево.
Так как операции имеют свой приоритет, да ещё и скобки если будут, то простым массивом не отделаться.

Comments

[Федор]

Спасибо! До деревьев еще не дошел - буду изучать :)

Answer 2

[xmoonlight]

var value = '(5+7)*10'.match(/(^[0-9*\/\\(\\)+-]+$)/);
var res = new Function('return '+ value );
console.log(res()); // => 120

или
Используйте обратную польскую запись.

Comments

[Adamos]

Отличие от eval - только в контексте. Автор как раз спрашивает... не знаю, что именно, но не eval.

[xmoonlight]

Adamos, так тут и нет eval'а никакого...

[Adamos]

xmoonlight, new Function(), получая строковый аргумент, выполняет те же действия, что eval, только не в текущем контексте, а в глобальном. Автор спрашивает = имеет ли смысл разбирать самому? Хотя почему он это спрашивает, если знает, что eval - зло, я не понимаю.

[sim3x]

xmoonlight, https://stackoverflow.com/questions/4599857/are-ev...

Что никак не говорит, что такой код преемлем на продакшене

[xmoonlight]

sim3x,

Crockford's point is that eval is often misused (depends on your perspective), so it should be excluded from a "best practices" JavaScript. I do agree, however, that it is useful for such uses as JSON or arithmetic expression parsing when input is first properly validated. Even Crockford uses it in his JSON library json2.js.

[Федор]

Спасибо за мысли! Единственное забыл добавить, что в этом калькуляторе ввод возможен только по нажатию кнопок (аля как калькулятор на mac os или ios)

[Adamos]

xmoonlight, вот если бы у вас второй строчкой стояла регулярка, оставляющая от строки исключительно формулу - возражений бы не было...

[xmoonlight]

Adamos, а чем поможет регулярка (если захотят подменить - подменят)?

[Adamos]

xmoonlight, признаться, в данном случае я и опасность eval обосновать не берусь ;)

[xmoonlight]

Adamos, ну о том и речь... :)

[xmoonlight]

Adamos, поправил ответ, чтобы возражений не было... ;)

[xmoonlight]

sim3x, вот в коде нашёл

Answer 3

[Александр Симонов]

Есть такая штука как Builder.
Что-то типа:

class Calc{
 public float Result;

 public Calc Add(float a){
  Result += a;
  return this;
 }

 public float GetResult() {
  return Result;
 }
}

Использовать можно как-то так

var a = new Calc().Add(2).Add(5).Minus(4).GetResult();

Только вот это всё нужно делать, парься, что вам там пользователи навводили

Адаптировать под JS, я думаю, сможете

Comments

[Федор]

Интересный подход! Спасибо, буду изучать