slicker way of writing IF statement

Question

can you please advise me on a slicker way of saying:

if (PK.ToString() == "id" || "Id" || "ID" || "iD"))
{

}

as this throws a || operator cant be applied to instances of bool and string

finished asking the question but cant post because Q does not meet requirements please ignore this attempt to fill white space

@Ramanlfc Why? This is clearly a case insensitive comparison that the OP wants. — juharr, Nov 05 '15 at 15:55
Use String.toUpper - https://msdn.microsoft.com/en-us/library/ewdd6aed(v=vs.110).aspx — rghome, Nov 05 '15 at 15:56
`if (PK.ToString().Equals("id", StringComparison.OrdinalIgnoreCase)) {...}` — Dmitry Bychenko, Nov 05 '15 at 15:57

juharr · Answer 1 · 2015-11-05T16:06:32.883

2

Use the string.Compare overload that lets you specify that you want case insensitive comparison

if(string.Compare(PK.ToString(), "ID", true) == 0)

Other options include converting to upper or lower case

if(PK.ToString().ToLower() == "id")

and

if(PK.ToString().ToUpper() == "id")

But you need to be aware of the culture because translating to upper or lower case doesn't always have the expected results depending on the culture. In which case you might find it better to use ordinal or the invariant culture.

if(PK.ToString().Equals("ID", StringComparison.OrdinalIgnoreCase))

or

if(PK.ToString().Equals("ID", StringComparison.InvariantCultureIgnoreCase))

edited Nov 05 '15 at 16:06

answered Nov 05 '15 at 15:54

juharr

31,741
4
58
93

this is flagging condition must be of type bool – John Nov 05 '15 at 15:56
@John Yeah, forgot to compare it to 0. Fixed. – juharr Nov 05 '15 at 15:58

ergonaut · Answer 2 · 2015-11-05T16:00:48.520

1

Just convert it to lowercase:

PK.ToString().toLowerCase().Equals("id")

edited Nov 05 '15 at 16:00

answered Nov 05 '15 at 15:55

ergonaut

6,929
1
17
47

[This may cause issues](http://stackoverflow.com/a/1660232/1324033) – Sayse Nov 05 '15 at 15:59

slicker way of writing IF statement

2 Answers2