Building if condition from array

Question

I have following if condition:

if (!string.IsNullOrEmpty(str) &&
    (!str.ToLower().Contains("getmedia") && !str.ToLower().Contains("cmsscripts") && 
     !str.ToLower().Contains("cmspages") && !str.ToLower().Contains("asmx") &&
     !str.ToLower().Contains("cmsadmincontrols"))
  )

I am trying to create an array of keywords instead of putting multiple AND conditions, could you please help?

string[] excludeUrlKeyword = ConfigurationManager.AppSettings["ExcludeUrlKeyword"].Split(',');

for (int i = 0; i < excludeUrlKeyword.Length; i++)
{
    var sExcludeUrlKeyword = excludeUrlKeyword[i];
}

How to build the same if condition from the array?

Use LinQ Any or All

Oasis
– Oasis

2017-08-03 11:52:18 +00:00
Commented Aug 3, 2017 at 11:52 — Oasis
– Oasis, Commented Aug 3, 2017 at 11:52
can we write without LINQ ?

SmartestVEGA
– SmartestVEGA

2017-08-03 13:19:14 +00:00
Commented Aug 3, 2017 at 13:19 — SmartestVEGA
– SmartestVEGA, Commented Aug 3, 2017 at 13:19

Sergey Kalinichenko · Accepted Answer · 2017-08-03 12:30:17Z

3

You can use LINQ's All or Any method to evaluate a condition on array elements:

// Check for null/empty string, then ...
var lower = str.ToLower();
if (excludeUrlKeyword.All(kw => !lower.Contains(kw))) {
    ...
}

Note that this is not the fastest approach: you would be better off with a regex. As an added bonus, regex would prevent "aliasing", when you discard a string with a keyword appearing as part of a longer word.

If you would like to try regex approach, change ExcludeUrlKeyword in the config file from comma-separated getmedia,cmsscripts,cmspages,asmx to pipe-separated getmedia|cmsscripts|cmspages|asmx, so that you could feed it directly to regex:

var excludeUrlRegex = ConfigurationManager.AppSettings["ExcludeUrlKeyword"];
if (!Regex.IsMatch(str.ToLower(), excludeUrlRegex)) {
    ...
}

edited Aug 3, 2017 at 12:30

answered Aug 3, 2017 at 11:46

Sergey Kalinichenko

729k85 gold badges1.2k silver badges1.6k bronze badges

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

Comments

fubo · Accepted Answer · 2017-08-03 11:46:27Z

1

Linq Any should do this

if (!string.IsNullOrEmpty(str) && !excludeUrlKeyword.Any(x => str.ToLower().Contains(x)))

answered Aug 3, 2017 at 11:46

fubo

46.2k20 gold badges109 silver badges139 bronze badges

Collectives™ on Stack Overflow

Building if condition from array

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related