[feat] 활동 연혁 CRUD 기능 구현 및 테스트 #95

jucheonsu · 2025-08-22T14:59:30Z

Issue 📌

[feat] 활동 연혁 API 생성 #86

🏛️ Entity

ActivityHistory → Activity
Section

🔗Repository

ActivityHistoryRepository → ActivityRepository
SectionRepository

📄 DTO

ActivityHistoryRequestDto → ActivityRequestDto
ActivityHistoryResponseDto → ActivityResponseDto
GroupedActivityHistoryDto → ActivityItemDto SectionItemDto

🧠 Service

ActivityHistoryService → ActivityService
ActivityHistoryServiceImpl → ActivityServiceImpl

🌐 Controller

AdminActivityHistoryController → AdminActivityController
ActivityHistoryController → ActivityController

✅ Test

ActivityHistoryServiceTest → ActivityServiceTest

🔨 Setting

application.yml

server:
  port: 8080 // 8081에서 변경
spring:
  application:
    profiles:  // 삭제
      active: credentials  // 삭제

⚙️ 주요 변경 사항

날짜 필드 추가
- activityDate
활동 목록 정렬 (오름차순)
- 연도 → 섹션 ID → 활동 날짜
엔티티 분리
- Activity → Activity Section

📷 Image

Postman

---

---

---

IntelliJ

ysw789 · 2025-08-24T12:41:51Z

src/main/resources/application.yml

@@ -1,8 +1,10 @@
 server:
-  port: 8080
+  port: 8081


포트번호 8080로 원복해주세요

네 확인헀습니다

ysw789 · 2025-08-24T12:45:41Z

src/main/resources/application.yml

+  profiles:
+    active: credentials # 추가 (제외시키면 application-credentials.yml 못 찾음)


프로파일 설정은 실행명령으로 설정해주세요

네 확인했습니다

ysw789 · 2025-08-24T12:53:51Z

src/main/java/dmu/dasom/api/domain/activity/entity/ActivityHistory.java

+    private String section;
+    private String title;
+    private String award;


ActivityHistoryRequestDto에 필드 크기 제약조건을 설정했다면 그를 저장하는 속성의 크기도 그에 맞게 설정해주세요.
length 설정이 없으면 기본적으로 길이가 255로 설정되기 때문에 의도치 않게 길이가 긴 항목도 저장될 수 있습니다.

네 확인했습니다 수정하겠습니다

ysw789 · 2025-08-24T12:56:51Z

src/main/java/dmu/dasom/api/domain/activity/dto/GroupedActivityHistoryDto.java

+    @Getter @Builder
+    @Schema(name = "SectionItemDto", description = "섹션별 활동 목록")
+    public static class SectionItemDto {
+
+        @Schema(description = "활동 섹션", example = "교내 경진대회")
+        private final String section;
+
+        @Schema(description = "활동 목록")
+        private final List<ActivityItemDto> activities;
+    }


이너 클래스 대신 외부로 분리해주세요

네 확인했습니다 수정하겠습니다

ysw789 · 2025-08-24T12:56:59Z

src/main/java/dmu/dasom/api/domain/activity/dto/GroupedActivityHistoryDto.java

+    @Getter @Builder
+    @Schema(name = "ActivityItemDto", description = "개별 활동 목록")
+    public static class ActivityItemDto {
+
+        @Schema(description = "활동 연혁 고유 ID", example = "1")
+        private final Long id;
+
+        @Schema(description = "활동 제목", example = "컴퓨터 공학부 경진대회")
+        private final String title;
+
+        @Schema(description = "수상 내역", example = "최우수상")
+        private final String award;
+
+        public static ActivityItemDto toDto(ActivityHistory history) {
+            return ActivityItemDto.builder()
+                    .id(history.getId()).title(history.getTitle())
+                    .award(history.getAward())
+                    .build();
+        }
+    }


이너 클래스 대신 외부로 분리해주세요

네 확인했습니다 수정하겠습니다

ysw789 · 2025-08-24T13:03:13Z

src/main/java/dmu/dasom/api/domain/activity/service/ActivityHistoryServiceImpl.java

+        List<ActivityHistory> histories = historyRepository.findAll();
+        return GroupedActivityHistoryDto.groupedActivityHistoryDto(histories);


조회와 반환을 한 줄로 합쳐서 정리해주세요

네 확인했습니다 수정하겠습니다

ysw789 · 2025-08-24T13:04:56Z

src/main/java/dmu/dasom/api/domain/activity/service/ActivityHistoryServiceImpl.java

+    }
+
+    @Override
+    public ActivityHistoryResponseDto createHistory(ActivityHistoryRequestDto requestDto) {


생성 메소드에서 반환값이 필수가 아니라면 리턴 타입을 void 로 설정해도 무방할 것 같습니다

네 수정하겠습니다

ysw789 · 2025-08-24T13:05:07Z

src/main/java/dmu/dasom/api/domain/activity/service/ActivityHistoryServiceImpl.java

+    }
+
+    @Override
+    public ActivityHistoryResponseDto updateHistory(Long id, ActivityHistoryRequestDto requestDto) {


갱신 메소드에서 반환값이 필수가 아니라면 리턴 타입을 void 로 설정해도 무방할 것 같습니다

네 수정하겠습니다

ysw789 · 2025-08-24T13:05:44Z

src/main/java/dmu/dasom/api/domain/activity/service/ActivityHistoryServiceImpl.java

+        ActivityHistory history = requestDto.toEntity();
+        ActivityHistory savedHistory = historyRepository.save(history);


dto 파싱과 저장을 한 줄로 합쳐서 정리해주세요

네 수정하겠습니다

ysw789 · 2025-08-24T13:12:26Z

src/main/java/dmu/dasom/api/domain/activity/dto/ActivityHistoryRequestDto.java

+    public ActivityHistory toEntity() {
+        return ActivityHistory.builder()
+                .year(this.year)
+                .section(this.section)
+                .title(this.title)
+                .award(this.award)
+                .build();
+    }


ActivieyHistory의 팩토리 메소드를 ActivieyHistory 클래스 내부로 옮기고 Service 계층에서 Dto -> Entity 로 변환할 때 ActivieyHistory.create() 를 호출해서 객체 인스턴스를 생성/초기화 하도록 구성해주세요

네 수정하겠습니다

ysw789 · 2025-08-24T13:23:31Z

작업 고생하셨습니다~ 추가적인 피드백 드리겠습니다!

GroupedActivityHistoryDto 클래스의 경우를 보시다시피 여러 DTO를 중첩 초기화하는 과정에서 코드가 장황해질 수 있습니다.
이를 개선하기 위해 각 DTO 클래스 내부에 팩토리 메소드를 만들어두고, 루트 DTO에서는 해당 메소드를 호출해 초기화하도록 구성하는 것이 좋습니다. 이렇게 하면 객체 생성 로직이 DTO 내부로 캡슐화되어 외부에서는 메소드 호출만으로 객체를 만들 수 있고, 클래스 구조 변경에도 영향이 최소화됩니다. 이 방법은 객체지향 설계의 기본적인 원칙(캡슐화, 책임 분리)에도 부합하는 방법입니다!
활동 연혁에 대한 정렬 기준이 필요하다면 ActivieyHistory 엔티티에 sortOrder(정렬 기준) 필드를 추가해 프론트엔드에서 해당 값을 기준으로 정렬하도록 제공해줘도 좋을 것 같습니다. 이 부분은 회의 때 논의해보세용

jucheonsu · 2025-08-24T14:14:38Z

상세한 피드백 감사합니다! 빠르게 고쳐서 다시 올리겠습니다!

jucheonsu · 2025-08-26T07:13:35Z

수정 완료했습니다 확인 부탁드리겠습니다!

dohy-eon · 2025-08-26T09:22:45Z

확인했습니다 @jucheonsu

jucheonsu · 2025-08-26T09:46:11Z

네 감사합니다

jucheonsu added 4 commits August 22, 2025 23:26

feat: 활동 연혁 엔티티 정의 (DASOMBE-15)

070c798

feat: 활동 연혁 리포지토리 인터페이스 정의 (DASOMBE-15)

2d827cd

feat: 활동 연혁 DTO 정의 (DASOMBE-15)

6899774

feat: 활동 연혁 서비스 로직 구현 (DASOMBE-15)

8f497a7

jucheonsu self-assigned this Aug 22, 2025

jucheonsu added FEAT CHORE TEST labels Aug 22, 2025

jucheonsu linked an issue Aug 22, 2025 that may be closed by this pull request

[feat] 활동 연혁 API 생성 #86

Open

jucheonsu changed the title ~~[feat] 활동 연혁 CRUD 기능 구현 및 테스트 추가~~ [feat] 활동 연혁 CRUD 기능 구현 및 테스트 Aug 22, 2025

jucheonsu requested review from dohy-eon, hodoon and ysw789 August 22, 2025 15:07

jucheonsu added 3 commits August 23, 2025 00:29

feat: 활동 연혁 API 컨트롤러 구현 (DASOMBE-15)

38ab213

test: 활동 연혁 서비스 단위 테스트 추가 (DASOMBE-15)

903de41

chore: 활동 연혁 기능 관련 환경 설정 (DASOMBE-15)

181ed4e

jucheonsu force-pushed the feat/DASOMBE-15-activity-history branch from 38e795b to 181ed4e Compare August 22, 2025 15:43

jucheonsu changed the title ~~[feat] 활동 연혁 CRUD 기능 구현 및 테스트~~ [feat] 활동 연혁 CRUD 기능 구현 및 테스트 (DASOMBE-15) Aug 22, 2025

jucheonsu changed the title ~~[feat] 활동 연혁 CRUD 기능 구현 및 테스트 (DASOMBE-15)~~ [feat] 활동 연혁 CRUD 기능 구현 및 테스트 Aug 22, 2025

jucheonsu changed the base branch from main to dev August 23, 2025 04:22

ysw789 requested changes Aug 24, 2025

View reviewed changes

hodoon approved these changes Aug 25, 2025

View reviewed changes

dohy-eon approved these changes Aug 25, 2025

View reviewed changes

jucheonsu added 5 commits August 26, 2025 13:33

refactor: 엔티티 계층 리팩토링 (DASOMBE-15)

4fdba6b

refactor: 리포지토리 계층 리팩토링 (DASOMBE-15)

1c73794

refactor: DTO 계층 리팩토링 및 분리 (DASOMBE-15)

27c1687

remove: 사용하지 않는 파일 제거 (DASOMBE-15)

fb365ca

refactor: 서비스 계층 리팩토링 (DASOMBE-15)

77d1538

refactor: 컨트롤러 계층 리팩토링 (DASOMBE-15)

9145987

jucheonsu force-pushed the feat/DASOMBE-15-activity-history branch from c4ea801 to a3827a0 Compare August 26, 2025 05:38

jucheonsu added 3 commits August 26, 2025 14:59

test: 테스트 코드 수정 (DASOMBE-15)

0b4d239

chore: 환경설정 변경 (DASOMBE-15)

2a6a2d7

rename: Activity로 변경 (DASOMBE-15)

87a1187

jucheonsu force-pushed the feat/DASOMBE-15-activity-history branch from a3827a0 to 87a1187 Compare August 26, 2025 06:06

jucheonsu closed this Aug 26, 2025

jucheonsu deleted the feat/DASOMBE-15-activity-history branch August 26, 2025 06:07

jucheonsu restored the feat/DASOMBE-15-activity-history branch August 26, 2025 06:10

jucheonsu deleted the feat/DASOMBE-15-activity-history branch August 26, 2025 06:11

jucheonsu restored the feat/DASOMBE-15-activity-history branch August 26, 2025 06:12

jucheonsu reopened this Aug 26, 2025

jucheonsu requested a review from ysw789 August 26, 2025 07:00

jucheonsu added REFACTOR REMOVE RENAME labels Aug 26, 2025

ysw789 approved these changes Aug 26, 2025

View reviewed changes

jucheonsu merged commit adf631c into dev Aug 26, 2025
2 checks passed

jucheonsu removed REFACTOR CHORE TEST REMOVE RENAME labels Aug 26, 2025

		profiles:
		active: credentials # 추가 (제외시키면 application-credentials.yml 못 찾음)

		List<ActivityHistory> histories = historyRepository.findAll();
		return GroupedActivityHistoryDto.groupedActivityHistoryDto(histories);

		ActivityHistory history = requestDto.toEntity();
		ActivityHistory savedHistory = historyRepository.save(history);

[feat] 활동 연혁 CRUD 기능 구현 및 테스트 #95

[feat] 활동 연혁 CRUD 기능 구현 및 테스트 #95

Uh oh!

Conversation

jucheonsu commented Aug 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Issue 📌

🏛️ Entity

🔗Repository

📄 DTO

🧠 Service

🌐 Controller

✅ Test

🔨 Setting

⚙️ 주요 변경 사항

📷 Image

Postman

IntelliJ

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ysw789 commented Aug 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jucheonsu commented Aug 24, 2025

Uh oh!

jucheonsu commented Aug 26, 2025

Uh oh!

dohy-eon commented Aug 26, 2025

Uh oh!

jucheonsu commented Aug 26, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

jucheonsu commented Aug 22, 2025 •

edited

Loading

ysw789 commented Aug 24, 2025 •

edited

Loading