2017-03-08 156 views
0

我不知道为什么函数不会调用。最初它来自onsubmit =“return checkFilled()”的表单,但我也尝试用一个按钮调用onclick事件,它永远不会进入函数。下面是代码:javascript函数不被调用

function checkFilled() { 
 
    alert("Entered"); //just to test if entering function 
 
    return false; 
 
    var currentBox = document.getElementById("livCost").value; 
 
    if (currentBox == null) { 
 
    alert("Not all preferences are filled out!"); 
 
    return false; 
 
    } 
 
    currentBox = document.getElementById("travOps").value; 
 
    else if (currentBox == "Select") { 
 
    alert("Not all preferences are filled out!"); 
 
    return false; 
 
    } else 
 
    return true; 
 
}
<form onsubmit="return checkFilled()" method="post" action="recommendScript.php"> 
 

 
    <p>Cheap living cost : 
 
    <select name="livingCost" id="livCost"> 
 
       <option selected>Select</option> 
 
       <option value="1">1</option> 
 
       <option value="2">2</option> 
 
      <p>Travel Opportunities : 
 
      <select name="travelOps" id="travOps"> 
 
       <option selected>Select</option> 
 
       <option value="1">1</option> 
 
       <option value="2">2</option>  
 
      </select></p> 
 
    <input type="submit" value="Submit" class="loginButton" />

为什么不能这样运行?

+0

在Javascript控制台中是否有任何错误? – Barmar

+0

@AlonEitan之前有一个alert()。 – Barmar

+0

您是否错过了您尝试运行的代码中的关闭脚本标记,或者仅仅是将代码复制到您的问题的问题? – Calum

回答

-5

首先,改变<script><script type="text/javascript">,使其更安全,在你的js代码的最后,请务必把结束脚本标签</script>

+2

您是否尝试过验证这是否解决了问题? – mmgross

0

请务必关闭脚本标签和表单标签,如果仍然不工作打开JavaScript控制台以查找任何语法错误

+0

您是否尝试过验证这是否解决了问题?其实你的答案的最后一部分是一个宝贵的建议,所以我删除了我的downvote。 – mmgross

+0

不,我没有但在提交时只会在表单标记关闭时触发,否则没有表单提交,我现在将测试您的代码 –

+0

这里是完整的代码https://jsfiddle.net/wjugs42b/关闭脚本标签解决了一半的问题,并在if和else之间移动一行,如果解决了另一半,你可能想给我一个投票。 –