[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

[f41t3r]

2. Po co ta "małpa": 

A po to żeby mi brzydkich Notice-ów nie wyświetlał

3. Czy na prawde potrzebujesz tak komplikować zapytanie:

Póki sie nie wymyśli czegoś wydajniejszego to raczej tak

być może nie ma tam wartości typu %Error% tylko jakieś całkiem inne 

Niestety są

Kod

(...)


Array ( [MsgDate] => 2006-02-27 [MsgTime] => 05:36:42 [MsgPriority] => Local7.Error [MsgHostname] => hpr-s10 [MsgText] => %LINK-3-UPDOWN: Interface FastEthernet0/19, changed state to down ) Array ( [MsgDate] => 2006-02-27 [MsgTime] => 05:36:44 [MsgPriority] => Local7.Error [MsgHostname] => hpr-s10 [MsgText] => %LINK-3-UPDOWN: Interface FastEthernet0/19, changed state to up ) Array ( [MsgDate] => 2006-02-27 [MsgTime] => 05:37:12 [MsgPriority] =>


(...)

P.S. switcha zrobiłem no i wygląda faktycznie troche bardziej przejrzyście.

Pozdro.

[nospor]

A po to żeby mi brzydkich Notice-ów nie wyświetlał

TomASS to wie. mu chodzilo oto, ze przypsianie tekstu do zmiennej, nie generuje brzydkuch noticów

być może nie ma tam wartości typu %Error% tylko jakieś całkiem inne 

Niestety są

Niestety nie ma
Pokaz mi w tym zrzucie gdzie znajdzisz : %Error% ? ja widze co najwyzej Local7.Error. Dla php to jest roznica. On nie szuka danych tak jak mysql przy pomocy %. Przejrzyj se zestaw tych funkcji, a znajdziesz coś do szukania
http://pl.php.net/manual/pl/ref.strings.php

[f41t3r]

Hmm dobrze wiedzieć

P.S.

Dziwne bo mi notice wyświetla jak se odznacze w formularzu pare rzeczy

Kod

Notice: Undefined index: priority3 in c:\program files\apc\enterprise manager\html\syslogd2\index2.php on line 99

Notice: Undefined index: priority4 in c:\program files\apc\enterprise manager\html\syslogd2\index2.php on line 99

Notice: Undefined index: priority5 in c:\program files\apc\enterprise manager\html\syslogd2\index2.php on line 99

Czegoś pewnie znowu nie wiem :/... standard.

Pozdro.

[nospor]

hmmm... no to ja sam sie czegos dowiedzialem
Myslalem ze ta @ bedzie dotyczyc tylko przupisania, ale wynika, ze ona dotyczy calego etapu generowania przypisania, czyli i w tym przypadku funkcji mysql_escape_string() . No ale takich rzeczy tak czy siak nie powinno sie tuszowac malpą, ale pisac poprawnie: czyli najpierw sprawdzic czy dany index jest, a dopiero potem go użyć

[f41t3r]

Na moim poziomie pisanie całkowicie poprawnych skryptów jest na razie nie możliwe. Niestety. :/

Btw. mam takiego switcha:

[PHP] pobierz, plaintext 
<?php
 
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)) {
 
	switch ($col_value) {
	case 'Lpr.Error':
		$kolor="#FEFF87";
		break;
	case 'Warning':
		$kolor='#87DD91';
		break;
	case 'Alert':
		$kolor='#CC8787';
		break;
	case 'Critical':
		$kolor='#DD8787';
		break;
	case 'Emergency':
		$kolor='#DD8787';
		break;
	default:
		$kolor='#dddddd';};
 
	print "\t<tr bgcolor='$kolor'>\n";
	foreach ($line as $col_value) {
		  print "\t<td>$col_value</td>\n";
	}
	print "\t</tr>\n";
	}
	print "</table>\n";
?>
[PHP] pobierz, plaintext 

I mam pytanie czy muszę case'y dla wszystkich rodzajów errorów wypisywać ?
np.

Kod

case 'Lpr.Error':
  $kolor="#FEFF87";
  break;
case 'Local7.Error':
  $kolor="#FEFF87";
  break;
case 'Lolcal2.Error':
  $kolor="#FEFF87";
  break;

Czy można to jakoś zastąpić żeby tylko do tego wyrazu Error się odwoływał ?

[nickers]

Ja znam tylko taką możliwość:

[PHP] pobierz, plaintext 
<?php
 
switch( $col_value )
{
	case 'Lpr.Error':
	case 'Local7.Error':
	case 'Lolcal2.Error':
 
		 $kolor="#FEFF87";
 
	break;
}
 
 
?>
[PHP] pobierz, plaintext 

Musisz wypisać wszystkie mozliowści, ale przynajmniej nie kopiujesz kodu.

Mam nadzieje, ze to nie jest cały kod, bo nie inicjujesz nigdzie: $col_value.

[f41t3r]

Aha no cóż, tego się obawiałem, no ale nic. Dzięki za odpowiedzi!

Pozdro.

[nospor]

mozna zachachmęcić

[PHP] pobierz, plaintext 
<?php
 
if (strpos($col_value, 'Error') !==false)
	$kolor='#FEFF87';
else
switch ($col_value) {
	case 'Warning':
		$kolor='#87DD91';
		break;
	case 'Alert':
		$kolor='#CC8787';
		break;
	case 'Critical':
		$kolor='#DD8787';
		break;
	case 'Emergency':
		$kolor='#DD8787';
		break;
	default:
		$kolor='#dddddd';};
?>
[PHP] pobierz, plaintext 

[f41t3r]

Wygląda rozsądniej niż to co ja teraz próbowałem kombinować tzn. pomyślałem że można wrzucić:

[PHP] pobierz, plaintext 
<?php
 
similar_text($col_value, 'Error', $p1);
similar_text($col_value, 'Warning', $p2);
similar_text($col_value, 'Critical', $p3);
similar_text($col_value, 'Emergency', $p4);
similar_text($col_value, 'Alert', $p5);
 
if ($p1>$p2 && $p1>$p3 && $p1>$p4 && $p1>$p5) {
print '#asdasd'
}
 
?>
[PHP] pobierz, plaintext 

poprawiam
---
nospor

i potem reszta ifów.

Jutro się jeszcze tym pobawie, dzięki jeszcze raz.

EDIT: Działa, dzięki wszystkim.

Ten post edytował f41t3r 9.03.2006, 11:01:16