ciclo if

[magistrus13]

Ciao a tutti,

ho un'altra domanda da porvi:

se avvio il seguente codice [PHP]:

if ($ritardo > 180){
echo " <td>Ritardo sostenuto!</td>";
}elseif ($ritardo > 120 || $ritardo < 180 ){
echo " <td>Ritardo moderato </td>";
} elseif(ritardo<120){
echo " <td>in orario! </td>";

mi scrive accanto alla colonna ritardo sempre il solito messaggio perché controlla solo il primo: come è possivbile vedere nella tabella il primo è realmente in ritardo moderato ma gli altri dovrebbbero darmi in orario!!!! (vedi tabella)

come posso fare?


inoltre se volessi dare un colore in base al tipo di ritardo dove devo inserire il codice del colore?


grazie a tutti

[Grino]

if ($ritardo < 120){
    echo " <td style=\"background-color: green\">In orario!</td>";
}elseif ($ritardo < 180 ){
    echo " <td style=\"background-color: orange\">Ritardo moderato </td>";
} else{
   echo " <td style=\"background-color: red\">Forte ritardo </td>";
} 

[bstefano79]

è sbagliata la logica degli if

tutti i valori che che mostri in tabella risultano veri alla domanda o sono maggiori di 120 o sono minori di 180 (infatti sono tutti minori di 180)

[maxxl]

Penso sia un problema di elseif....

Soluzioni

if ($ritardo > 180){
echo " <td>Ritardo sostenuto!</td>";
}else{
if ($ritardo > 120 || $ritardo < 180 ){
echo " <td>Ritardo moderato </td>";
} else{
echo " <td>in orario! </td>";
}
}

oppure

fai tre if separati

oppure

usi un select case

per il colore

echo " <td bgcolor='FFFFFF'>Ritardo sostenuto!</td>";

[bstefano79]

if ($ritardo > 180){
echo " <td>Ritardo sostenuto!</td>";
}elseif ($ritardo > 120){
echo " <td>Ritardo moderato </td>";
} else{
echo " <td>in orario! </td>"; 

[magistrus13]

grazie davvero Grino

[magistrus13]

grazie a tutti siete mitici

[Lingo70]

Comunque secondo me l'errore principale dello script che avevi fatto sta nel fatto che avevi usato l'operatore || invece di && nel secondo if.

[bstefano79]

Originariamente inviato da Lingo70
Comunque secondo me l'errore principale dello script che avevi fatto sta nel fatto che avevi usato l'operatore || invece di && nel secondo if.

no con && è sbagliato comunque 180 non entrerebbe in nessun ramo