0

creo un array donde tiene años (2012,2013..) y al querer sacar los elementos de la tabla no me arroja nada este es el codigo que uso

for($i=0;$i<count($array);$i++){
    $sql = "SELECT SUM(`VENTAS`) VENTAS FROM EDORESULTADO WHERE ANO='".$array[$i]."'";
    $resultado=mysqli_query($conexion,$sql);
    $array=array();
    $index=0;
    while($row=mysqli_fetch_assoc($resultado)){    
        echo '<td class="tg-s6z2">'.$row['VENTAS'].'</td>';

    }
}

y lo mas curioso es que si quito el array si funciona

$sql = "SELECT SUM(`VENTAS`) VENTAS FROM EDORESULTADO WHERE ANO='2012'";
10
  • podrias ver la estructura de $array, mediante var_dump($array), primero hay que saber si de verdad estas accediendo a los valores de años el 22 sep. 2018 a las 19:44
  • ya lo hice si puedo acceder al array de hecho imprimo la variable y la sentencia es correcta si imprime SELECT SUM(VENTAS) VENTAS FROM EDORESULTADO WHERE ANO='2012'; pero no se al ejecutar no lo hace la sentencia el 22 sep. 2018 a las 19:47
  • Podrias postear lo que te arroja acceder a tu $array completo? Eso enriquecera tu publicacion el 22 sep. 2018 a las 19:50
  • echo var_dump($arra); este es con el dump array(7) { [0]=> string(9) "2012" [1]=> string(9) "2013" [2]=> string(9) "2014" [3]=> string(9) "2015" [4]=> string(9) "2016" [5]=> string(9) "2017" [6]=> string(9) "2018" } el 22 sep. 2018 a las 19:53
  • 1
    Si tu array se llama $array realmente (¿no se llamará $arra?), puedes hacerlo así: foreach($array as $v){ $sql = "SELECT SUM(VENTAS) VENTAS FROM EDORESULTADO WHERE ANO=$v"; // ... resto del código } Si la columna ANO es VARCHAR entonces pones las comillas simples: $sql = "SELECT SUM(VENTAS) VENTAS FROM EDORESULTADO WHERE ANO='$v";`. De todos modos, tu código es no óptimo (no es buena práctica ejecutar consultas en bucles), y además es inseguro (deberías usar consultas preparadas). Pero esas son dos cuestiones aparte.
    – A. Cedano
    el 22 sep. 2018 a las 19:59

1 respuesta 1

1

Vamos a suponer que este es tu array (por favor verifica que los nombres de variable sean correctos);

$array=array(2012,2013,2014,2015);

Puedes sacar los valores del array separados por coma con implode:

$sqlParams=implode(",",$array);

Y modificar la consulta, usando IN. De ese modo te evitas enviar una consulta por cada elemento (lo cual afectaría el rendimiento). El código va así tal cual, sin necesidad del antiguo bucle for o foreach:

$sql = "SELECT SUM(`VENTAS`) VENTAS FROM EDORESULTADO WHERE ANO IN($sqlParams)";
$resultado=mysqli_query($conexion,$sql);
while($row=mysqli_fetch_assoc($resultado)){    
    echo '<td class="tg-s6z2">'.$row['VENTAS'].'</td>';

}

Si en tu base de datos la columna ANO no es numérica, puedes crear los parámetros IN rodeados de comillas simples del siguiente modo:

$sqlParams = "'" . implode ( "', '", $array ) . "'"; 

Aquí la consulta sería así:

SELECT 
    SUM(`VENTAS`) VENTAS 
FROM EDORESULTADO 
WHERE ANO IN('2012', '2013', '2014', '2015');

Si por el contrario, ANO es del tipo INT, lo dejas como está al principio. La consulta que se va a producir se parecerá a esto:

SELECT 
    SUM(`VENTAS`) VENTAS 
FROM EDORESULTADO 
WHERE ANO IN(2012, 2013, 2014, 2015);

Nota sobre la seguridad: Tu código puede ser altamente vulnerable a ataques de inyección SQL. Cuando se manejan datos que proviene de otras fuentes conviene el uso de consultas preparadas.

2
  • muchas gracias A.Cedano lo que necesito es sacar la suma de cada año separado pero pueden variar los año es por eso que saco primero los año y los agrego a un array despues hacer el for o el foreach para que la sentencia cambie los años y asi sacarlos por cada año el 22 sep. 2018 a las 20:23
  • @ANDRESPALMAHERNANDEZ realmente no entendí bien lo que quieres en realidad ¿podrías explicarlo mejor en la pregunta, poniendo un ejemplo? Si es como dices entonces lo que (parece) que habría que hacer es agrupar por años... sería algo totalmente distinto entonces. Intenta esto y me cuentas: SELECT SUM(VENTAS) VENTAS FROM EDORESULTADO WHERE ANO IN($sqlParams) GROUP BY ANO
    – A. Cedano
    el 22 sep. 2018 a las 20:25

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.