Это плохой код?

Question

[stepan132]

Здравствуйте. Помогите разрешить наш с другом спор. Он прислал мне свой калькулятор, я сказал, что это плохой код, он-же утверждает обратное. Вот сам код:

let input = document.querySelector('.calculator-input'),
    buttons = document.querySelectorAll('.calculator-button'),
    clearInputButton = document.querySelector('.clearInput'),
    calculateButton = document.querySelector('.calculate');

let insertToInput = (value) => {
    if (input.value.length > 12 || value.toString().length > 12) {
        input.value = 'Number too large';
        return;
    }
    input.value += value;
}
let clearInput = () => {
    input.value = '';
}
let calculate = (str) => {
    clearInput();
    let res = eval(str);
    if (/\/0$/g.test(str) || isNaN(res)) {
        return insertToInput('Error');
    } else if (/0\.\d\+0\.\d/g.test(str)) {
        return insertToInput(res.toFixed(1));
    } else if (/0\.\d\*0\.\d/g.test(str)) {
        let numsArr = str.split('*');
        let firstNum = numsArr[0];
        let secondNum = numsArr[1];
        return insertToInput((firstNum * 10) * (secondNum * 10) / (10 * 10));
    }
    insertToInput(res);
}

for (let button of buttons) {
    button.addEventListener('click', (e) => {
    let calcSymbol = e.target.getAttribute('data-symbol');
        insertToInput(calcSymbol);
    });
}
clearInputButton.addEventListener('click', clearInput);
calculateButton.addEventListener('click', () => {
    let stroke = input.value;
    calculate(stroke);
});

Заранее спасибо за ответ.

Comments

[Stalker_RED]

VicTHOR, вот тебе дают торт, который прокисший и подгоревший одновременно, и ты понимаешь что он плох, но критиковать низя, потому что ты сам не умеешь печь торты. Все верно?

[alex-1917]

а что значит хороший плохой? если дело в длине, то я за 48 секунд укоротил ваш код ровно в 2,3 раза! что дальше?

[NiyazNA]

Stalker_RED, Ну, легко сказать, что код плохой. Это и так наиболее вероятно, учитывая, что они с другом новички.

[lloydbanks]

Stalker_RED, не надо самому для этого печь торты. Достаточно предложить другой.

[Stalker_RED]

lloydbanks, сюрприз: можно и не предлагать. Можно просто сказать "мне не нравится этот торт", даже когда ты не умеешь печь торты и если у тебя нет другого торта на замену.

[lloydbanks]

Stalker_RED, сюрприз: мы находимся на ресурсе где обсуждаются технические вопросы, а не кулинарные рецепты. Критиковать без разумных аргументов - много ума не надо, а предложить лучшее решение уже другое дело.

[Stalker_RED]

lloydbanks, вы на полном серьезе предлагаете мне сделать лабораторку за какого-то студента, который об этом даже не просил?

Предлагаю для начала погуглить полную версию этой цитаты о критике.

[lloydbanks]

Stalker_RED, Зачем же сразу лабораторку. Вы ведь сами ответили на данный вопрос более развернуто после того как автор спросил у вас в комментариях. То, что подобный вопрос требует конкретики это предсказуемо и к гадалке не ходи.

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

[Stalker_RED]

lloydbanks, это не автор спросил )

Answer 1

[Stalker_RED]

да, плохой.

Comments

[Родион Альметов]

а конкретнее можно? Чисто из любопытства)

[Stalker_RED]

Родион Альметов, мне лениво пытаться его запустить.

Ну вот навскидку

let insertToInput = (value) => {
   // ...
    input.value += value;
}

Эта штука может выдавать довольно странные результаты, в зависимости от того, что считали на предыдущем шаге.

if (/0\.\d\+0\.\d/g.test(str)) {
        return insertToInput(res.toFixed(1));
}

Если 0.2 + 0.11, то уже не попадет в регулярку.
Если целая часть больше нуля (2.5, например), то тоже не попадкет в регулярку.

Слудующий блок кода, там где умножение на 10 и потом деление на 10*10, это что за бред вообще?

let stroke = ...wtf?

Да и одна только идея считать через eval() чего стоит.

[alex-1917]

в целях поддержания спора: Код хороший.