Un consejo, crea los casos de uso por separado de tus controladores.
Creemos una clase CasosUsoProductos
y dentro del él el caso de uso setStock
con los parametros $id
del producto y $cantidad
class CasoUsoProductos{
public static setStock($id,$cantidad){
$producto=Productos::find($id);//conseguimos el producto a editar
//validamos la existencia de valor
if($cantidad <= 0 && $producto->cantidad < ($cantidad*-1)){
throw new Exception('Cantidad no existente');
}
$producto->cantidad=$producto->cantidad+$cantidad;
$producto->save();
return $producto;
}
public function agregarACarrito($id,$id_carrito,$cantidad){
...
}
}
Llamamos a nuestra función de la siguiente manera:
public function update(Request $request,$id){
$producto=CasoUsoProductos::setStock($id,$request->cantidad);
return redirect('retirar_producto');
}
No olvides importar el namespace de CasoUsoProductos
.
Dicha funcion funcionaria mas o menos asi:
Producto_ID=1, stock de 8: CasoUsoProductos::setStock(1,2); -> stock resultante 10
Producto_ID=4, stock de 10: CasoUsoProductos::setStock(4,-3); -> stock resultante 7
Producto_ID=5, stock de 3: CasoUsoProductos::setStock(5,-7); -> error:Cantidad no existente
Producto_ID=7, stock de 4: CasoUsoProductos::setStock(7,0); -> error:Cantidad no existente
En todo caso si no quieres ordenar tu codigo y deseas mantener tu formato solo edita la funcion update
;
public function update(Request $request,$id){
$cantidad=$request->cantidad; //vefirica que el numero sea negativo o solo lograras agregar y no restar
$producto=Productos::find($id);
if($cantidad <= 0 && $producto->cantidad < ($cantidad*-1)){
throw new Exception('Cantidad no existente');
}
$producto->cantidad=$producto->cantidad+$cantidad;
$producto->save();
return redirect('retirar_producto');
}
Productos::
ya estás teniendo una referencia a la tabla, lo que haces es repetirlo dos veces de distinta forma.