Jqueryコードでエラーが発生しました

Question

このコードは機能しません。

おそらく私は間違ったことをしましたか？誰でもこのコードをチェックできますか？それはjqueryを使ったからですか？

$(document).ready(function showCart(next) { 
    var ca = document.getElementById("cartArea"); 
    var params = ""; 
    for (i = 0; i < document.clientCart.elements.length; i++) { 
     param = getElemValue(document.clientCart.elements[i]); 
     if (param != "") params += param + "&"; 
    } 
    if (next) params += "Next=1"; 
    ca.innerHTML = postIt(params); 
    makePie(); 
} 
}); 

Answer 1

あなたは余分に}を持っています。

ので

$(document).ready(function showCart(next) { 
    var ca = document.getElementById("cartArea"); 
    var params = ""; 
    for (i = 0; i < document.clientCart.elements.length; i++) { 
     param = getElemValue(document.clientCart.elements[i]); 
     if (param != "") params += param + "&"; 
    } 
    if (next) params += "Next=1"; 
    ca.innerHTML = postIt(params); 
    makePie(); 
}); 

は、少なくともいずれかの構文上の問題を持っていません。

Answer 2

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

$(document).ready(function(next){ 
    var ca = document.getElementById("cartArea"); 
    var params = ""; 
    for (i = 0; i < document.clientCart.elements.length; i++) { 
     param = getElemValue(document.clientCart.elements[i]); 
     if (param != "") params += param + "&"; 
    } 
    if (next) params += "Next=1"; 
    ca.innerHTML = postIt(params); 
    makePie(); 
}); 

Answer 3

あなたは1つのあまりにも多くのクロージング括弧を持っている：

$(document).ready(function showCart(next) { 
    var ca = document.getElementById("cartArea"); 
    var params = ""; 
    for (i = 0; i < document.clientCart.elements.length; i++) { 
     param = getElemValue(document.clientCart.elements[i]); 
     if (param != "") params += param + "&"; 
    } 
    if (next) params += "Next=1"; 
    ca.innerHTML = postIt(params); 
    makePie(); 
} // <-------------- that shouldn't be there 
}); 

また、あなたの関数でdocsnextに応じて常にjQueryとなりますので、なぜそれを削除しませんそして、単にjQueryをjQuery（またはnoconflictモードでない場合は$）と参照してください。

はまた、あなたが

$(document).ready(function showCart(next) { 

を簡素化することができますのでご注意：

$(function showCart(next) { 

や不要なnextパラメータを捨て、関数式名

$(function() {