2017-03-08 14 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 type="text/javascript"><script>を変更し、あなたのJSコードの最後に、必ずあなたのスクリプトタグとフォームタグを閉じてください閉じるスクリプトタグ</script>

+2

これでこれが問題を解決することを確認しようとしましたか? – mmgross

0

を入れて作ります、それでもまだ動作していない場合は、javascript consoleを使用して構文エラーを探してください。

+0

これが問題を解決することを確認しようとしましたか?実際にあなたの答えの最後の部分は貴重な助言です、私は私のdownvoteを削除しました。 – mmgross

+0

いいえ私は提出しませんでしたが、フォームタグが閉じられている場合にのみ送信されます。提出するフォームがない場合は、今すぐあなたのコードをテストします –

+0

ここは完全なコードですhttps://jsfiddle.net/wjugs42b/スクリプトタグは半分の問題を解決し、ifとelseの間で1行移動すると、他の半分が解決すれば、あなたは私に投票をしたいかもしれません。 –

関連する問題