这看起来正确吗？

Question

所以我不久前做了这个（我相信是在这里的帮助下，但我不记得了——不相关）来上课。它应该根据一天中的时间返回一条消息，然后返回一条随机消息。随机消息部分工作正常。这是我感到困惑的一天中的时间消息。告诉我你的想法...看起来合适吗？

var hr = new Date();

if (hr < 12) {
    alert("Good Morning");
}
if ((hr - 12) < 6) {
    alert("Good Afternoon");
}
if (hr >= 18) {
    alert("Good Evening");
}

msg = new Array
msg[1]="how are you"
msg[2]="whazzup"
msg[3]="how have you been"
msg[4]="how has your day been going"
msg[5]="hey y'all"

random_num = (Math.round((Math.random()*4)+1))

alert(msg[random_num]); 

非常感谢所有帮助和建议。谢谢。

Answer 1

您应该调用 getHours() 来返回小时。您的 hr 实际上是一个 Date 对象：

var d = new Date();
var hr = d.getHours();

// Etc...
if (hr < 12) {
    alert("Good Morning");
}
if ((hr - 12) < 6) {
    alert("Good Afternoon");
}
if (hr >= 18) {
    alert("Good Evening");
}

Answer 2

var hr = (new Date()).getHours();
if (hr < 12)
{
   alert("Good Morning");
}
else if (hr < 18)
{
   alert("Good Afternoon");
}
else if (hr >= 18)
{
   alert("Good Evening");
}

对于数组：

msg = [null]; //start off with index 0 as null
msg.push("how are you");
msg.push("whazzup");
msg.push("how have you been");
msg.push("how has your day been going");
msg.push("hey y'all");

Answer 3

因为您将相同的表达式与多个结果进行比较，所以我建议使用 switch 语句，因为它更干净，并且提供了更大的灵活性，而不是一堆带有 else if 之类的意大利面条代码。

Answer 4

// proper way to get the current hour
var hr = new Date().getHours();

// use if elseif
if(hr < 12) {
    alert("Good Morning");
} else if(hr < 18) {
    alert("Good Afternoon");
} else {
    alert("Good Evening");
}

// use var so we don't pollute
var msg = [
    "how are you",
    "whazzup",
    "how have you been",
    "how has your day been going",
    "hey y'all"
];

var random_num = Math.round(Math.random()*4);

alert(msg[random_num]);