将代码临时编写成更DRY的方法

Improvising Code Into A More DRY Approach?

本文关键字：DRY 方法 代码      更新时间：2023-09-26

我正在格式化函数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;
};

这里是需要第三个参数"category"的地方。

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, 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;
};
相关文章：