Pregunta General

[dontripi]

Hola compañeros del foro esta pregunta poco tiene que ver con php en si, pero si tiene que ver por que es sobre programacion, lo que pasa es que yo desarrollo sobre php y he visto que uso mucho los IF's para tomar muchas decisiones en mis aplicaciones, no se si esto es malo o es bueno, alguien que pudiera decirme si existe un limite para su uso o si uso execivo es perjudicial para la ejecucion de la aplicacion, en fin si alguien puede comentar algo me sería de gran ayuda, gracias por todo.


Att:


Dontripi

[guiweb]

yo tambien los uso...
creo que alguien alguna vez me criticó porque dijo que programaba de manera "cotidiana" es decir comparado a mi vida diaria ej:
si como se me sacia el hambre, si tengo ganas de ir al baño, si tire la cadena, si sonrei.
osea una programación estructurada está orientada a tomar ejemplos "Cotidianos".
Esto no es lo mismo es POO donde he visto disminuido el uso de mis IF.
bueno esa es mi opinion... seria interesante que alguien mas aporte..

Saludos

[dontripi]

quien mas se atreve a comentar

[Fridureiks]

ES que tu pregunta es muy amplia, hay que ver en que momento es que utilizas muchas condiciones... a que llamas muchas condiciones, etc...

Quiza te ayude colocar un ejemplo de un codigo hecho por ti en el que utilices "muchas condiciones"...

Saludos.

[mauled]

Siii!! Seria bueno que postearas un ejemplo, para que de esta forma pueda haber un mejor debate del tema

Si tienes un superbloque de if´s, lo que puedes hacer es usar switch(), y de esta forma estaria mucho "mejor" programado esa parte de tu código.

Saludillos.

[dontripi]

bueno el codigo que les muestro aqui toma los valores de un mes
para un año determinado y lo esta comparando con el mismo mes pero en otro año
y esta diciendo que si alguno de los dos valores de las fechas es igual a cero
ponga el resultado de una vez en cero sino pues que haga lo que tiene que hacer

Código PHP:

  if($sumvalen == 0 OR $sumvalenant == 0) 
              $difvalene = 0;             
            else  
             $difvalene = number_format(((($sumvalen/$sumvalenant)-1)*100),1,".",",")."%"; 
                  
                if($sumunden == 0 OR $sumundenant == 0) 
                   $difuniene = 0; 
                else     
                 $difuniene = number_format(((($sumunden/$sumundenant)-1)*100),1,".",",")."%"; 
                  
                if ($sumvalfeb == 0 OR $sumvalfebant == 0) 
                 $difvalfeb = 0; 
                else  
                 $difvalfeb = number_format(((($sumvalfeb/$sumvalfebant)-1)*100),1,".",",")."%"; 
                  
                if($sumundfeb == 0 OR $sumundfebant == 0) 
                  $difunifeb = 0; 
                else   
                 $difunifeb = number_format(((($sumundfeb / $sumundfebant)-1)*100),1,".",",")."%"; 
                  
                if ($sumvalmar == 0 OR $sumvalmarant == 0) 
                 $difvalmar = 0;  
                else  
                 $difvalmar = number_format(((($sumvalmar / $sumvalmarant)-1)*100),1,".",",")."%"; 
                 
                if ($sumundmar == 0 OR $sumundmarant == 0) 
                 $difunimar = 0; 
                else                  
                 $difunimar = number_format(((($sumundmar / $sumundmarant)-1)*100),1,".",",")."%"; 
                  
                if ($sumvalabr == 0 OR $sumvalabrant == 0) 
                 $difvalabr = 0; 
                else  
                 $difvalabr = number_format(((($sumvalabr / $sumvalabrant)-1)*100),1,".",",")."%"; 
                 
                if ($sumundabr == 0 OR $sumundabrant == 0) 
                 $difuniabr = 0; 
                else  
                 $difuniabr = number_format(((($sumundabr / $sumundabrant )-1)*100),1,".",",")."%"; 
                if ($sumvalmay == 0 OR $sumvalmayant == 0) 
                 $difvalmay = 0; 
                else  
                 $difvalmay = number_format(((($sumvalmay / $sumvalmayant)-1)*100),1,".",",")."%"; 
                if ($sumundmay == 0 OR $sumundmayant == 0) 
                 $difunimay = 0; 
                else  
                 $difunimay = number_format(((($sumundmay / $sumundmayant)-1)*100),1,".",",")."%"; 
                  
                if ($sumvaljun == 0 OR $sumvaljunant == 0) 
                 $difvaljun = 0; 
                else  
                 $difvaljun = number_format(((($sumvaljun / $sumvaljunant)-1)*100),1,".",",")."%"; 
                  
                if ($sumundjun == 0 OR $sumundjunant == 0) 
                 $difunijun = 0; 
                else  
                 $difunijun = number_format(((($sumundjun / $sumundjunant)-1)*100),1,".",",")."%"; 
                 
                if ($sumvaljul == 0 OR $sumvaljulant == 0) 
                 $difvaljul = 0; 
                else  
                 $difvaljul = number_format(((($sumvaljul / $sumvaljulant)-1)*100),1,".",",")."%"; 
                  
                if ($sumundjul == 0 OR $sumundjulant == 0) 
                 $difunijul = 0; 
                else  
                 $difunijul = number_format(((($sumundjul / $sumundjulant)-1)*100),1,".",",")."%"; 
                  
                if ($sumvalago == 0 OR $sumvalagoant == 0) 
                 $difvalago = 0; 
                else  
                 $difvalago = number_format(((($sumvalago / $sumvalagoant)-1)*100),1,".",",")."%"; 
                  
                if ($sumundago == 0 OR $sumundagoant == 0) 
                 $difuniago = 0; 
                else  
                 $difuniago = number_format(((($sumundago / $sumundagoant)-1)*100),1,".",",")."%"; 
                  
                if ($sumvalsep == 0 OR $sumvalsepant == 0) 
                 $difvalsep = 0; 
                else  
                 $difvalsep = number_format(((($sumvalsep / $sumvalsepant)-1)*100),1,".",",")."%"; 
                  
                if ($sumundsep == 0 OR $sumundsepant == 0) 
                 $difunisep = 0; 
                else  
                 $difunisep = number_format(((($sumundsep / $sumundsepant)-1)*100),1,".",",")."%"; 
                  
                if ($sumvaloct == 0 OR $sumvaloctant == 0) 
                 $difvaloct = 0; 
                else  
                 $difvaloct = number_format(((($sumvaloct / $sumvaloctant)-1)*100),1,".",",")."%"; 
                  
                if ($sumundoct == 0 OR $sumundoctant == 0) 
                 $difunioct = 0; 
                else  
                 $difunioct = number_format(((($sumundoct / $sumundoctant)-1)*100),1,".",",")."%"; 
                 
                if ($sumvalnov == 0 OR $sumvalnovant == 0) 
                 $difvalnov = 0; 
                else  
                 $difvalnov = number_format(((($sumvalnov / $sumvalnovant)-1)*100),1,".",",")."%"; 
                 
                if ($sumundnov == 0 OR $sumundnovant == 0) 
                 $difuninov = 0; 
                else                  
                 $difuninov = number_format(((($sumundnov / $sumundnovant)-1)*100),1,".",",")."%"; 
                  
                if ($sumvaldic == 0 OR $sumvaldicant == 0) 
                 $difvaldic = 0; 
                else  
                 $difvaldic = number_format(((($sumvaldic / $sumvaldicant)-1)*100),1,".",",")."%"; 
                 
                if ($sumunddic == 0 OR $sumunddicant == 0) 
                 $difunidic = 0; 
                else  
                 $difunidic = number_format(((($sumunddic / $sumunddicant)-1)*100),1,".",",")."%"; 
                  
                  
                 $sumtotunddif = ($difuniene + $difunifeb + $difunimar + $difuniabr + $difunimay 
                                   + $difunijun + $difunijul + $difuniago + $difunisep + $difunidoct +  
                                   $difuninov + $difunidic); 
                  
                  
                 $sumtotvaldif = ($difvalene + $difvalfeb + $difvalmar + $difvalabr + $difvalmay 
                                   + $difvaljun + $difvaljul + $difvalago + $difvalsep + $difvaldoct +  
                                   $difvalnov + $difvaldic);  
   

pues ese es el codigo amigos espero me puedan decir algo gracias, de todos modos voy a evaluar
la posibilidad de utilizar switch de antemano muchas gracias por todo

[Fridureiks]

Mmm, asi como se ve, parece algo aparatoso jeje. Pero no entiendo bien que tarea realiza, podrias explicar un poco mejor? parametros de entrada, parametros de salida (resultados).... quiza haya alguna forma mas simple de hacer lo que deseas, pero para eso debemos antes entender lo que deseas hacer.

Saludos.

[nicolaspar]

No es malo utilizarlos si están justificados y optimizados.
Un ejemplo simple es cuando se valida un dato de entrada haciendo:

Código PHP:

  if( !empy($dato) ){
 ...
 proceso
 ...
}else{
echo "error";
}  
   

Para mi esta tarea, por ejemplo, es mejor así:

Código PHP:

  if( empy($dato) ){ echo "error"; exit; }
proceso
...  
   

Pero el código que expresas es muy extenso como dice Federico como para leerlo en 1 min (tiempo que le doy a la lectura de un thread como máximo :P) y comprenderlo diciéndote si es correcto o no el uso de tantos ifs.

[Fridureiks]

Por cierto, no viene al caso, pero dejo este link que tiene unas pequeñas reglas para ser mas prolijos-claros, a la hora de escribir codigo con php.

Saludos.

[pintix]

El problema en el ejemplo que has puesto no es tanto el abuso de los if, como que el código es muy poco mantenible. Si analizas tu código verás que haces lo mismo una y otra vez, en esencia, esto:

Código PHP:

  if ($sumundabr == 0 OR $sumundabrant == 0)  
                 $difuniabr = 0;  
                else   
                 $difuniabr = number_format(((($sumundabr / $sumundabrant )-1)*100),1,".",",")."%";  
   

Siempre que se vea que se está haciendo algo repetidamente, quiere decir que es mejor ponerlo en un bucle o convertirlo en una función, o, si es más complejo, convertirlo en una clase por que tu código será más reutilizable y mantenible. Imagínate que en un momento avanzado del desarrollo tienes que cambiar algo de uno de los pasos, para hacerlo deberías cambiar todas las veces donde está utilizado.

Lo mejor en este caso sería poner los datos en un array y a partir de ahí recorrerlo con un foreach y aplicar la operación que se repite y en cada uno de los pasos sumar el resultado a una variable.