の追加機能は、JSファイル

Question

に私はJSファイルでこの機能を持っていたし、すべてがうまく働いていた他の機能を無効にされています

function check_acco_form() 
{ 
    var name=$("#name").val(); 
     var institution=$("#institution").val(); 
     var year=$("#year").val(); 

     //PNR Data 
    var pnr1=$("#pnr1").val(); 
    var pnr2=$("#pnr2").val(); 
    // Arrival date info 
    var arr_year=$("#arr_year").val(); 
    var arr_month=$("#arr_month").val(); 
    var arr_date=$("#arr_date").val(); 
    //Departure date info 
    var dep_year=$("#dep_year").val(); 
    var dep_month=$("#dep_month").val(); 
    var dep_date=$("#dep_date").val(); 

    var numericExpression = /^[0-9]+$/; 

     //Name, institution and year must not be empty 
     if(name=="" || institution=="" || year=="") 
    { 
     alert("One or more fields are empty."); 
       return; 
     } 
    //PNR must be all numbers 
    if(!pnr1.match(numericExpression) || !pnr2.match(numericExpression)) 
    { 
     alert("A PNR number consists of 10 digits only. Please enter again."); 
     $("#pnr1").val(""); 
     $("#pnr2").val(""); 
     return; 
    } 
    if(pnr1.length!=3 || pnr2.length!=7) 
    { 
     alert('Invalid PNR Number.'); 
     $("#pnr1").val(""); 
     $("#pnr2").val(""); 
     return; 
    } 

    if((arr_month==dep_month && dep_date<arr_date) || (dep_month<arr_month)) 
    { 
     alert('Invalid dates.Please check again.'); 
     return; 
    } 

    //Test passed. Store in database; 
    URL="saveAcco.php"; 
    parameters="name="+name+"&ins="+institution+"&year="+year+"&pnr="+pnr1+""+pnr2+"&dateArr="+arr_year+"-"+arr_month+"-"+arr_date+"&dateDep="+dep_year+"-"+dep_month+"-"+dep_date; 
    $.get(URL+"?"+parameters,function(data){ 
     $("#msg_box").html(data); 
     if(data=="Your changes have been saved." || data=="Your data has been saved and is pending approval.") 
       { 
      $("#acco_status").html('<br/><b>Accomodation Approval Status</b> : <span style="padding:3px;background-color:#f4fb3c">Approval Pending</span><br/><br/>'); 
       } 
     $("#msg_box").fadeIn("slow",function(){ 
      setTimeout('fadeOutMsgBox();',3000); 
       }); 
      }); 
} 

私は、この関数に少し変更を加えた（変数のmobile_num」を追加しましたそして、「train_num」、条件は、ユーザーが数字のみを入力するとjQueryの変更機能を取得をしたことを確認するには「が」含まれています）、次のコードをもたらしたもの：変更後

function check_acco_form() 
{ 
    //Personal Information 
    var name=$("#name").val(); 
    var institution=$("#institution").val(); 
    var year=$("#year").val(); 

    //Contact Information 
    var mobile_num=$("#mobile").val(); 

     //PNR Data 
    var pnr1=$("#pnr1").val(); 
    var pnr2=$("#pnr2").val(); 

    //Train Number 
    var train_num=$("#trainnum").val(); 

    // Arrival date info 
    var arr_year=$("#arr_year").val(); 
    var arr_month=$("#arr_month").val(); 
    var arr_date=$("#arr_date").val(); 
    //Departure date info 
    var dep_year=$("#dep_year").val(); 
    var dep_month=$("#dep_month").val(); 
    var dep_date=$("#dep_date").val(); 

    var numericExpression = /^[0-9]+$/; 

    //Name, institution and year must not be empty. 
    if(name=="" || institution=="" || year=="") 
    { 
     alert("One or more fields are empty."); 
     return; 
    } 

    //PNR can be empty but if entered must be all numbers 
    if(pnr1!="" and pnr2!="") 
    { 
     if(!pnr1.match(numericExpression) || !pnr2.match(numericExpression)) 
     { 
      alert("A PNR number consists of 10 digits only. Please enter again."); 
      $("#pnr1").val(""); 
      $("#pnr2").val(""); 
      return; 
     } 

     if(pnr1.length!=3 || pnr2.length!=7) 
     { 
      alert('Invalid PNR Number.'); 
      $("#pnr1").val(""); 
      $("#pnr2").val(""); 
      return; 
     } 
    } 

    //Train number can be empty but if entered must be all numbers 
    if(train_num!="") 
    { 
     if(!train_num.match(numericExpression)) 
     { 
      alert("Train number must consits of digits only"); 
      $("#trainnum").val(""); 
      return; 
     } 
    } 

    //Mobile num can be empty but must be all numbers 
    if(mobile_num!="") 
    { 
     if(!mobile_num.match(numericExpression)) 
     { 
      alert("Invalid mobile number"); 
      $("#mobile_num").val(""); 
      return; 
     } 
     if(mobile_num.length!=10) 
     { 
      alert('A mobile number consists of 10 digits.Please enter again.'); 
      return; 
     } 
    } 

    if((arr_month==dep_month && dep_date<arr_date) || (dep_month<arr_month)) 
    { 
     alert('Departure date cannot be before arrival date.Please check again.'); 
     return; 
    } 

    //Test passed. Store in database; 
    URL="saveAcco.php"; 
    parameters="name="+name+"&ins="+institution+"&year="+year+"&pnr="+pnr1+""+pnr2+"&dateArr="+arr_year+"-"+arr_month+"-"+arr_date+"&dateDep="+dep_year+"-"+dep_month+"-"+dep_date+"&mobile="+mobile_num+"&train_num="+train_num; 
    $.get(URL+"?"+parameters,function(data){ 
     $("#msg_box").html(data); 
     if(data=="Your changes have been saved." || data=="Your data has been saved and is pending approval.") 
       { 
      $("#acco_status").html('<br/><b>Accomodation Approval Status</b> : <span style="padding:3px;background-color:#f4fb3c">Approval Pending</span><br/><br/>'); 
      $("#acco_letter_print").html('Download accomodation letter <a href="PDF/acco_print.php" target="_blank">here</a>'); 
      $("#acco_letter_print").fadeIn();   
       } 
     $("#msg_box").fadeIn("slow",function(){ 
      setTimeout('fadeOutMsgBox();',3000); 
       }); 
      }); //End of get function 

} 

、中に突然のすべての機能この機能のjsファイルは、この関数。フォーラムを検索すると、私はこのディスカッションを見つけました：JavaScript function causing all other functions not to work inside js fileこのエラーは、予約語を使用している可能性があることを示しています。しかし、私はコード内で変数として使用される予約語を見つけることができません。何が問題なのでしょうか？

Answer 1

あなたはそこでこれを持っている：

if(pnr1!="" and pnr2!="") 

それは次のようになります。

if(pnr1!="" && pnr2!="") 

このような任意の構文エラーがもののためにあなたのエラーコンソールを確認してください、全体のことが失敗しますこのように、彼らはすぐに原因を指摘します。これに

setTimeout('fadeOutMsgBox();',3000); 

：

setTimeout(fadeOutMsgBox,3000); 

この意志はさておき、これを変更し、直接関数参照を渡すだけでなくsetTimeout()に文字列を渡すことがないようにしようとして

---

問題を少なくし、スコープ内のどこにでも関数を置くことができるようにするために、グローバルである必要はありません（文字列の場合のように）。