🚢오늘의 회고

굳

---

✔ 오늘 한 일

• yum yum trend 좋아요 기능 리팩토링

🔍좋아요 기능 리팩토링!

주말동안 진행한 게시글 좋아요 기능에 대해 팀원들의 추가적인 의견이 있어 이를 반영하고자 좋아요 기능을 리팩토링 하였다.

• 기존 : ArticleController의 getArticle API에서 Like에 대한 데이터를 담아서 프론트 단으로 넘겨준다.

// 예시
@GetMapping("/articles")
    public List<ArticleResponseDto> getArticles(@AuthenticationPrincipal UserDetailsImpl userDetails) {
        return articleService.getArticles();
    }

• 변경 : Like에 대한 정보를 다루는 LikeController 클래스를 만들어서 Like에 대한 정보는 프론트 단에 따로 보내준다.

// 예시
@GetMapping("/likes")
    public List<LikeResponseDto> getLikes(@AuthenticationPrincipal UserDetailsImpl userDetails) {
        return likeService.getLikes(userDetails.getUser().getId());
    }

• 변경하게된 이유 : article에 대한 코드가 더 늘어나게 될 경우 코드를 수정하기 위해서
  article, like 두 기능에 대한 코드를 수정해야 하는 반면에 변경 후와 같이 controller를 나눠서 코딩해 놓으면 각각의 기능에 대해 각각 수정을 취하면 되기 때문에 리팩토링을 실시하였다.

• 변경 이후: controller를 나눈 만큼 그만큼 api가 두배가 되었기 때문에 코드적으로는 더 길어졌고 데이터도 중복해서 보내는 항목들이 발생했다. 하지만, 기능별로 controller를 구분한 결과 코드 클래스와 변수명을 볼때 기능을 더 직관적으로 파악할 수 있게 되었다.

// ArticleController
@RestController
@RequiredArgsConstructor
public class ArticleController {

    private final ArticleService articleService;

    @GetMapping("/articles")
    public List<Article> getArticles() {
        return articleService.getArticles();
    }

    @GetMapping("/articles/{id}")
    public Article getArticle(@PathVariable Long id) {
        return articleService.getArticle(id);
    }

    @PostMapping("/articles")
    public void createArticle(@AuthenticationPrincipal UserDetailsImpl userDetails,
                                @RequestParam("text") String text,
                                @RequestParam("location") String locationJsonString,
                                @RequestParam("hashtagNameList") List<String> hashtagNameList,
                                @RequestParam("imageFileList") List<MultipartFile> imageFileList) {

        articleService.createArticle(userDetails.getUser(), text, new LocationRequestDto(locationJsonString), hashtagNameList, imageFileList);
    }
}


// LikeController
@RestController
@RequiredArgsConstructor
public class LikeController {
    private final LikeService likeService;

    @GetMapping("/likes")
    public List<LikeResponseDto> getLikes(@AuthenticationPrincipal UserDetailsImpl userDetails) {
        return likeService.getLikes(userDetails.getUser().getId());
    }

    @GetMapping("/likes/guest")
    public List<LikeResponseDto> getLikesGuest() {
        return likeService.getLikesGuest();
    }

    @GetMapping("/likes/{id}")
    public LikeResponseDto getLike(@PathVariable Long id,
                                      @AuthenticationPrincipal UserDetailsImpl userDetails) {
        return likeService.getLike(id, userDetails.getUser().getId());
    }

    @GetMapping("/likes/guest/{id}")
    public LikeResponseDto getLikeGuest(@PathVariable Long id) {
        return likeService.getLikeGuest(id);
    }

    @GetMapping("/profile/likes/{id}")
    public List<LikeResponseDto> getLikesUser(@PathVariable Long id) {
        return likeService.getLikesUser(id);
    }

    @GetMapping("/profile/likes")
    public List<LikeResponseDto> getLikes(@PathVariable Long id) {
        return likeService.getLikesUser(id);
    }


    @PutMapping("/articles/like")
    public void increaseLikeCount(@AuthenticationPrincipal UserDetailsImpl userDetails,
                                  @RequestParam("articleId") Long articleId) {
        likeService.increaseLikeCount(userDetails.getId(), articleId);
    }

    @PutMapping("articles/unlike")
    public void decreaseLikeCount(@AuthenticationPrincipal UserDetailsImpl userDetails,
                                  @RequestParam("articleId") Long articleId) {
        likeService.decreaseLikeCount(userDetails.getId(), articleId);
    }
}

🔍@JsonIgnore

서로 연관관계를 매핑하면 서로 무한히 참조하는 상황이 발생해 버린다.
이 때, @JsonIgnore를 이용하면 값이 출력되지 않게 하여 무한히 참조하는 상황을 방지해줄 수 있다.
즉, @JsonIgnore가 달려있는 컬럼은 json데이터에서 null로 들어간다.

---

👀Todo List

내일은 조금 더 체계적으로 공부합시다.
배운 것도 미리미리 노트해놓고...

• ⛏프로젝트
  • 회원가입,articleController 테스트 코드 작성
  • 예외처리 어떻게 작성할지 구상해보기