1 
 renderControl: function () {
        var that = this;
        var _CountryCol = new CountryCol();
        var _ComboCol = new ComboCol();

        _CountryCol.fetch({
            success: function (data) {
                that.$("#ctr").select2({
                    placeholder: "Select a Country",
                    allowClear: true,
                    data: JSON.parse(JSON.stringify(data)),
                });
            }
        });
        _ComboCol.fetch({
            data: { id: 'status' },
            success: function (data) {
                that.$("#sta").select2({
                    placeholder: "Select a status",
                    allowClear: true,
                    data: JSON.parse(JSON.stringify(data)),
                });
            }
        });
        _ComboCol.fetch({
            data: { id: 'marital' },
            success: function (data) {
                that.$("#mrt").select2({
                    placeholder: "Select a marital",
                    allowClear: true,
                    data: JSON.parse(JSON.stringify(data)),
                });
            }
        });
        _ComboCol.fetch({
            data: { id: 'daytype' },
            success: function (data) {
                that.$("#dty").select2({
                    placeholder: "Select a type",
                    allowClear: true,
                    data: JSON.parse(JSON.stringify(data)),
                });
            }
        });
        this.$("#hpn, #hmn, #icn").numeric({ decimal: false, negative: false });
        this.$('#dob').datepicker({
            dateFormat: 'dd/mm/yy'
        });
    },

哪里可以缩短？代码运行良好，只是调用多个ajax看起来麻烦繁琐，代码看起来也不太优雅，如果有人对此有更好的想法，可以分享主干代码的方式，这样更容易管理，谢谢

更新1：

通过参考 Marc 的想法，我进一步减少了代码，但不确定这是否是正确的方法，任何反馈都会很棒

       renderControl: function () { //use to render special control
            var that = this;
            var _CountryCol = new CountryCol();
            var _ComboCol = new ComboCol();
            $.when(
                _CountryCol.fetch(),
                _ComboCol.fetch({ data: { id: 'status' } }),
                _ComboCol.fetch({ data: { id: 'marital' } }),
                _ComboCol.fetch({ data: { id: 'daytype' } })).done(
                function (country, status, marital, daytype) {
                    that.populateSelect('#ctr', "Select a country", country);
                    that.populateSelect("#sta", "Select a status", status);
                    that.populateSelect("#mrt", "Select a marital", marital);
                    that.populateSelect("#dty", "Select a type", daytype); 
            });

            this.$("#hpn, #hmn, #icn").numeric({ decimal: false, negative: false });
            this.$('#dob').datepicker({
                dateFormat: 'dd/mm/yy'
            });
        },
        populateSelect: function (selector, placeholder, collection) {
            debugger;
            this.$(selector).select2({
                placeholder: placeholder,
                allowClear: true,
                data: JSON.parse(JSON.stringify(collection[0]))
            });
        },
4

2 回答 2

2

正如@mike 所说，使用监听器。

为此，您需要在 fetch 之前添加代码，例如：

_CountryCol.on('sync', function(collection) {
  that.$("#ctr").select2({
    placeholder: "Select a Country",
    allowClear: true,
    data: JSON.stringify(collection.toJSON())
  });
});
_CountryCol.fetch()

虽然这个解决方案只是将代码从一个地方移动到另一个地方，并没有真正削减任何代码。将重复的代码放入函数中会：

populateSelect = function(selector, placeholder, collection) {
  that.$(selector).select2({
    placeholder: placeholder,
    allowClear: true,
    data: JSON.stringify(collection.toJSON())
  });
}

_CountryCol.on('sync', function(collection) {
  populateSelect("#ctr", "Select a Country", collection);
});
_CountryCol.fetch();

_ComboCol.on('sync', function(collection) {
  populateSelect("#sta", "Select a status", collection)
});
_ComboCol.fetch({data: {id: 'status'}});
于 2013-06-20T09:47:19.823 回答
1

不确定这里的整体情况，但通常在 Backbone 中，最好使用集合上的侦听器处理对获取的响应，而不是fetch方法本身的回调。这样，您可能只需要编写一次回调代码。

于 2013-06-20T09:10:57.583 回答