Question

我在NodeJS中有这个函数，我在其中过滤元素：

// Remove duplicates and expired
var data = (result[0].concat(result[1]).concat(result[3])).filter((event, index, self) => self.findIndex((t) => {
                    var eventDate = new Date(event.start_time);
                    return t.id === event.id && todayDate.setHours(0,0,0,0) <= eventDate.setHours(0,0,0,0);
                }) === index)

                // Remove invalid
                .filter(function(event){
                    if(~this.indexOf(event.id)){
                        return false;
                    } else {
                        return true;
                    }
                }, result[2])

                // Sort array by date
                .sort(function(a,b) {
                    return new Date(a.start_time).getTime() - new Date(b.start_time).getTime();
                })

                // Filter promoted
                // PROBLEM
                .filter(function(event){
                    if(~this.indexOf(event.id)){
                        event.promoted = true;
                        console.log(filtered.indexOf(event));
                        // MOVE THIS TO TOP
                    }
                    return true;
                }, result[4]);

现在，在最后一个过滤方法中，我从lis（结果[4]）中找到带有ID的元素并添加属性＆＃34;提升＆＃34;。

我还希望将所有提升的事件立即移到数组的顶部，因为我的数组中的索引而不是为此作业启动另一个循环。

我该怎么做？因为使用拼接对我来说没有用

还有关于此功能质量的任何建议吗？

谢谢：）

Answer 1

您不应使用filter来更改数组元素。对此有map。

如果你想根据多个属性对元素进行排序，你可以这样做。

//clean
const invalid = result[2],
    promoted = result[4];

var data = result[0]
   .concat(result[1], result[3])
   .filter(dedup) //remove duplicated
   //set promoted flag
   .map(event => {
       event.promoted = promoted.indexOf(event.id) > -1
       return event
   })
   //sort by promoted then by start_time  
   .sort((a, b) => (b.promoted - a.promoted) || (a.start_time - b.start_time))

将元素数组位置更改为顶部

1 个答案: