Feat: 백오피스 개발 by MoonMinHyuk1 · Pull Request #165 · ODOICHON/server

MoonMinHyuk1 · 2023-05-23T07:50:33Z

주요 작업 내용

백오피스 api 개발

작업 내용

CheckList

CI를 통과했나요?
리뷰어를 등록했나요?
참고 레퍼런스가 있을 경우, PR 혹은 댓글로 남겼나요?

전달 사항

간단한 시나리오는 노션 테크블로그 파트에 적어두었습니다.
개발 후에 바로 pr을 날리고 싶었지만 테스트때문에 ci jacoco 오류가 떠서 날릴 수 없었습니다..
모호한 부분이 있을 수 있어서 계속 수정할 계획입니다!
태민님께서 의견주신 웹소켓도 적용 고려해보겠습니다^^

sonarqubecloud · 2023-05-23T07:55:41Z

SonarCloud Quality Gate failed.

0 Bugs
0 Vulnerabilities
0 Security Hotspots
22 Code Smells

72.6% Coverage
0.0% Duplication

dldmsql · 2023-05-23T11:25:24Z

src/main/kotlin/com/example/jhouse_server/domain/record/controller/RecordController.kt

+    fun getRecord(
+        @PathVariable("record_id") recordId: Long,
+        request: HttpServletRequest,
+        @PageableDefault(size = 10) pageable: Pageable


페이징 처리에서 디폴드 값이 10인 이유가 있나요..? 위에 API는 4로 지정되어 있어서 여쭤봅니다.

레코드 단건 조회에서의 페이징은 댓글에 대한 페이징이어서 제가 임의로 10개로 설정해두었습니다!

dldmsql · 2023-05-23T11:30:37Z

src/main/kotlin/com/example/jhouse_server/domain/record/repository/RecordRepository.kt

+
+interface RecordRepository: JpaRepository<Record, Long>, RecordRepositoryCustom {
+
+    @Query("select dtype from record where id = :recordId", nativeQuery = true)


컬럼을 하나만 추출하고 싶을 때에는 정적 쿼리로 사용해야 하는 군요..!

이건 사실 슈퍼-서브 클래스 사용 시 dtype을 알 방도가 없어서 사용했습니다😅

dldmsql · 2023-05-23T11:36:21Z

src/main/kotlin/com/example/jhouse_server/domain/record/repository/RecordRepository.kt

+    @Query("select r from Record r where r.createdAt >= :weekAgo and r.status = 'WAIT'")
+    fun findWeekAgo(@Param("weekAgo") weekAgo: LocalDateTime): List<Record>
+
+    @EntityGraph(attributePaths = ["user"])


해당 어노테이션의 사용은 처음 접하는 것 같아요.!
Record 객체 조회 시, user 객체를 한 번에 fetch join으로 불러오면 성능에서 이점으로 작용할 것 같아요 👍🏻

dldmsql · 2023-05-23T11:40:46Z

src/main/kotlin/com/example/jhouse_server/domain/record/service/RecordServiceImpl.kt

+    }
+
+    override fun getHotRecords(): RecordHotResDto {
+        val weekAgo = LocalDateTime.now().minusWeeks(7)


LocalDateTime 라이브러리에 minusWeeks() 함수가 있는지는 몰랐어요. 유용한 함수들이 많군요

dldmsql · 2023-05-23T11:42:03Z

src/main/kotlin/com/example/jhouse_server/domain/record/service/RecordServiceImpl.kt

+        return when (condition.type) {
+            "all" -> {
+                val records = recordRepository.findRecords(condition, pageable)
+                RecordPageResDto(records)
+            }
+            "odori" -> {
+                val records = odoriRepository.findOdoris(condition, pageable)
+                RecordPageResDto(records)
+            }
+            "retro" -> {
+                val records = retrospectionRepository.findRetrospections(condition, pageable)
+                RecordPageResDto(records)
+            }
+            "tech" -> {
+                val records = technologyRepository.findTechnologies(condition, pageable)
+                RecordPageResDto(records)
+            }
+            else -> throw ApplicationException(NOT_FOUND_EXCEPTION)


when에서 case에 해당하는 부분을 문자열 값으로 사용하기 보다는 명시적으로 enum 변수를 사용하는 건 어떨까요? 유지보수에도 용이할 것 같습니다!

오! 바로 적용 가겠습니다~~

dldmsql · 2023-05-23T11:42:34Z

src/main/kotlin/com/example/jhouse_server/domain/record/service/RecordServiceImpl.kt

+
+        return when (recordRepository.findDType(recordId)) {
+            "O" -> {
+                val odori = record as Odori


타입스크립트가 생각나는 문법이군요 .. 👍🏻

dldmsql · 2023-05-23T11:43:34Z

src/main/kotlin/com/example/jhouse_server/domain/record/service/RecordServiceImpl.kt

+        val record = recordRepository.findByIdWithUser(recordId)
+            .orElseThrow { ApplicationException(NOT_FOUND_EXCEPTION) }


record의 pk 값인 id로 조회 후, exception 처리를 하는 것이라면 findByIdOrThrow() 함수를 사용해도 되지 않을까 합니다!

레코드의 유저 정보를 사용하기 때문에 페치 조인으로 한번에 불러오기 위해 사용했습니다!

dldmsql · 2023-05-23T11:49:11Z

src/main/kotlin/com/example/jhouse_server/domain/record_comment/entity/RecordComment.kt

+    var ref: Long,
+
+    var step: Long,
+
+    var level: Long,


LinkedList가 생각나는 구조인데, 맞을까요..?

무한 대댓글 머리 빠지는 줄 알았어요...

ㅋㅋㅋㅋ앗!! 코드 상의 구조는 굉장히 이쁜데요?!! 역시 갓민혁😎

dldmsql · 2023-05-23T11:50:40Z

...n/kotlin/com/example/jhouse_server/domain/record_comment/service/RecordCommentServiceImpl.kt

+        val record = recordRepository.findByIdOrThrow(recordCommentReqDto.recordId)
+        var parent: RecordComment? = null
+        val refCount = recordCommentRepository.findNextRef(record)
+        var ref: Long = if(refCount == null) 1 else (refCount + 1).toLong()
+        var step: Long = 1
+        var level: Long = 1
+        var allChildrenSize: Long = 0
+        if(recordCommentReqDto.parentId != null) {
+            parent = recordCommentRepository.findByIdOrThrow(recordCommentReqDto.parentId)
+
+            val parentStep: Long = parent.step
+            val parentAllChildrenSize: Long = parent.allChildrenSize
+            step = parentStep + parentAllChildrenSize + 1
+            ref = parent.ref
+            level = parent.level + 1
+
+            val comments = recordCommentRepository.findSameRef(record, ref)
+            updateAllChildrenSize(parent)
+            updateStep(comments, step)
+        }
+        return recordCommentRepository.save(RecordComment(recordCommentReqDto.content, record, user, ref, step, level, allChildrenSize, parent)).id


미녁씨... 눈에 확 들어오는 코드 ... 너무 멋있는 거 아닌가요 ? 😊👍🏻⭐️

dldmsql · 2023-05-23T11:51:19Z

src/main/kotlin/com/example/jhouse_server/domain/record_review/dto/RecordReviewDto.kt

+    @JsonFormat(pattern = "MM.dd.yyyy")
+    @JsonProperty("create_at")
+    val createAt: LocalDateTime


포맷팅까지 해주는 센스 ... 🥺

YoonTaeMinnnn

혼자서 이렇게 많은 api 뚝딱뚝딱 만들어줘서 감사합니다! 고생하셨어요!
우릉비씨가 제 마음을 대변하는 리뷰를 해주셔서 저는 딱히 코멘트 안달었어요!! ㅎ..ㅎ🤗

dldmsql · 2023-05-23T15:43:06Z

ㅋㅋㅋ 저의 리뷰가 좀 야무지긴 했죠?? 😎 @YoonTaeMinnnn

MoonMinHyuk1 added 30 commits May 14, 2023 21:33

feat: admin type 추가

6fc8292

feat: record review domain

d1b966c

feat: record review api

d00000b

feat: admin type 추가

ab8eddb

feat: record comment api

3d5e304

feat: record category domain

ecb86fd

feat: record category api

37d5ab0

feat: record domain

87afc09

feat: record dto

5069d5d

feat: record api

c0e857c

refactor: page 반환타입 수정

a00f181

feat: record dto

c6d2eee

feat: record domain

440d52b

refactor: status 조건 추가

01e2cd7

feat: scheduler

54be018

feat: record review apply domain

6170e97

feat: record review apply dto

7bc8493

feat: record review apply api

2ab041d

feat: record review domain

42fbd72

feat: record review dto

630103c

feat: record review error

2ad7b13

feat: record review api

89a045d

restore

0db42f1

Merge branch 'dev' into feat/back-office

118afc7

feat: scheduling

b749f1f

Merge branch 'dev' into feat/back-office

93fd4d8

feat: record comment domain

6e394c1

feat: record comment dto

c5dd45c

feat: record comment api

63ca333

feat: record comment

ec3753d

MoonMinHyuk1 added 12 commits May 23, 2023 00:39

refactor: 변수명 변경

692867d

test: service test

4fca1c7

test: controller test

e6b4a93

docs: record review api 명세서

6b6dec2

refactor: refactor dto

b9487c6

refactor: 쿼리 수정

6a7c31a

test: service test

2a17265

refactor: 변수 타입 변경

654d451

test: controller test

492ad76

docs: record category api 명세서

4426912

docs: record api 명세서

10dee1d

docs: api 명세서

ae789a7

MoonMinHyuk1 requested review from YoonTaeMinnnn and dldmsql May 23, 2023 07:50

dldmsql reviewed May 23, 2023

View reviewed changes

dldmsql approved these changes May 23, 2023

View reviewed changes

YoonTaeMinnnn approved these changes May 23, 2023

View reviewed changes

This comment was marked as duplicate.

Sign in to view

MoonMinHyuk1 merged commit 0ec7738 into dev May 23, 2023


		interface RecordRepository: JpaRepository<Record, Long>, RecordRepositoryCustom {

		@Query("select dtype from record where id = :recordId", nativeQuery = true)

		val record = recordRepository.findByIdWithUser(recordId)
		.orElseThrow { ApplicationException(NOT_FOUND_EXCEPTION) }

Conversation

MoonMinHyuk1 commented May 23, 2023

주요 작업 내용

작업 내용

CheckList

전달 사항

Uh oh!

sonarqubecloud bot commented May 23, 2023

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!

MoonMinHyuk1 May 23, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

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!

YoonTaeMinnnn left a comment

Choose a reason for hiding this comment

Uh oh!

This comment was marked as duplicate.

Uh oh!

dldmsql commented May 23, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Comments

MoonMinHyuk1 May 23, 2023 •

edited

Loading