В чем отличия моего рабочего и нерабочего решения?

Question

[Nerevar96]

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

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

Первый

let time = prompt('Который сейчас час?', '');

let day = prompt('Какой сейчас день недели?', '');

let result_Time
let result_Day
let overeng

if ( time < 10 || time > 19 ) {
    result_Time = false;
} else {
    result_Time == true;
}

if ( day == 'понедельник' || day == 'вторник' || day == 'среда' || day == 'четверг' || day == 'пятница' ) {
    result_Day = true
} else if ( day == 'суббота' || day == 'воскресенье' ) {
    result_Day = false
}

if (result_Time == true, result_Day == true) {
    alert('открыто')
} else {
    alert('закрыто')
}

Второй:

let time = prompt('Который сейчас час?', '');

let day = prompt('Какой сейчас день недели?', '');

let result_Time = 0;
let result_Day
let overeng = 0

if ( time < 10 || time > 19 ) {
    overeng = 0;
} else {
    overeng == ++overeng;
}

if ( day == 'понедельник' || day == 'вторник' || day == 'среда' || day == 'четверг' || day == 'пятница' ) {
    overeng = ++overeng;
} else if ( day == 'суббота' || day == 'воскресенье' ) {
    overeng = overeng - 1;
}

if (overeng == 2) {
    alert('открыто')
} else {
    alert('закрыто')
}

Comments

[WapSter]

Что по твоему делает запятая в этом куске кода if (result_Time == true, result_Day == true)
Откуда вообще такие идеи берутся

Введи в консоли браузера

if (false, true) {
  console.log('Вариант 1')
}
if (true, false) {
  console.log('Вариант 2')
}
if (true, true) {
  console.log('Вариант 3')
}
if (false, false) {
  console.log('Вариант 4')
}

и подумай, что ты делаешь не так

Answer 1

[Алексей Уколов]

if (result_Time == true && result_Day == true)
Ну вы учебник-то не закрывайте, раз у вас есть склонность к придумыванию собственного синтаксиса.

spoiler

let time = prompt('Который сейчас час?', '');
let day = prompt('Какой сейчас день недели?', '');

const isWorkingTime = time >= 10 && time <= 19;
const isWorkingDay = day == 'понедельник' || day == 'вторник' || day == 'среда' || day == 'четверг' || day == 'пятница';

if (isWorkingTime && isWorkingDay) {
    alert('открыто')
} else {
    alert('закрыто')
}

Comments

[Nerevar96]

if (result_Time == true && result_Day == true)
Да, я тут ошибся, пробовал разные варианты, и с || и с &&, результат был тем же