Estoy intentando actualizar un registro de la forma en la que se muestra en la documentación de laravel y también de algunos sitios web y vídeos que miré, pero no pude, no me sale ningun error pero tampoco se actualizan los campos.
Entonces pensé que de la manera como agregro un usuario a la base de datos,
public function store(Request $request)
{
$usuarios = new Usuario();
$usuarios->nombre = $request->Nombre;
$usuarios->apellidos = $request->Apellidos;
$usuarios->sexo = $request->Sexo;
$usuarios->edad = $request->Edad;
$usuarios->correo = $request->Correo;
$usuarios->save();
return redirect('usuarios');
}
también podría actualizarlo solo cambiando la primera línea de la función update
public function edit($id)
{
$usuario = Usuario::findOrFail($id);
return view('usuarios.edit' , compact('usuario'));
}
/**
* Update the specified resource in storage.
*
* @param \Illuminate\Http\Request $request
* @param int $id
* @return \Illuminate\Http\Response
*/
public function update(Request $request, $id)
{
$usuario = Usuario::findOrFail($id);
$usuario->nombre = $request->Nombre;
$usuario->apellidos = $request->Apellidos;
$usuario->sexo = $request->Sexo;
$usuario->edad = $request->Edad;
$usuario->correo = $request->Correo;
$usuario->save();
return redirect('usuarios');
}
cabe resaltar que de esa manera me actualiza los datos perfectamente , pero mi pregunta es: ¿está mal actualizarlo de esa manera? o si no es una mala práctica hacerlo de esa manera.
= new User()
estas creando un nuevo registro, pero por el contrario si deseas actualzar entonces no creas una nueva instancia de tu claseUser
sino que como muestras casi al final debes hacer$user = User::findOrFail();
para de este modo actualizar siempre y cuando encuentre coincidencia por ejemplo por medio del$id
y en caso contrario mostrar una excepción