Как избежать повторения кода в операторе if / else внутри AJAX

Question

Я пытаюсь сделать if / else внутри 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

Попробуйте тоже правильно,

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!');
                }
            });

Answer 2

Единственное изменение заключается в том, используете ли вы 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!');
    }
  });
}