2017-07-18 66 views
-6

我希望我的网站根据当前时间显示“打开”或“关闭”。显示打开/关闭标志

我的问题是什么都不显示了在网站上,所以我试图找出什么毛病只能getElementById一次指定的元素,已申报代码

<script type="text/javascript"> 
var today = new Date(), 
var open = "Open", 
var closed = "Closed", 
var display = document.getElementById('hours-display'); 
if (today.getHours() >= 9 && today.getHours() < 18) { 
display.innerHTML = open; 
} else { 
display.innerHTML = closed; 
} 
</script> 


<div id="hours-display"></div> 
+6

而你的问题是.....? – Gavin

+0

似乎你已经回答了你自己的问题。 –

+3

Soooo不应该'var closed =“打开''是'var closed =”关闭“'否则输出将会打开'不管是否使用open或closed变量... – NewToJS

回答

0

。此外,您使用逗号代替分号

<div id="hours-display"></div> 
<script> 
    var today = new Date(); 
    var display = document.getElementById("hours-display"); 
    if (today.getHours() >= 9 && today.getHours() < 18) { 
    display.innerHTML = "Open"; 
    } else { 
    display.innerHTML = "Closed"; 
    } 
</script> 
+0

小心解释这些变化或为什么这可能有用吗?或者您是否期望人们使用它而不在乎了解这些变化? – NewToJS

+0

这是一个客户,汽车经销商,他们真的很想 – Kevin

0

您声明变量的方式是问题。你应该从其他变量中删除变量,因为你使用的逗号是这样的:

var today = new Date(), 
open = "Open", 
closed = "Closed", 
display = document.getElementById('hours-display'); 
if (today.getHours() >= 9 && today.getHours() < 18) { 
    display.innerHTML = open; 
} else { 
    display.innerHTML = closed; 
} 
+0

你试过吗? Theres多一个错误在脚本 – samdd

+0

那么,我发现了错误,并更新了我的答案。 – Richboy

+0

并且脚本标记需要低于DIV – samdd