2010-12-07 2 views
0

Am j Подтвердите подтверждение.jConfirm alert - плагин jQuery

Мой первый 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; 
           } 
          }); 
         } 
        }); 
       } 
      } 
     }); 
    }); 
}); 

Что мне не хватает?

ответ

2

Не уверен, что если это все, но эта часть:

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, ...), чтобы иметь какое-то название, что означает что-то полезное для кого-то читать это в будущее.

+0

Спасибо, icyrock. Можете ли вы показать бит повторного факторинга, разделив их на более мелкие функции. – Sreedhar 2010-12-07 06:29:09