Buenas.
Tengo el siguiente código:
<?php
class ControladorDeEjemplo extends Controller
{
public function store(StoreData $request)
{
$data = $request->all();
/*
bloque de
instrucciones
bastante extenso 1
*/
if ($condicionDeError) {
return response()->json(['success' => false, 'error' => 'Descripcion del error']);
}
/*
bloque de
instrucciones
bastante extenso 2
*/
return response()->json(['success' => true]);
}
}
y quiero sacar el bloque de instrucciones a otra función, ya que el método original que quiero arreglar tiene mas de 120 lineas de extensión. Entonces quiero hacer algo como esto:
<?php
class ControladorDeEjemplo extends Controller
{
public function store(StoreData $request)
{
$data = $request->all();
funcion($data);
/*
bloque de
instrucciones
bastante extenso 1
*/
return response()->json(['success' => true]);
}
public public function Funcion($datos)
{
/*
bloque de
instrucciones
bastante extenso 2
*/
$variable1;
$variable2;
if ($condicionDeError) {
return response()->json(['success' => false, 'error' => 'Descripcion del error']);
}
}
}
Pero algunos problemas que tengo son:
El primer bloque de instrucciones consulta a la base de datos y asigna las variables necesarias para que el
/* bloque de instrucciones bastante extenso 2 */
funcione (que en este ejemplo serían las$variable1
y$variable2
)Si no se usa
return
cuando se llama a la funciónFuncion()
dentro destore()
, no devuelve el error, pero si se usa, no continúa la rutina hasta el final, a pesar de que estén correctas las condiciones.
Esto se puede arreglar con una condición de esta forma. (esta es una idea)
<?php
if(get_class($result = funcion()) == "Response"){
return $result;
}
Pero no tengo forma de pasar los resultados de la función al contexto anterior de "store" sin convertirlas en variables de la clase.
Otra idea es usar excepciones y devolver los datos como arreglos. Como lo indica http-exceptions. Pero esto implica modificar el código que interpreta la salida en javascript.
Me gustaría saber si existe una forma de hacer esto elegantemente :P
$variable1
y$variable2
? ¿algún motivo para incluir más de 120 líneas en un método de un controlador?store
para hacerlo mas comprensible.