전체 글

문제 재기 만약에 어떤 메서드에서 에러를 터트린다면 orElseThrow라는 이름으로 함수 이름을 끝내는게 적합한지에 대한 코드 리뷰를 받았다. 여태 명시적으로 메서드를 작성하는게 좋고, 클린코드를 읽고 난 이후 현대 IDE에서 자동완성기능이 정말 좋아졌기 때문에 굳이 메서드를 축약형으로 쓴다거나 줄이기 위해서 애를 쓸 필요 없다고도 생각을 했다. 제이슨님의 의견 유쾌한 스프링방에서 제이슨님이 두 가지 정도의 의견을 제시해주셨다. 모든 메서드는 Exception을 발생시킬 가능성을 내포하고 있다. 만약에 저 메서드에서 Member 값이 null 값이 들어가게 되면 NullPointException이 발생하는 등 모든 메서드들은 항상 Exception이 발생할 가능성이 내재되어 있다는 것이다. update..

Refactoring 기존에 Riot API를 호출하는 부분은 LolSearchAdapter라는 네이밍으로 아래와 같이 개발이 되어있었다. @Service public class LolSearchAdapter { @Value("${lol.api.key}") private String apiKey; private final RestTemplate restTemplate; @Autowired LolSearchAdapter(RestTemplate restTemplate) { this.restTemplate = restTemplate; } /** * User의 게임 목록을 가져옴 * @param matchId * @return */ public MatchRecord searchMatch(String matchI..

문제 QueryDSL을 공부하던 중에 QueryDSL을 이용해서 Join 하는 코드를 개발하는 중이었는데, 이 코드가 제대로 동작하지 않았다. 문제가 되는 코드는 다음과 같았다. @Test @DisplayName("조인 Error가 발생함") void searchErrorTest() { Team teamA = new Team("teamA"); Team teamB = new Team("teamB"); em.persist(teamA); em.persist(teamB); Member member1 = new Member("member1", 10, teamA); Member member2 = new Member("member2", 20, teamA); Member member3 = new Member("memb..