I am trying to loop through an array and filter out all the items that do not match specific values.

For example I have this array:

const emails = ["[email protected]", "[email protected]", "[email protected]", "[email protected]", "[email protected]", "[email protected]"];

I would like to filter out emails that end in"

*@hotmail *@gmail

---

I have given it a go and got this but this doesn't work:

const filtered = emails.filter((email) => {
  return !email.includes('@hotmail') || !email.includes('@gmail');
});

---

The preferred output from the example above would be:

["[email protected]", "[email protected]", "[email protected]"]

I am trying to loop through an array and filter out all the items that do not match specific values.

For example I have this array:

const emails = ["[email protected]", "[email protected]", "[email protected]", "[email protected]", "[email protected]", "[email protected]"];

I would like to filter out emails that end in"

*@hotmail. *@gmail.

---

I have given it a go and got this but this doesn't work:

const filtered = emails.filter((email) => {
  return !email.includes('@hotmail.') || !email.includes('@gmail.');
});

---

The preferred output from the example above would be:

["[email protected]", "[email protected]", "[email protected]"]

• javascript
• arrays

Share Improve this question Follow asked Jan 27, 2017 at 12:58 user3180997user3180997 1,92633 gold badges2121 silver badges3737 bronze badges 1

• What about return !(email.endsWith('@hotmail.') || email.endsWith('@gmail.'))? – sehrob Commented Jan 27, 2017 at 13:00

Add a ment  | 

1 Answer 1

Sorted by: Reset to default 10

Replace the or || with an and &&

If you select all emails which do not contain hotmail OR do not contain gmail, you'll get all of them which don't contain both which isn't your objective.
You want to get all of those that do not contain both hotmail and gmail instead!

const filtered = emails.filter((email) => {
  return !email.includes('@hotmail.') && !email.includes('@gmail.');
});