[SOLUCIONADO] Problema con if en php no imprime correctamente

[self]

Hola , estoy haciendo un script para mostrar el tiempo en un sitio por extraño motivo me imprime la imagen mostlycloudy cuando por parametro esta viniendo OVERCAST y deberia imprimir la imagen fog.

Espero alguien me ayude, gracias

Código PHP:

  if (Date("G") > 06 && Date("G") <= 18){ 
 
        if ($row['tiempo'] == "Clear"){ 
          echo '<img src="./sites/all/themes/arctica_lafuente/weather/Clear.png">'; 
       } elseif ($row['tiempo'] == "Partly Cloudy" || "Mostly Sunny" || "Partly Sunny" || "Mostly Cloudy"){ 
          echo '<img src="./sites/all/themes/arctica_lafuente/weather/mostlycloudy.png">'; 
       } elseif ($row['tiempo'] == "Fog" || "Hazy" || "Overcast"){ 
          echo '<img src="./sites/all/themes/arctica_lafuente/weather/fog.png">'; 
       } elseif ($row['tiempo'] == "Sleet" || "Rain"){ 
          echo '<img src="./sites/all/themes/arctica_lafuente/weather/rain.png">'; 
       } elseif ($row['tiempo'] == "Chance Rain" || "Chance Sleet"){ 
          echo '<img src="./sites/all/themes/arctica_lafuente/weather/chancerain.png">'; 
       }  elseif ($row['tiempo'] == "Storms"){ 
          echo '<img src="./sites/all/themes/arctica_lafuente/weather/storms.png">'; 
       }else{ 
          echo '<img src="./sites/all/themes/arctica_lafuente/weather/na.png">'; 
       } 
       }else { 
        if ($row['tiempo'] == "Clear"){ 
          echo '<img src="./sites/all/themes/arctica_lafuente/weather/nocheclear.png">'; 
       } elseif ($row['tiempo'] == "Partly Cloudy" || "Mostly Sunny" || "Partly Sunny" || "Mostly Cloudy"){ 
          echo '<img src="./sites/all/themes/arctica_lafuente/weather/nochecloudy.png">'; 
       } elseif ($row['tiempo'] == "Fog" || "Hazy" || "Overcast"){ 
          echo '<img src="./sites/all/themes/arctica_lafuente/weather/fog.png">'; 
       } elseif ($row['tiempo'] == "Sleet" || "Rain"){ 
          echo '<img src="./sites/all/themes/arctica_lafuente/weather/rain.png">'; 
       } elseif ($row['tiempo'] == "Chance Rain" || "Chance Sleet"){ 
          echo '<img src="./sites/all/themes/arctica_lafuente/weather/chancerain.png">'; 
       } elseif ($row['tiempo'] == "Storms"){ 
          echo '<img src="./sites/all/themes/arctica_lafuente/weather/storms.png">'; 
       } else{ 
        echo '<img src="./sites/all/themes/arctica_lafuente/weather/na.png">'; 
       } 
       }  
   

[bNd170]

Hasta donde yo se, esto no vale

Código PHP:

if($row['tiempo'] == "Chance Rain" || "Chance Sleet")
 

Ahi el php esta interpretando que debe entrar si $row['tiempo'] vale Chance Rain o "Chance Sleet", ese ultimo string es como decirle a PHP que entre si 1 o si "a", es decir, es algo que no puede ser evaluado y por lo tanto lo da por afirmativo y entra.

La solucion pasa por poner $row['tiempo'] en todos los casos o hacer un switch que es mucho mas sencillo y comodo

[self]

Me podrías poner une ejemplo??
muchas gracias

[bNd170]

Si, claro.

Código PHP:

          switch ($row['tiempo']) { 
                case 'Clear': 
                        echo '<img src="./sites/all/themes/arctica_lafuente/weather/Clear.png">';  
                        break; 
                case 'Partly Cloudy': 
                case 'Mostly Sunny': 
                case 'Partly Sunny': 
                case 'Mostly Cloudy': 
                        echo '<img src="./sites/all/themes/arctica_lafuente/weather/mostlycloudy.png">';  
                        break; 
        }  
   

En un switch, empieza a evaluar de arriba hacia abajo el script y no para hasta terminar o hasta enontrar un break, lo bueno del switch es que si encuentra una coincidencia, entra en todos los siguientes, por lo que podemos anidar posibles grupos de casos, como el que te he ilustrado justo arriba.

Para clear, como era una simple coincidencia, plantamos un break, por si es clear, que se detenga ahi el switch.

Para el grupo que tiene que devolver mostlycloudy dejamos sin código de respuesta tres de ellas puesto que entrando por una de ellas, seguira ejecutando el codigo hasta llegar al final o encontrarse un break, por lo que si es alguna de esas 4 posibilidades, ejecutará ese echo.

[self]

muchas gracias