Optimize my code!

[ledtear]

How can I streamline the following:

if ($a_row[field1] == $var) {
print "x";
}
if ($a_row[field2] == $var) {
print "x";
}
if ($a_row[field3] == $var) {
print "x";
}
if ($a_row[field4] == $var) {
print "x";
}
if ($a_row[field5] == $var) {
print "x";
}

[illzz]

Try using a Switch structure.

http://us2.php.net/switch

[Horizon88]

I presume those are just placeholders and not the actual values, because otherwise if you're just printing "x", literally, whenever each key matches $var, you could do:

foreach ($a_row as &$val) {
   echo ($val == $var) ? 'x' : '';
}

Untested, but that ought to work as well. (For reference, uses a foreach() loop and the ternary operator.)

[illzz]

Here is the syntax if you want to use Switches:

switch ($var) {
case $a_row[field1] :
    print "x";
    break;
case $a_row[field2] :
    print "x";
    break;
case $a_row[field3] :
    print "x";
    break;
}