Como nota, el collect()
no hace falta sobre el resultado de PrivateService::get();
pues la consulta ya te devolverá una colección.
Objetivo: llevar un contador de los servicios para cada estatus.
No necesitas iterar para contar, ni el if
para filtrar. Puedes asignar el resultado directamente a la variable, filtrando la colección con un where()
y aplicandole count()
, que devuelve cantidad de elementos.
$collection = PrivateService::get();
$activeServices = $collection->where('status','=', ServiceStatus::$active)->count();
$canceledServices = $collection->where('status','=', ServiceStatus::$canceled)->count();
$completedServices = $collection->where('status','=', ServiceStatus::$finished)->count();
Observaciones: No hay error, dentro de la función actualiza la variable, pero fuera, siempre da 0
Siempre da 0 porque es el valor que le asignaste a $activeServices
en ese ámbito, la $activeServices
que usas como contador está limitada al ámbito local de esa función. Podes leer más en Ámbito de las variables.
Si por alguna razón, aún quieres cambiar el valor de una variable externa
dentro de la función, podés pasarle a cada variable local de la función, la referencia a la variable original con este símbolo &
, de modo que cuando se modifique dentro de la función la variable asignada a la referencia, se cambie también el valor de la variable original.
$activeServices = 0;
$canceledServices = 0;
$completedServices = 0;
PrivateService::all()->each( function ( $item ) use ( &$activeServices, &$canceledServices, &$completedServices ) {
if( $item['status'] == ServiceStatus::$active ) $activeServices++;
if( $item['status'] == ServiceStatus::$finished ) $completedServices++;
if( $item['status'] == ServiceStatus::$canceled ) $canceledServices++;
});