Feature/jaino/#67 #68

jeongjaino · 2023-12-22T11:43:55Z

1. 📄 관련된 이슈 및 소개

#66 설문 작성 Validation 구현

2. 🔥변경된 점

작성한 설문일 경우 들어가지지 않음.
주관식 답변일 경우 10자 이상으로 작성하도록 구현

3. 📸 스크린샷(선택)

4. 💡알게된 혹은 궁금한 사항들

…Migration

…ration

…ository로 이동

tgyuuAn · 2023-12-23T00:52:07Z

설문 작성 시 surveyId랑 userId 를 이용하여 유효성 검증을 한다 라는 말에서 약간 이상하다고 생각이 들어서 곰곰히 생각해보았는데요,

PR 설명을 보니까 설문 작성이 아니라 설문 응답에 대한 유효성 검증이었군요!

추후에 응답한 설문 내용 수정 기능을 도입한다고 하면, 응답한 surveyId랑 userId를 이용해서 값을 불러와서 수정할 화면에 띄워준 뒤,

새로운 값을 입력받고 완료 버튼을 누르면 db에 update하는 형식으로 하면 수정 기능도 도입할 수 있겠네요!

tgyuuAn

진호상 고생하셨습니다!

소소한 코멘트 남겨봤습니다. 제 코멘트에 대한 진호님의 의견 궁금해요!

이번 PR도 수고하셨습니다~ 👍👍👍

tgyuuAn · 2023-12-23T00:52:53Z

core/data/src/main/java/com/wap/wapp/core/data/repository/survey/SurveyRepository.kt


-    suspend fun getSurveyForm(eventId: Int): Result<SurveyForm>
+    suspend fun isSubmittedSurvey(eventId: Int, userId: String): Result<Boolean>


네이밍 아주 좋아요 !

tgyuuAn · 2023-12-23T00:56:43Z

core/network/src/main/java/com/wap/wapp/core/network/model/survey/SurveyResponse.kt

        surveyedAt = LocalDateTime.parse(
            this.surveyedAt,
            DateTimeFormatter.ISO_LOCAL_DATE_TIME,


String -> LocalDateTime으로 변환하는 메소드 같은데 Util로 빼면 공지사항 등록 로직에서도 사용할 수 있지 않을까요 ??!

이게 애매하긴 하네요

확실히 유틸로 빼면 작성되는 코드가 많이 줄 것 같아요, 사용하는 구간이 많아서요

대체적으로 살펴보니까 LocalDateTime Parse, Format 하는 부분이 대체적으로

data (Repository에서 DataSource로 전달할 때 파싱 변환 )

network (Repo에서 UseCase로 전달할 때 포메팅 변환, [하지만 network 모델 클래스 로컬 함수로 등록되어 있음]) 이라

data, network는 따로 common 모듈을 참조하지 않기 때문에
각각 모듈에 DateUtil을 만들어야 해요 ㅠㅠ

data, network 각 모듈에서 data는 2번 파싱, network는 3번 포메팅을 진행하고 있어요 ! (방금 다뒤져봄 후다다)

몇번 중첩 참조되는지 기준을 정하면 더 좋을 것 같아요 !
약간 3번 넘으면 유틸클래스 만들기 ! 이런느낌으로다가 하하

태규상은 어떤거 같아요 ?

3번 참조되면 유틸 클래스로 만드는 것 좋습니다.

근데 이 부분은 명확하게 Event등록이랑 중복되는 부분이니까

common모듈까지 가지 않더라도,

feature 모듈 내에서라도 빼면 좋을 것 같다는 생각이 들어요.

의견 궁금합니다잇

기능 모듈 안에서도 3번 이상 참조되면 유틸로 빼는 걸로 하죠 !

tgyuuAn · 2023-12-23T01:00:33Z

...ment/src/main/java/com/wap/wapp/feature/management/check/navigation/SurveyCheckNavigation.kt

+object SurveyCheckRoute {
+    const val route = "survey/check/{surveyId}"
+}


혹시 따로 SurveyCheckRoute 객체에다가 변수를 할당해준 이유가 있을까요?

이렇게 될 경우,

route = SurveyCheckRoute.route

SurveyCheckRoute라는 객체 이름 뒤에 route라는 프로퍼티 명을 한 번 더 지정해주어야 하는데요,

확장성의 이유라고 하면 혹시 SurveyCheckRoute에 또 어떤 프로퍼티가 들어갈 수 있을까요 ??

이전에 작성한 코드를 참조하면서 구현해봤는데,
따로 기능이 추가될 일 없으면 Object를 선언할 이유는 없는 것 같네요.

지금 다른 Navigation 관련 코드를 확인하면 다 최상위로 선언해주셔서,
저도 스타일 따라서 구현하는게 맞는 것 같아요 !

tgyuuAn · 2023-12-23T01:03:57Z

feature/survey/src/main/java/com/wap/wapp/feature/survey/answer/SurveyAnswerViewModel.kt

    fun setNextQuestion() {
        val currentQuestionNumber = _questionNumber.value
-        val lastQuestionNumber = surveyAnswerList.value.size
+        val lastQuestionNumber = _surveyForm.value.surveyQuestionList.lastIndex

-        if(currentQuestionNumber == lastQuestionNumber) { // 마지막 질문일 경우 제출
+        if (currentQuestionNumber == lastQuestionNumber) { // 마지막 질문일 경우 제출
            submitSurvey()
            return
        }


setNextQuestion에서 submitSurvey를 호출하고 있는데,

1-1. 마지막 질문이 아닐 경우, 다음 질문으로 넘어감.
1-2. 마지막 질문일 경우 응답을 제출함.

의 로직을 결정하는 건 View딴에서 결정하게 하는 것이 어떨까요?

setNextQuestion 함수에서 설문 응답까지 핸들링하고 있는 것 같아요.

굳이 필요성을 못느낀다면 함수 명을 바꿔보는 쪽도 고려해볼 법 하네요.

#64 (comment)
(이전 피알에서 이어지는 내용이에요 ! )

맞죠,, 함수는 하나의 기능만 해야 하고, 네이밍만 보고도 내용이 유추가 되어야 하는데,
해당 함수는 네이밍 면에서 부족한 것 같아요

근데 저도 어렵네요 어떤게 정답일까요
(View딴에서 함수를 호출(이전 피알 참조), 현재 함수명 수정 -> setNextQuestionIfLastQuestionSubmit)

�지금 제가 보기에는 View에서 호출하는 걸로 다시 수정하는게 나을 것 같아요.
로직의 가독성이 더 좋고, 지금은 로직이 숨겨져 있어서 파악하기 어려울 것 같다는 생각이 드네요

jeongjaino · 2023-12-23T03:31:17Z

제대로입니다 태규상 !!

jeongjaino added 18 commits December 22, 2023 14:19

[REFACTOR] #65 : SurveyFormDataSource 구현 및 설문조사 형식 폼 저장 및 불러오는 로직 모두 …

622c893

…Migration

[REFACTOR] #65 : SurveyFormRepository 구현 및 설문조사 형식 폼 저장 및 불러오는 로직 Mig…

6b720a2

…ration

[CHORE] #65 : managementRepository, surveyRepository -> surveyFormRep…

2f5aaed

…ository로 이동

[CHORE] #65 : surveyForm network model move to survey/form

e26381a

[CHORE] #65 : surveyResponse field answerList to surveyAnswerList

2786891

[FEATURE] #65 : Survey DTO 구현

2a42c2e

[FEATURE] #65 : SurveyDataSource 설문 조사 응답 로직 구현

826ee5f

[FEATURE] #65 : SurveyDataRepository 설문 조사 응답 모델 DTO 변경 로직 구현

7cd3aab

[FEATURE] #65 : 설문 응답 등록 유스케이스 구현

bd8c446

[FEATURE] #65 : 설문 확인 화면 NavArgument 등록

944ad02

[FEATURE] #65 : 설문 확인 화면 NavArgument 등록

cfce195

[FEATURE] #65 : 관리화면에서 설문 확인 화면으로 전환 구현

cf67be9

[CHORE] #65 : 사용하지 않는 클래스 소거

a2eb2e9

[FEATURE] #66 : 제출한 설문인지 판단하는 로직 구현

0b63a55

[FEATURE] #66 : 제출한 설문인지 확인하는 유스케이스 구현

e1848f3

[FEATURE] #66 : 제출 기록에 따라, 작성화면으로 전환 로직 구현

7ca9e3f

[FEATURE] #66 : 제출 기록에 따라, 작성화면으로 전환 로직 구현

e15c559

[FEATURE] #66 : 주관식 응답이 10글자 이상일 경우, 버튼이 활성화 되도록 구현

964db3e

jeongjaino added 🚀진호🚀 ESTP 정진호 23세 🌱기능🌱 새로운 기능 두두둥장! 🧹허드렛일🧹 종속성 추가, 기능과 무관한 코드 작성 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. labels Dec 22, 2023

jeongjaino self-assigned this Dec 22, 2023

jeongjaino linked an issue Dec 22, 2023 that may be closed by this pull request

[FEATURE]: 설문 작성 Validation 로직 구현 #66

Closed

2 tasks

tgyuuAn approved these changes Dec 23, 2023

View reviewed changes

jeongjaino added 3 commits December 23, 2023 13:17

[CHORE] #67 : Survey Check Route Object to const val

f2477ec

[STYLE] #67 : 단일문 함수 block to equal로 변경

9dfc6d6

[REFACTOR] #67 : 다음 질문 넘기는 로직 ViewModel to View로 전환

ce29f57

[CHORE] #67 : String to LocalDateTime 변환 유틸리티 구현

690a3d3

tgyuuAn added 🌟머지 해주세요🌟 코드 리뷰가 완료된 뒤 PR을 올린사람이 Merge를 하면 되는 단계입니다. and removed 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. labels Dec 23, 2023

jeongjaino merged commit bf957c8 into develop Dec 23, 2023

tgyuuAn deleted the feature/jaino/#67 branch February 6, 2024 08:20

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/jaino/#67 #68

Feature/jaino/#67 #68

jeongjaino commented Dec 22, 2023

tgyuuAn commented Dec 23, 2023

tgyuuAn left a comment •

edited

Loading

tgyuuAn Dec 23, 2023

tgyuuAn Dec 23, 2023

jeongjaino Dec 23, 2023

tgyuuAn Dec 23, 2023 •

edited

Loading

jeongjaino Dec 23, 2023

tgyuuAn Dec 23, 2023

jeongjaino Dec 23, 2023

tgyuuAn Dec 23, 2023

jeongjaino Dec 23, 2023

jeongjaino commented Dec 23, 2023


		suspend fun getSurveyForm(eventId: Int): Result<SurveyForm>
		suspend fun isSubmittedSurvey(eventId: Int, userId: String): Result<Boolean>

Feature/jaino/#67 #68

Feature/jaino/#67 #68

Conversation

jeongjaino commented Dec 22, 2023

1. 📄 관련된 이슈 및 소개

2. 🔥변경된 점

3. 📸 스크린샷(선택)

4. 💡알게된 혹은 궁금한 사항들

tgyuuAn commented Dec 23, 2023

tgyuuAn left a comment • 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

tgyuuAn Dec 23, 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

jeongjaino commented Dec 23, 2023

tgyuuAn left a comment •

edited

Loading

tgyuuAn Dec 23, 2023 •

edited

Loading