jConfirm alert - jQuery-plugin

Bin jConfirm für Benutzer Bestätigung.

Meine erste jConfirm doesnt stop für die Aktion des Benutzers und passt einfach 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?

InformationsquelleAutor Sreedhar | 2010-12-07
  1. 2

    Nicht sicher, ob dies alle, aber dieses Teil:

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

    wahrscheinlich nicht tun, was Sie wollen. Es beendet die function(ans) { ... } - Funktion, während Sie wahrscheinlich wollen, um zu beenden das ganze-handler, d.h. $("#UpdateJobHandler").click(function () { ... }. Wenn ja, würden Sie brauchen, zu tun, ähnlich zu dem, was Sie tun unten - d.h. setzen Sie das ganze in function(ans) { ... }, nach der Rückkehr. Wahrscheinlich am besten zu trennen, in kleinere Funktionen.

    EDIT: Etwas entlang diesen Linien:

        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 tun, etwas ähnliches für alle success Funktionen.

    Weiteres Beispiel, schreiben Sie Instances prüfen, wie diese:

                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, ...) haben einige Namen, die bedeutet, etwas sinnvolles zu jemand, der dieses liest, in die Zukunft.

    • Dank icyrock,Können Sie bitte zeigen Sie etwas von der Probe re-factoring dieses durch Aufteilung in kleinere Funktionen.
    • Siehe den edit.
    InformationsquelleAutor icyrock.com
Schreibe einen Kommentar