No meu Controller tenho o seguinte mapeamento para exibir o formulário de edição do cadastro de uma pessoa:
@GetMapping("/mostraEmpresaEdicao/{idpessoa}")
public ModelAndView mostraEmpresaEdicao(@PathVariable("idpessoa") Long idpessoa) {
Optional<Pessoa> pessoa = pessoaRepository.findById(idpessoa);
ModelAndView modelAndView = new ModelAndView("cadastro/editarpessoa");
modelAndView.addObject("pessoaobj", pessoa.get());
return modelAndView;
}
A pessoa é carregada no formulário editarpessoa.html. Qual a melhor forma de editar essa pessoa? Supondo que é possível editar todos os campos de uma pessoa. Eu posso colocar a ação desse formulário para o meu método de salvar, que eu também utilizo para salvar um novo cadastro?
@RequestMapping(method=RequestMethod.POST, value="/salvarpessoa")
public ModelAndView salvar(Pessoa pessoa) {
pessoaRepository.save(pessoa);
return pessoas();
}
OBS: O pessoas() é um método para listar os cadastros.
editarpessoa.html com a ação para /salvarpessoa, utilizada também em novos cadastros:
<form action="#"
th:action="@{/salvarpessoa}"
th:object="${pessoaobj}"
method="post">
<label>Id</label>
<input name="id" readonly="readonly" th:field="*{id}">
<label>Nome</label>
<input name="nome" th:field="*{nome}">
<label>Sobrenome</label>
<input name="sobrenome" th:field="*{sobrenome}">
<button class="btn waves-effect waves-light" type="submit" name="salvar">Salvar
<i class="material-icons right">send</i>
</button>
</form>
Ou o mais correto é criar um mapeamento próprio para o editar, alterando a action do formulário para ele?
@PostMapping("/editarpessoa/{idpessoa}")
public String editar(Pessoa pessoaedicao) {
Optional<Pessoa> pessoa = pessoaRepository.findById(pessoaedicao.getId());
pessoa.get().setNome(pessoaedicao.getNome());
pessoa.get().setSobrenome(pessoaedicao.getSobrenome());
pessoaRepository.save(pessoa.get());
return "cadastro/cadastropessoa";
}
Caso o ideal seja criar um método separado, é correto utilizar o set() para alterar os campos da entidade e depois o save(), ou não há problema em usar apenas o save()? No fundo, o Spring Data vai ver que o ID já existe e vai dar o update?
@PostMapping("/editarpessoa/{idpessoa}")
public String editar(@PathVariable("idpessoa") Long idpessoa, Pessoa pessoaedicao) {
Optional<Pessoa> pessoa = pessoaRepository.findById(idpessoa);
pessoaRepository.save(pessoa.get());
return "cadastro/cadastropessoa";
}