Почему на срабатывает цикл калькулятора?

Question

[Evgenij_nechujveter]

do {
        function calculator() {


            function add(a, b) {
                return a + b;
            }

            function sub(a, b) {
                return a - b;
            }

            function mul(a, b) {
                return a * b;
            }

            function div(a, b) {
                return a / b;
            }

            var operand1 = prompt("Введите первое число: ");
            var sign = prompt("Введите знак арифметической операции: + - * /");
            var operand2 = prompt("Введите второе число: ");
            var result;

            operand1 = parseInt(operand1);
            operand2 = parseInt(operand2);

            switch (sign) {
                case "+":
                    result = add(operand1, operand2);
                    break;
                case "-":
                    result = sub(operand1, operand2);
                    break;
                case "*":
                    result = mul(operand1, operand2);
                    break;
                case "/":
                    result = div(operand1, operand2);
                    break;

                default:
                    document.write("<p><b style='color:red'>" + sign + "</b> - не является знаком арифметической операции.");
            }


            document.write("<p>" + operand1 + " " + sign + " " + operand2 + " = " + result);
        }
        setTimeout(calculator, 5000);
    } while (calculator);

Comments

[Stalker_RED]

Вы взяли пример на бейсике из книжки которой 30 лет и пытаетесь переписать на js?

1. Не стоит объявлять функции add, sub, mul... в каждом цикле. Их можно вынести в начало.
2. setTimeout в каждом цикле не нужен. Используйте setInterval один раз.
3. document.write - тоже сомнительный выбор.

[Evgenij_nechujveter]

Stalker_RED: я не гений, пока только начал учить js...

Answer 1

[Stalker_RED]

Не загружается потому, что просто зависает. У вас же бесконечный цикл while (calculator) который никогда не станет false, и он будет жрать процессор беспрерывно. Кроме того, он нигде не получает пользовательский ввод. Т.е. не слушает какие кнопки нажаты и не читает никакой текст. Он просто очень быстро повторяется и забивает память новыми калькуляторами (из-за setTimeout в каждом цикле).

Лучший вариант - разобрать это всё на части (типа декомпозиция), потом почитать учебник и 60% кода выбросить.

Еще раз:
1. Не стоит объявлять функции add, sub, mul... в каждом цикле. Их можно вынести в начало.
2. setTimeout в каждом цикле не нужен. Используйте setInterval один раз.
3. document.write - тоже сомнительный выбор. выводите в консоль (console.log) или в какой-то блок, вместо тог чтоб пытаться писать в документ.

Comments

[Evgenij_nechujveter]

Это дз) так мне так и надо, чтобы он бесконечно выполнялся. То есть:
1. человек вводит что он хочет сделать (умножить, разделить и т.д.)
2. Вводит 1-вое число
3. Вводит 2-рое число
4. Получает результат через document.write на самой странице
5. Потом идет задержка(ну скажем секунд 5)
6. После опять пункт 1 и т.д.

[Stalker_RED]

0. не надо. вместо этого есть обработка событий.
1. у вас нет ввода
4. хрень. от document.write масса проблем. Ничего не выведется или страница сломается.
5. у вас нет задержки. совсем нет. ядро процессора загружается на 100%.

Answer 2

[Negwereth]

Охохо, а что вы хотите в цикле делать-то? Вы просто космическими темпами тут память забиваете.

Comments

[Evgenij_nechujveter]

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

[Evgenij_nechujveter]

сейчас у меня просто даже страница не загружается