[2025-05-19] 멘토링 코드 리뷰 by dlsrks1021 · Pull Request #60 · prgrms-be-devcourse/NBE5-7-2-Team02

dlsrks1021 · 2025-05-19T09:28:20Z

No description provided.

* feat: cors 설정 * feat: SecurityConfig 인가 설정 * feat: RefreshToken, RefreshTokenBlackList 기능 * chore: Java 스타일 수정 * feat: RefreshToken, RefreshTokenBlackList 기능 * chore: Java 스타일 수정 * refactor: dto 패키지 구조 변경 --------- Co-authored-by: github-actions <>

* feat: 게시물 수정 DTO 구현 * feat: PostCreateResponse -> PostResponse 이름 변경 * feat: Post update 메서드 구현 * feat: PostMapper update 메서드 구현 * feat: ImageUploader 게시물 폴더별 이미지 삭제 메서드 구현 * feat: PostTagRepository deleteAllByPost 추가 * feat: SuccessCode 추가 * feat: ErrorCode 추가 및 변경 * chore: ErrorCode 변경에 따른 코드 수정 * feat: ChatRoomRepository deleteByPost 추가 * feat: PostService 게시물 수정, 삭제 메서드 구현 * feat: PostController 게시물 수정, 삭제 api 구현 * chore: yml env 파일 설정 추가 * chore: Java 스타일 수정 * fix: merge 충돌 처리 중 잘못 삭제된 중괄호 복구 * refactor: CollectionUtils.isEmpty() 사용하여 이미지 리스트 체크 로직 개선 * chore: Java 스타일 수정 --------- Co-authored-by: github-actions <>

* ✨ feat: 마이페이지 기능 구현 ♻️ refactor: 팔로우 리팩토링

* refactor: PostController에 @AuthenticationPrincipal 적용 * chore: Java 스타일 수정 --------- Co-authored-by: github-actions <>

* ✨ feat: 마이페이지 구현[FE] ♻️ refactor: API연동에 따른 리팩토링[BE]

BaeRoNuI · 2025-05-19T10:33:22Z

+        Member member = checkUserLogin.checkUserLogin(userDetails);
+
+        ChatRoom chatRoom = chatRoomService.checkChatRoomExists(chatRoomId);
+
+        ChatMember chatMember = chatMemberRepository.findByChatRoomAndMember(chatRoom, member);
+
+        if (chatMember == null) {
+            throw new ErrorException(CHAT_MEMBER_NOT_ENTER);
+        }


line 82 ~ 90 의 코드와 완전히 동일한 코드인데요, 메소드로 뽑아낼 수 있는 방법을 고민하는건 어떨까요?

ChatMemberService 의 update ChatMember 도 동일하네요
ChatMemeber Service 에 관련 메소드를 작성하고, 여기선 chatMemberRepository 를 직접 의존하는것이 아닌 chatMemberService 를 의존하는것은 어떤가요

BaeRoNuI · 2025-05-19T10:37:55Z

        Member member =
                memberRepository
                        .findById(memberId)
                        .orElseThrow(() -> new ErrorException(NOT_FOUND_MEMBER));


이런로직은 그 특성상 자주 쓰일수밖에 없죠.
메소드로 빼는게 좋아보입니다.

BaeRoNuI · 2025-05-19T10:41:18Z

+                .isFollowing(isFollowing)
+                .isOwner(isOwner)


자바 네이밍 컨벤션을 지켜주세요ㅠㅠ
필드값의 경우 boolean 은 isXxx(혹은 hasXxx 등)이 아닌 xxx 로 작성합니다.

BaeRoNuI · 2025-05-19T11:26:29Z

+    INVALID_ACCESS_SIGNATURE(UNAUTHORIZED, "JWT-001", "잘못된 Access Token 서명입니다."),
+    EXPIRED_ACCESS_TOKEN(UNAUTHORIZED, "JWT-002", "만료된 Access Token 토큰입니다."),
+    UNSUPPORTED_ACCESS_TOKEN(UNAUTHORIZED, "JWT-003", "지원되지 않는 Access Token 토큰입니다."),
+    INVALID_ACCESS_TOKEN(UNAUTHORIZED, "JWT-004", "유효하지 않은 Access Token 토큰입니다."),
+    INVALID_REFRESH_TOKEN(UNAUTHORIZED, "JWT-005", "유효하지 않은 Refresh 토큰입니다."),
+    INVALID_REFRESH_SIGNATURE(UNAUTHORIZED, "JWT-006", "잘못된 Refresh Token 서명입니다."),
+    EXPIRED_REFRESH_TOKEN(UNAUTHORIZED, "JWT-007", "만료된 Refresh Token 토큰입니다."),
+    UNSUPPORTED_REFRESH_TOKEN(UNAUTHORIZED, "JWT-008", "지원되지 않는 Refresh Token 토큰입니다."),


401, 403 관련은 에러코드, 일반적인 response 정도만 내려주고 자세한 이유는 로깅만 하는것이 좋습니다.

가정 : access token 만료되었으며, refresh token은 만료되지 않은 상태

클라이언트 api 요청 -H Authorization: Bearer [access token]

서버는 401 응답

클라이언트는 다시 refresh token 을 가지고 token refresh 요청 - 여기선 /token/refresh 로 만드신 것 같네요

서버는 access token refresh 하여 반환

이후 클라이언트는 다시 access token 활용

* feat: 좋아요 누르기 및 취소 api 구현 * feat: 좋아요 누르기 및 취소 api 연동 * chore: Java 스타일 수정 * refactor: 로그인 상태에서만 좋아요 누를 수 있도록 수정 및 알림 추가 --------- Co-authored-by: github-actions <>

* ✨ fix: 마이페이지 위치수정[FE] ♻️ refactor: 프로필 변경방식에 따른 리팩토링[BE]

* feat: ChatRoomController, ChatRoomService 기능 추가 * chore: Java 스타일 수정 * feat: 채팅방 프론트 구현 * feat: 프론트 수정 * fix: 사용하지 않는 파일 삭제 * feat: 네비게이션 바 채팅방 아이콘 추가 * refactor: SecurityConfig 수정 * feat: 게시글 채팅방 아이콘 추가, 기능 구현 * chore: Java 스타일 수정 * chore: 세미콜론, .vscode 파일 복구 * 패키지 추가 --------- Co-authored-by: github-actions <> Co-authored-by: dlsrks1021 <dlsrks1021@gmail.com>

* 🐛 fix: 언팔로우 버그 수정[FE]

* refactor: 코드 정리 * chore: 오타 수정

ksw733 · 2025-05-23T07:14:24Z

-        run: |
-          echo "${{ secrets.SCHEMA_SQL_B64 }}" | base64 -d > ${{ env.PROJECT_DIR }}/src/test/resources/schema.sql
-
      - name: Run tests


* fix: 프로그래머스 검증 예외처리 * chore: Java 스타일 수정 * refactor: 코드 리뷰 수정 * chore: Java 스타일 수정 --------- Co-authored-by: github-actions <>

* chore: 필요없는 주석, 로그 삭제 * chore: Java 스타일 수정 * refactor: 채팅방 목록 이미지 삭제 * refactor: 채팅방 목록에서 Room ID, Post ID 삭제 * refactor: 웹소켓 연결 시 불필요한 로그 삭제 * refactor: Optional -> List 로 변경 * refactor: 채팅방 프론트 불필요한 로그 주석처리 * chore: Java 스타일 수정 * refactor: 게시글 제목이 늘어나도 채팅방 버튼 크기 고정 * refactor: 게시글 제목이 늘어나면 ...로 함축 표시 * refactor: 피드에서 참여시 채팅 중복 참여 메시지 삭제 & 채팅방으로 리다이렉트 --------- Co-authored-by: github-actions <>

* fix: 채팅방 중복 접속 삭제 * refact: 주석 삭제

* fix: 채팅 멤버 목록 표시 수리 * refact: 테스트 에러 부분 삭제

* feat: 채팅 참가 멤버수 도메인 추가 * feat: 마지막 채팅 시간 도메인 추가 * feat: 프론트 채팅 멤버수, 마지막 채팅 시간 표시 * refact: chatroom column 추가 * chore: Java 스타일 수정 * refact: Count 쿼리 적용 * refact: 마지막 채팅 시간 관련 도메인 수정 * refact: 도메인 수정으로 인한 DDL 파일 작성

* test: OAuth 테스트 코드 * chore: Java 스타일 수정 * fix: package 파일 삭제 * test: OAuth 테스트 코드 --------- Co-authored-by: github-actions <>

* refactor: 채팅방 나가기 버튼 추가 * refactor: 불필요한 부분, 주석 삭제 * refactor: 프론트 나가기 버튼 기능 구현 * feat: 백엔드 채팅방 나가기 기능 구현 * chore: Java 스타일 수정 * refactor: 삭제된 기능 테스트 부분 삭제

ksw733 and others added 10 commits May 15, 2025 12:37

마이페이지 기능 구현, 팔로우 리팩토링 (#39)

17bfcaf

* ✨ feat: 마이페이지 기능 구현 ♻️ refactor: 팔로우 리팩토링

feat: PostController @AuthenticationPrincipal 기반 인증 방식으로 수정 (#48)

4714265

* refactor: PostController에 @AuthenticationPrincipal 적용 * chore: Java 스타일 수정 --------- Co-authored-by: github-actions <>

feat: 게시글 조회 기능 구현 (#45)

953e965

feat: 채팅 멤버, 메세지 구현 (#44)

1462cfe

마이페이지 구현[FE],API연동에 따른 리팩토링[BE] (#51)

99cb3ed

* ✨ feat: 마이페이지 구현[FE] ♻️ refactor: API연동에 따른 리팩토링[BE]

feat: 멤버 게시글 조회 (#53)

4925200

feat: 로그인, 회원가입 프론트 연동 (#55)

44ae331

feat: 마이페이지 게시글 조회 (FE) (#58)

ecc7cab

BaeRoNuI reviewed May 19, 2025

View reviewed changes

dlsrks1021 and others added 14 commits May 20, 2025 09:27

feat: 비회원 접근에 대한 홈 화면 수정 (#64)

bb99298

fix: logout, refresh 버그 수정 (#65)

23c6d8b

feat: 게시물 작성, 수정, 삭제 페이지 구현 및 연동 (#66)

63e72dc

fix: 마이페이지 화면 수정 및 이미지 업로드 리팩토링 (#70)

ad70758

* ✨ fix: 마이페이지 위치수정[FE] ♻️ refactor: 프로필 변경방식에 따른 리팩토링[BE]

fix: 무한스크롤 정지 현상 해결 (#73)

2a4bf34

chore: 코드 정리 (#76)

5fc1cfd

feat: 채팅 웹소켓 송수신 기능 구현 (#77)

40bfd35

fix: 배포 환경 시험 중 버그들 수정 (#79)

3e5af88

chore: 로그인 만료 에러 메세지 수정 (#83)

8d02e62

bug: 언팔로우 동작 버그 수정 (#84)

cca5333

* 🐛 fix: 언팔로우 버그 수정[FE]

🐛 fix: snakeCase 오타 수정[FE] (#86)

1a746b1

chore: 오타 수정 (#88)

4eed7b4

* refactor: 코드 정리 * chore: 오타 수정

ksw733 reviewed May 23, 2025

View reviewed changes

dlsrks1021 and others added 4 commits May 26, 2025 16:34

fix: 2차 프로젝트 배포 버전 (#91)

ac3d4a4

chore: BaseResponse 삭제 (#94)

972b4f5

refactor: PK 컬럼명을 entity_id에서 id로 변경 (#97)

4afcd31

fix: 프로그래머스 검증 예외처리 (#105)

78b89da

* fix: 프로그래머스 검증 예외처리 * chore: Java 스타일 수정 * refactor: 코드 리뷰 수정 * chore: Java 스타일 수정 --------- Co-authored-by: github-actions <>

audwls239 and others added 19 commits June 10, 2025 16:17

design: 피드에 작성 날짜 추가 (#109)

c98deb3

refactor: BrowserTest에 DBRider 적용 (#108)

0a58bac

design: 피드내용에 줄바꿈 적용 (#110)

ac94119

refactor: 첨부 이미지가 10장 이상인 경우 알림창 및 이미지 초기화 (#113)

789691e

fix: deleteByChatRoom 복원 (#122)

d858afb

feat: 채팅 멤버 목록 클릭 시 마이페이지로 이동, 타인 상태변경 버튼 삭제, 내 상태변경 기능 주석처리 (#119)

010b210

fix: 채팅방 중복 접속 삭제 (#120)

685fa0c

* fix: 채팅방 중복 접속 삭제 * refact: 주석 삭제

fix: 채팅 멤버 목록 표시 수정 (#118)

01f6f33

* fix: 채팅 멤버 목록 표시 수리 * refact: 테스트 에러 부분 삭제

fix: 채팅 멤버 목록 클릭 시 마이페이지로 이동, 타인 상태변경 버튼 삭제, 내 상태변경 기능 주석처리 (#128)

ea7fd1c

chore: thymeleaf 관련 dependency 삭제 (#124)

5b9a585

feat: 채팅방 프론트 최신순 정렬 (#130)

7fa1efe

chore: 코틀린 스타일 CI 추가 (#134)

ce09233

feat: 피드 수정/삭제 시 참조를 잃은 태그 또한 삭제 (#129)

5a46fab

fix: migration test 삭제 (#139)

bd7b60f

test: OAuth 테스트 코드 (#136)

8846f77

* test: OAuth 테스트 코드 * chore: Java 스타일 수정 * fix: package 파일 삭제 * test: OAuth 테스트 코드 --------- Co-authored-by: github-actions <>

feat: 모집글 관련 기능 구현 및 api 연동 (#137)

91a8920

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[2025-05-19] 멘토링 코드 리뷰#60

[2025-05-19] 멘토링 코드 리뷰#60
dlsrks1021 wants to merge 47 commits into
review_0519from
dev

dlsrks1021 commented May 19, 2025

Uh oh!

BaeRoNuI May 19, 2025

Uh oh!

BaeRoNuI May 19, 2025

Uh oh!

BaeRoNuI May 19, 2025

Uh oh!

BaeRoNuI May 19, 2025

Uh oh!

BaeRoNuI May 19, 2025

Uh oh!

ksw733 May 23, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Uh oh!

Conversation

dlsrks1021 commented May 19, 2025

Uh oh!

BaeRoNuI May 19, 2025

Choose a reason for hiding this comment

Uh oh!

BaeRoNuI May 19, 2025

Choose a reason for hiding this comment

Uh oh!

BaeRoNuI May 19, 2025

Choose a reason for hiding this comment

Uh oh!

BaeRoNuI May 19, 2025

Choose a reason for hiding this comment

Uh oh!

BaeRoNuI May 19, 2025

Choose a reason for hiding this comment

Uh oh!

ksw733 May 23, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants