2015-10-17 41 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

相关问题