refactor(admin): 주석 및 로그 삭제

[HyemIin]

작업 요약

getUser 주석 및 로그 삭제

Issue Link

문제점 및 어려움

해결 방안

Reference

Summary by CodeRabbit

릴리스 노트

• Chores (유지보수)
  • 여러 서비스에서 미사용 코드 제거
  • 불필요한 로깅 문장 최적화
  • 빌드 설정 형식 정규화

이 릴리스는 내부 코드 정리를 포함하며, 사용자에게 영향을 주는 기능 변경사항은 없습니다.

[coderabbitai]

개요

여러 서비스 클래스에서 주석 처리된 getUser() 헬퍼 메서드를 제거하고 OrderService의 로그 문을 정리했습니다. build.gradle에서 의존성 표기법의 따옴표 형식을 변경했습니다. 런타임 동작은 변경되지 않습니다.

변경 사항

코호트 / 파일	변경 요약
빌드 설정 nowait-app-admin-api/build.gradle	의존성 표기법의 따옴표 형식을 이중 따옴표에서 단일 따옴표로 변경 (의미론적 영향 없음)
서비스 클래스 정리 nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/menu/service/MenuService.java, nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/store/service/StoreServiceImpl.java, nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/reservation/service/ReservationService.java, nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/service/StorePaymentServiceImpl.java	주석 처리된 private User getUser(MemberDetails) 헬퍼 메서드 제거
OrderService 로그 정리 nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/order/service/OrderService.java	findAllOrders() 메서드에서 log.info() 문 제거 및 주석 처리된 코드 블록 정리

코드 리뷰 예상 노력

🎯 1 (단순) | ⏱️ ~5분

• 모든 변경 사항이 주석 처리된 코드 제거 또는 로그 문 정리로 구성됨
• 활성 코드 로직에 영향 없음
• 반복적인 패턴의 정리 (여러 서비스에서 동일한 메서드 제거)

제안 라벨

refactor

제안 리뷰어

• Jjiggu

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목은 변경 사항의 주요 목표를 명확하게 설명합니다. 주석 및 로그 삭제라는 리팩토링 작업이 모든 수정된 파일에서 일관되게 수행되었습니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/#331/N+1해결

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3bbda2d and 1baf365.

📒 Files selected for processing (6)

• nowait-app-admin-api/build.gradle (1 hunks)
• nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/menu/service/MenuService.java (0 hunks)
• nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/order/service/OrderService.java (0 hunks)
• nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/reservation/service/ReservationService.java (0 hunks)
• nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/store/service/StoreServiceImpl.java (0 hunks)
• nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/service/StorePaymentServiceImpl.java (0 hunks)

💤 Files with no reviewable changes (5)

• nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/reservation/service/ReservationService.java
• nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/service/StorePaymentServiceImpl.java
• nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/store/service/StoreServiceImpl.java
• nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/menu/service/MenuService.java
• nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/order/service/OrderService.java

🔇 Additional comments (1)

nowait-app-admin-api/build.gradle (1)

37-37: 따옴표 스타일 일관성 개선.

파일 내 다른 프로젝트 의존성 선언(34-36줄)과 동일하게 작은따옴표로 통일되어 일관성이 좋습니다.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}