2017-07-18 51 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