2015-10-17 73 views
0

我不能讓我的JS/HTML寫的,如果x &Ÿ單選按鈕被選中......檢查是否選中了兩個單選按鈕,然後寫入。 [HTML/JS]

我的代碼:

http://dumptext.com/6rR25ynt

+2

請更多的細節和具體職位代碼更新你的問題在這裏。 –

+0

你從不聲明或分配x或y,也不嘗試打印它們?你確實由於某種原因檢查他們是虛假的,然後什麼都不做。也沒有Id「S4A」的元素,所以之前的條件也不會做任何事情。 – diadical

回答

0

有幾件事情代碼錯誤: 您定義了函數Pris(),但您錯過了右括號,因此該函數不可用。此外,如果是測試是否檢查單選按鈕是不正確的。 測試此項:

<script type="text/javascript"> 
    function Pris() { 

     var S4A = document.getElementById("S4A"); 
     var S5A = document.getElementById("S5A"); 

     var Skade1 = document.getElementById("Skade1"); 
     var Skade2 = document.getElementById("Skade2"); 
     var Skade3 = document.getElementById("Skade3"); 
     var Skade4 = document.getElementById("Skade4"); 

     if (S4A.checked && Skade1.checked){ 
      document.write("2412kr") 
     } 
    } 
</script> 
+0

你打我:) –

+0

謝謝你,這工作! – TheNeedingMan

0

這裏有幾件事情正在進行。首先,在您的比較,我想你想看看是否都被檢查:

if (S4A && Skade1 == checked){ 

但要做到這一點應該是這樣的:

if (S4A.checked && Skadel1.checked){ 

其次,你必須在一個if塊這是指變量底部還沒有被定義爲:

if((x == false) && (y == false)){ 
} 

另外,作爲一個一般的筆記,你要使用===而不是==儘可能多的,因爲==自動類型強制。

最後,這實際上是我發現的第一個錯誤,您的Pris函數沒有右括號。它看起來像if((x...etc塊沒有給出它自己的關閉}。不匹配的大括號會阻止您的代碼執行。

希望這會有所幫助。

編輯補充:working jsbin

+0

即使我在第一條評論中得到了答案,我也會爲你評論,因爲你解釋了爲什麼以及代碼應該如何工作,而不僅僅是給我答案,謝謝! – TheNeedingMan

相關問題