C# "or" operator

Question

I have this code:

if (textBox1.Text == "one" || "two")

I have tried to use || and | to add more strings, but it says that it cannot be applied to operands of type "bool" and "string". How can I make this work? Thank you.

what are the diferent cases? – Giannis Paraskevopoulos Jul 25 '13 at 13:40 — Giannis Paraskevopoulos, Jul 25 '13 at 13:40

score 6 · Answer 1 · answered Jul 25 '13 at 13:40

6

Try this

if (textBox1.Text == "one" || textBox1.Text == "two")

answered Jul 25 '13 at 13:40

Sriram Sakthivel

72,067
7
111
189

@SandroMilhano OP -- my guess is you were doing `textBox1.Text == "one" || "two"`. You can't do that -- the left expression will get evaluated to a boolean, then you're left trying to evaluate `boolean || "two"`, resulting in the error you mentioned. – tnw Jul 25 '13 at 13:51
@tnw Exactly. that's what me too suspect! – Sriram Sakthivel Jul 25 '13 at 13:53
Yes I was trying to do that :) – Sandro Milhano Jul 25 '13 at 14:23

score 5 · Accepted Answer · answered Jul 25 '13 at 13:41

5

or alternatively :

var strings = new List<string>() {"one", "two", "thee", .... "n"};
if(strings.Contains(textBox1.Text)){
}

answered Jul 25 '13 at 13:41

Tigran

61,654
8
86
123

Thanks! That's even better for what I need. – Sandro Milhano Jul 25 '13 at 13:44
Can you tell me why this is failing: `(c.Contains('All') || c.Contains('')) ? "[ClientText] LIKE '%'" : "[ClientText] = '" + c + "'";` cannot convert string to bool error. – SearchForKnowledge Mar 12 '15 at 20:15

score 5 · Answer 3 · edited May 23 '17 at 12:24

5

You can't combine operators in the way I suspect you are trying:

if (textBox1.Text == "one" || "two")

You need to qualify each condition as follows:

if (textBox1.Text == "one" || textBox1.Text == "two")

There are ways to make this easier to do, see answers to this question for an alternative way to do it

edited May 23 '17 at 12:24

Community

1
1

answered Jul 25 '13 at 13:42

Trevor Pilley

16,156
5
44
60

score 0 · Answer 4 · answered Jul 25 '13 at 13:41

0

I'd recommend to use:

var options = new [] { "one", "two" };
if (options.Contain(textBox1.Text))
    ...

answered Jul 25 '13 at 13:41

D.R.

20,268
21
102
205

C# "or" operator

4 Answers4