Consejo para reducir código

[miguellara]

Buenas, esta es una pregunta bastante simple, resulta que en mis scripts utilizo muchisimas veces condiciones del tipo:

Código PHP:

  if(isset($_POST["Submit"]) && $_POST["Submit"]=="Ingresar"){tal cosa}  
   

y llega un momento que se hace pesadisima la lectura del codigo por tantos $_POST, if, isset, etc. que se repiten

Alguien sabe alguna opción mas sintáctica para este tipo de código?

Muchas gracias.

[turco_7]

osea vos tenes muchos if ? y queres saber como podes hacer para disminuir tu cantidad ? o no entendi bien ???

[miguellara]

buenas, gracias por responder :)

si, generalmente uso muchos if / elseif, y paso muchas variables por url y post, mi inquietud viene por el lado de la comodidad visual, por ejemplo si tengo que llamar 16 veces a la variable $_GET["accion"] no se visualiza tan cómodo como llamar 16 veces a la variable $accion, se entiende? Ese tipo de detallitos...

Por eso me surgio la inquietud de como hacen otros programadores en estos casos...

Saludos

[turco_7]

entonces lo qu deberias hacer es lo siguiente ..
al principio del documento, asignale a la variable $accion el valor de $_GET['accion']

osea:

Código PHP:

  $accion = $_GET['accion'];  
   

de este modo, cada ves que la necesites la llamas como $accion.. y hace lo mismo con POST ... nose si te ahorras mucho cogido, pero te puede ser mas facil analizar tu codigo, y

con el tema de los if, tendrias que ver de que manera armas la condicion ..
por ejemplo:

Código PHP:

  <?php
if($A == 0){
      if($b == 0){
            echo "tal cosa";
     }
}
?>   

seria mejor que lo hagas de esta forma:

Código PHP:

  <?
if($A == 0 && $b == 0){
     echo "tal cosa";
}
?>   

con tema de los if/elseif, nose si es que hay muchas alternativas... pero podrias cambiarlo por un swicht case ... no te parece ?

Código PHP:

  <?php
if($i = 0){
      echo "i = 0";
}elseif ($i = 1){
      echo "i = 1";
}elseif ($i = 2){
      echo "i = 2";
}elseif ($i = n){
      echo "i = n";
}
 
switch ($i) {
    case 0:
        echo "i = 0";
        break;
    case 1:
        echo "i = 1";
        break;
    case 2:
        echo "i = 2";
        break;
    case n:
        echo "i = n";
        break;
}
?>   

Espero que esto de una forma u otra te ayude a simplicar tu codigo... crees que no es lo mejor para vos, proba haciendo funciones .. saludos !!!

[living]

Siempre se puede reducir código, hacerlo lo más intuitivo posible pero cuanto más se complica una aplicación más "feo" queda todo . No te agobies mucho por eso, lo importante es ir poniendo comentarios y seguir consejos como los que te ha dado Turco para que sea sencillo poder hacer modificaciones en el futuro y no volverse loco.

[Kaopectate]

Hola miguellara.

No estoy seguro de haberte entendido bien, pero si tus consultas tienen siempre la misma estructura, yo usaría una función. Por ejemplo:

Código PHP:

  function test_value($item, $val){ 
 return (isset($_POST[$item]) && $_POST[$item] == $val); 
}  
   

Eso cambiaría tu código a:

Código PHP:

  if (test_value("Submit", "Ingresar")){tal cosa}  
   

Saludos.

[miguellara]

Muchas gracias a todos por los consejos!