Skip to content

Conversation

@johnbosco0414
Copy link
Collaborator

@johnbosco0414 johnbosco0414 commented Sep 24, 2025

제목(필수): [TYPE]: 제목 예) [FEAT]: 회원가입 기능 추가

제목 규칙 자세히 보기
  • 형식: [TYPE]: 제목
  • 제한: 50자 이내, 첫 글자 대문자, 명령문
  • TYPE: FEAT FIX REFACTOR COMMENT STYLE TEST CHORE INIT

무엇을 / 왜

  • 무엇(What):
  1. Scenario/Repository에 ScenarioRepository, SceneTypeRepository, SceneCompareRepository를 구현하였습니다.
  2. 시나리오 도메인의 각 기능을 Scenario/Service에 구현했습니다. 다만, 타 도메인과 연동이 필요한 부분은 Mocking 처리하였으며, 타 도메인 완성시 추후 구현할 예정입니다.
  3. Servcie 구현 과정에서 비교분석(Compare)의 로직이 수정됐습니다.
  4. Scenario/Service에 각 기능을 구현함에 따라 Mocking처리했던 Controller에 Service를 연결해주었습니다.
  5. JSON 파싱을 위해 Global 디렉토리에 JsonConfig를 생성하여 Object Mapper 메서드를 추가했습니다.
  6. ErrorCode에 3개의 ErrorCode를 추가했습니다.
  • 왜(Why):
  1. Entity 구성과 일관성을 유지하면서 각 기능별 캡슐화를 위함입니다.
  2. MVP구현을 위해서입니다.
  3. 기존에 염두해두었던 로직이 기획당시의 설계를 제대로 반영하지 못하고 있었기 때문입니다.
  4. 기존 Controller가 샘플데이터로 Mocking되어있었기때문입니다.
  5. 시나리오 도메인뿐만 아니라 타 도메인에서도 JSON 파싱이 필요한 경우가 있기 때문입니다.
  6. 예외 처리시 더 구체적으로 Code를 전달하기 위함입니다.

어떻게(요약) — 3줄 이내

  • Service Layer를 구현하였으며, 구현 과정에서 비교분석(Compare)의 로직을 변경하였고 이에 따라 ERD의 변경이 필요합니다. JSON 파싱을 위해 Global디렉토리에 JsonConfig를 생성하여 Object Mapper 메서드를 추가하였습니다. 추후 타 도메인 완성시 추가적인 업데이트가 있을 예정입니다.

영향 범위

  • API 변경
  • DB 마이그레이션
  • Breaking Change
  • 보안/권한 영향
  • 문서/가이드 업데이트 필요

체크리스트

  • 타입 라벨 부착 (FEAT/FIX/REFACTOR/COMMENT/STYLE/TEST/CHORE/INIT)
  • 로컬/CI 테스트 통과
  • 영향도 점검 완료
  • 주석/문서 반영(필요 시)

ToDo (선택)

  • 할 일 1
  • 할 일 2

스크린샷/증빙(선택)

이슈 연결 (자동)

Closes #21

@johnbosco0414 johnbosco0414 self-assigned this Sep 24, 2025
@johnbosco0414 johnbosco0414 added the FEAT 새로운 기능 추가 label Sep 24, 2025
@johnbosco0414
Copy link
Collaborator Author

추후 리팩토링시 Controller의 ApiResponse를 ResponseEntity로 바꿀 예정

Copy link
Collaborator

@lcs9317 lcs9317 left a comment

Choose a reason for hiding this comment

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

좋습니다 고생하셨습니다

Copy link
Collaborator

@Gooraeng Gooraeng left a comment

Choose a reason for hiding this comment

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

확인 완료했습니다 고생많으셨습니다!

@johnbosco0414 johnbosco0414 merged commit 0580d48 into main Sep 24, 2025
3 checks passed
@lcs9317 lcs9317 deleted the sceneService branch September 25, 2025 01:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FEAT 새로운 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT]: 시나리오 Service Layer 구현

4 participants