Не выводится время для определения времени суток?

Question

[Александр AS]

Доброй ночи!
Пытаюсь вывести в консоль текущее времени суток.
Скрипт работал минут 20-30, затем начал выдавать undefined
В чем беда?

function getHour(hours) {
    
    if (hours > 4 && hours < 12) {
        return ('Доброе утро');
    } else if (hours <= 12 && hours > 16) {
        return ('Добрый день');
    } else if (hours <= 16 && hours > 21) {
        return ('Добрый вечер');
    } else if (hours < 21 && hours > 4) {
        return ('Доброй ночи');
    }
    
}
let hours = new Date().getHours();
console.log(getHour());

Спасибо!

Answer 1

[Богдан Жувак]

У тебя условия какие-то странные. Скорее всего ты хотел вот это

function getHour(hours) {

  if (hours > 4 && hours < 12) {
    return 'Доброе утро';
  } else if (hours >= 12 && hours < 16) {
    return 'Добрый день';
  } else if (hours >= 16 && hours < 21) {
    return 'Добрый вечер';
  } else if (hours >= 21 || hours < 4) {
    return 'Доброй ночи';
  }
}
let hours = new Date().getHours();
console.log(getHour(hours));

Comments

[Александр AS]

Да, это.
Больше спасибо!

Answer 2

[Developer]

hours не передаётся в функцию в качестве аргумента.

Проблема в том, что нет последнего
else return "Привет!"

Посмотри, у тебя функция не возвращает результат для hours < 4, поэтому для hours меньше 4 будет undefined

Comments

[Александр AS]

изменил, но всё равно ничего не изменилось

[Developer]

Александр AS, а какие конкретные значения hours передаются?

[Александр AS]

Developer, на данный момент никакие.
Пробую разными путями, но на выходе только undefined вижу

[Developer]

Александр AS, изменил ответ. Поправь код