Variables vs Propiedades de Instancia
El error en tu intento fue que declaraste tus colecciones como si fuesen una variable cualquiera:
public function __construct() {
$ajustes = Ajustes::All();
$categorias = Categoria::All();
}
Acción que limita su existencia al scope del constructor. Apenas éste se ejecuta la referencia a ellas se pierde y quedan a dispósición del garbage collector como memoria liberable.
(Edit: esto en lo que respecta a fast-cgi. Si se usa un gestor como php-pm o roadrunner seguirán en alguna parte del éter hasta reiniciar el proceso)
Para adosar estas colecciones como propiedad de la instancia, esto se asigna con $this->{nombre}
.
/** @var \Illuminate\Database\Eloquent\Collection */
protected $ajustes;
/** @var \Illuminate\Database\Eloquent\Collection */
protected $categorias;
public function __construct() {
$this->ajustes = Ajustes::All();
$this->categorias = Categoria::All();
}
Ojo que lo correcto es declarar esas propiedades de instancia con su respectiva visibilidad. Si estás usando PHP 7.4 puedes además declarar su tipo sin tener que recurrir a las anotaciones en docblocks.
(Al dejarlas como protected
las hago visibles para cualquier controller que herede del actual y no necesite declarar nada distinto en su constructor).
Comportamiento de compact
Esto nos lleva a un segundo problema, y es que, si bien podrás evitar ejecutar la consulta {Model}::All
en cada método, tu uso de compact
asume que hay dos variabes en el scope con nombres $ajustes
y $categorias
y te oligará a ponerlas como tal
public function index(Request $request){
$ajustes = $this->ajustes;
$categorias = $this->categorias;
return view('welcome',compact('ajustes','categorias'));
}
Puedes modificar eso para darle a la vista un array de variables explícito:
public function index(Request $request){
return view('welcome', [
'ajustes' => $this->ajustes,
'categorias' => $this->categorias
]);
}
Compartir variables a todas las vistas
Si los otros métodos usan esas mismas variables para sus respectivas vistas, tendrás código repetido:
public function item(Request $request, int $item_id){
return view('item', [
'ajustes' => $this->ajustes,
'categorias' => $this->categorias
'item_id' => $item_id
]);
}
Para este caso sería más limpio compartir esos valores a todas las vistas usando el método share
en el facade View
<?php
namespace App\Http\Controllers;
use Illuminate\Support\Facades\View;
class EjemploController extends Controller {
/** @var \Illuminate\Database\Eloquent\Collection */
protected $ajustes;
/** @var \Illuminate\Database\Eloquent\Collection */
protected $categorias;
public function __construct() {
$this->ajustes = Ajustes::All();
$this->categorias = Categoria::All();
View::share('ajustes', $this->ajustes);
View::share('categorias', $this->categorias);
}
public function item(Request $request, int $idem_id){
return view('idem_id', [
'idem_id' => $idem_id
]);
}
El método EjemploController::item
ahora sólo necesita declarar en la vista sus variables particulares. Las otras dos ya están inyectadas
Caché de Colecciones
Como bien dice @Omar Alvarado, poner la consulta en el constructor implica ejecutarla aunque quieras mostrar una vista que no utiliza esos datos.
Lo correcto sería, en cambio, poner esa operación en el método handle
de un middleware. En particular, de un middleware que intervenga solamente en el grupo de rutas que utiliza tus variables de selectores.
Pero, siendo bien específicos, para este caso de uso sería incluso mejor aprovechar que parece tratarse de entidades que se modifican muy de vez en cuando, y dejar las colecciones en caché.
Dónde poner ese caché? Para algo tan general se podría poner en el método boot
de AppServiceProvider
use Illuminate\Support\Facades\Cache;
use Illuminate\Support\Facades\View;
use Illuminate\Support\ServiceProvider;
class AppServiceProvider extends ServiceProvider
{
/**
* Bootstrap any application services.
*/
public function boot(): void
{
$categorias = Cache::remember('categorias', 600, function () {
return Categoria::all();
});
View::share('categorias', $categorias);
}
}
Pero si implementas la idea de manejar todo esto en un middleware, entonces ahí es el lugar óptimo