Using indexOf to filter an array

Question

I am trying to output the first two objects in the events array using indexOf.

This doesn't return anything:

var whiteList=['css','js'];

var events =[
    {file: 'css/style.css', type:'css'},
    {file: 'js/app.js', type:'js'},
    {file: 'index/html.html', type:'html'}
];

var fileList= events
    .filter(function(event){
    return event.type.indexOf(whiteList) >- 1 
  })

console.log(fileList);

If I change the function like this, it returns the css and js object, although I expected it to return the html object.

var fileList= events
    .filter(function(event){
    return event.type.indexOf('html') 
  })

event.type.indexOf('html') returns 0 (the position of the 'h') - 0 is interpreted as false. — J. Titus
– J. Titus, Commented Oct 10, 2016 at 20:23

Nenad Vracar · Accepted Answer · 2016-10-10 20:23:40Z

15

You are doing it wrong, it should go like this.

var whiteList = ['css', 'js'];

var events = [{
  file: 'css/style.css',
  type: 'css'
}, {
  file: 'js/app.js',
  type: 'js'
}, {
  file: 'index/html.html',
  type: 'html'
}];

var fileList = events.filter(function(event) {
  return whiteList.indexOf(event.type) > -1
})

console.log(fileList)

answered Oct 10, 2016 at 20:23

Nenad Vracar

122k16 gold badges160 silver badges184 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Chef1075 Over a year ago

This is perfect, and I was able to use it for another piece of code I was writing. Thanks!

Nina Scholz · Accepted Answer · 2016-10-10 20:39:03Z

7

With ES6, you could use Set for faster access with larger data sets.

var whiteList = ['css', 'js'],
    whiteSet = new Set(whiteList),
    events = [{ file: 'css/style.css', type: 'css' }, { file: 'js/app.js', type: 'js' }, { file: 'index/html.html', type: 'html' }],
    fileList = events.filter(event => whiteSet.has(event.type));

console.log(fileList);

answered Oct 10, 2016 at 20:39

Nina Scholz

388k26 gold badges367 silver badges417 bronze badges

Collectives™ on Stack Overflow

Using indexOf to filter an array

2 Answers 2

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related