Question

我在函数 inputCategories 中格式化数组，并且无法正确添加“category”的第三个参数 - 强制我多次复制该函数。

以下是当前状态：

使用参数调用函数。

$scope.categories = inputCategories($scope.item.categories, $scope.settings.categories);

function inputCategories (input, settings) {
    var updatedSettings = [];
    angular.forEach(input, function(obj) {
        updatedSettings.push({"category": obj, "ticked": true});
    });
    var list = updatedSettings.concat(settings);
    list.sort(function(a, b) {
        return (a.category > b.category) - (a.category < b.category);
    });
    for ( var i = 1; i < list.length; i++ ){
        if(list[i-1].category == list[i].category) {
            list.splice(i,1);
        }
    }
    return list;
};

以下是需要第三个“类别”参数的地方。

function inputCategories (input, settings) {
    var updatedSettings = [];
    angular.forEach(input, function(obj) {
        updatedSettings.push({****"category"****: obj, "ticked": true});
    });
    var list = updatedSettings.concat(settings);
    list.sort(function(a, b) {
        return (a.****category**** > b.****category****) - (a.****category**** < b.****category****);
    });
    for ( var i = 1; i < list.length; i++ ){
        if(list[i-1].****category**** == list[i].****category****) {
            list.splice(i,1);
        }
    }
    return list;
};

我认为我遇到的问题是因为我混合了字符串和变量，这是一个字符串，在第四行的对象内...？

Answer 1

也许你可以这样做：

function inputCategories (input, settings, category) {
    var updatedSettings = [];
    angular.forEach(input, function(obj) {
        var setting = { "ticked": true };
        setting[category] = obj;
        updatedSettings.push(setting);
    });
    var list = updatedSettings.concat(settings);
    list.sort(function(a, b) {
        return (a[category] > b[category]) - (a[category] < b[category]);
    });
    for ( var i = 1; i < list.length; i++ ){
        if(list[i-1][category] == list[i][category]) {
            list.splice(i,1);
        }
    }
    return list;
};

将代码简化为更干燥的方法？

1 个答案: