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.
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.