1

Tengo el siguiente código en mi controlador de Laravel:

public function show(){
    $user=User::where("id",Auth()->id())->first();
    $vendedores1=User::where("sponsor",Auth()->id())->get();
    $amount1=User::where("sponsor",Auth()->id())->count();
    $amount2=0;
    $amount3=0;
    $vendedores2;
    $vendedores3;
    foreach ($vendedores1 as $vendedor1) {
        $vendedores2=User::where("sponsor",$vendedor1->id)->get();
        $amount2=+User::where("sponsor",$vendedor1->id)->count();
            foreach ($vendedores2 as $vendedor2) {
                $vendedores3=User::where("sponsor",$vendedor2->id)->get();
                $amount3=+User::where("sponsor",$vendedor2->id)->count();
            }
    }
    $cantidad=$amount1+$amount2+$amount3;
    return view("/fuerzadeventa",compact("user","vendedores1","vendedores2","vendedores3","amount1","amount2","amount3","cantidad"));
}

Cuando pongo el dd($vendedores2) dentro del scope del foreach, éste tiene contenido, pero al poner el mismo dd() fuera del foreach (después de éste), me arroja un arreglo vacío.

dd($vendedores2) dentro del foreach:

introducir la descripción de la imagen aquí

dd($vendedores2) fuera del foreach:

introducir la descripción de la imagen aquí

17
  • ¿Estás seguro que dd($vendedores2) tras el foreach no te arroja el mismo resultado que el último dd($vendedores2) de dentro? Commented el 28 jun. 2018 a las 6:08
  • Sí, estoy seguro, me parece muy extraño en realidad. Commented el 28 jun. 2018 a las 6:11
  • Agregué imágenes de lo que me lanza el dd() dentro y fuera del scope Commented el 28 jun. 2018 a las 6:15
  • Tendría que ver la implementación de la clase User. Sólo se me ocurre que estés guardando resultados o algo en variables estáticas y consiguientes llamadas a User::where estén modificando su valor. Commented el 28 jun. 2018 a las 6:15
  • Cual sería la forma correcta para que los resultados se vayan sumando a una variable? Commented el 28 jun. 2018 a las 6:16

1 respuesta 1

1

En tu código hay que cambiar o mejorar lo siguiente:

  • Existen algunas consultas repetidas que pueden unirse en una o eliminarse (un get() seguido de un count() de la misma consulta).
  • Estás usando =+ en vez de += para sumar datos.
  • El contenido de las variables $vendedores2 y $vendedores3 contendrá el valor que tuvieron durante la última iteración y no el agregado.

Tu código podría quedar tal que así:

<?php
public function show()
{
    $user = User::where("id",Auth()->id())->first();
    /* En vez de usar colecciones usaremos matrices */
    $vendedores1 = User::where(
      'sponsor',
      Auth()->id()
    )->get()->all();
    $vendedores2 = [];
    $vendedores3 = [];
    foreach ($vendedores1 as $vendedor1) {
        $vendedores = User::where(
          'sponsor',
          $vendedor1->id
        )->get()->all();
        foreach ($vendedores as $vendedor) {
            /* Agregamos sin repetición al vendedor */
            $vendedores2[$vendedor->id] = $vendedor;
            $subvendedores = User::where(
              'sponsor',
              $vendedor->id
            )->get()->all();
            foreach($subvendedores as $subvendedor) {
                $vendedores3[$subvendedor->id] = $subvendedor;
            }
        }
    }
    /* En vez de repetir la consulta User::where("sponsor",Auth()->id())->get()
      usamos count con tu resultado */
    $amount1 = count($vendedores1);
    /* Lo mismo con el resto, no hace falta ir sumando porque a estas alturas
       tendremos el total */
    $amount2 = count($vendedores2);
    $amount3 = count($vendedores3);
    $cantidad = $amount1 + $amount2 + $amount3;
    return view(
      "/fuerzadeventa",
      compact(
        "user",
        "vendedores1",
        "vendedores2",
        "vendedores3",
        "amount1",
        "amount2",
        "amount3",
        "cantidad"
      )
    );
}

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.