j-アラートの確認 - jQueryプラグイン

Question

ユーザー確認のためにjを確認してください。

私の最初のjConfirmは、ユーザーの操作のために停止せず、次へ進むだけです。

マイコード：

$(function() { 

    $("#UpdateJobHandler").click(function() { 

     var JobHander = getJobHandler(); 
     if (JobHander.MaxInstances == 0) { 
       jConfirm('Continue?', 'Current Maximum Instances', function (ans) { 
        if (!ans) 
         return; 
       }); 
     } 

     var json = $.toJSON(JobHander); 

     $.ajax({ 
      url: '../Metadata/JobHandlerUpdate', 
      type: 'POST', 
      dataType: 'json', 
      data: json, 
      contentType: 'application/json; charset=utf-8', 
      success: function (data) { 

       var message = data.Message; 
       var alertM = data.MessageType; 
       if (alertM == 'Error') { 
        $("#resultMessage").html(message); 

       } 

       if (alertM == 'Success') { 
        $("#resultMessage").empty(); 
        alert(alertM + '-' + message); 
        action = "JobHandler"; 
        controller = "MetaData"; 
        loc = "../" + controller + "/" + action; 
        window.location = loc; 
       } 

       if (alertM == "Instances") { 
        jConfirm(message, 'Instances Confirmation?', function (answer) { 
         if (!answer) 
          return; 
         else { 
          var JobHandlerNew = getJobHandler(); 
          JobHandlerNew.FinalUpdate = "Yes"; 
          var json = $.toJSON(JobHandlerNew); 
          $.ajax({ 

           url: '../Metadata/JobHandlerUpdate', 
           type: 'POST', 
           dataType: 'json', 
           data: json, 
           contentType: 'application/json; charset=utf-8', 
           success: function (data) { 

            var message = data.Message; 
            $("#resultMessage").empty(); 
            alert(alertM + '-' + message); 
            action = "JobHandler"; 
            controller = "MetaData"; 
            loc = "../" + controller + "/" + action; 
            window.location = loc; 
           } 
          }); 
         } 
        }); 
       } 
      } 
     }); 
    }); 
}); 

私は何をしないのですか？これがすべてである場合

Answer 1

わからないが、この部分：

if (JobHander.MaxInstances == 0) { 
      jConfirm('Continue?', 'Current Maximum Instances', function (ans) { 
       if (!ans) 
        return; 
      }); 
    } 

はおそらく、あなたがやりたいことはありません。 function(ans) { ... }関数を終了していますが、ハンドラ全体を終了したい場合があります（$("#UpdateJobHandler").click(function() { ... }）。もしそうなら、あなたは以下のことと同様にする必要があります。つまり、返品後にすべてをfunction(ans) { ... }に入れます。おそらく、より小さな関数に分割するのが最も良いでしょう。

EDIT：これらの線に沿って何か：

function afterContinue() { 
     var json = $.toJSON(JobHander); 

     $.ajax({ 
      // ... all other lines here ... 
     }); 
    } 

    if (JobHander.MaxInstances == 0) { 
      jConfirm('Continue?', 'Current Maximum Instances', function (ans) { 
       if (ans) { 
        afterContinue(); 
       } 
      }); 
    } 

あなたはすべてのsuccess機能について同様のことを行うことができます。

別の例として、次のようなInstancesチェックを書き換えることができます。

  function afterInstances() { 
         var JobHandlerNew = getJobHandler(); 
         JobHandlerNew.FinalUpdate = "Yes"; 

         // ... and everything under else branch ... 
      } 

      if (alertM == "Instances") { 
       jConfirm(message, 'Instances Confirmation?', function (answer) { 
        if (answer) { 
         afterInstances(); 
        } 
       }); 
      } 

重要 - この中を読んで誰かに何か役に立つことを意味し、いくつかの名前を持つようにする方法（afterContinue、afterInstancesを、...）の名前を変更します未来。