If (something) or (somethingElse) problem 2

[wokkie]

Hi,

I am having trouble using OR in an If loop. I have

if (cboCeaseType.SelectedValue.ToString () == "2") or (cboCeaseType.SelectedValue.ToString () == "5" ){
do something
} else {

do something else
}

It doesn't like this. How do I have an OR in an If loop? What should the above be?

Cheers

Ian

 

[Rhys666]

c# is more java-like than vb-like and uses java-like operators such as && (VB 'And') and || (VB 'Or').


Rhys
"When one burns one's bridges, what a very nice fire it makes" -- Dylan Thomas
"As to marriage or celibacy, let a man take the course he will. He will be sure to repent" -- Socrates

 

[wokkie]

Thanks, I have being using VBA for years and often get mixed up!

Thanks again

 

[chiph]

You'll also need to add an additional set of parenthesis, since the if statement wants them around the whole expression:

if ((cboCeaseType.SelectedValue.ToString() == "2") ||
    (cboCeaseType.SelectedValue.ToString() == "5"))
{

Another suggestion would be to put the constant part of your comparison on the left side of the comparison operator. That way if you leave one of the equals signs off, you'll get a "Can't assign value to constant" error. Looks a little odd, but works because the comparison operator is reflexive.

if (("2" == cboCeaseType.SelectedValue.ToString()) ||
    ("5" == cboCeaseType.SelectedValue.ToString()))
{

Chip H.



If you want to get the best response to a question, please check out FAQ222-2244 first