1

Edit: Pongo la salida de algunas variables que se ven en el codigo

$ventas:

array(18) { [0]=> array(20) 
            { ["id"]=> string(1) "9" 
            [0]=> string(1) "9" ["codigo"]=> string(5) "10001" 
            [1]=> string(5) "10001" ["id_cliente"]=> string(1) "1" 
            [2]=> string(1) "1" ["id_vendedor"]=> string(2) "17" 
            [3]=> string(2) "17" ["productos"]=> string(107) "[{"id":"61","descripcion":"Cera EDItada","cantidad":"1","stock":"68","precio":"10841.6","total":"10841.6"}]" 
            [4]=> string(107) "[{"id":"61","descripcion":"Cera EDItada","cantidad":"1","stock":"68","precio":"10841.6","total":"10841.6"}]" ["servicios"]=> string(160) "[{"descripcion":"Perfilado","precio":"250","id":"27"},{"descripcion":"Otros","precio":"299","id":"31"},{"descripcion":"Corte y barba","precio":"222","id":"32"}]" 
            [5]=> string(160) "[{"descripcion":"Perfilado","precio":"250","id":"27"},{"descripcion":"Otros","precio":"299","id":"31"},{"descripcion":"Corte y barba","precio":"222","id":"32"}]" ["precio_productos"]=> string(7) "10841.6" 
            [6]=> string(7) "10841.6" ["precio_servicios"]=> string(3) "771" 
            [7]=> string(3) "771" ["metodo_pago"]=> string(8) "Efectivo" 
            [8]=> string(8) "Efectivo" ["fecha"]=> string(19) "2020-01-01 16:40:15" 
            [9]=> string(19) "2020-01-01 16:40:15" 
         }

$usuarios

array(2) { [0]=> array(18)
         {["id"]=> string(1) "1" 
          [0]=> string(1) "1" ["nombre"]=> string(21) "Usuario Administrador" 
          [1]=> string(21) "Usuario Administrador" ["usuario"]=> string(6) "admin1" 
          [2]=> string(6) "admin1" ["password"]=> string(60) "$2a$07$hsdfhdhf..." 
          [3]=> string(60) "$2a$07$hsdfhdhfsdhfh27..." ["perfil"]=> string(8) "Especial" 
          [4]=> string(8) "Especial" ["foto"]=> string(34) "vistas/img/usuarios/admin1/772.png" 
          [5]=> string(34) "vistas/img/usuarios/admin1/772.png" ["estado"]=> string(1) "1"
          [6]=> string(1) "1" ["ultimo_login"]=> string(19) "2020-03-11 09:16:39" 
          [7]=> string(19) "2020-03-11 09:16:39" ["fecha"]=> string(19) "2020-03-11 09:16:39" 
          [8]=> string(19) "2020-03-11 09:16:39" 
        }

Estoy intentando recuperar en un arrays todas las ventas que realizo el usuario. Al array quiero agregarle los pares nombreUsuario => precioProductos

Al intentar el siguiente código, yo esperaba que el array arrayProdVendedores me traiga las 15 ventas que el usuario realizó, según lo que tengo en la base de datos. Pero la linea var_dump($arrayProdVendedores) me devuelve un array con un único elemento, que es la ultima venta realizada.

(Ya le hice var_dump a las variables usuarios y ventas y todo lo que traen es correcto.)

$ventas = ControladorVentas::ctrMostrarVentas($item,$valor);
$usuarios = ControladorUsuarios::ctrMostrarUsuarios($item,$valor);

$arrayVendedores = array();
$arrayProdVendedores = array();



foreach($ventas as $key => $valueVentas){

    foreach ($usuarios as $key => $valueUsuario){

        if($valueUsuario["id"] == $valueVentas["id_vendedor"]){


            #Capturamos nombres y precios en un mismo array
            $arrayProdVendedores[$valueUsuario["nombre"]] = $valueVentas["precio_productos"];

        } 


   }


}


var_dump($arrayProdVendedores);
16
  • 1
    Creo que podrías resolverlo de este modo: $subArray=array ($valueVentas["precio_productos"], $valueVentas["precio_servicios"]); $arrayServVendedores[$valueUsuario["nombre"]] [] = $subArray; He usado un $subArray para más claridad. La diferencia básica es esto [] en $arrayServVendedores que evitará que a cada paso se sobre-escriban los datos. Y, te recomiendo que uses nombres de variables menos kiloméeeeetricos.
    – A. Cedano
    Commented el 11 mar. 2020 a las 17:05
  • 1
    Jaja muchas gracias, en breve lo pruebo y les cuento como me fue. Me olvido fácil para qué sirven mis códigos, prefiero variables claras que ganar un code golf
    – Agustin G.
    Commented el 11 mar. 2020 a las 18:17
  • 2
    Mira la diferencia, si cambias $arrayProdVendedores por $mVendors o cambias $valueUsuario por $mName o cambias $valueVentas por $mSales, etc. Cuando veas las variables sabrás en seguida de qué se trata y si adoptas una convención de nombre en inglés las variables no sólo serán más cortas, sino que el código queda internacionalizado. Antes yo tenía la costumbre de indicar el tipo de dato que era, si array, si string, si entero... pero un día leí no sé donde que eso no servía para nada realmente y con el tiempo he comprendido que es cierto. De todos modos es una opción personal.
    – A. Cedano
    Commented el 11 mar. 2020 a las 18:28
  • 1
    Te agradezco, lo voy a tener en cuenta para las cosas que haga de aqui en adelante. Tu comentario/respuesta no me funciono, cabe aclarar que ya habia intentado ponerle [] al final del nombre del array y no tuve exito. De todas maneras no entiendo que estoy haciendo en $arrayServVendedores[$valueUsuario["nombre"]] [] = $subArray;, en esa linea yo deberia guardarle solo el precio de los servicios, y segun lo que me pasaste, estoy guardando los dos.
    – Agustin G.
    Commented el 11 mar. 2020 a las 18:32
  • 1
    En el código original querías guardar los dos valores ¿no? En mi código es lo mismo, sólo que yo crée en $subArray un array con los dos valores y luego se lo paso al array final. Y el uso de [] es para que anide un sub-array en cada iteracción, solventando el problema principal que era que se sobre-escribía siempre el mismo índice. De todos modos leyendo tu pregunta da la impresión que mareas demasiado los datos y creo que todo el proceso podría simplificarse, pero para ayudarte en ese sentido habría que entender el contexto completo y tener una pequeña muestra de los datos.
    – A. Cedano
    Commented el 11 mar. 2020 a las 18:37

5 respuestas 5

1

He sacado en limpio datos esenciales de cada array y el código funciona como te había dicho anteriormente.

Veamos:

$usuarios=
    array (
            array ("id"=>"1", "nombre"=>"Usuario Administrador"), 
            array ("id"=>"17","nombre"=>"Pepe")
        );

$ventas=array 
    (
        array (
                "id_vendedor"=>"17", 
                "precio_productos"=>"1",
                "precio_servicios"=>"10"                
            ),
        array (
                "id_vendedor"=>"17", 
                "precio_productos"=>"2",
                "precio_servicios"=>"20"                

            ),
        array (
                "id_vendedor"=>"18", 
                "precio_productos"=>"3",
                "precio_servicios"=>"30"                

            ),
        array (
                "id_vendedor"=>"17", 
                "precio_productos"=>"4",
                "precio_servicios"=>"40"                

            ),
    );

$mUser=array();
foreach($ventas as $key => $valueVentas){
    foreach ($usuarios as $key => $valueUsuario){
        if($valueUsuario["id"] == $valueVentas["id_vendedor"]){
            $subArray=array($valueVentas["precio_productos"], $valueVentas["precio_servicios"]);
            $mUser[$valueUsuario["nombre"]][]=$subArray;
        }
    }
}
var_dump($mUser); 

La salida es esta:

array(1) {
  ["Pepe"]=>
  array(3) {
    [0]=>
    array(2) {
      [0]=>
      string(1) "1"
      [1]=>
      string(2) "10"
    }
    [1]=>
    array(2) {
      [0]=>
      string(1) "2"
      [1]=>
      string(2) "20"
    }
    [2]=>
    array(2) {
      [0]=>
      string(1) "4"
      [1]=>
      string(2) "40"
    }
  }
}

He puesto datos ficticios. El vendedor Pepe tiene asociados los precios de productos y servicios. Si quieres ponerle un nombre a cada clave o bien organizar el resultado de otro modo me dices.

Aquí se ve más claro, poniendo nombres a cada clave:

$mUser=array();
foreach($ventas as $key => $valueVentas){
    foreach ($usuarios as $key => $valueUsuario){
        if($valueUsuario["id"] == $valueVentas["id_vendedor"]){
            $subArray=array("productos"=>$valueVentas["precio_productos"], "servicios"=>$valueVentas["precio_servicios"]);
            $mUser[$valueUsuario["nombre"]][]=$subArray;
        }
    }
}
var_dump($mUser);    

Salida:

array(1) {
  ["Pepe"]=>
  array(3) {
    [0]=>
    array(2) {
      ["productos"]=>
      string(1) "1"
      ["servicios"]=>
      string(2) "10"
    }
    [1]=>
    array(2) {
      ["productos"]=>
      string(1) "2"
      ["servicios"]=>
      string(2) "20"
    }
    [2]=>
    array(2) {
      ["productos"]=>
      string(1) "4"
      ["servicios"]=>
      string(2) "40"
    }
  }
}
7
  • No por nada tienes 65.2K de reputacion. Alabado seas, gracias
    – Agustin G.
    Commented el 11 mar. 2020 a las 20:21
  • @Mani me alegro de que te haya servido. De todos modos revisa el tipo de fetch que estás aplicando para traer los datos. Tienes un montón de datos repetidos que lo único que hacen es sobrecargar la memoria. Eso puede ser problemático, sobre todo en conexiones concurrentes o en consultas que manejen muchos datos. Saludos.
    – A. Cedano
    Commented el 11 mar. 2020 a las 20:23
  • Lo hice siguiendo los pasos de un tutorial de udemi, en el modelo las unicas 3 lineas de la funcion mostrarVentas son: 1) $stmt = Conexion::conectar()->prepare("SELECT * FROM $tabla ORDER BY id ASC"); - 2) $stmt -> execute(); - 3) return $stmt -> fetchAll();
    – Agustin G.
    Commented el 11 mar. 2020 a las 20:36
  • Luego, desde el controlador, unicamente tengo $respuesta = ModeloVentas::mdlMostrarVentas($tabla,$item,$valor); y un return $respuesta;... tendria que aplicarle alguna funcion a esta $respuesta para que los datos se organicen de una forma mas optima?
    – Agustin G.
    Commented el 11 mar. 2020 a las 20:39
  • @Mani En la etapa 3) que mencionas, puedes pasarle PDO::FETCH_ASSOC al método fetchAll() para que te traiga los datos únicamente como array asociativo. Por ejemplo: return $stmt -> fetchAll(PDO::FETCH_ASSOC);
    – A. Cedano
    Commented el 11 mar. 2020 a las 22:03
1

Solo agrega [] y con eso se soluciona tu problema.

$arrayProdVendedores[$valueUsuario["nombre"]][] = $valueVentas["precio_productos"];
$arrayServVendedores[$valueUsuario["nombre"]][] = $valueVentas["precio_servicios"];

Saludos

4
  • 1
    No, ya lo había probado sin exito a eso.
    – Agustin G.
    Commented el 11 mar. 2020 a las 18:34
  • 1
    O ya intentaste con un array_push? Commented el 11 mar. 2020 a las 18:40
  • 1
    $arrayProdVendedores = array_push($valueUsuario["nombre"], $valueVentas["precio_productos"]);
    – Agustin G.
    Commented el 11 mar. 2020 a las 18:45
  • 1
    un var dump a eso retorna NULL
    – Agustin G.
    Commented el 11 mar. 2020 a las 18:46
1

Acabo de probar esto

$valueUsuario["nombre"] = "Gerardo";
$valueVentas["precio_productos"] = 1;
$valueVentas["precio_servicios"] = 1;

for ($i=0; $i < 10; $i++) { 
    $arrayProdVendedores[$valueUsuario["nombre"]][] = $valueVentas["precio_productos"];
    $arrayServVendedores[$valueUsuario["nombre"]][] = $valueVentas["precio_servicios"];
    $valueVentas["precio_productos"] += 1;
    $valueVentas["precio_servicios"] += 1;
}

    print_r($arrayProdVendedores);

    print_r($arrayServVendedores);

Y me da como resultado esto y creo que es lo que buscas

Array
(
    [Gerardo] => Array
        (
            [0] => 1
            [1] => 2
            [2] => 3
            [3] => 4
            [4] => 5
            [5] => 6
            [6] => 7
            [7] => 8
            [8] => 9
            [9] => 10
        )

)
Array
(
    [Gerardo] => Array
        (
            [0] => 1
            [1] => 2
            [2] => 3
            [3] => 4
            [4] => 5
            [5] => 6
            [6] => 7
            [7] => 8
            [8] => 9
            [9] => 10
        )

)
1

yo creo que el problema viene porque este valor $valueUsuario["nombre"] es siempre el mismo entonces el lugar de añadir todos los registros los va machacando por eso sale siempre el ultimo. prueba intentado no generar un array asociativo o con la funcion push de php

1
  • 1
    En realidad no, porque la variable "ventas" me retorna muchos elementos, algunos tienen un id_vendedor y otros tiene uno diferente, ya que tengo ventas realizadas de ambos vendedores. La cosa es que por cada uno de ellos, me muestra la ultima venta en lugar de ir agregando, nisiquiera quiero que los sume, quiero que quede algo como ["oscar" => 200, "oscar" =>500, "agustin" => 400, "oscar" =>200]
    – Agustin G.
    Commented el 11 mar. 2020 a las 19:06
0

El problema es que necesitas un array, no lo estás inicializando y solo estás agregando un valor.

En versiones anteriores de PHP podías agregar elementos a un array no definido sin mayor problema, en versiones recientes ya no es posible.

$array[] = 'algo'; // En versiones anteriores funcionaba

$array = []; // Ahora debes inicializar
$array[] = 'algo'; // Solo así puedes agregar elementos

Solución a tu problema:

$ventas = ControladorVentas::ctrMostrarVentas($item,$valor);
$usuarios = ControladorUsuarios::ctrMostrarUsuarios($item,$valor);

$arrayVendedores = array();
$arrayProdVendedores = array();

foreach($ventas as $key => $valueVentas){

    foreach ($usuarios as $key => $valueUsuario){

        if($valueUsuario["id"] == $valueVentas["id_vendedor"]){
            // Verificar que existe el array
            if(!isset($arrayProdVendedores[$valueUsuario["nombre"]])) {
                // No existe, entonces se crea
                $arrayProdVendedores[$valueUsuario["nombre"]] = [];
            }

            #Capturamos nombres y precios en un mismo array
            // Con los corchetes al final agregas un nuevo elemento
            $arrayProdVendedores[$valueUsuario["nombre"]][] = $valueVentas["precio_productos"];

        } 
   }
}

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.