if else ??

Anon

hi,

there were 3 pages as 1.php, 2.php and 3.php and all pages contains the table which i work on.
i want,
if a person enters 1.php, FIRST PART bgcolor will be black and others are white,
if 2.php, SECOND PART black ...

anyway what should my if condition

<?
$selected_bgcolor = "#000000";
$not_selected_bgcolor = "#FFFFFF";

<table>
<tr>
<td bgcolor="<? if (condition){
echo "$selected_bgcolor";
}
else {echo "$not_selected_bgcolor"; }
?>">FIRST PART</td>
<td bgcolor="<? if (condition){
echo "$selected_bgcolor";
}
else {echo "$not_selected_bgcolor"; }
?>">SECOND PART</td>
<td bgcolor="<? if (condition){
echo "$selected_bgcolor";
}
else {echo "$not_selected_bgcolor"; }
?>">THIRD PART</td>
</tr>
</table>

Anon

You are doing too much work. It'd be easier to do it this way.

<?
$selected_bgcolor = "#000000";
$section1_bgcolor = "#FFFFFF";
$section2_bgcolor = "#FFFFFF";
$section3_bgcolor = "#FFFFFF";

if ($condition = "1.php") {
$section1_bgcolor = $selected_color;
}
else if ($condition = "2.php") {
$section2_bgcolor = $selected_color;
}
else {
$section3_bgcolor = $selected_color;
}

<table>
<tr>
<td bgcolor="<? echo $section1_bgcolor; ?>">FIRST PART</td>
<td bgcolor="<? echo $section2_bgcolor; ?>">SECOND PART</td>
<td bgcolor="<? echo $section3_bgcolor; ?>">THIRD PART</td>
</tr>
</table>

This way sets all of your sections to the unselected color, then just does one statement to change whichever section matches your criteria. You could also use a switch statement instead of an if..else,

switch($condition) {
 case "1.php"
  $section1_bgcolor = $selected_color;
 break;
 case "2.php"
  $section2_bgcolor = $selected_color;
 break;
 case "3.php"
  $section3_bgcolor = $selected_color;
 break;
}

mithril

In the if else structure, double equal signs should have been used🙂 The way it's currently written the first condition will always evaluate true. Other than that typo Rob is completely correct🙂

Geoff A. Virgo

Anon

Damn, I always do that. ;-> Thanks for the catch.