Skip to content

Conversation

@JangDongHo
Copy link
Member

변경 사항

코드 컨벤션을 NestJS 스타일 가이드에 맞춰 전반적으로 리팩토링했습니다.

1. 파일 및 디렉토리 구조

  • requestrequests (복수형으로 통일)
  • responseresponses (복수형으로 통일)

2. 엔드포인트 URL 컨벤션

// 변경 전
POST /api/node/reading-rooms/get/campus
POST /api/node/reading-rooms/get/reading-room

// 변경 후
POST /api/node/reading-rooms/campuses/list
POST /api/node/reading-rooms/list
POST /api/node/reading-rooms/detail

3. DTO 네이밍 컨벤션

// 변경 전
GetDepartmentRequestDto
UpsertDepartmentRequestDto

// 변경 후
ListDepartmentsDto
UpsertDepartmentDto

4. 서비스 메소드 네이밍

// 변경 전
getCampusListCard()
getCollegeListCard()

// 변경 후
campusesListCard()
collegesListCard()

5. 접근 제어자 정리

  • 컨트롤러 메소드: public (HTTP 엔드포인트)
  • 서비스 메소드:
    • public: 외부에서 호출되는 메소드
    • private: 내부 구현 메소드

@JangDongHo JangDongHo merged commit d9af382 into main Mar 20, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants