switch Case Question

[Pedro]

Hi all,

Why this does'nt work:

switch(strExtenscao)
{
case ((".odo") || (".odo").ToUpper()):
//CODE
case ((".odb") || (".odb").ToUpper()):
//CODE
}

I will be gratful, probably is easy to solve. But I'm new with this language.

--
Thanks ,

Pedro

[Kevin Spencer]

switch(strExtenscao)
{
case ".odo":
case ".ODO":
//CODE
break;
case ".odb":
case ".ODB":
//CODE
break;
}

--
HTH,

Kevin Spencer
Microsoft MVP
..Net Developer
You can lead a fish to a bicycle,
but it takes a very long time,
and the bicycle has to *want* to change.

"Pedro" <Pe***@discussions.microsoft.com> wrote in message
news:60**********************************@microsof t.com...

Hi all,

Why this does'nt work:

switch(strExtenscao)
{
case ((".odo") || (".odo").ToUpper()):
//CODE
case ((".odb") || (".odb").ToUpper()):
//CODE
}

I will be gratful, probably is easy to solve. But I'm new with this
language.

--
Thanks ,

Pedro

[Jon Skeet [C# MVP]]

Pedro <Pe***@discussions.microsoft.com> wrote:

Hi all,

Why this does'nt work:

switch(strExtenscao)
{
case ((".odo") || (".odo").ToUpper()):
//CODE
case ((".odb") || (".odb").ToUpper()):
//CODE
}

I will be gratful, probably is easy to solve. But I'm new with this
language.

The "cases" have to be constants. In this case, you could either write:

switch (strExtenscao)
{
case ".odo":
case ".ODO":
...
break;
case ".odb":
case ".ODB":
...
break;
}

or:

switch (strExtenscao.ToUpper())
{
case ".ODO":
...
break;
case ".ODB":
...
break;
}

Note that the latter version covers cases like ".odO" which the first
doesn't.

--
Jon Skeet - <sk***@pobox.com>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too

[D. Valdez]

What about?

switch (strExtenscao.ToUpper())
{
case ".ODO":
...
break;
case ".ODB":
...
break;
}

[Pedro]

Geat is work!

--
Thanks all,

Pedro
"D. Valdez" wrote:

What about?

switch (strExtenscao.ToUpper())
{
case ".ODO":
...
break;
case ".ODB":
...
break;
}

[Jon Skeet [C# MVP]]

D. Valdez <dv*****@oslaw.com> wrote:

What about?

switch (strExtenscao.ToUpper())
{
case ".ODO":
...
break;
case ".ODB":
...
break;
}

Um, that's what I posted, isn't it?

--
Jon Skeet - <sk***@pobox.com>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too

[Pedro]

Hi all,

Yours solutions was helpful.
--
Thanks ,

Pedro
"Jon Skeet [C# MVP]" wrote:

D. Valdez <dv*****@oslaw.com> wrote:

What about?

switch (strExtenscao.ToUpper())
{
case ".ODO":
...
break;
case ".ODB":
...
break;
}

Um, that's what I posted, isn't it?

--
Jon Skeet - <sk***@pobox.com>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too

[Ken Wilson]

On Thu, 29 Dec 2005 21:55:36 -0000, Jon Skeet [C# MVP]
<sk***@pobox.com> wrote:

D. Valdez <dv*****@oslaw.com> wrote:

What about?

switch (strExtenscao.ToUpper())
{
case ".ODO":
...
break;
case ".ODB":
...
break;
}

Um, that's what I posted, isn't it?

No. Your solution did not solve the problem of a mixed-case value. It
assumed the value would be only uppercase or lowercase.

Ken Wilson
Seeking viable employment in Victoria, BC

[Bill Butler]

"Ken Wilson" <kw*********@NOshaw.SPAMca> wrote in message
news:lp********************************@4ax.com...

On Thu, 29 Dec 2005 21:55:36 -0000, Jon Skeet [C# MVP]
<sk***@pobox.com> wrote:

D. Valdez <dv*****@oslaw.com> wrote: <snip>Um, that's what I posted, isn't it?

No. Your solution did not solve the problem of a mixed-case value. It
assumed the value would be only uppercase or lowercase.

I suggest you re-read Jon's Post
************************************************** ****
The "cases" have to be constants. In this case, you could either write:

switch (strExtenscao)
{
case ".odo":
case ".ODO":
...
break;
case ".odb":
case ".ODB":
...
break;
}

or:

switch (strExtenscao.ToUpper())
{
case ".ODO":
...
break;
case ".ODB":
...
break;
}

Note that the latter version covers cases like ".odO" which the first
doesn't.
************************************************** ******

Looks like the second version covers it to me

Bill

[Jon Skeet [C# MVP]]

Bill Butler <qw****@asdf.com> wrote:

<snip>

Looks like the second version covers it to me

In particular, the second version looks to me to be letter-for-letter
what D. Valdez posted :) (Even down to the indentation of the cases by
three characters instead of four...)

--
Jon Skeet - <sk***@pobox.com>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too

[Ken Wilson]

On Thu, 5 Jan 2006 03:57:52 -0000, Jon Skeet [C# MVP]
<sk***@pobox.com> wrote:

Bill Butler <qw****@asdf.com> wrote:

<snip>

Looks like the second version covers it to me

In particular, the second version looks to me to be letter-for-letter
what D. Valdez posted :) (Even down to the indentation of the cases by
three characters instead of four...)

My apologies Jon. I missed the second option that was part of your
post. You are, indeed, correct and take care of my concern for mixed
case.

Ken Wilson
Seeking viable employment in Victoria, BC