将对象属性作为函数参数传递？ [关闭]答案

【问题标题】：Passing object properties as function argument? [closed]将对象属性作为函数参数传递？ [关闭]
【发布时间】：2016-10-20 22:38:58
【问题描述】：

我有这三个基本上做同样事情的函数。他们从 Promise 中获取数据，并为每个函数创建一个具有相同属性的对象。

我想把它晒干：

var getFiles = function() {
    return wg.getFiles().then(function(data) {
        processData(data, {
            type: "File",
            title: x.filename,
            created: x.Created,
            path: x.path
        })
    })
}

var getEvents = function() {
    return wg.getEvents().then(function(data) {
        processData(data, {
            type: "Event",
            title: x.Title,
            created: x.Created,
            path: x.path
        })
    })
}

var getFeedback = function() {
    return wg.getFeedback().then(function(data) {
        processData(data, {
            type: "Review",
            title: "by " + x.Author,
            created: x.Created,
            path: x.path
        })
    })
}
var processData = function(data, props) {
    var x = _(data)
    .map(function(x) {return props})
    .value()
.map(function(x) {
    activties.push(x)
})
}

我想通过将processData 函数更改为类似这样的方式来解决这个问题：

var processData = function(data, props) {
    var x = _(data)
    .map(function(x) {
        return {
            type: x[props[0]],
            title: x[props[1]],
            created: x[props[3]],
            path: "/" + x[props[4]]
        }
    })
    .value()
.map(function(x) {
    activties.push(x)
})
}

那么我可以这样称呼它：

var getFiles = function() {
    return wg.getFiles().then(function(data) {
        processData(data, ['filename', 'created', ['FileRef']['lookupValue']])

    })
}

这就是我的想法，但如果有人有更好的建议，我很乐意。

【问题讨论】：

出于好奇，您认为您当前的方法有问题吗？当然，processData() 方法看起来有点难看，但为了让其他所有内容都可读，我会说它很好。
您可以颠倒您的 processData fn 中的参数顺序。从函数（数据、道具）到函数（道具、数据），然后根据需要从部分应用程序中受益。
@Archer 我不会说我认为有什么问题，但如果可能或足够简单来完成，我确实喜欢保持我的代码干燥。如果试图把它干掉会导致额外的 20 行代码，那么是的，这不值得付出努力。但如果它足够简单，我想学习如何做。
请注意我投票结束，因为这主要是基于意见的，我想说你所拥有的很好。
只是好奇，x 是什么，activities 是什么？你为什么不return 来自承诺的processData(…) 的结果？您当前的代码似乎无法正常工作。

标签： javascript jquery object dry

【解决方案1】：

不要传递属性名，传递函数：

function getFiles() {
    return wg.getFiles().then(processData("File", _.property("filename")));
}

function getEvents() {
    return wg.getEvents().then(processData("Event", _.property("Title")));
}

function getFeedback() {
    return wg.getFeedback().then(processData("Review", function(x) { return "by " + x.Author; })));
}

function processData(type, makeTitle) {
    return function(data) {
        return data.map(function(x) {
            return {
                type: type,
                title: makeTitle(x),
                created: x.Created,
                path: x.path
            };
        });
    };
}

Promise.all([getFiles(), getEvents(), getFeedback()])
.then(function(arrays) {
    var activities = _.flatten(arrays);
    …
});

为方便起见，我使用了一些下划线/lodash 函数，但您也可以不使用它们。

【讨论】：

我注意到，通过清理从 wg 对象返回的属性并使它们保持一致，它使整个问题变得更简单。
是的，但解决问题的方法保持不变 :-) 如果不清理它们，您只需对每个差异使用一个回调函数。
我不太了解_.prop() 和makeTitle(x) 以及它们今天的工作方式。我正在尝试查找它的文档，但我什么也没看到。
@Batman 哎呀，我把它和R.prop 混淆了，我的意思是_.property。 makeTitle 只是作为参数传递的函数，就像 getFeedback 中的 function (x) { return … } 表达式
@Batman $.when 不会创建 arrays 数组，而是创建 passes multiple arguments。所以要么使用_.flatten(arguments)，要么直接写到$.when(getFiles(), getEvents(), getFeedback()).then(function(files, events, feedbacks) { var activities = files.concat(events, feedbacks); })