Problemas php imagenes cada dia de la semana

[karonte]

Tengo un codigo php listo para funcionar dentro de un bloque de phpnuke pero no se por que razon no me muestra la imagen el php es este :

<?PHP

if (eregi("block-imagenes.php", $PHP_SELF)) {
Header("Location: index.php");
die();
}

$dia_semana=date("w");

if ($dia_semana==0) {
$imagen="domingo.jpg";
$dia="Domingo";
} elseif ($dia_semana==1) {
$imagen="lunes.jpg";
$dia="Lunes";
} elseif ($dia_semana==2) {
$imagen="martes.jpg";
$dia="Martes";
} elseif ($dia_semana==3) {
$imagen="miercoles.jpg";
$dia="Miércoles";
} elseif ($dia_semana==4) {
$imagen="jueves.jpg";
$dia="Jueves";
} elseif ($dia_semana==5) {
$imagen="viernes.jpg";
$dia="Viernes";
} elseif ($dia_semana==6) {
$imagen="sabado.jpg";
$dia="Sábado";
}

echo "<img scr=/images/$imagen alt=$dia>";
?>

[josemi]

Hola,

Supongo que existiran esos ficheros en ese directorio, ¿verdad? Verifica el codigo HTML que te genera el fichero php (ya sabes, "Ver codigo fuente" en el navegador). O haz un echo de las variables $dia_semana, $dia e $imagen antes del echo del tag img para ver su valor.

Tambien te recuerdo que es muy recomendable poner los valores del los atributos de los tags de HTML entre comillas.

Y otro apunte mas, esta vez sobre el estilo de programacion. El codigo te quedaria mas limpio si en lugar de usar el chorizo de ifs, usas el switch. Y todavia mas limpio si usas un array (o dos) y usas el valor del date("w") como indice (parece que Cluster y yo estemos haciendo campaña a favor del uso de arrays).

Suerte.

[Cluster]

Y otro apunte mas, esta vez sobre el estilo de programacion. El codigo te quedaria mas limpio si en lugar de usar el chorizo de ifs, usas el switch. Y todavia mas limpio si usas un array (o dos) y usas el valor del date("w") como indice (parece que Cluster y yo estemos haciendo campaña a favor del uso de arrays).

pues sip .. jeje. Queda mas limpio el código y se identifican los errores mejor .. de ahí a tener esos datos en un fichero hay un paso solo ... Imagina si quieres dar esa fecha en varios idiomas? .. Un array para cada idioma y listo .. cambias segun el idioma q usas ..

La filosofia de todo esto de arrays principalmente es separar lo mas posible el "código" de los "datos" o por lo menos tener los "datos" lo mas agrupados posibles.

Un saludo,