Калькулятор на JS

[ATVStudio]

Здравствуйте!
Закодил простой калькулятор на js! В принципе основные моменты работают... но есть одно но... почему работает вот этот код?

Код: [Выделить]

                     case '<=':
                                if (display.textContent = display.textContent.slice(0, -1)) {
                                } else {
                                        display.textContent = '0';
                                }
                                break;
Почему срабатывает данное условие?? не пойму!!!
В архиве приложил весь код!

[Лия]

if (display.textContent = display.textContent.slice(0, -1)) {

Может вы хотели

Код: [Выделить]

if (display.textContent == display.textContent.slice(0, -1)) {?

[ATVStudio]

Цитата: ATVStudio от Сегодня в 16:45:29
if (display.textContent = display.textContent.slice(0, -1)) {
Может вы хотели
Код: [Выделить]
if (display.textContent == display.textContent.slice(0, -1)) {
?

Нет... код работает как и должен... вопрос почему он работает ведь тут условие и есть код выполнение а после условия пусто...

[Лия]

вопрос почему он работает ведь тут условие и есть код выполнение а после условия пусто...

А почему это он не должен работать?
Как же работает ваш код:
Сначала выполняется

Код: [Выделить]

display.textContent = display.textContent.slice(0, -1))затем, присвоенное display.textContent значение подставляется в if
Если оно истинно (true) - ничего не выполняется, если ложно (false), выполняется else
Ваш код можно переписать так, и ничего не изменится:

Код: [Выделить]

                        case '<=':
                                display.textContent = display.textContent.slice(0, -1);
                                if (display.textContent) { } else { display.textContent = '0'; }
                                break;Или даже так:

Код: [Выделить]

                        case '<=':
                                display.textContent = display.textContent.slice(0, -1);
                                if (!display.textContent) { display.textContent = '0'; }
                                break;Так понятнее?
Пользователь добавил сообщение 07 Июля 2024, 17:54:42:Но я не рекомендую использовать присвоение в if, если нет понимания, как оно работает и чего вы пытаетесь добиться

[ATVStudio]

Код: [Выделить]
                        case '<=':
                                display.textContent = display.textContent.slice(0, -1);
                                if (!display.textContent) { display.textContent = '0'; }
                                break;
Так понятнее?

Да понятно.. я просто не мог понять что условие по факту и выполняло код.. что это и так работает...
Пользователь добавил сообщение 07 Июля 2024, 18:33:02:

Но я не рекомендую использовать присвоение в if, если нет понимания, как оно работает и чего вы пытаетесь добиться

Изначально я делал так

Код: [Выделить]

   
                      case '<=':
                                if (display.textContent !== '') {display.textContent = display.textContent.slice(0, -1)}
                                else {display.textContent = '0'}
                                break;
но оно не корректно работало..

[Лия]

Изначально я делал так
Код: [Выделить]

   
                      case '<=':
                                if (display.textContent !== '') {display.textContent = display.textContent.slice(0, -1)}
                                else {display.textContent = '0'}
                                break;


но оно не корректно работало..

Вы сначала сравниваете, а потом присваиваете, а надо наоборот
Я выше скидывала код, как правильно
Но и ваш можно легко исправить:

Код: [Выделить]

                      case '<=':
                                if (display.textContent.slice(0, -1) !== '') {display.textContent = display.textContent.slice(0, -1)}
                                else {display.textContent = '0'}
                                break;


[ATVStudio]

Но и ваш можно легко исправить:
Код: [Выделить]
                      case '<=':
                                if (display.textContent.slice(0, -1) !== '') {display.textContent = display.textContent.slice(0, -1)}
                                else {display.textContent = '0'}
                                break;

Да понял ошибку! Спасибо!