[php] Kolorowanie warunkowe w pętli z wynikami Opcje

[f41t3r]

Mam problem z kolorowaniem wyników wyświetlanych przez skrypt mysql. Chodzi mi o to ażeby wiersze zawierające konkretne wartości tj. error, warning, critical itp. były innego odpowiedniego koloru.

Tak wygląda dotychdczasowy skrypt, który musze się przyznać, że nie działa za dobrze tj. podczas warunku cały czas zwraca wartość false i wykonuje mi tylko elsa czyli podkreśla wszystkie linie na #dddddd

[PHP] pobierz, plaintext 
<?php
 
 
@ $query="select * from syslogd.syslogd where
MsgDate='".mysql_escape_string($HTTP_GET_VARS['bydate'])."' AND (Msghostname 
 
LIKE '".mysql_escape_string($HTTP_GET_VARS['Urzadzenie'])."' AND Msgpriority 
 
LIKE '".mysql_escape_string($HTTP_GET_VARS['priority1'])."' OR Msgpriority 
 
LIKE '".mysql_escape_string($HTTP_GET_VARS['priority2'])."' oR Msgpriority 
 
LIKE '".mysql_escape_string($HTTP_GET_VARS['priority3'])."' OR Msgpriority 
 
LIKE '".mysql_escape_string($HTTP_GET_VARS['priority4'])."' OR Msgpriority 
 
LIKE '".mysql_escape_string($HTTP_GET_VARS['priority5'])."') ORDER BY 
 
'".mysql_escape_string($HTTP_GET_VARS['order'])."'";
$result=mysql_query("$query");
 
print "<table border=0>\n";
	print "<tr><td><a>Date</a></td><td><a >Time</a></td><td><a 
 
>Priority</a></td><td><a >Hostname</a></td><td><a>Text</a></td></tr>";
	while ($line = mysql_fetch_array($result, MYSQL_ASSOC)) {
	print "\t<tr>\n";
	foreach ($line as $col_value) {
		  print "\t<td bgcolor=";
 
		if ($col_value=="%Alert%") {
		print "#DD8787";
} 
 
		elseif ($col_value=="%Emergency%"){
		print "#DD6767";
}
 
		elseif ($col_value=="%Critical%"){
		print "#DD5757";
}
 
		elseif ($col_value=="%Error%"){
		print "#87DD91";
}
 
		elseif ($col_value=="%Warning%"){
		print "#FEFF87";
}
 
 
else {
	print "#dddddd";
};
 
 
print ">$col_value</td>\n";
	}
	print "\t</tr>\n";
	}
	print "</table>\n";
 
?>
[PHP] pobierz, plaintext 

Jak na razie utknąłem i nie wiem co dalej z tym zrobić.
Proszę o pomoc.

[TomASS]

1. Zamiast

[HTML] pobierz, plaintext 
<td bgcolor="....">
[HTML] pobierz, plaintext 

ładniej dać:

[HTML] pobierz, plaintext 
<tr style="background-color: .....">
[HTML] pobierz, plaintext 

2. Po co ta "małpa":

[PHP] pobierz, plaintext 
<?php
@ $query=....
?>
[PHP] pobierz, plaintext 

3. Czy na prawde potrzebujesz tak komplikować zapytanie:

[SQL] pobierz, plaintext 
SELECT * 
FROM syslogd.syslogd
 WHERE MsgDate='".mysql_escape_string($HTTP_GET_VARS['bydate'])."' AND (Msghostname 
 
LIKE '".mysql_escape_string($HTTP_GET_VARS['Urzadzenie'])."' AND Msgpriority 
 
LIKE '".mysql_escape_string($HTTP_GET_VARS['priority1'])."' OR Msgpriority 
 
LIKE '".mysql_escape_string($HTTP_GET_VARS['priority2'])."' OR Msgpriority 
 
LIKE '".mysql_escape_string($HTTP_GET_VARS['priority3'])."' OR Msgpriority 
 
LIKE '".mysql_escape_string($HTTP_GET_VARS['priority4'])."' OR Msgpriority 
 
LIKE '".mysql_escape_string($HTTP_GET_VARS['priority5'])."') ORDER BY
[SQL] pobierz, plaintext 

4. Zrób sobie:

[PHP] pobierz, plaintext 
<?php
 
$query....
$result = mysql_query($query);
while ($row = mysql_fetch_array($result)){
 print_r($row);
}
 
?>
[PHP] pobierz, plaintext 

I zobacz co Ci się wyświetla - być może nie ma tam wartości typu %Error% tylko jakieś całkiem inne - a jakie? To właśnie zobaczysz.
5. Zamiast tyle ifów/elsów możesz elegancko to napisać w tabeli albo dzięki switch.
6 Zamiast

[PHP] pobierz, plaintext 
<?php
print "#DD6767";
?>
[PHP] pobierz, plaintext 

pisz

[PHP] pobierz, plaintext 
<?php
print '#DD6767';
?>
[PHP] pobierz, plaintext 

7. Wyrzuć tą książkę - ona się do niczego nie nadaje (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)