Why not use more readable code? In my opinion, it is much more convenient to maintain.
private Boolean HasValidEnding(String input) { if (input.EndsWith("SEC",StringComparison.Ordinal) || input.EndsWith("PRI",StringComparison.Ordinal)) { if (!input.Equals("SIGSEC",StringComparison.Ordinal)) { return true; } } return false; }
or in one line
private Boolean HasValidEnding(String input) { return (input.EndsWith("SEC",StringComparison.Ordinal) || input.EndsWith("PRI",StringComparison.Ordinal)) && !input.Equals("SIGSEC",StringComparison.Ordinal); }
Not that I did not use regular expressions, but in this case I would not use them.
source share