71 feat/student activity list #77

KimTaeO · 2023-10-26T05:00:01Z

💡 개요

동아리와 관계없이 학생 활동 전체를 조회하는 api를 작성하였습니다

📃 작업내용

Pageable을 파라미터로 받는 queryAllStudentActivity service
QueryAllStudentActivity controller
QueryAllStudentResponse DTO

esperar · 2023-10-26T05:02:13Z

...kotlin/team/msg/domain/student/presentation/data/response/QueryAllStudentActivityResponse.kt

+data class QueryAllStudentActivityResponse(
+    val activityId: UUID,
+    val title: String,
+    val userId: UUID,
+    val userName: String,
+    val approveStatus: ApproveStatus
+)


앞서 설명해드렸어야했는데 Response는 공통 DomainResponse 스크립트에 관리하는게 맞는 것 같아요 기본 인자를 가지고있는 DomainResponse와 ListResponse, DetailResponse 클래스를 한 스크립트에 같이 삽입하고 companion object에 정의된 of 함수를 통해 response를 생성해서 관리하는게 좋을 것 같습니다.

공통 DomainResponse로 처리하였습니다 8f2228c

esperar · 2023-10-26T05:03:08Z

bitgouel-api/src/main/kotlin/team/msg/domain/student/presentation/StudentActivityController.kt

+    fun queryAllStudentActivity(pageable: Pageable): ResponseEntity<Page<QueryAllStudentActivityResponse>> {
+        val result = studentActivityService.queryAllStudentActivity(pageable)
+        return ResponseEntity.status(HttpStatus.OK).body(result)


여긴 왜 result인가요? 다른곳에서는 response로 쓰이는데 일관성을 잘 맞춰주세요

변경하였습니다
e0668c1

@KimTaeO 아직 변경 안된 것 같습니다!

이번에는 정말 변경했습니다
06268d5

esperar · 2023-10-26T05:08:09Z

bitgouel-api/src/main/kotlin/team/msg/domain/student/service/StudentActivityServiceImpl.kt

+    override fun queryAllStudentActivity(pageable: Pageable): Page<QueryAllStudentActivityResponse> {
+        val user = userUtil.queryCurrentUser()
+
+        val studentActivities = studentActivityRepository.findAll(pageable)


ToOne 관계에서는 페이징 관련 조회라도 조인을 사용해서 N + 1을 해결해도되며 @EntityGraph를 사용하는 것도 방법일 수 있습니다. 추가로 @BatchSize 어노테이션을 사용해 최대 배치 사이즈를 지정해서 성능을 향상시키는 여러가지 방법을 고려해보셨으면 좋겠습니다.

이 글을 참고해보세요

일단은 StudentActivityRepository에서 findAll할 경우에 @entitygraph를 통하여 N+1 문제를 해결하였으며, @batchsize는 ~ToMany관계에서 사용을 주로 한다고 하더군요. 나중에 전체 조회가 아닌 학생 조회를 하게 된다면 학생과 학생활동 간의 양방향 매핑을 하여 적용을 하는 것을 고려 해보도록 하겠습니다 그리고 다른 코드에서도 N + 1 문제가 발생하는 코드들이 있더군요 그 부분은 따로 Issue를 통해 해결해보도록 하겠습니다
0f7b5f4

bitgouel-api/src/main/kotlin/team/msg/domain/student/service/StudentActivityServiceImpl.kt

KimTaeO added 5 commits October 25, 2023 23:03

create :: query all student activity resopnse

86bb0a1

add :: query all student activity service

65039dd

update :: 함수명 규약대로 변경

a8afa4a

add :: query all student activity controller

35bf322

add :: query all student activity security config

397a27c

KimTaeO added 1️⃣ Priority: High 우선순위 - 상 ✨ Feature 신규 기능 labels Oct 26, 2023

KimTaeO self-assigned this Oct 26, 2023

KimTaeO requested review from esperar, JuuuuHong and ani2689 as code owners October 26, 2023 05:00

KimTaeO linked an issue Oct 26, 2023 that may be closed by this pull request

학생 활동 정보 리스트 #71

Closed

esperar reviewed Oct 26, 2023

View reviewed changes

ani2689 reviewed Oct 26, 2023

View reviewed changes

bitgouel-api/src/main/kotlin/team/msg/domain/student/service/StudentActivityServiceImpl.kt Show resolved Hide resolved

KimTaeO and others added 7 commits October 26, 2023 14:50

create :: student activity response

8f2228c

update :: 함수 리턴타입 새 response로 변경

835a88b

update :: controller response타입 변경

e0668c1

add :: student activity 객체 조회 시 student도 같이 조회

0f7b5f4

add :: 주석 추가

a965d4a

docs :: 주석 내용 변경

af8be2f

update :: query all student activity 변수명

06268d5

esperar approved these changes Oct 28, 2023

View reviewed changes

JuuuuHong approved these changes Oct 29, 2023

View reviewed changes

KimTaeO merged commit 0a2f70c into master Oct 30, 2023
1 check passed

KimTaeO deleted the 71-feat/student-activity-list branch October 30, 2023 00:17

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

71 feat/student activity list #77

71 feat/student activity list #77

KimTaeO commented Oct 26, 2023 •

edited

Loading

esperar Oct 26, 2023

KimTaeO Oct 27, 2023

esperar Oct 26, 2023

KimTaeO Oct 27, 2023

JuuuuHong Oct 27, 2023

KimTaeO Oct 27, 2023

esperar Oct 26, 2023

KimTaeO Oct 27, 2023

71 feat/student activity list #77

71 feat/student activity list #77

Conversation

KimTaeO commented Oct 26, 2023 • edited Loading

💡 개요

📃 작업내용

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

KimTaeO commented Oct 26, 2023 •

edited

Loading