Alternative to this code?

[Dave Thomas]

I know there is a better way to do something like this in PHP:

if (($color == "red") || ($color == "blue") || ($color == "white") ||
($color == "yellow")) { .. }

Is there a way I can condense that so I don't have to write $color
everytime?

[]

$colors = array(
'red',
'blue',
'white',
'yellow
);
if (in_array($colo r, $colors))
{
//...
}
"Dave Thomas" <ge****@hotmail .com> wrote in message
news:1d******** ************@ad elphia.com...
|I know there is a better way to do something like this in PHP:
|
| if (($color == "red") || ($color == "blue") || ($color == "white") ||
| ($color == "yellow")) { .. }
|
| Is there a way I can condense that so I don't have to write $color
| everytime?

[Colin McKinnon]

Dave Thomas wrote:

I know there is a better way to do something like this in PHP:

if (($color == "red") || ($color == "blue") || ($color == "white") ||
($color == "yellow")) { .. }

Is there a way I can condense that so I don't have to write $color
everytime?

switch ($color) {
case 'red':
case 'blue':
case 'yellow':
case 'white':
print "yes, its one on those\n";
break;
case 'black':
default:
print "No, its something else\n";
break;
}

HTH

C.

[kholm]

Yet another method:

$pattern = '/^(red|blue|whit e|yellow)$/';

if (preg_match($pa ttern, $color)) {
...
}

[drwxr-xr-x]

On 15 Aug 2005 13:40:06 -0700, kh***@dis.umsme d.edu wrote:

Yet another method:

$pattern = '/^(red|blue|whit e|yellow)$/';

if (preg_match($pa ttern, $color)) {
...
}

Just _what_ are you nattering on about?
Oh, I see the problem:
X-Trace: posting.google. com

Straighten up and fly right:
http://groups.google.com/googlegroup...html#summarize
http://oakroadsystems.com/genl/unice.htm#quote

[Cristian Gutierrez]

Hoy en la tarde, Dave Thomas dijo:

I know there is a better way to do something like this in PHP:

if (($color == "red") || ($color == "blue") || ($color == "white") ||
($color == "yellow")) { .. }

Is there a way I can condense that so I don't have to write $color
everytime?

,----
| // can also be loaded from somewhere else, say, a db or file
| $available_colo rs = array('red', 'blue', 'white', 'yellow');
|
| if (in_array($colo r, $available_colo rs)) {
| ...
| }
`----

--
Cristian Gutierrez /* cr******@dcc.uc hile.cl */
"Vampirewar e: /n/, a project, capable of sucking the lifeblood out of
anyone unfortunate enough to be assigned to it, which never actually
sees the light of day, but nonetheless refuses to die."

[Ken Robinson]

Dave Thomas <ge****@hotmail .com> wrote in news:1dqdnQiykp 9dSJ3eRVn-
[e-mail]:

I know there is a better way to do something like this in PHP:

if (($color == "red") || ($color == "blue") || ($color == "white") ||
($color == "yellow")) { .. }

Is there a way I can condense that so I don't have to write $color
everytime?

You could use a switch statement:
<?php
switch ($color) {
case 'red':
case 'blue':
case 'white':
case 'yellow':
//
// do your stuff here
//
break;
case 'somethingelse' :
echo 'what color is this???';
break;
}
?>

Ken

[]

| Just _what_ are you nattering on about?
| Oh, I see the problem:

so you just jump into this conversation for the sole purpose OF nattering
about HOW someone else posts?!!! don't be a tard! get a good news reader and
you won't have to worry about following the thread. even BETTER, mind your
OWN behavior AND actually post content that addresses the op's concern.
that's more offensive than having someone not summarizing/referencing a
previous post to which he comments.

foad.