PHP-hispano.net

Bajo mi punto de vista, incluir como parámetros de entrada la fecha y un separador, es obligar a la función a realizar unas operaciones por las que no debería preocuparse. Es decir, que lo ideal sería obtener el horóscopo directamente a partir del día y del mes.

Por otra parte, otras cosas que he visto:

$dia="$separa[0]";
$mes="$separa[1]";

No te hacen falta esas comillas dobles.


if($mes[0] != "0"){
$mes="$mes";
}else{
$mes="$mes[1]";
}

Esto lo podrías solucionar haciendo directamente un $mes = intval($mes);

La manera que usas de comparar las fechas concatenando día y mes, aunque no creo que sea la mejor solución, resulta efectivo.

En cambio esto:

if($total >= 0){
if($total <= 120){ $horoscopo = "Capricornio"; }else
if($total <= 219){ $horoscopo = "Acuario"; }else
if($total <= 320){ $horoscopo = "Piscis"; }else

son demasiados ifs anidados. Cuando veas cosas de este estilo en un código, recuerda que en la mayoría de los casos, hay una forma mejor de hacerlo sin necesidad de usar tanto if else.

Podrías probar a hacer algo así:


if($total >= 0){
if($total <= 120) return 'Capricornio';
if($total <= 219) return 'Acuario';
}

y así con todos. Directamente devuelves el valor una vez que lo has encontrado y así te evitas tantas llaves y else.

Por otra parte, sería conveniente que revisaras los valores de entrada. Es decir, que no haya un mes 13, o un día 40.
Échale un vistazo a la función checkdate.

Te voto negativo en principio a la espera de que hagas algunos cambios para que la función esté mejor y más completa ;)

Podría ser una buena idea ampliar la función para calcular el horóscopo chino también... :P

Es muy original el modo en que lo calculas, pero el código como dice zubyc es mejorable

Lo mismo opino.

Actualizalo pq la idea la verdad esque es original.