Efficient method of searching for multiple values in a variable using indexOf()?

Question

I am building a condition script that searches a comma-separated list of email recipients. There can be multiple recipients and the condition script searches for three (3) specific addresses.

This is not a full-fledged script - it can only include a conditional statement. Our issue is that this field has a character limit of 255... and we have reached it.

Here is the current condition:

email.recipients.toLowerCase().indexOf('email_1') >= 0 || email.recipients.toLowerCase().indexOf('email_2') >= 0 || email.recipients.toLowerCase().indexOf('email_3') >= 0

This is within our character limit - and works - but now we need to add another email address to search for. Is there any way of shortening this condition?

Can't you use regexps??? Should it be on one line (one statement)? — ibrahim mahrir
– ibrahim mahrir, Commented Feb 10, 2017 at 15:23
This is not a full-fledged script - it can only include a conditional statement. - can you be more specific about what this means? What is permitted, precisely? — patstuart
– patstuart, Commented Feb 10, 2017 at 15:24

patstuart · Accepted Answer · 2017-02-10 16:18:34Z

1

If you can use ES6 (you don't need to worry about Internet Explorer), you can use an arrow function:

['email_1','email_2','email_3'].some(e=>email.recipients.toLowerCase().indexOf(e)+1);

edited Feb 10, 2017 at 16:18

answered Feb 10, 2017 at 15:30

patstuart

1,9901 gold badge20 silver badges30 bronze badges

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

Comments

Bergi · Accepted Answer · 2017-02-10 15:34:06Z

0

Depending on the particular email addresses, a regex solution might be simpler:

/email_[1-3]/i.test(email.recipients)

answered Feb 10, 2017 at 15:34

Bergi

671k162 gold badges1k silver badges1.5k bronze badges

Comments

Kaddath · Accepted Answer · 2017-02-10 15:34:33Z

0

Not the perfect answer of course, but you can shorten a little bit with using an alias (be careful about var conflicts, because you cannot use the var keyword inside this expression, meaning the var will be global, it also might not work in strict mode i think).

Just made an answer because it was too long for a comment.. hope ite helps.

var email = {}; email.recipients = 'email_1,email_2,email_3';
console.log((a = email.recipients.toLowerCase()) && (a.indexOf('email_1') >= 0 || a.indexOf('email_2') >= 0 || a.indexOf('email_3') >= 0));

answered Feb 10, 2017 at 15:34

Kaddath

6,1511 gold badge11 silver badges24 bronze badges

Comments

BenG · Accepted Answer · 2017-02-10 15:57:56Z

0

use some:-

['email_1', 'email_2', 'email_3'].some(function(e) {
  return email.recipients.toLowerCase().indexOf(e) >= 0;
});

edited Feb 10, 2017 at 15:57

answered Feb 10, 2017 at 15:27

BenG

15.2k5 gold badges48 silver badges62 bronze badges

Collectives™ on Stack Overflow

Efficient method of searching for multiple values in a variable using indexOf()?

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related