Skip to content

[REFACTOR] 코드 일관성 개선#88

Merged
soomin0209 merged 16 commits into
devfrom
refactor/code-consistency
May 11, 2026
Merged

[REFACTOR] 코드 일관성 개선#88
soomin0209 merged 16 commits into
devfrom
refactor/code-consistency

Conversation

@soomin0209

@soomin0209 soomin0209 commented May 11, 2026

Copy link
Copy Markdown
Contributor

@coderabbitai ignore

📝 작업 내용

네이밍 통일

  • BidController URL 경로 변수명 auction_idauctionId
  • AuctionSearchCondition 페이지 파라미터명 pageSizesize, 기본값 10 → 20
  • bid 도메인 예외 패키지명 exceptionsexception
  • eventBridge 패키지명 → eventbridge (Java 패키지 소문자 관례)
  • GlobalExceptionHandler 메서드명 handle 접두어 + camelCase로 통일

구조 정리

  • 전 도메인 dto 디렉토리 request / response 구조로 분리

응답 형식 통일

  • 컨트롤러 성공 응답 메시지 ~했습니다 형식으로 통일
  • 경매 취소 응답 타입 BaseResponse<Void>로 통일
  • 테스트 코드 응답 메시지 assertions 동기화

불필요한 코드 제거

  • 미사용 import, @NonNull, @Column(nullable = true), 빈 괄호(()) 제거
  • String.valueOf() 불필요한 래핑 제거
  • 미사용 파라미터 제거 (BidQueryService userDetails)
  • AsyncConfig 제거 → threads.virtual.enabled: true 설정으로 @Async의 기본 executor가 가상 스레드 기반

버그 수정

  • 경매 상태 필터 API 문서 설명 오류 수정 (CLOSEDDONE / NO_BID)

✅ 체크리스트 (Checklist)

  • 브랜치 이름 규칙을 준수했나요? (예: feat/login)
  • 코딩 컨벤션을 준수했나요?
  • 기능에 대한 테스트 코드를 작성/수행했나요?
  • 불필요한 주석이나 로그(console.log)를 제거했나요?

💬 기타 사항

디렉토리 구조 변경으로 인해 수정 파일이 파일이 142개라 코드래빗 리뷰는 스킵하겠습니다

@soomin0209 soomin0209 added BugFix 버그를 고쳤습니다 Refactor 코드 개선 labels May 11, 2026

@imprity imprity left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR 제목만 맞게 고치시면 완벽할 거 같습니다! LGTM!!

@soomin0209 soomin0209 changed the title Refactor/code consistency [REFACTOR] 코드 일관성 개선 May 11, 2026
@soomin0209 soomin0209 merged commit 9acb4a0 into dev May 11, 2026
2 checks passed
@soomin0209 soomin0209 deleted the refactor/code-consistency branch May 11, 2026 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BugFix 버그를 고쳤습니다 Refactor 코드 개선

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants