From b2db3cabe759e1cbfb1831fef02d55e8784f1c3e Mon Sep 17 00:00:00 2001 From: Lucas Nascimento Date: Tue, 13 Dec 2022 20:33:35 -0300 Subject: [PATCH 1/2] criando os logs nas classes de servicos, exluindo imports nao utilizados nas classes --- .idea/libraries/Maven__log4j_log4j_1_2_17.xml | 13 ++++++++ pom.xml | 1 - .../service/impl/ConsultaService.java | 24 ++++++++++---- .../service/impl/DentistaService.java | 33 +++++++++++++++---- .../service/impl/EnderecoService.java | 15 +++++++-- .../service/impl/PacienteService.java | 30 +++++++++++++---- 6 files changed, 93 insertions(+), 23 deletions(-) create mode 100644 .idea/libraries/Maven__log4j_log4j_1_2_17.xml diff --git a/.idea/libraries/Maven__log4j_log4j_1_2_17.xml b/.idea/libraries/Maven__log4j_log4j_1_2_17.xml new file mode 100644 index 0000000..e383c1b --- /dev/null +++ b/.idea/libraries/Maven__log4j_log4j_1_2_17.xml @@ -0,0 +1,13 @@ + + + + + + + + + + + + + \ No newline at end of file diff --git a/pom.xml b/pom.xml index 4986e1a..45e6827 100644 --- a/pom.xml +++ b/pom.xml @@ -63,7 +63,6 @@ spring-boot-starter-validation - log4j log4j diff --git a/src/main/java/com/dh/trabalhoIntegrador/service/impl/ConsultaService.java b/src/main/java/com/dh/trabalhoIntegrador/service/impl/ConsultaService.java index bb2ecdd..1b978ab 100644 --- a/src/main/java/com/dh/trabalhoIntegrador/service/impl/ConsultaService.java +++ b/src/main/java/com/dh/trabalhoIntegrador/service/impl/ConsultaService.java @@ -2,36 +2,38 @@ import com.dh.trabalhoIntegrador.exception.ResourceNotFoundException; import com.dh.trabalhoIntegrador.model.Consulta; -import com.dh.trabalhoIntegrador.model.Paciente; import com.dh.trabalhoIntegrador.model.dto.ConsultaDTO; import com.dh.trabalhoIntegrador.repository.ConsultaRepository; import com.dh.trabalhoIntegrador.service.IService; import com.fasterxml.jackson.databind.ObjectMapper; +import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.http.HttpStatus; import org.springframework.http.ResponseEntity; import org.springframework.stereotype.Service; -import java.sql.Timestamp; -import java.time.Instant; + import java.util.ArrayList; import java.util.List; import java.util.Optional; @Service public class ConsultaService implements IService { - + static final Logger logger = LogManager.getLogger(ConsultaService.class); @Autowired ConsultaRepository consultaRepository; @Override public Consulta salvar(Consulta consulta){ + logger.info("Salvando uma consulta"); Consulta consultaSalva = consultaRepository.save(consulta); return consultaSalva; } @Override public ConsultaDTO buscar(Long id) throws ResourceNotFoundException { + logger.info("Buscando um consulta por Id"); ObjectMapper mapper = new ObjectMapper(); Consulta consulta = consultaRepository.findById(id).orElseThrow(() -> {return new ResourceNotFoundException(""); }); @@ -41,27 +43,31 @@ public ConsultaDTO buscar(Long id) throws ResourceNotFoundException { @Override public ConsultaDTO alteracaoTotal(ConsultaDTO consultaDTO) throws ResourceNotFoundException { + logger.info("Buscando consulta para alteracao"); Consulta consulta = consultaRepository.findByCodConsulta(consultaDTO.getCodConsulta()).orElseThrow(() -> new ResourceNotFoundException("Consulta inexistente na base de dados, verifique.")); - + logger.info("Efetuando alteracao de consulta"); Consulta consultaUpdate = consulta; consultaUpdate.setCodConsulta(consultaDTO.getCodConsulta()); consultaUpdate.setDataConsulta(consultaDTO.getDataConsulta()); consultaUpdate.setDentista(consultaDTO.getDentista()); consultaUpdate.setPaciente(consultaDTO.getPaciente()); + logger.info("Salvando consulta alterada com sucesso"); consultaRepository.save(consultaUpdate); + logger.info("Retornando lista de consulta"); return consultaDTO; } @Override public List buscarTodos() { + logger.info("Buscando todas as Consultas"); List listConsulta = consultaRepository.findAll(); List consultasDTOList = new ArrayList<>(); ObjectMapper mapper = new ObjectMapper(); for(Consulta c: listConsulta){ consultasDTOList.add(mapper.convertValue(c, ConsultaDTO.class)); } - + logger.info("Retornando lista de consultas"); return consultasDTOList ; } @@ -71,23 +77,29 @@ public ResponseEntity deletar(Long id) { } public ResponseEntity deletar(String codConsulta){ + logger.info("Buscando consulta pelo código de consulta para deletar"); Optional consulta = consultaRepository.findByCodConsulta(codConsulta); if (consulta.isEmpty()){ + logger.error("Retorno da pesquisa vazio, código de consulta não existe "); return new ResponseEntity("Codigo de Consulta não existe", HttpStatus.BAD_REQUEST); } consultaRepository.deleteById(consulta.get().getId()); + logger.info("Consulta deletada com sucesso"); return new ResponseEntity("Consulta excluida com sucesso com sucesso", HttpStatus.OK); } public ResponseEntity buscarCodConsulta(String codConsulta){ + logger.info("Buscar consulta por código de consulta"); ObjectMapper mapper = new ObjectMapper(); Optional consulta = consultaRepository.findByCodConsulta(codConsulta); if (consulta.isEmpty()){ + logger.error("Código de consulta não encontrado"); return new ResponseEntity("Consulta não encontrada", HttpStatus.BAD_REQUEST); } Consulta consultaPesquisada = consulta.get(); ConsultaDTO consultaDTO = mapper.convertValue(consultaPesquisada, ConsultaDTO.class); + logger.info("Consulta realizada com sucesso"); return new ResponseEntity(consultaDTO,HttpStatus.OK); } diff --git a/src/main/java/com/dh/trabalhoIntegrador/service/impl/DentistaService.java b/src/main/java/com/dh/trabalhoIntegrador/service/impl/DentistaService.java index 8587d8f..dd37f41 100644 --- a/src/main/java/com/dh/trabalhoIntegrador/service/impl/DentistaService.java +++ b/src/main/java/com/dh/trabalhoIntegrador/service/impl/DentistaService.java @@ -1,48 +1,53 @@ package com.dh.trabalhoIntegrador.service.impl; import com.dh.trabalhoIntegrador.exception.ResourceNotFoundException; -import com.dh.trabalhoIntegrador.model.Paciente; import com.dh.trabalhoIntegrador.model.dto.DentistaDTO; -import com.dh.trabalhoIntegrador.model.dto.PacienteDTO; import com.dh.trabalhoIntegrador.repository.DentistaRepository; import com.dh.trabalhoIntegrador.model.Dentista; import com.dh.trabalhoIntegrador.service.IService; import com.fasterxml.jackson.databind.ObjectMapper; +import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.http.HttpStatus; import org.springframework.http.ResponseEntity; import org.springframework.stereotype.Service; -import java.sql.Timestamp; -import java.time.Instant; import java.util.ArrayList; import java.util.List; import java.util.Optional; @Service public class DentistaService implements IService { - + static final Logger logger = LogManager.getLogger(DentistaService.class); @Autowired DentistaRepository dentistaRepository; @Override public DentistaDTO buscar(Long id) throws ResourceNotFoundException { + logger.info("Busanco dentista por ID"); ObjectMapper mapper = new ObjectMapper(); - Dentista dentista = dentistaRepository.findById(id).orElseThrow(() -> new ResourceNotFoundException("Id inexistente na base de dados, verifique.")); + Dentista dentista = dentistaRepository.findById(id).orElseThrow(() -> + new ResourceNotFoundException("Id inexistente na base de dados, verifique.")); + logger.info("Retornando dentista pesquisado"); return mapper.convertValue(dentista, DentistaDTO.class); } @Override public Dentista salvar(Dentista dentista) throws ResourceNotFoundException { + logger.info("Verificando se matricula do dentista existe, antes de salvar"); Dentista dentistaExists = dentistaRepository.findByNumMatricula(dentista.getNumMatricula()).orElseThrow(() -> new ResourceNotFoundException("RG já encontra-se cadastrado na base de dados.")); + logger.info("Salvando dentista"); Dentista dentistaSalvo = dentistaRepository.save(dentista); + logger.info("Retornando dentista salvo"); return dentistaSalvo; } @Override public List buscarTodos() { + logger.info("Buscando todos os dentistas"); List dentistaList = dentistaRepository.findAll(); List dentistaDTOList = new ArrayList<>(); @@ -50,53 +55,65 @@ public List buscarTodos() { for (Dentista d : dentistaList) { dentistaDTOList.add(mapper.convertValue(d, DentistaDTO.class)); } + logger.info("Retornando lista de dentistas cadastrados"); return dentistaDTOList; } @Override public ResponseEntity deletar(Long id) { + logger.info("Buscando dentista para deletar"); Optional dentista = dentistaRepository.findById(id); if (dentista.isEmpty()) { + logger.error("Id informado para o dentista não existe"); return new ResponseEntity("Id informado não existe", HttpStatus.BAD_REQUEST); } dentistaRepository.deleteById(id); + logger.info("Dentista excluido com sucesso"); return new ResponseEntity("Dentista excluido com sucesso!", HttpStatus.OK); } public ResponseEntity buscarPorNumMatricula(String numMatricula) { ObjectMapper mapper = new ObjectMapper(); + logger.info("Procurando dentista por numero de matricula"); Optional dentista = dentistaRepository.findByNumMatricula(numMatricula); if (dentista.isEmpty()) { + logger.error("Numero de matricula informado não existe, retorno vazio"); return new ResponseEntity("Matricula inexistente, Dentista não encontrado", HttpStatus.BAD_REQUEST); } Dentista dentistaProcurado = dentista.get(); DentistaDTO dentistaDTO = mapper.convertValue(dentistaProcurado, DentistaDTO.class); + logger.info("Retornando dentista pesquisado"); return new ResponseEntity(dentistaDTO, HttpStatus.OK); } public ResponseEntity alteracaoPacial(DentistaDTO dentistaDTO) { ObjectMapper mapper = new ObjectMapper(); + logger.info("Pesquisando dentista para alteracao"); Optional dentistaOptional = dentistaRepository.findByNumMatricula(dentistaDTO.getNumMatricula()); if (dentistaOptional.isEmpty()) { + logger.error("Matricula de dentista informado não existe"); return new ResponseEntity("A matricula informada não existe", HttpStatus.NOT_FOUND); } Dentista dentista = dentistaOptional.get(); - + logger.info("Procedendo com alteracao de dentista"); if (dentistaDTO.getNome() != null) { dentista.setNome(dentistaDTO.getNome()); } if (dentista.getSobrenome() != null) { dentista.setSobrenome(dentistaDTO.getSobrenome()); } + logger.info("Salvando alteracao de dentista"); DentistaDTO dentistaAlterado = mapper.convertValue(dentistaRepository.save(dentista), DentistaDTO.class); + logger.info("Retornando Dentista alterado com sucesso"); return new ResponseEntity(dentistaAlterado, HttpStatus.OK); } public DentistaDTO alteracaoTotal(DentistaDTO dentistaDTO) throws ResourceNotFoundException { Dentista dentista = dentistaRepository.findByNumMatricula(dentistaDTO.getNumMatricula()).orElseThrow(() -> { + logger.error("Matricula de dentista informado não existe"); return new ResourceNotFoundException("RG inexistente na base de dados, verifique."); }); @@ -104,7 +121,9 @@ public DentistaDTO alteracaoTotal(DentistaDTO dentistaDTO) throws ResourceNotFou dentistaUpdate.setNome(dentistaDTO.getNome()); dentistaUpdate.setSobrenome(dentistaDTO.getSobrenome()); dentistaUpdate.setNumMatricula(dentistaDTO.getNumMatricula()); + logger.info("Salvando alteracao de dentista"); dentistaRepository.save(dentistaUpdate); + logger.info("Retornando Dentista alterado com sucesso"); return dentistaDTO; } } diff --git a/src/main/java/com/dh/trabalhoIntegrador/service/impl/EnderecoService.java b/src/main/java/com/dh/trabalhoIntegrador/service/impl/EnderecoService.java index f1a5779..85c1c83 100644 --- a/src/main/java/com/dh/trabalhoIntegrador/service/impl/EnderecoService.java +++ b/src/main/java/com/dh/trabalhoIntegrador/service/impl/EnderecoService.java @@ -1,9 +1,9 @@ package com.dh.trabalhoIntegrador.service.impl; import com.dh.trabalhoIntegrador.model.Endereco; -import com.dh.trabalhoIntegrador.model.dto.EnderecoDTO; import com.dh.trabalhoIntegrador.repository.EnderecoRepository; -import com.fasterxml.jackson.databind.ObjectMapper; +import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.http.HttpStatus; import org.springframework.http.ResponseEntity; @@ -13,15 +13,18 @@ @Service public class EnderecoService { - + static final Logger logger = LogManager.getLogger(EnderecoService.class); @Autowired EnderecoRepository repository; public ResponseEntity salvarEndereco(Endereco endereco) { + logger.info("Iniciando salvamento de endereco"); try { + logger.info("Salvando novo endereco"); Endereco enderecoSalvo = repository.save(endereco); return new ResponseEntity("Endereco com id " + enderecoSalvo.getId() + " foi cadastrado com sucesso.", HttpStatus.CREATED); }catch (Exception e) { + logger.error("Erro ao salvar endereco"); return new ResponseEntity("Erro ao cadastrar endereço, tente novamente.", HttpStatus.BAD_REQUEST); } } @@ -29,20 +32,26 @@ public ResponseEntity salvarEndereco(Endereco endereco) { public ResponseEntity buscarEnderecoPorId(Long id){ Optional endereco = repository.findById((id)); + logger.info("Iniciando busca de endereco por Id"); if (endereco.isEmpty()) { + logger.error("Id de endereco informado não existe"); return new ResponseEntity("O endereço informado não existe.", HttpStatus.BAD_REQUEST); } repository.findById(id); + logger.info("Endereco encontrado com sucesso"); return new ResponseEntity("Endereço encontrado com sucesso", HttpStatus.OK); } public ResponseEntity deletarEndereco(Long id) { + logger.info("Pesquisando endereco para deletar"); Optional endereco = repository.findById(id); if(endereco.isEmpty()) { + logger.error("Endereco não encontrado"); return new ResponseEntity("O endereço informado não existe.", HttpStatus.BAD_REQUEST); } repository.deleteById(id); + logger.info("Endereco excluido com sucesso"); return new ResponseEntity("Endereço excluído com sucesso.", HttpStatus.OK); } diff --git a/src/main/java/com/dh/trabalhoIntegrador/service/impl/PacienteService.java b/src/main/java/com/dh/trabalhoIntegrador/service/impl/PacienteService.java index 1dc4a92..29a92f1 100644 --- a/src/main/java/com/dh/trabalhoIntegrador/service/impl/PacienteService.java +++ b/src/main/java/com/dh/trabalhoIntegrador/service/impl/PacienteService.java @@ -7,6 +7,8 @@ import com.dh.trabalhoIntegrador.model.Paciente; import com.dh.trabalhoIntegrador.service.IService; import com.fasterxml.jackson.databind.ObjectMapper; +import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.http.HttpStatus; import org.springframework.http.ResponseEntity; @@ -20,21 +22,24 @@ @Service public class PacienteService implements IService { - + static final Logger logger = LogManager.getLogger(EnderecoService.class); @Autowired PacienteRepository pacienteRepository; public PacienteDTO buscar(Long id) throws ResourceNotFoundException { + logger.info("Pesquisando paciente por Id"); ObjectMapper mapper = new ObjectMapper(); Paciente paciente = pacienteRepository.findById(id).orElseThrow(() -> new ResourceNotFoundException("RG inexistente na base de dados, verifique.")); + logger.info("Retornando paciente pesquisado"); return mapper.convertValue(paciente, PacienteDTO.class); } @Override public List buscarTodos() { + logger.info("Buscando lista com todos os pacientes"); List pacienteList = pacienteRepository.findAll(); List pacienteDTOList = new ArrayList<>(); @@ -42,35 +47,43 @@ public List buscarTodos() { for(Paciente p : pacienteList){ pacienteDTOList.add(mapper.convertValue(p, PacienteDTO.class)); } + logger.info("Retornando lista de pacientes"); return pacienteDTOList; } @Override public ResponseEntity deletar(Long id) throws ResourceNotFoundException { + logger.info("Procurando paciente para deletar"); Paciente paciente = pacienteRepository.findById(id).orElseThrow(() -> new ResourceNotFoundException("Id não encontrado, verifique.")); pacienteRepository.deleteById(id); + logger.info("Paciente deletado com sucesso"); return new ResponseEntity("Paciente "+ paciente.getNome() + "excluído com sucesso.", HttpStatus.OK); } public Paciente salvar(Paciente paciente) throws CadastroInvalidoException, ResourceNotFoundException { + logger.info("Procurando paciente por RG"); Optional pacienteExists = pacienteRepository.findByRg(paciente.getRg()); Paciente pacienteSalvo = null; if(pacienteExists.isEmpty()){ + logger.info("Salvando novo paciente"); paciente.setDataCadastro(Timestamp.from(Instant.now())); pacienteSalvo = pacienteRepository.save(paciente); }else{ + logger.error("Paciente já existente, não foi possível efetuar novo cadastro"); throw new CadastroInvalidoException("Rg já existente na base de dados."); } return pacienteSalvo; } public PacienteDTO buscarPorRg(String rg) throws ResourceNotFoundException { + logger.info("Busanco paciente por RG"); ObjectMapper mapper = new ObjectMapper(); Paciente paciente = pacienteRepository.findByRg(rg).orElseThrow(() -> { + logger.error("Paciente não existe na base de dados"); return new ResourceNotFoundException("RG inexistente na base de dados, verifique."); }); - + logger.info("Retornando paciente salvo"); return mapper.convertValue(paciente, PacienteDTO.class); @@ -78,12 +91,14 @@ public PacienteDTO buscarPorRg(String rg) throws ResourceNotFoundException { public PacienteDTO alteracaoPacial(PacienteDTO pacienteDTO){ ObjectMapper mapper = new ObjectMapper(); + logger.info("Procurando paciente para alteracao"); Optional pacienteOptional = pacienteRepository.findByRg(pacienteDTO.getRg()); if(pacienteOptional.isEmpty()){ + logger.error("Paciente nao existe, não foi possivel efetuar alteracao"); return null; } Paciente paciente = pacienteOptional.get(); - + logger.info("Paciente encontrado, efetuando alteracao"); if(pacienteDTO.getNome() != null){ paciente.setNome(pacienteDTO.getNome()); } @@ -96,19 +111,22 @@ public PacienteDTO alteracaoPacial(PacienteDTO pacienteDTO){ if(pacienteDTO.getSobrenome() != null){ paciente.setSobrenome(pacienteDTO.getSobrenome()); } - + logger.info("Salvando paciente alterado"); return mapper.convertValue(pacienteRepository.save(paciente), PacienteDTO.class); } public PacienteDTO alteracaoTotal(PacienteDTO pacienteDTO) throws ResourceNotFoundException { - - Paciente paciente = pacienteRepository.findByRg(pacienteDTO.getRg()).orElseThrow(() -> {return new ResourceNotFoundException("RG inexistente na base de dados, verifique.");}); + logger.info("Procurando por paciente para efetuar alteracao"); + Paciente paciente = pacienteRepository.findByRg(pacienteDTO.getRg()).orElseThrow(() -> { + logger.error("Paciente não encontrado"); + return new ResourceNotFoundException("RG inexistente na base de dados, verifique.");}); Paciente pacienteUpdate = paciente; pacienteUpdate.setNome(pacienteDTO.getNome()); pacienteUpdate.setSobrenome(pacienteDTO.getSobrenome()); pacienteUpdate.setEndereco(pacienteDTO.getEndereco()); pacienteUpdate.setRg(pacienteDTO.getRg()); + logger.info("Salvando paciente alterado"); pacienteRepository.save(pacienteUpdate); return pacienteDTO; } From 3a8dd59a001a25ed5207aa8ea964b1c6ab412046 Mon Sep 17 00:00:00 2001 From: Lucas Nascimento Date: Tue, 13 Dec 2022 20:37:58 -0300 Subject: [PATCH 2/2] excluindo imports nao utilizados nas classes de controller --- .../dh/trabalhoIntegrador/controller/DentistaController.java | 2 -- .../dh/trabalhoIntegrador/controller/EnderecoController.java | 2 -- .../dh/trabalhoIntegrador/controller/PacienteController.java | 1 - 3 files changed, 5 deletions(-) diff --git a/src/main/java/com/dh/trabalhoIntegrador/controller/DentistaController.java b/src/main/java/com/dh/trabalhoIntegrador/controller/DentistaController.java index 2bf5a9b..b43f9b8 100644 --- a/src/main/java/com/dh/trabalhoIntegrador/controller/DentistaController.java +++ b/src/main/java/com/dh/trabalhoIntegrador/controller/DentistaController.java @@ -3,7 +3,6 @@ import com.dh.trabalhoIntegrador.exception.CadastroInvalidoException; import com.dh.trabalhoIntegrador.exception.ResourceNotFoundException; import com.dh.trabalhoIntegrador.model.Dentista; -import com.dh.trabalhoIntegrador.model.Paciente; import com.dh.trabalhoIntegrador.model.dto.DentistaDTO; import com.dh.trabalhoIntegrador.service.impl.DentistaService; import org.springframework.beans.factory.annotation.Autowired; @@ -14,7 +13,6 @@ import javax.validation.Valid; import java.io.Serializable; import java.util.List; -import java.util.Optional; @RestController @RequestMapping("/dentista") diff --git a/src/main/java/com/dh/trabalhoIntegrador/controller/EnderecoController.java b/src/main/java/com/dh/trabalhoIntegrador/controller/EnderecoController.java index cc52ad6..b192a93 100644 --- a/src/main/java/com/dh/trabalhoIntegrador/controller/EnderecoController.java +++ b/src/main/java/com/dh/trabalhoIntegrador/controller/EnderecoController.java @@ -1,13 +1,11 @@ package com.dh.trabalhoIntegrador.controller; import com.dh.trabalhoIntegrador.model.Endereco; -import com.dh.trabalhoIntegrador.model.dto.EnderecoDTO; import com.dh.trabalhoIntegrador.service.impl.EnderecoService; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.http.ResponseEntity; import org.springframework.web.bind.annotation.*; -import java.util.List; @CrossOrigin("*") @RestController diff --git a/src/main/java/com/dh/trabalhoIntegrador/controller/PacienteController.java b/src/main/java/com/dh/trabalhoIntegrador/controller/PacienteController.java index d9f0ed6..f2f14fb 100644 --- a/src/main/java/com/dh/trabalhoIntegrador/controller/PacienteController.java +++ b/src/main/java/com/dh/trabalhoIntegrador/controller/PacienteController.java @@ -13,7 +13,6 @@ import javax.validation.Valid; import java.io.Serializable; import java.util.List; -import java.util.Optional; @CrossOrigin("*") @RestController