Entrar Ainda não tem acesso?
Solucionado (ver solução)
Solucionado
(ver solução)
6
respostas

Erro 500 Internal Server Error na requisição de retornar por Id

Referente ao curso Spring Boot API REST: construa uma API, no capítulo Métodos PUT, DELETE e tratamento de erro e atividade Detalhando tópicos

por Gabriel Mota Melo
| 48k xp | 4 posts
Estagiário de TI

Estou com um problema, na hora que faço a requisição por Id (método: detalhar) pois está me retornando o erro 500, só que quando eu faço a requisição do findAll(método: lista) ele me retorna todos os tópicos normalmente, segue os trechos:

Erro da requisição: Erro da requisição no postmanRequisição do findAll: Requisição do findAll

TopicosController:

package br.com.alura.forum.controller;

import br.com.alura.forum.controller.dto.DetalhesDoTopicoDTO;
import br.com.alura.forum.controller.dto.TopicoDTO;
import br.com.alura.forum.controller.form.TopicoForm;
import br.com.alura.forum.model.Topico;
import br.com.alura.forum.repository.CursoRepository;
import br.com.alura.forum.repository.TopicoRepository;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.http.ResponseEntity;
import org.springframework.web.bind.annotation.*;
import org.springframework.web.util.UriComponentsBuilder;

import javax.validation.Valid;
import java.net.URI;
import java.util.List;

@RestController
@RequestMapping(value = "/topicos")
public class TopicosController {

    @Autowired
    private TopicoRepository topicoRepository;

    @Autowired
    private CursoRepository cursoRepository;

    @GetMapping
    public List<TopicoDTO> lista(String nomeCurso) {
        if (nomeCurso == null){
            List<Topico> topicos = topicoRepository.findAll();
            return TopicoDTO.converter(topicos);
        } else {
            List<Topico> topicos = topicoRepository.findByCursoNome(nomeCurso);
            return TopicoDTO.converter(topicos);
        }
    }

    @PostMapping
    public ResponseEntity<TopicoDTO> cadastrar(@RequestBody @Valid TopicoForm form, UriComponentsBuilder uriComponentsBuilder){
        Topico topico = form.converter(cursoRepository);
        topicoRepository.save(topico);
        URI uri = uriComponentsBuilder.path("/topicos/{id}").buildAndExpand(topico.getId()).toUri();
        return ResponseEntity.created(uri).body(new TopicoDTO(topico));
    }

    @GetMapping(value = "/{id}")
    public DetalhesDoTopicoDTO detalhar(@PathVariable Long id){
        Topico topico = topicoRepository.getReferenceById(id);
        return new DetalhesDoTopicoDTO(topico);
    }

}

DetalhesDoTopicoDTO

package br.com.alura.forum.controller.dto;

import br.com.alura.forum.model.StatusTopico;
import br.com.alura.forum.model.Topico;

import java.time.LocalDateTime;
import java.util.ArrayList;
import java.util.List;
import java.util.stream.Collectors;

public class DetalhesDoTopicoDTO {

    private Long id;
    private String titulo;
    private String mensagem;
    private LocalDateTime dataCriacao;
    private String nomeAutor;
    private StatusTopico status;
    private List<RespostaDTO> respostas;

    public DetalhesDoTopicoDTO(Topico topico){
        this.id = topico.getId();
        this.titulo = topico.getTitulo();
        this.mensagem = topico.getMensagem();
        this.dataCriacao = topico.getDataCriacao();
        this.nomeAutor = topico.getAutor().getNome();
        this.status = topico.getStatus();
        this.respostas = new ArrayList<>();
        this.respostas.addAll(topico.getRespostas().stream().map(RespostaDTO::new).collect(Collectors.toList()));
    }

    public Long getId() {
        return id;
    }

    public String getTitulo() {
        return titulo;
    }

    public String getMensagem() {
        return mensagem;
    }

    public LocalDateTime getDataCriacao() {
        return dataCriacao;
    }

    public String getNomeAutor() {
        return nomeAutor;
    }

    public StatusTopico getStatus() {
        return status;
    }

    public List<RespostaDTO> getRespostas() {
        return respostas;
    }
}

Já revirei o fórum todo e a internet inteira em busca da solução desse problema e nada...Aguardo retorno!

6 respostas
por Rodrigo da Silva Ferreira Caneppele
| 5043k xp | 8841 posts
Instrutor Instrutor

Oi Gabriel,

Nessa linha:

Topico topico = topicoRepository.getReferenceById(id);

Não utilize o método getReferenceById, pois ele não carrega a entidade do banco de dados e sim devolve uma referência para a entidade contendo apenas o id. Esse método deve ser utilizado apenas quando desejamos carregar uma entidade para seta-la como relacionamento em outra entidade.

Para os casos onde desejamos carregar uma entidade pelo id e acessar de fato os seus atributos, utilize o método findById:

Topico topico = topicoRepository.findById(id).get();

Bons estudos!

por Gabriel Mota Melo
| 48k xp | 4 posts
Estagiário de TI

Então Rodrigo,

Alterei essa linha e o erro ainda persiste:

Erro postmanTopicosController :

package br.com.alura.forum.controller;

import br.com.alura.forum.controller.dto.DetalhesDoTopicoDTO;
import br.com.alura.forum.controller.dto.TopicoDTO;
import br.com.alura.forum.controller.form.TopicoForm;
import br.com.alura.forum.model.Topico;
import br.com.alura.forum.repository.CursoRepository;
import br.com.alura.forum.repository.TopicoRepository;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.http.ResponseEntity;
import org.springframework.web.bind.annotation.*;
import org.springframework.web.util.UriComponentsBuilder;

import javax.validation.Valid;
import java.net.URI;
import java.util.List;

@RestController
@RequestMapping(value = "/topicos")
public class TopicosController {

    @Autowired
    private TopicoRepository topicoRepository;

    @Autowired
    private CursoRepository cursoRepository;

    @GetMapping
    public List<TopicoDTO> lista(String nomeCurso) {
        if (nomeCurso == null){
            List<Topico> topicos = topicoRepository.findAll();
            return TopicoDTO.converter(topicos);
        } else {
            List<Topico> topicos = topicoRepository.findByCursoNome(nomeCurso);
            return TopicoDTO.converter(topicos);
        }
    }

    @PostMapping
    public ResponseEntity<TopicoDTO> cadastrar(@RequestBody @Valid TopicoForm form, UriComponentsBuilder uriComponentsBuilder){
        Topico topico = form.converter(cursoRepository);
        topicoRepository.save(topico);
        URI uri = uriComponentsBuilder.path("/topicos/{id}").buildAndExpand(topico.getId()).toUri();
        return ResponseEntity.created(uri).body(new TopicoDTO(topico));
    }

    @GetMapping(value = "/{id}")
    public DetalhesDoTopicoDTO detalhar(@PathVariable Long id){
        Topico topico = topicoRepository.findById(id).get();
        return new DetalhesDoTopicoDTO(topico);
    }

}
por Rodrigo da Silva Ferreira Caneppele
| 5043k xp | 8841 posts
Instrutor Instrutor

Agora o problema é por conta de no seu DTO você estar carregando junto as respostas do Topico, que é um relacionamento LAZY.

Nesse caso você vai precisar criar um método no repository para carregar o topico junto com as respostas então:

@Query("SELECT t FROM Topico t LEFT JOIN FETCH t.respostas")
public Topico carregarComRespostas(Long id);

E altere no controller para chamar esse método ao invés do findById.

por Gabriel Mota Melo
| 48k xp | 4 posts
Estagiário de TI

Cara, o erro ainda persiste:

Insira aqui a descrição dessa imagem para ajudar na acessibilidadeTopicoRepository:

package br.com.alura.forum.repository;

import br.com.alura.forum.model.Topico;
import org.springframework.data.jpa.repository.JpaRepository;
import org.springframework.data.jpa.repository.Query;

import java.util.List;

public interface TopicoRepository extends JpaRepository<Topico, Long> {

    List<Topico> findByCursoNome(String nomeCurso);

    @Query("SELECT t FROM Topico t LEFT JOIN FETCH t.respostas")
    public Topico carregarComRespostas(Long id);

}

TopicosController:

package br.com.alura.forum.controller;

import br.com.alura.forum.controller.dto.DetalhesDoTopicoDTO;
import br.com.alura.forum.controller.dto.TopicoDTO;
import br.com.alura.forum.controller.form.TopicoForm;
import br.com.alura.forum.model.Topico;
import br.com.alura.forum.repository.CursoRepository;
import br.com.alura.forum.repository.TopicoRepository;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.http.ResponseEntity;
import org.springframework.web.bind.annotation.*;
import org.springframework.web.util.UriComponentsBuilder;

import javax.validation.Valid;
import java.net.URI;
import java.util.List;

@RestController
@RequestMapping(value = "/topicos")
public class TopicosController {

    @Autowired
    private TopicoRepository topicoRepository;

    @Autowired
    private CursoRepository cursoRepository;

    @GetMapping
    public List<TopicoDTO> lista(String nomeCurso) {
        if (nomeCurso == null){
            List<Topico> topicos = topicoRepository.findAll();
            return TopicoDTO.converter(topicos);
        } else {
            List<Topico> topicos = topicoRepository.findByCursoNome(nomeCurso);
            return TopicoDTO.converter(topicos);
        }
    }

    @PostMapping
    public ResponseEntity<TopicoDTO> cadastrar(@RequestBody @Valid TopicoForm form, UriComponentsBuilder uriComponentsBuilder){
        Topico topico = form.converter(cursoRepository);
        topicoRepository.save(topico);
        URI uri = uriComponentsBuilder.path("/topicos/{id}").buildAndExpand(topico.getId()).toUri();
        return ResponseEntity.created(uri).body(new TopicoDTO(topico));
    }

    @GetMapping(value = "/{id}")
    public DetalhesDoTopicoDTO detalhar(@PathVariable Long id){
        Topico topico = topicoRepository.carregarComRespostas(id);
        return new DetalhesDoTopicoDTO(topico);
    }

}
solução!
por Rodrigo da Silva Ferreira Caneppele
| 5043k xp | 8841 posts
Instrutor Instrutor

Ah foi mal, esqueci de filtrar pelo id na querie :D

@Query("SELECT t FROM Topico t LEFT JOIN FETCH t.respostas WHERE t.id = :id")
public Topico carregarComRespostas(Long id);
por Gabriel Mota Melo
| 48k xp | 4 posts
Estagiário de TI

Deu certo cara, agora funcionando corretamente!

Obrigado!!