Overview

팔로우 등의 기능을 구현하시기 위해 많이 고민해주셨을 것 같습니다. 고생 많으셨습니다. 👍
RESTful API 외에도 GraphQL이라는 게 있는데요. 페이스북에서 팔로우 관련 내용을 효율적으로 처리하기 위해 고안한 방법입니다. 관련하여 추가적으로 살펴보셔도 좋을 것 같습니다.
- 참고 자료

시연

시연 관련 음성이 저장되지 않은 것으로 보이는데요. 음성이 포함되면 좀더 좋았을 것 같습니다. 이후 시연 시 확인 부탁드립니다!

ERD

like 관련 테이블에서 post 또는 comment 별 like 개수를 집계할 수 있어 각 테이블의 like 개수를 굳이 저장하지 않아도 될 것 같습니다.
- 하지만 성능 상의 문제로 현업에서 구현해주신 부분과 같이 구현하기도 하는데요. 이를 반정규화라고 합니다. 참고 부탁드립니다.
mysql에서 길이가 긴 텍스트를 저장하고 싶을 떄 text / mediumtext / longtext 등의 데이터타입이 있습니다. introduction 컬럼은 이런 데이터타입이 조금 더 적절할 수 있을 것 같습니다.
좋아요에 대한 정책에 따라 관련 테이블 설계도 조금 바뀔 수 있을 것 같은데요. 만약 한 명의 user에게 한 번의 좋아요만 허용된다면, user_id와 post_id 등으로 복합키 설정을 하는 것도 좋을 것 같습니다.

API

전반적으로 API 설계를 잘 해주셨습니다.
unfollow 기능을 like와 다른 게 follow와 동일 API 스펙으로 하신 이유가 있으실까요? 혼선을 방지하기 위해서라도 unfollow로 url을 바꾸거나 DELETE 메소드를 사용하는 게 좋을 것 같습니다.

프로젝트 전반

응답에는 에러 관련 내용을 간단하게 서술하고, 구체적인 validation fail 메시지 등을 로그로 보여주셨는데, 해당 내용을 응답에 전달해도 좋을 것 같습니다~

https://github.com/proLmpa/newsfeed/blob/0688afa4c3209d75fbc4d0366e875b92c4c81bfe/src/main/java/com/newsfeed/board/advice/GlobalControllerAdvice.java#L12

상황에 맞게 여러 exception에 대한 처리를 정의해도 좋을 것 같습니다. 프로젝트 전반에 관련 코멘트 드린 대로 응답에 구체적인 에러 원인을 전달하는 게 예시가 될 수 있을 것 같습니다 😄
- 참고 자료

https://github.com/proLmpa/newsfeed/blob/0688afa4c3209d75fbc4d0366e875b92c4c81bfe/src/main/java/com/newsfeed/board/email/service/EmailServiceImpl.java#L47

이런 값은 설정 파일이나 상수로 관리하는 걸 추천드립니다~

https://github.com/proLmpa/newsfeed/blob/0688afa4c3209d75fbc4d0366e875b92c4c81bfe/src/main/java/com/newsfeed/board/email/repository/EmailService.java#L1-L4

repository와 service단의 역할은 다르기 때문에 Service interface가 repository package에 위치한 것은 적절하지 않은 것으로 보입니다.

https://github.com/proLmpa/newsfeed/blob/0688afa4c3209d75fbc4d0366e875b92c4c81bfe/src/main/java/com/newsfeed/board/follow/service/FollowService.java#L48-L51

http 응답을 만들어주는 레이어는 controller단이기 때문에 다소 부적절해 보입니다.

https://github.com/proLmpa/newsfeed/blob/0688afa4c3209d75fbc4d0366e875b92c4c81bfe/src/main/java/com/newsfeed/board/follow/service/FollowService.java#L37-L40

해당 영역의 에러 때문에 throw 선언을 해주신 걸까요? 아래와 같은 처리도 가능하긴 합니다.

참고 자료

try {
    UserEntity followingUser = findFollowingUser(followingUserId); // 팔로우를 할 대상의 ID
    UserEntity followerUser = userDetails.getUser(); // 팔로우를 하는 사람의 ID (본인 ID) Long id

    if (followingUser.getId().equals(userDetails.getId())) {
        throw new Exception("CANNOT_FOLLOW_YOURSELF");
    } else {
        FollowEntity followEntity = new FollowEntity(followingUser, followerUser);
        followRepository.save(followEntity);
        return ResponseEntity.ok().body(new ApiResponseDto(HttpStatus.OK.value(), "SUCCESS_FOLLOW_USER"));
    }
} catch (UnexpectedRollbackException e) {
       // 해당 Exception 관련 처리
} catch (Exception e) {
    // 위의 Exception 제외 모든 Exception 관련 처리
    return ResponseEntity.badRequest().body(new ApiResponseDto(HttpStatus.BAD_REQUEST.value(), e.getMessage()));
}

https://github.com/proLmpa/newsfeed/blob/0688afa4c3209d75fbc4d0366e875b92c4c81bfe/src/main/java/com/newsfeed/board/user/service/UserService.java#L58-L62

데이터베이스 조건절로도 풀어낼 수 있는 부분으로 보입니다. 가령 입력 받은 config 값이고 createdAt이 현재 시간 3분 전보다 큰 entity를 검색해 존재하는 경우 인증되는 형태가 될 수 있을 것 같습니다.

https://github.com/proLmpa/newsfeed/blob/0688afa4c3209d75fbc4d0366e875b92c4c81bfe/src/main/java/com/newsfeed/board/user/service/UserService.java#L81-L89

보안 상의 이유로 어느 쪽이 틀렸는지 알려주지 않기도 합니다.

proLmpa / newsfeed

프로젝트 피드백 - 작성자: 이승은 튜터 #9

Overview

시연

ERD

API

프로젝트 전반