jConfirm alert - jQuery-Plugin

Question

Am jConfirm Benutzer zur Bestätigung.

Mein erster jConfirm doesnt Anschlag Handlungs Benutzer und geht nur zum nächsten.

Mein Code:

    $(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;
                                }
                            });
                        }
                    });
                }
            }
        });
    });
});

Was bin ich?

Answer 1

Nicht sicher, ob das alles ist, aber dieser Teil:

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

macht wahrscheinlich nicht das, was Sie wollen. Es verläßt die function(ans) { ... } Funktion, während Sie wahrscheinlich den ganzen Handler, das heißt $("#UpdateJobHandler").click(function () { ... } verlassen mögen. Wenn ja, würden Sie brauchen, um ähnlich zu tun, was Sie tun unten - das heißt setzen das Ganze in function(ans) { ... }, nach der Rückkehr. Wahrscheinlich in kleinere Funktionen am besten trennen.

EDIT: Etwas in diese Richtung:

    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();
                }
            });
    }

Sie können etwas Ähnliches tun für alle success Funktionen.

Ein weiteres Beispiel, können Sie die Instances Prüfung wie diese neu schreiben:

            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();
                    }
                });
            }

Wichtig -. Benennen Sie die Methoden (afterContinue, afterInstances, ...) einen Namen zu haben, das bedeutet, etwas Nützliches zu jemandem dies in der Zukunft zu lesen