[chore] 프로필 수정 API 변경사항 적용 및 일부 로직 변경 #239

t1nm1ksun · 2024-09-03T07:09:08Z

Related issue 🛠

closed [chore] 프로필 수정 API 변경사항 적용 및 일부 로직 변경 #238

Work Description ✏️

프로필 수정 API 변경사항을 적용하고 하단 버튼 로직을 변경하였습니다.
- 프로필 수정 API - isDefaultImage requestBody 추가
- 기존 값에서 하나 이상의 변경 사항이 있을 때에만 버튼 활성화가 가능하도록 수정

Screenshot 📸

Screen_recording_20240912_095701.mp4

Uncompleted Tasks 😅

N/A

To Reviewers 📢

안드데로 파이띵 ㅜ

jihyunniiii

자세한 건 낼 보겠슴 ㅜㅜ

jihyunniiii · 2024-09-03T07:44:02Z

app/src/main/java/org/sopt/dateroad/data/repositoryimpl/AuthRepositoryImpl.kt

+            isDefaultImage = if (editProfile.image.isNullOrEmpty()) {
+                "true"
+            } else {
+                "false"
+            }.toRequestBody("application/json".toMediaType())


Suggested change

isDefaultImage = if (editProfile.image.isNullOrEmpty()) {

"true"

} else {

"false"

}.toRequestBody("application/json".toMediaType())

isDefaultImage = Json.encodeToString(if (editProfile.image.isNullOrEmpty()) {

true

} else {

false

}).toRequestBody("application/json".toMediaType())

일케 하면 "true" 이런 거 안 써두 되지롱
근데 여기 if문 너무 많아서 복잡해지는 것 같아서 고민 좀 해보겠슴다
오늘 너무 바빠서 낼 할게오 ㅠㅠ,,

image를 매핑할 때 https:// 를 추출해주면 좋을 것 같숩니다

또,, 생각해보니까 isNullOrEmpty()의 리턴 값은 Boolean 값이니 이런 식으로 쓸 수도 있겠다 싶네용!

Suggested change

isDefaultImage = if (editProfile.image.isNullOrEmpty()) {

"true"

} else {

"false"

}.toRequestBody("application/json".toMediaType())

isDefaultImage = editProfile.image.isNullOrEmpty().toString().toRequestBody("application/json".toMediaType())

jihyunniiii

집 가서 마저 달게오 ㅜㅜ

jihyunniiii · 2024-09-04T10:52:18Z

app/src/main/java/org/sopt/dateroad/data/dataremote/util/Constraints.kt

@@ -42,6 +42,7 @@ object ApiConstraints {
    const val DATES = "dates"
    const val TIME = "time"
    const val NEAREST = "nearest"
+    const val ISDEFAULTIMAGE = "isDefaultImage"


Suggested change

const val ISDEFAULTIMAGE = "isDefaultImage"

const val IS_DEFAULT_IMAGE = "isDefaultImage"

이런 식으로 하는 게 좋을 것 같숩니당

jihyunniiii · 2024-09-04T10:52:39Z

app/src/main/java/org/sopt/dateroad/data/mapper/todata/ProfileMapper.kt

@@ -6,5 +6,6 @@ import org.sopt.dateroad.domain.model.Profile
 fun Profile.toEditProfile(): EditProfile = EditProfile(
    name = this.name,
    tags = this.tag,
-    image = this.imageUrl
+    image = this.imageUrl,
+    isDefaultImage = true


이거 디폴트 값을 주신 이유가 있나요????

jihyunniiii · 2024-09-04T10:54:28Z

app/src/main/java/org/sopt/dateroad/data/repositoryimpl/AuthRepositoryImpl.kt

+            isDefaultImage = if (editProfile.image.isNullOrEmpty()) {
+                "true"
+            } else {
+                "false"
+            }.toRequestBody("application/json".toMediaType())


image를 매핑할 때 https:// 를 추출해주면 좋을 것 같숩니다

jihyunniiii · 2024-09-04T10:55:48Z

app/src/main/java/org/sopt/dateroad/data/repositoryimpl/AuthRepositoryImpl.kt

+            isDefaultImage = if (editProfile.image.isNullOrEmpty()) {
+                "true"
+            } else {
+                "false"
+            }.toRequestBody("application/json".toMediaType())


또,, 생각해보니까 isNullOrEmpty()의 리턴 값은 Boolean 값이니 이런 식으로 쓸 수도 있겠다 싶네용!

Suggested change

isDefaultImage = if (editProfile.image.isNullOrEmpty()) {

"true"

} else {

"false"

}.toRequestBody("application/json".toMediaType())

isDefaultImage = editProfile.image.isNullOrEmpty().toString().toRequestBody("application/json".toMediaType())

jihyunniiii · 2024-09-04T11:05:13Z

app/src/main/java/org/sopt/dateroad/domain/model/EditProfile.kt

-    val image: String? = ""
+    val image: String? = "",
+    val isDefaultImage: Boolean = true


Data Layer의 모델은 데이터에 대한 처리를 위해 (보통 서버 통신) 필요한 data class를 작성하지만, Domain Layer의 모델은 이를 가공하여 뷰에서 사용하는 or UI로직에 필요한 data class를 작성한다고 했었던 거 기억나시나용?? 그렇기 때문에 isDefaultImage의 값은 필요가 없다는 생각인데,,! 어떻게 생각하시나요??

jihyunniiii

쓰다보니 또 코리 폭탄이 되어버렸네요 ㅠ 죄송합니다.
프로필 수정 로직도 변경했으니까 이거 이슈랑 PR에 추가해주고 스크린샷도 넣어주시면 감사하겠습니다 (좀 자세히 써주셍 ㅋㅋ)

jihyunniiii · 2024-09-06T12:41:14Z

app/src/main/java/org/sopt/dateroad/presentation/ui/profile/ProfileScreen.kt

+        ProfileType.EDIT -> {
+            val currentImage = uiState.profile.imageUrl
+            val currentTags = uiState.profile.tag
+            Log.d("ㅋㅋ", "프로필 태그: $currentTags, 수정 태그: ${uiState.editProfile.tags}")


로그 지워주세용

jihyunniiii · 2024-09-06T12:52:42Z

...in/java/org/sopt/dateroad/presentation/ui/component/textfield/DateRoadTextFieldWithButton.kt

Component는 전반적으로 사용되는 부분이니까 수정 전 논의 부탁드립니당 ㅠㅠ

jihyunniiii · 2024-09-06T12:53:18Z

...in/java/org/sopt/dateroad/presentation/ui/component/textfield/DateRoadTextFieldWithButton.kt

@@ -37,6 +37,7 @@ fun DateRoadTextFieldWithButton(
    modifier: Modifier = Modifier,
    validateState: TextFieldValidateResult = TextFieldValidateResult.Basic,
    title: String? = null,
+    titleHint: String? = null,


titleDescription 어떠셔요? 힌트라기보다는 단순히 설명이라는 느낌이 강해서,,!

jihyunniiii · 2024-09-06T13:00:15Z

...in/java/org/sopt/dateroad/presentation/ui/component/textfield/DateRoadTextFieldWithButton.kt

@@ -78,7 +85,10 @@ fun DateRoadTextFieldWithButton(
                    .padding(vertical = 16.dp),
                value = value,
                onValueChange = {
-                    if (it.length <= maxLength) onValueChange(it)
+                    val filteredValue = it.filter { char ->
+                        char.toString().matches(Regex("[ㄱ-ㅎ가-힣a-zA-Z0-9]"))


Regex 객체 생성은 CPU 리소스를 크게 잡아먹는 연산 중 하나입니다! 그래서 이런 식으로 검사를 진행할 때마다 Regex 객체를 생성해주게 되면 성능이 크게 저하되는 문제가 발생합니다 ㅠㅠ Companion Object 등의 싱글턴 패턴을 활용해서 객체를 한 번만 초기화 시켜주고 검사를 진행할 때 matches만 호출해주는 것이 더 효과적이에요!!

그리구 정규식 내부에 있는 string 상수화 진행해주세용

jihyunniiii · 2024-09-06T13:01:31Z

...in/java/org/sopt/dateroad/presentation/ui/component/textfield/DateRoadTextFieldWithButton.kt

                onValueChange = {
-                    if (it.length <= maxLength) onValueChange(it)
+                    val filteredValue = it.filter { char ->


지금은 해당 컴포넌트를 프로필 수정/등록 뷰에서만 사용해서 큰 문제가 없겠지만 추후 확장성을 생각한다면 해당 함수를 호이스팅 해주는 것이 좋아 보입니다! 다른 뷰에서는 특수문자를 막을 필요가 없을 수도 있으니까요!

이게 제가 위에서 컴포넌트 수정 전 논의를 먼저해달라고 한 가장 큰이유이기도 함!!

jihyunniiii · 2024-09-06T13:13:18Z

app/src/main/java/org/sopt/dateroad/presentation/ui/profile/ProfileContract.kt

+        data class FetchProfile(
+            val fetchProfileLoadState: LoadState,
+            val editProfile: EditProfile,
+            val nicknameValidateResult: TextFieldValidateResult,
+            val isEnrollButtonEnabled: Boolean
+        ) : ProfileEvent()


서버 통신이 완료될 시점에 nicknameValidateResult 값과 isEnrollButtonEnabled (얜 어차피 초기 값이 false라 따로 지정 안 해줘도 될 것 같숨다?)을 지정해야할 이유가 딱히 없어보여요! profileType을 통해 탑바 등을 바꿔줄 때 같이 해줘도 될 것 같다는 생각 입니다.

…-profile-screen

t1nm1ksun added 8 commits September 1, 2024 15:23

[chore] #234 닉네임 텍스트 필드 힌트 stringResource 추가

6ee81c9

[chore] #234 닉네임 텍스트 필드 타이틀 힌트 추가

30bcde7

[chore] #234 ktlintFormat

c3ba146

[chore] #234 닉네임 입력 정규식 추가

3224d33

[chore] #234 fetch profile 인자 추가

de09f93

[chore] #238 불필요한 DTO 삭제

43ffa91

[chore] #238 isDefaultImage 추가

30e07b2

[chore] #238 isDefaultImage.toRequestBody

1051512

t1nm1ksun added 🐻‍❄️ 민석 🐻‍❄️ 💗 chore 💗 labels Sep 3, 2024

t1nm1ksun added this to the [2차 스프린트] QA 반영 milestone Sep 3, 2024

t1nm1ksun self-assigned this Sep 3, 2024

t1nm1ksun requested a review from a team as a code owner September 3, 2024 07:09

jihyunniiii reviewed Sep 3, 2024

View reviewed changes

jihyunniiii reviewed Sep 5, 2024

View reviewed changes

jihyunniiii reviewed Sep 6, 2024

View reviewed changes

Merge remote-tracking branch 'refs/remotes/origin/develop' into chore…

4c269e5

…-profile-screen

jihyunniiii changed the title ~~[chore] 프로필 수정 API 변경사항 적용~~ [chore] 프로필 수정 API 변경사항 적용 및 일부 로직 변경 Sep 11, 2024

jihyunniiii self-assigned this Sep 11, 2024

jihyunniiii added the 🐸 지현 🐸 label Sep 11, 2024

jihyunniiii added 9 commits September 11, 2024 20:13

[chore] #238 기존 프로필 값과 변경할 프로필 값이 같은 경우 프로필 등록하기 버튼 비활성화

91800ce

[chore] #238 닉네임 관련 에러 메시지 변경

43b6d33

[chore] #238 사용하지 않는 Constraints 제거 및 네이밍 변경

84fd17a

[chore] #238 EditProfile data class 내 isDefaultImage 값 제거

850f9cd

[chore] #238 patchEditProfile 함수 간결화

f9d44f8

[chore] #238 titleHint -> titleDescription

2725703

[chore] #238 DateRoadTextFieldWithButton title null 처리

e6bb094

[chore] #238 Regex 객체 재생성 이슈 해결

5d5cf05

[chore] #238 ktlintFormat

3b582fb

jihyunniiii approved these changes Sep 13, 2024

View reviewed changes

t1nm1ksun merged commit 7b5cf96 into develop Sep 13, 2024
1 check passed

t1nm1ksun deleted the chore-profile-screen branch September 13, 2024 07:24

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[chore] 프로필 수정 API 변경사항 적용 및 일부 로직 변경 #239

[chore] 프로필 수정 API 변경사항 적용 및 일부 로직 변경 #239

t1nm1ksun commented Sep 3, 2024 •

edited by jihyunniiii

Loading

jihyunniiii left a comment

jihyunniiii Sep 3, 2024

jihyunniiii Sep 4, 2024

jihyunniiii Sep 4, 2024

jihyunniiii left a comment

jihyunniiii Sep 4, 2024

jihyunniiii Sep 4, 2024

jihyunniiii Sep 4, 2024

jihyunniiii Sep 4, 2024

jihyunniiii Sep 4, 2024

jihyunniiii left a comment

jihyunniiii Sep 6, 2024

jihyunniiii Sep 6, 2024

jihyunniiii Sep 6, 2024

jihyunniiii Sep 6, 2024

jihyunniiii Sep 6, 2024

jihyunniiii Sep 6, 2024

jihyunniiii Sep 6, 2024

jihyunniiii Sep 6, 2024

	const val ISDEFAULTIMAGE = "isDefaultImage"
	const val IS_DEFAULT_IMAGE = "isDefaultImage"

[chore] 프로필 수정 API 변경사항 적용 및 일부 로직 변경 #239

[chore] 프로필 수정 API 변경사항 적용 및 일부 로직 변경 #239

Conversation

t1nm1ksun commented Sep 3, 2024 • edited by jihyunniiii Loading

Related issue 🛠

Work Description ✏️

Screenshot 📸

Uncompleted Tasks 😅

To Reviewers 📢

jihyunniiii left a 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

jihyunniiii left a 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

jihyunniiii left a 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

Choose a reason for hiding this comment

Choose a reason for hiding this comment

t1nm1ksun commented Sep 3, 2024 •

edited by jihyunniiii

Loading