javascript - 将对象属性作为函数参数传递？

Question

关闭。这个问题是 opinion-based 。目前不接受答案。

---

想要改进这个问题吗？更新问题，以便 editing this post 可以用事实和引文来回答它。 .

已关闭 6 年前。

Improve this question

我有这三个函数，它们基本上做同样的事情。他们获取来自 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']])

    })
}

这就是我的想法，但如果有人有更好的想法，我会开放。

Answer 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 函数，但您也可以不使用它们。