ファンクションメモリをクリアする必要があります。 jQueryランニング関数が何度も実行されました

Question

最初は、クリックイベントが複数回発生していた問題がありましたが、私は下のコードに示すように、おそらくはunbind()とone()を使用してこれを克服できました。

私がここに持っているのは、普段使うことのできるモーダルウィンドウを開くコードです。いくつかのケースでは、パスワードフォームを含むさまざまな用途に使用します。

私はあなたがHTMLを必要としないと思いますので、投稿しません。

ボタン、またはアクションウィンドウが必要とされるようにするとき、私はこのような関数を呼び出す：

showModalAlert(type, theWidth, theHeight, title, html, confirmThis, denyThis) 

最初の三つの変数は、ウィンドウがどのように見えるかを決定titleとhtml内容を決定し、 confirmThisおよびdenyThisは、この関数を呼び出す直前に設定された関数で、confirmのウィンドウで、confirmまたはdenyのボタンが押された場合のアクションを決定します。

securityウィンドウの場合、確認ボタンは、単純なパスワードフォームを送信し、データベースからユーザーIDを返す「署名する」ボタンに置​​き換えられます。ユーザーIDが正常に返された場合、スクリプトはプログラムでconfirmボタンを押し、モーダルウィンドウの開始時の呼び出しに従って関数を実行します。

私の問題があれば、誤ったパスワードが入力された、またはユーザーがブラウザウィンドウを更新せず、後で、ウィンドウをキャンセルすると、正しいパスワードを再入力し、confirmThis()関数が2回実行（または何回されていることです間違ったパスワード/キャンセルアクションが実行されたため）。

これは明らかに、毎回confirmThis機能を「記憶」していることを示しています。私が言ったように、最初は、パスワードの成功の機能がconfirmItは二回、one()の豊富な使用は、これを修正したクリックして

、それは今、間違いなく一度だけconfirmItをクリックしているが、それはまだ機能、複数の時間を実行しています。

この機能をクリアして1回だけ行うことができますか？

そこから私は、モーダルウィンドウを呼び出しています関数は次のようになります。

$('#saveDelivery').click(function() { 
      function confirmIt() { 
       formData = (JSON.stringify($('#delDetail').serializeObject())); 
       saveData(formData); 
       $('#saveDelivery').removeClass('centreLoader'); 
      }; 
      showModalAlert('security', '300px', '185px', 'Security!', 'You need to "Sign" this action.', confirmIt, ''); 
    }); 

それは単にsaveDelivery要素のクリックは、confirmThis関数は、この時点で宣言されていますし、AJAXフォームを送信

実際showModalAlert機能は以下の通りです：

function showModalAlert(type, theWidth, theHeight, title, html, confirmThis, denyThis)  { 

// stuff that opens the alert window \\ 

if (confirmThis == '') { 
    $('#confirmIt').one('click', function() { $('#closeAlert').one('click').click(); }); 
} else { 
    $('#confirmIt').one('click', function() { confirmThis(); $('#closeAlert').one('click').click(); }); 
}; 
if (denyThis == '') { 
    $('#denyIt').one('click', function() { $('#closeAlert').one('click').click(); $('#signIt').unbind(); }); 
} else { 
    $('#denyIt').one('click', function() { denyThis(); $('#closeAlert').one('click').click(); $('#signIt').unbind(); }); 
}; 

if (type == "confirm") { 
    $('.closeAlert, .signItForm').hide(); 
}; 
if (type == "alert") { 
    $('.alertConfirm, .signItForm').hide(); 
}; 
if (type == "fixedAlert") { 
    $('.closeAlert, .alertConfirm, .signItForm').hide(); 
}; 
if (type == "security") { 
    $('.signItForm').show(); 
    $('.closeAlert').hide(); 
    $('#confirmIt').hide(); 
    $('#signIt').unbind().fadeTo('fast',1); 
}; 
}; 

$('#signIt').live('click', function() { 
var formData = (JSON.stringify($('.secureSign').serializeObject())); 
var signitPwd = $('#signItpwd').val(); 
var jsonURL = "/jsonout/getdata.aspx?sql=SELECT id, password FROM users WHERE password ='" + signitPwd + "' LIMIT 1&output=json&usedb=new&labelName=any&fileName="; 

$.getJSON(jsonURL, function (data) { 
    if (data.length > 0) { 
     $('.savingUserID').val(data[0].id); 
     $('#confirmIt').one('click').click(); 
     $('#signIt').fadeTo('fast', 0); 
     $('#confirmIt').show(); 
    } else { 
     $('#signIt').fadeTo('fast', 0); 
     $('#confirmIt').one('click').show(); 
     $('.closeAlert').show(); 
     $('.alertConfirm, .signItForm').hide(); 
     $('#alertTitle').html("Error!"); 
     $('#alertContent').css({ 'text-align': 'center' }).html("Password Denied"); 
    }; 
}); 
}); 

Answer 1

私から$.oneの理解では、単にONCEイベントを実行します。イベントに2回バインドすると、即座に2回実行されますが、それ以上は実行されません。

例：http://jsfiddle.net/qCwMH/（ボタンをクリックすると、イベントが4回実行されます）。

saveDeliveryをクリックするたびに、別のイベントを#confirmItにバインドしています。

あなたができることは、モーダル関数の開始時にconfirmItとdenyItからイベントをバインド解除することです（つまり、$('#confirmIt, #denyIt').unbind('click');）。その上にビルドするのではなく、その関数が呼び出されるたびに新しいものを割り当てます。理想的なバインド/バインド解除は他のオプションより多くのリソースを使用しますが、おそらくそれを開始しようとしていますか？