Оператор в js. Date, time?

Question

[SaberChazer]

На body у меня весит onload="myFunction()".
Взял где-то 50% с интернета.
Интересует то что оно как-то не совсем корректно например когда реальное время под утро может написать Good evening.
И эта проблема связана с ">, <, >=" Оператором.

function myFunction() {
    var greeting;
    var time = new Date().getHours();
    if (time < 10) {
        greeting = "Good morning";
    } else if (time < 20) {
        greeting = "Good day";
    } else {
        greeting = "Good evening";
    }
document.getElementById("random_txt").innerHTML = greeting;
}

<span id="random_txt"></span>

Comments

[SaberChazer]

Возможности загрузить на Codepen и JSFiddle нету.

[Rsa97]

Так добавьте вывод time и посмотрите, может у вас часовой пояс не тот.

Answer 1

[defriz]

function myFunction() {
    let greeting,
    time = new Date().getHours();
    if (time >= 6 && time <= 12) {
        greeting = "Good morning";
    } else if (time > 12 && time <= 18) {
        greeting = "Good day";
    } else if (time >= 0 && time < 6) {
        greeting = "Good night";
    } else {
        greeting = "Good evening";
    };
    document.getElementById("random_txt").innerHTML = greeting;
};

Comments

[Rsa97]

const greeting = () => `Good ${['night', 'morning', 'day', 'evening'][Math.floor(new Date().getHours() / 6)]}`;
greeting(); // "Good day"

[defriz]

Rsa97, я не стал выдумывать другие способы, просто переделал способ автора.

[Сергей Соколов]

Rsa97, а потом выяснится, что настоящая проблема ТС была в том, что это приветствие не обновляется вживую на странице, которую долго держат открытой.. )