Skip to content

refactor: 셔틀 버스 API 리팩토링 #2098

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 22 commits into from
Nov 27, 2025
Merged

refactor: 셔틀 버스 API 리팩토링 #2098

merged 22 commits into from
Nov 27, 2025

Conversation

@dh2906
Copy link
Contributor

@dh2906 dh2906 commented Nov 24, 2025 •
edited
Loading

🔍 개요

  • 셔틀 버스 업로드, 수정 API에 대한 리팩토링을 진행한다.

🚀 주요 변경 내용

  • 기존 util 패키지에 있는 모든 클래스는 ~~Parser 로 네이밍 되어있었으나, 아래와 같은 기준으로 이를 분리함.

    1. extractor : 엑셀에서 특정 셀의 내용을 가져오는 모듈
    2. parser : 문자열을 괄호 여부에 따라 이름, 세부 이름으로 분리하는 모듈 (정규표현식 활용)
    3. util : 범용성있게 사용할 수 있을만한 모듈

  • 업로드 API의 응답 포맷을 등하교, 생협에서 제공하는 포맷과 같이 통일

💬 참고 사항

✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

dh2906 added 17 commits November 18, 2025 11:50
@dh2906
chore: 개행 추가
c0f179f
@dh2906
refactor: 이너 클래스 명 변경
d6d84a4
@dh2906
docs: 스웨거 명세 예시 추가
19212ba
@dh2906
refactor: 이름, 세부 이름 파싱 로직 분리
d95df9f
@dh2906
refactor: 문자열 분리 로직 모델에서 호출하도록 수정
0422d99
@dh2906
refactor: 파서 패키지 이동
695adad
@dh2906
refactor: 엑셀 데이터 추출 클래스 명 Parser -> Extractor로 변경 및 패키지 이동
1ef2105
@dh2906
refactor: 셀 내용 추출 클래스 이름 변경 및 메소드 명 변경
a459143
@dh2906
refactor: 파일 업로드 미리보기 메소드 이름 변경
1b2bab6
@dh2906
refactor: 엑셀 셀 문자열 추출을 PoiExtractor로 통일
85e12c0
@dh2906
refactor: Extractor 클래스 stateful하도록 수정
c5a1cb7
@dh2906
refactor: PoiCellExtractor.extractStringValue() 호출하도록 수정
9c2a47f
@dh2906
refactor: 셀 내용 추출 시 서식에 대한 문제 방지
d115bfe
@dh2906
refactor: 응답 DTO 구조 변경
3254c09
@dh2906
refactor: 메소드 명 변경
414117e
@dh2906
chore: 코드 포맷팅
b886158
@dh2906
refactor: 서비스 메소드 명 수정
9bc0db6
@dh2906 dh2906 requested review from Soundbar91 and kih1015 November 24, 2025 05:07
@dh2906 dh2906 self-assigned this Nov 24, 2025
@dh2906 dh2906 added the 리팩터링 리팩터링을 위한 이슈입니다 label Nov 24, 2025
@github-actions
Copy link

github-actions bot commented Nov 24, 2025 •
edited
Loading

Unit Test Results

672 tests   671 ✔️  1m 17s ⏱️
165 suites      1 💤
165 files        0 ❌

Results for commit ce57dd8.

♻️ This comment has been updated with latest results.

asa9874
asa9874 requested changes Nov 25, 2025
View reviewed changes
Copy link
Member

@asa9874 asa9874 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다👍 코멘트 남겨뒀습니다

Soundbar91
Soundbar91 approved these changes Nov 26, 2025
View reviewed changes
Copy link
Collaborator

@Soundbar91 Soundbar91 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

.../java/in/koreatech/koin/admin/bus/shuttle/controller/AdminShuttleBusTimetableController.java Outdated
List<AdminShuttleBusTimetableResponse> response = adminShuttleBusExcelService
.previewShuttleBusTimetable(file);
AdminShuttleBusTimetableResponse response = adminShuttleBusExcelService
.getShuttleBusTimetablePreview(file);
Copy link
Collaborator

@Soundbar91 Soundbar91 Nov 26, 2025

Choose a reason for hiding this comment

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

C

줄 넘김 확인해주세요

Copy link
Contributor Author

@dh2906 dh2906 Nov 27, 2025

Choose a reason for hiding this comment

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

감사합니당!
수정 완료 했습니다. dd6c17c

@dh2906 dh2906 merged commit 2f774f8 into develop Nov 27, 2025
5 checks passed
@dh2906 dh2906 deleted the refactor/2090-shuttle-bus branch November 27, 2025 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Soundbar91 Soundbar91 Soundbar91 approved these changes

@asa9874 asa9874 asa9874 approved these changes

@kih1015 kih1015 Awaiting requested review from kih1015

Assignees

@dh2906 dh2906

Labels

리팩터링 리팩터링을 위한 이슈입니다

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[어드민] 셔틀 버스 어드민 기능을 리팩토링 한다.

4 participants

@dh2906 @Soundbar91 @asa9874