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";
}
Verificando o manual, o método save faz o persist se a entidade for nova, e o merge se ela já existir:
@Transactional
public <S extends T> S save(S entity) {
if (this.entityInformation.isNew(entity)) {
this.em.persist(entity);
return entity;
} else {
return this.em.merge(entity);
}
}
Então acredito que se os dados que eu vou editar são os mesmos de um novo cadastro, não uso algum tipo de form apenas com alguns desses dados, tanto faz setar os dados primeiro e depois salvar a entidade, ou usar apenas o save().
Optional<Pessoa> pessoa = pessoaRepository.findById(pessoaedicao.getId()); pessoa.get().setNome(pessoaedicao.getNome()); pessoa.get().setSobrenome(pessoaedicao.getSobrenome()); pessoaRepository.save(pessoa.get());
Como eu disse, apenas usando o save() funciona. Existe o mais correto? Carregar, setar e salvar, ou apenas salvar como abaixo?pessoaRepository.save(pessoaedicao);