4

He creado un switch con varios cases que según el valor me de un texto lo que sucede que debo hacerlo con múltiples campos y sería repetir el código que es algo extenso o ver de que forma simplificarlo que debe haber alguna.

switch ($s1) {
    case (($s1 >= 0) && ($s1 < 10)): $im1 = "Indefinido"; break; 
    case (($s1 >= 10) && ($s1 <= 20)): $im1 = "Definiendo";break; 
    case ($s1 > 20): $im1 = "Definido"; break;
}

El $s1 va relacionado a este código que obtiene todos los campos y los suma.

$s1=$s2=$s3=$s4= 0; 
for ($i = 0; $i < count($Datos); $i++) { 
    $s1 += $Datos[$i]["Valor1"];

Y así con cada linea la cuestión es que obtiene todo el valor de los campos en $Datos[$i]["Valor1"]; y después uso $s1 += $Datos[$i]["Valor1"]; para así llamarlo en el switch $s1 pero debo repetir el paso con $s2, $s3, $s4 y me preguntaba si me podéis echar un cable a simplificarlo.

Edición:

La linea que repito es:

El $s1 es $s1 += $Datos[$i]["Valor1"];

switch ($s1) {
    case (($s1 >= 0) && ($s1 < 10)): $im1 = "Indefinido"; break; 
    case (($s1 >= 10) && ($s1 <= 20)): $im1 = "Definiendo";break; 
    case ($s1 > 20): $im1 = "Definido"; break;
}

a

El $s2 es $s2 += $Datos[$i]["Valor2"];

switch ($s2) {
    case (($s2 >= 0) && ($s2 < 10)): $im2 = "Indefinido"; break; 
    case (($s2 >= 10) && ($s2 <= 20)): $im2 = "Definiendo";break; 
    case ($s2 > 20): $im2 = "Definido"; break;
}

En vez de crear tanta linea.

Código funcional

$Datos es obtenido de la base datos por eso extraigo la información en $Datos[$i][CELDA]

$s1=$s2=$s3=$s4=$s5=$s6=$s7=$s8= 0; for ($i = 0; $i < count($Datos); $i++) { 
$s1 += $Datos[$i]["Valor1"];$s2 += $Datos[$i]["Valor2"];
$s3 += $Datos[$i]["Valor3"];$s4 += $Datos[$i]["Valor4"];

$s5 += $Datos[$i]["Valor5"];$s6 += $Datos[$i]["Valor6"];
$s7 += $Datos[$i]["Valor7"];$s8 += $Datos[$i]["Valor8"];}

switch ($s1) {case (($s1 > 0) && ($s1 < 10)): $im1 = "Valor1 10"; break; case (($s1 >= 10) && ($s1 <= 20)): $im1 = "Valor1 20";break; case ($s1 > 20): $im1 = "Valor1 > 20"; break;}
switch ($s2) {case (($s2 > 0) && ($s2 < 10)): $im2 = "Valor2 10"; break; case (($s2 >= 10) && ($s2 <= 20)): $im2 = "Valor2 20";break; case ($s2 > 20): $im2 = "Valor2 > 20"; break;}
switch ($s3) {case (($s3 > 0) && ($s3 < 10)): $im3 = "Valor3 10"; break; case (($s3 >= 10) && ($s3 <= 20)): $im3 = "Valor3 20";break; case ($s3 > 20): $im3 = "Valor3 > 20"; break;}
switch ($s4) {case (($s4 > 0) && ($s4 < 10)): $im4 = "Valor4 10"; break; case (($s4 >= 10) && ($s4 <= 20)): $im4 = "Valor4 20";break; case ($s4 > 20): $im4 = "Valor4 > 20"; break;}

switch ($s5) {case (($s5 > 0) && ($s5 < 10)): $im5 = "Valor5 10"; break; case (($s5 >= 10) && ($s5 <= 20)): $im5 = "Valor5 20";break; case ($s5 > 20): $im5 = "Valor5 > 20"; break;}
switch ($s6) {case (($s6 > 0) && ($s6 < 10)): $im6 = "Valor6 10"; break; case (($s6 >= 10) && ($s6 <= 20)): $im6 = "Valor6 20";break; case ($s6 > 20): $im6 = "Valor6 > 20"; break;}
switch ($s7) {case (($s7 > 0) && ($s7 < 10)): $im7 = "Valor7 10"; break; case (($s7 >= 10) && ($s7 <= 20)): $im7 = "Valor7 20";break; case ($s7 > 20): $im7 = "Valor7 > 20"; break;}
switch ($s8) {case (($s8 > 0) && ($s8 < 10)): $im8 = "Valor8 10"; break; case (($s8 >= 10) && ($s8 <= 20)): $im8 = "Valor8 20";break; case ($s8 > 20): $im8 = "Valor8 > 20"; break;}

Y con ese código se me queda la variable echo $im8; entre otras para un uso especifico ese es el resultado funcional que tengo pero sin optimizar como se puede apreciar.

8
  • Hola Vicente, te recomendaría añadir un default, ya que si $s1 es menor que 0, no entraría en ninguno de los cases
    – rencinas
    Commented el 5 nov. 2018 a las 15:08
  • 2
    ¿Por qué no creas una función con el código común y luego la llamas para cada variable?
    – Alvaro Montoro
    Commented el 5 nov. 2018 a las 15:09
  • rencinas el valor no será nunca menor a 0 pero gracias por el detalle. @AlvaroMontoro cómo podría hacerlo ya que lo que quiero es no tener que copiar tanto código y si con eso lo simplifica.
    – Vicente
    Commented el 5 nov. 2018 a las 18:29
  • El switch es la parte que se repite para cada variable, ¿no? Entonces podrías moverlo a una función que devuelva el valor de $im y llamar a esa función para cada una de las variables $s que tengas (en un bucle o como prefieras). Básicamente es lo que te sugiere Carmen.
    – Alvaro Montoro
    Commented el 5 nov. 2018 a las 18:39
  • @AlvaroMontoro ahí edite me había saltado una paso crucial ya que el valor va cambiando a Valor1, Valor2 por eso cambia el $s1, $s2 al igual $im1 y $im2 y intento evitar crear múltiples párrafos solo cambiando esos valores.
    – Vicente
    Commented el 5 nov. 2018 a las 19:11

3 respuestas 3

4

Puedes crear una función que ejecute el código común, en este caso el bucle, y te devuelva el resultado, de modo que sólo tienes que pasarle el valor cómo parámetro cuando la llamas. Por ejemplo:

function estado($number) {
  if ($number>=20) {$txt = "Definido";}
  else if ($number>=10) {$txt = "Definiendo";}
  else if ($number>=0) {$txt = "Indefinido";}
  else {$txt="OutOfLimit";}
  return $txt;
}

$Datos[0]["Valor"]=2;
$Datos[1]["Valor"]=20;

$s1=0;
for ($i = 0; $i < count($Datos); $i++) { 
    $s1 += $Datos[$i]["Valor"];
    echo estado($s1)."<br>";
}

Resultado:

Indefinido
Definido

Edición tras aclaraciones de la pregunta: para distintas variable sirve igualmente porque a la función le pasamos el valor numérico que corresponde en cada caso y extraemos el resultado.

function estado($number) {
      if ($number>20) {$txt = "Definido";} //>20
      else if ($number>=10) {$txt = "Definiendo";} //de 10 a 20 
      else if ($number>=0) {$txt = "Indefinido";} //de 0 a 9
      else {$txt="OutOfLimit";} //otro valor(negativos)
      return $txt;
    }

    $Datos[0]["Valor1"]=2;
    $Datos[1]["Valor1"]=20;

    $Datos[0]["Valor2"]=10;
    $Datos[1]["Valor2"]=22;

    $Datos[0]["Valor3"]=5;
    $Datos[1]["Valor3"]=25;

    $s1=0; $s2=0; $s3=0;
    for ($i = 0; $i < count($Datos); $i++) { 
        $s1 += $Datos[$i]["Valor1"];
        $s2 += $Datos[$i]["Valor2"];
        $s3 += $Datos[$i]["Valor3"];
        //Aquí le das valor a todas las variables. Y luego ejecutas la función para cada variable
        echo estado($s1)."<br>".estado($s2)."<br>".estado($s3)."<br>";
    }
18
  • La cuestión es reducir el código ya que no solo debo usar $s1, usare $s2, $s3 y más y la intención es no ir copiando la linea. switch ($s1) { case (($s1 >= 0) && ($s1 < 10)): $im1 = "Indefinido"; break; case (($s1 >= 10) && ($s1 <= 20)): $im1 = "Definiendo";break; case ($s1 > 20): $im1 = "Definido"; break; } switch ($s2) { case (($s2 >= 0) && ($s2 < 10)): $im2 = "Indefinido"; break; case (($s2 >= 10) && ($s2 <= 20)): $im2 = "Definiendo";break; case ($s2 > 20): $im2 = "Definido"; break; }
    – Vicente
    Commented el 5 nov. 2018 a las 18:30
  • @Vicente ese código parece más completo que el de la pregunta, quizás sería bueno que editaras la pregunta para ponerlo y que se vea mejor el problema que estás planteando
    – Alvaro Montoro
    Commented el 5 nov. 2018 a las 18:41
  • Es el mismo código pero como comente duplico las lineas ya que no es solo para un valor.
    – Vicente
    Commented el 5 nov. 2018 a las 18:44
  • @AlvaroMontoro teniaz razón me había saltado código por mencionar ahí os edite .
    – Vicente
    Commented el 5 nov. 2018 a las 19:12
  • @carmen me había saltado código espero que puedan interpretarlo mejor ahora.
    – Vicente
    Commented el 5 nov. 2018 a las 19:13
3

Tienes un par de juegos de 8 variables ($s y $im) todas enumeradas del 1 al 8, en lugar de tener esas 16 variables, podrías tener dos arrays de tamaño 8 cada uno y acceder con un iterador/índice.

Vamos a ir por partes. Esto:

$s1=$s2=$s3=$s4=$s5=$s6=$s7=$s8= 0; 

se puede cambiar por el array mencionado arriba:

$s = [0,0,0,0,0,0,0,0];

Luego esta parte también se podría simplificar si la pones dentro de un bucle que se repita una vez por cada elemento del array:

for ($i = 0; $i < count($Datos); $i++) { 

    $s1 += $Datos[$i]["Valor1"];$s2 += $Datos[$i]["Valor2"];
    $s3 += $Datos[$i]["Valor3"];$s4 += $Datos[$i]["Valor4"];

    $s5 += $Datos[$i]["Valor5"];$s6 += $Datos[$i]["Valor6"];
    $s7 += $Datos[$i]["Valor7"];$s8 += $Datos[$i]["Valor8"];
}

Algo como esto:

for ($i = 0; $i < count($Datos); $i++) { 
  for ($j = 0; $j < 8; $j++) {
     $s[$j] += $Datos[$i]["Valor" . ($j+1)];
  }
}

Y finalmente puedes hacer algo parecido también para todos los switch:

switch ($s1) {case (($s1 > 0) && ($s1 < 10)): $im1 = "Valor1 10"; break; case (($s1 >= 10) && ($s1 <= 20)): $im1 = "Valor1 20";break; case ($s1 > 20): $im1 = "Valor1 > 20"; break;}
switch ($s2) {case (($s2 > 0) && ($s2 < 10)): $im2 = "Valor2 10"; break; case (($s2 >= 10) && ($s2 <= 20)): $im2 = "Valor2 20";break; case ($s2 > 20): $im2 = "Valor2 > 20"; break;}
switch ($s3) {case (($s3 > 0) && ($s3 < 10)): $im3 = "Valor3 10"; break; case (($s3 >= 10) && ($s3 <= 20)): $im3 = "Valor3 20";break; case ($s3 > 20): $im3 = "Valor3 > 20"; break;}
switch ($s4) {case (($s4 > 0) && ($s4 < 10)): $im4 = "Valor4 10"; break; case (($s4 >= 10) && ($s4 <= 20)): $im4 = "Valor4 20";break; case ($s4 > 20): $im4 = "Valor4 > 20"; break;}

switch ($s5) {case (($s5 > 0) && ($s5 < 10)): $im5 = "Valor5 10"; break; case (($s5 >= 10) && ($s5 <= 20)): $im5 = "Valor5 20";break; case ($s5 > 20): $im5 = "Valor5 > 20"; break;}
switch ($s6) {case (($s6 > 0) && ($s6 < 10)): $im6 = "Valor6 10"; break; case (($s6 >= 10) && ($s6 <= 20)): $im6 = "Valor6 20";break; case ($s6 > 20): $im6 = "Valor6 > 20"; break;}
switch ($s7) {case (($s7 > 0) && ($s7 < 10)): $im7 = "Valor7 10"; break; case (($s7 >= 10) && ($s7 <= 20)): $im7 = "Valor7 20";break; case ($s7 > 20): $im7 = "Valor7 > 20"; break;}
switch ($s8) {case (($s8 > 0) && ($s8 < 10)): $im8 = "Valor8 10"; break; case (($s8 >= 10) && ($s8 <= 20)): $im8 = "Valor8 20";break; case ($s8 > 20): $im8 = "Valor8 > 20"; break;}

Para los que puedes crear otro bucle que atraviese el array y vaya asignando valores:

$im = ["","","","","","","",""];

for ($j = 0; $j < 8; $j++) {
  switch ($s[$j]) {
    case (($s[$j] > 0) && ($s[$j] < 10)): $im[$j] = "Valor" . ($j+1) . " 10"; break; 
    case (($s[$j] >= 10) && ($s[$j] <= 20)): $im[$j] = "Valor" . ($j+1) . " 20";break; 
    case ($s[$j] > 20): $im[$j] = "Valor" . ($j+1) . " > 20"; break;}
}

Y todo junto se vería así:

$s = [0,0,0,0,0,0,0,0];
$im = ["","","","","","","",""];

for ($i = 0; $i < count($Datos); $i++) { 
  for ($j = 0; $j < 8; $j++) {
     $s[$j] += $Datos[$i]["Valor" . ($j+1)];
  }
}   

for ($j = 0; $j < 8; $j++) {
  switch ($s[$j]) {
    case (($s[$j] > 0) && ($s[$j] < 10)): $im[$j] = "Valor" . ($j+1) . " 10"; break; 
    case (($s[$j] >= 10) && ($s[$j] <= 20)): $im[$j] = "Valor" . ($j+1) . " 20";break; 
    case ($s[$j] > 20): $im[$j] = "Valor" . ($j+1) . " > 20"; break;
  }
}

Con ese código he hecho una prueba usando el siguiente set de $Datos:

$Datos = [
    [ "Valor1" => 1, "Valor2" => 2, "Valor3" => 3, "Valor4" => 4, "Valor5" => 5, "Valor6" => 6, "Valor7" => 7, "Valor8" => 8 ],
    [ "Valor1" => 1, "Valor2" => 2, "Valor3" => 3, "Valor4" => 4, "Valor5" => 5, "Valor6" => 6, "Valor7" => 7, "Valor8" => 8 ],
    [ "Valor1" => 1, "Valor2" => 2, "Valor3" => 3, "Valor4" => 4, "Valor5" => 5, "Valor6" => 6, "Valor7" => 7, "Valor8" => 8 ],
    [ "Valor1" => 1, "Valor2" => 2, "Valor3" => 3, "Valor4" => 4, "Valor5" => 5, "Valor6" => 6, "Valor7" => 7, "Valor8" => 8 ],
    [ "Valor1" => 1, "Valor2" => 2, "Valor3" => 3, "Valor4" => 4, "Valor5" => 5, "Valor6" => 6, "Valor7" => 7, "Valor8" => 8 ]
];

Y haciendo var_dump de $s e $im obtengo respectivamente:

array(8) {
  [0]=>
  int(5)
  [1]=>
  int(10)
  [2]=>
  int(15)
  [3]=>
  int(20)
  [4]=>
  int(25)
  [5]=>
  int(30)
  [6]=>
  int(35)
  [7]=>
  int(40)
}

array(8) {
  [0]=>
  string(9) "Valor1 10"
  [1]=>
  string(9) "Valor2 20"
  [2]=>
  string(9) "Valor3 20"
  [3]=>
  string(9) "Valor4 20"
  [4]=>
  string(11) "Valor5 > 20"
  [5]=>
  string(11) "Valor6 > 20"
  [6]=>
  string(11) "Valor7 > 20"
  [7]=>
  string(11) "Valor8 > 20"
}
12
  • Me ha servido tu código junto al de Carmen que me hace reducir bastante código pero tengo un problema he querido utilizar de tu código para remplazar el actual $s = [0,0,0,0,0,0,0,0]; for ($i = 0; $i < count($Datos); $i++) { $s1 += $Datos[$i]["Valor1"];$s2 += $Datos[$i]["Valor2"]; $s3 += $Datos[$i]["Valor3"]; ... más código. Pero no funciona y tampoco veo el error en tu código se ve bastante funcional... for ($i = 0; $i < count($Datos); $i++) { for ($j = 1; $j < 9; $j++) { $s[$i] += $Datos[$i]["Valor" . $j]; } }
    – Vicente
    Commented el 5 nov. 2018 a las 20:39
  • @Vicente Me acabo de dar cuenta de que hay un fallo. Porque podría excederse el tamaño del array porque estoy usando el índice incorrecto. He corregido estas líneas: for ($j = 0; $j < 8; $j++) { $s[$j] += $Datos[$i]["Valor" . ($j+1)]; Revisa el código de nuevo.
    – Alvaro Montoro
    Commented el 5 nov. 2018 a las 20:55
  • @Vicente después de corregir eso, he realizado un par de pruebas y funciona sin problemas (o al menos eso parece).
    – Alvaro Montoro
    Commented el 5 nov. 2018 a las 21:02
  • Pues por más que intento me esta dando valor de 0 y en el normal me carga los valores sin problema.
    – Vicente
    Commented el 5 nov. 2018 a las 21:32
  • ¿Cómo los cargas? Añade eso también a la pregunta. Por favor lee Cómo preguntar y cómo crear un ejemplo mínimo verificable.
    – Alvaro Montoro
    Commented el 5 nov. 2018 a las 21:39
2

Si quieres hacerlo de esa manera puedes utilizar variables variables http://php.net/manual/es/language.variables.variable.php Saludos!

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.