Question

我正在尝试在Ajax中执行if / else，但是正如您在脚本中看到的那样，我正在重复我的代码，我认为最好将其归档在一行中，或者在Ajax和老实说，我不知道该怎么做？有人可以帮助我或向正确的方向指点我吗？
在此先感谢：）

function RMAfunction() {
        var model = {
            Serienummer: $("#notavailable").val(),
            SelectedSerieText: $("#ddlSerial option:selected").text(),
            Kundenavn: $("#Kundenavn").val(),
            Ordrenummer: $("#Ordrenummer").val()
            }
            if (model.Serienummer === 'not available') {
                $.ajax({
                    type: 'POST',
                    url: '@Url.Action("ProcessRequestRMA", "Account")',
                    dataType: 'json',
                    data: {
                        Serienummer: model.Serienummer,
                        Kundenavn: model.Kundenavn,
                        Ordrenummer: model.Ordrenummer,
                    },
                    success: function (status) {
                        if (status) {

                            status.Serienummer = model.Serienummer;
                            status.Kundenavn = model.Kundenavn;
                            status.Ordrenummer = model.Ordrenummer;
                        }
                        else {
                            alert("Something Wrong");
                        }
                    },
                    error: function () {
                        console.log('something went wrong - debug it!');
                    }
                });
            }
            else {
                    $.ajax({
                    type: 'POST',
                    url: '@Url.Action("ProcessRequestRMA", "Account")',
                    dataType: 'json',
                    data: {
                        Serienummer: model.SelectedSerieText,
                        Kundenavn: model.Kundenavn,
                        Ordrenummer: model.Ordrenummer
                    },
                    success: function (status) {
                        if (status) {
                            status.Serienummer = model.SelectedSerieText;
                            status.Kundenavn = model.Kundenavn;
                            status.Ordrenummer = model.Ordrenummer;     
                        }
                        else {
                            alert("Something Wrong");
                        }
                    },
                    error: function () {
                        console.log('something went wrong - debug it!');
                    }
                });

            }
}

Answer 1

唯一的变化是基于Serienummer的原始值在SelectedSerieText的属性中使用model还是Serienummer。这样，您可以只使用使用该值的三元表达式。试试这个：

function RMAfunction() {
  var model = {
    Serienummer: $("#notavailable").val(),
    SelectedSerieText: $("#ddlSerial option:selected").text(),
    Kundenavn: $("#Kundenavn").val(),
    Ordrenummer: $("#Ordrenummer").val()
  }
  var hasSerieNummer = model.Serienummer === 'not available';

  $.ajax({
    type: 'POST',
    url: '@Url.Action("ProcessRequestRMA", "Account")',
    dataType: 'json',
    data: {
      Serienummer: hasSerieNummer ? model.Serienummer : model.SelectedSerieText,
      Kundenavn: model.Kundenavn,
      Ordrenummer: model.Ordrenummer,
    },
    success: function(status) {
      if (status) {
        status.Serienummer = hasSerieNummer ? model.Serienummer : model.SelectedSerieText;
        status.Kundenavn = model.Kundenavn;
        status.Ordrenummer = model.Ordrenummer;
      } else {
        alert("Something Wrong");
      }
    },
    error: function() {
      console.log('something went wrong - debug it!');
    }
  });
}

通过使用model对象中的函数包含确定返回哪个标识符的逻辑，可以进一步减少此问题：

function RMAfunction() {
  var model = {
    Serienummer: $("#notavailable").val(),
    SelectedSerieText: $("#ddlSerial option:selected").text(),
    Kundenavn: $("#Kundenavn").val(),
    Ordrenummer: $("#Ordrenummer").val(),
    GetIdentifier: function() {
      return this.Serienummer === 'not available' ? this.Serienummer : this.SelectedSerieText;
    }
  }

  $.ajax({
    type: 'POST',
    url: '@Url.Action("ProcessRequestRMA", "Account")',
    dataType: 'json',
    data: {
      Serienummer: model.GetIdentifier(), // usage here
      Kundenavn: model.Kundenavn,
      Ordrenummer: model.Ordrenummer,
    },
    success: function(status) {
      if (status) {
        status.Serienummer = model.GetIdentifier(); // and here
        status.Kundenavn = model.Kundenavn;
        status.Ordrenummer = model.Ordrenummer;
      } else {
        alert("Something Wrong");
      }
    },
    error: function() {
      console.log('something went wrong - debug it!');
    }
  });
}

Answer 2

尝试此方法也正确

var model = {
        Serienummer: $("#notavailable").val(),
        SelectedSerieText: $("#ddlSerial option:selected").text(),
        Kundenavn: $("#Kundenavn").val(),
        Ordrenummer: $("#Ordrenummer").val()
        }
        if (model.Serienummer === 'not available') {
            var data = {
                    Serienummer: model.Serienummer,
                    Kundenavn: model.Kundenavn,
                    Ordrenummer: model.Ordrenummer,
                };
        }else{
            var data = {
                Serienummer: model.SelectedSerieText,
                    Kundenavn: model.Kundenavn,
                    Ordrenummer: model.Ordrenummer
            };
        }
            $.ajax({
                type: 'POST',
                url: '@Url.Action("ProcessRequestRMA", "Account")',
                dataType: 'json',
                data: JSON.stringify(data);
                success: function (status) {
                    if (status) {
                        if (model.Serienummer === 'not available') {
                            status.Serienummer = model.Serienummer;
                        }else{
                            status.Serienummer = model.SelectedSerieText;
                        }
                            status.Kundenavn = model.Kundenavn;
                            status.Ordrenummer = model.Ordrenummer;
                    }
                    else {
                        alert("Something Wrong");
                    }
                },
                error: function () {
                    console.log('something went wrong - debug it!');
                }
            });

如何避免在AJAX中的if / else语句中重复代码

2 个答案: