feat: 프로필 수정 API 연결 #230

junseo511 · 2024-08-27T07:21:04Z

📌𝘐𝘴𝘴𝘶𝘦𝘴

closed feat: 프로필수정 뷰 api 연결 #227

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

닉네임 검증 추가
프로필 수정 추가
바텀시트 대략적인 추가

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

Screen_recording_20240827_162001.mp4

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

TODO: 프로필 이미지 수정 추가, 에러 핸들링 추가
아직 마이페이지쪽에서 완성된게 없어서 실시간 테스트는 어려울 것 같네요...! 이미지 관련도 답변좀 들어봐야할듯
너무 양이 많아져서 중간에 끊었다가 갑니다

s9hn · 2024-08-27T07:29:56Z

app/src/main/java/com/teamwss/websoso/data/remote/response/UserNicknameValidityResponseDto.kt

+@Serializable
+data class UserNicknameValidityResponseDto(
+    @SerialName("isValid")
+    val isValid: Boolean


r: 트레일링콤마

s9hn · 2024-08-27T07:31:35Z

app/src/main/java/com/teamwss/websoso/data/repository/UserRepository.kt

+        return userApi.getNicknameValidity(nickname).isValid
+    }
+
+    suspend fun patchUserProfile(avatarId: Int?, nickname: String?, intro: String?, genrePreferences: List<String>) {


c: 패치요?

엥 save인데 뭐지

s9hn · 2024-08-27T07:32:04Z

app/src/main/java/com/teamwss/websoso/ui/myPage/MyPageFragment.kt

+            val intent = ProfileEditActivity.getIntent(
+                requireContext(),
+                nickname = "밝보",
+                introduction = "만나서 반가워요!",
+                avatarImageUrl = "https://mblogthumb-phinf.pstatic.net/MjAyMjA3MDdfMTgg/MDAxNjU3MTIwODE3MDU5.4sNUX1NFnBHQsQ8xrq6Fd2mrVrtyipj6H9aLuJIpyj0g.h-orck6dDWA-ErMcplHgzh-2bPPk7TEAJwxrnNr5qoQg.PNG.ssankal78/청명.png?type=w800",
+                genrePreferences = arrayListOf("fantasy", "romance"),


c: 객체로 묶는게 이쁠 것 같아요. uimodel을 넘기시죠

여기 관련 확장함수를 그냥 추가하는게 예쁠 것 같습니다 👍

s9hn · 2024-08-27T07:34:00Z

app/src/main/java/com/teamwss/websoso/ui/profileEdit/ProfileEditActivity.kt

+        )
+        binding.tvProfileEditNickname.isSelected = uiState.profile.nicknameModel.nickname.isNotEmpty()
+        binding.tvProfileEditNicknameError.text = uiState.nicknameEditResult.message
+        val defaultState = uiState.nicknameEditResult != NicknameEditResult.VALID_NICKNAME && uiState.nicknameEditResult != NicknameEditResult.NONE


c: sealed 혹은 enum들은 가급적이면 다 임포트해주세요

오 훨씬 깔끔해지네요 소소팁 감사 👍

s9hn · 2024-08-27T07:37:26Z

app/src/main/java/com/teamwss/websoso/ui/profileEdit/model/ProfileEditUiModel.kt

+    ;
+
+    companion object {
+        fun String.checkNicknameValidity(): NicknameEditResult {


c: 이넘클래스의 역할에 대해 다시 생각해봐도 좋겠어요

값을 나열해주는 클래스에서 로직이 들어간다면 어색할 수 있겠네요 🤔 이런 비즈니스 로직같은 친구는 다른 곳에 있어야 할 것 같습니다

m6z1

수고하셨습니다! 질문 남겨놓은 게 많아서 코멘트 확인 후에 어프루브 드리겠슴다 ( _ _ 👍

m6z1 · 2024-08-27T07:28:02Z

app/src/main/java/com/teamwss/websoso/data/mapper/UserMapper.kt

+    return UserProfileRequestDto(
+        avatarId = avatarId,
+        nickname = nickname,
+        intro = intro,
+        genrePreferences = genrePreferences,
+    )
+}


a: 요 부분은 저도 리뷰 받은 거긴 한데 request를 사용할 때 매퍼를 만들어야 하나? 라는 리뷰를 받았어서 엔티티 클래스 생성하지 않고 바로 requestDto로 넘겨줘도 될 듯 함다

사실 저도 현재 크게 분리할 필요성을 느끼지 못한 부분이긴 함당

m6z1 · 2024-08-27T07:30:54Z

app/src/main/java/com/teamwss/websoso/ui/profileEdit/ProfileEditActivity.kt

+    companion object {
+        private const val NICKNAME = "NICKNAME"
+        private const val INTRODUCTION = "INTRODUCTION"
+        private const val AVATAR_IMAGE_URL = "AVATAR_IMAGE_URL"
+        private const val GENRE_PREFERENCES = "GENRE_PREFERENCES"
+
+        fun getIntent(context: Context, nickname: String, introduction: String, avatarImageUrl: String, genrePreferences: List<String>): Intent {
+            return Intent(context, ProfileEditActivity::class.java).apply {
+                putExtra(NICKNAME, nickname)
+                putExtra(INTRODUCTION, introduction)
+                putExtra(AVATAR_IMAGE_URL, avatarImageUrl)
+                putStringArrayListExtra(GENRE_PREFERENCES, genrePreferences as ArrayList<String>)
+            }
+        }
+    }


a: activity 내에서 데이터 통신 데이터를 사용하지 않고, intent 객체에 데이터 값을 받아오는 이유가 무엇인가욥? 코드 봤을 때, 조회 api를 사용하면 뷰가 훨씬 가벼워질 것 같다는 생각이 드는데!

이미 이전 뷰에서 무조건 가지고 있어야 할 정보를 위해 네트워크를 한번 더 찌르는게 성능이나 리소스 사용 면에서 좋지 못하다고 생각했습니다. 만약 가독성을 위해서라면 Serializable로 이번 수정에서 직렬화를 진행할 계획입니다 :)

m6z1 · 2024-08-27T07:33:27Z

app/src/main/java/com/teamwss/websoso/ui/profileEdit/ProfileEditActivity.kt

+    private fun updateDuplicateCheckButton(uiState: ProfileEditUiState) {
+        val isEnable = uiState.profile.nicknameModel.nickname.isNotEmpty() && uiState.nicknameEditResult == NicknameEditResult.NONE
+        binding.tvProfileEditNicknameCheckDuplicate.setTextColor(
+            if (isEnable) AppCompatResources.getColorStateList(this, R.color.primary_100_6A5DFD).defaultColor
+            else AppCompatResources.getColorStateList(this, R.color.gray_200_AEADB3).defaultColor
+        )
+        binding.tvProfileEditNicknameCheckDuplicate.setBackgroundResource(
+            if (isEnable) R.drawable.bg_profile_edit_primary_50_radius_12dp
+            else R.drawable.bg_profile_edit_gray_70_radius_12dp
+        )
+        binding.tvProfileEditNicknameCheckDuplicate.isEnabled = isEnable
+    }


c: 버튼이 활성화되기 위한 로직을 뷰에서 사용해야 하나요? 뷰모델에서 isEnabled 의 상태를 가지는 건 어떨지?

확실히 뷰모델에서 가지는게 맞겠군요

m6z1 · 2024-08-27T07:34:26Z

app/src/main/java/com/teamwss/websoso/ui/profileEdit/ProfileEditActivity.kt

+        binding.tvProfileEditNicknameCount.text = getColoredText(
+            getString(R.string.profile_edit_nickname_max_count, uiState.profile.nicknameModel.nickname.length),
+            listOf(uiState.profile.nicknameModel.nickname.length.toString()),
+            AppCompatResources.getColorStateList(this, R.color.gray_300_52515F).defaultColor
+        )
+        binding.tvProfileEditNickname.isSelected = uiState.profile.nicknameModel.nickname.isNotEmpty()
+        binding.tvProfileEditNicknameError.text = uiState.nicknameEditResult.message


c: 여기 binding 으로 한 번 묶으면 가독성이 훨씬 좋아질 것 같아요

m6z1 · 2024-08-27T07:39:12Z

app/src/main/java/com/teamwss/websoso/ui/profileEdit/ProfileEditActivity.kt

                }.also { websosoChip -> binding.wcgProfileEditPreferGenre.addChip(websosoChip) }
        }
    }
+
+    private fun setupNicknameEditText() {


r: 리스너에 대한 코드니 메서드명이 달라야 할 것 같아요

on-what-action 이 맞을까여

m6z1 · 2024-08-27T07:39:21Z

app/src/main/java/com/teamwss/websoso/ui/profileEdit/ProfileEditActivity.kt

+        }
+    }
+
+    private fun setupIntroductionEditText() {


r: 여기도 마찬가지

m6z1 · 2024-08-27T07:44:36Z

app/src/main/java/com/teamwss/websoso/ui/profileEdit/ProfileEditViewModel.kt

+                    avatarId = if (previousProfile.avatarImageUrl == currentProfile.avatarImageUrl) null else 1,
+                    nickname = if (previousProfile.nicknameModel.nickname == currentProfile.nicknameModel.nickname) null else currentProfile.nicknameModel.nickname,
+                    intro = if (previousProfile.introduction == currentProfile.introduction) null else currentProfile.introduction,
+                    genrePreferences = currentProfile.genrePreferences.map { it.tag },


c: 이 로직 말고 다른 방법은 없는 건가욥??

비교하는 함수를 하나 분리하는 것 이외에 크게 달라질 것은 없을 것 같습니다...! 기존과 같은 경우 서버에서 null값을 요구해서요

m6z1 · 2024-08-27T08:15:17Z

app/src/main/java/com/teamwss/websoso/ui/profileEdit/model/ProfileEditUiModel.kt

+        fun String.checkNicknameValidity(): NicknameEditResult {
+            return when {
+                this.length !in 2..10 -> INVALID_NICKNAME_LENGTH
+                this.contains(Regex("^\\s|\\s$|[^\\w가-힣-_]|[ㄱ-ㅎㅏ-ㅣ]")) -> INVALID_NICKNAME_SPECIAL_CHARACTER


c: 검증 로직을 탈 때마다 정규식을 새롭게 생성하는 걸로 보이는데 성능 향상 고려하면 좋을 듯해유

좋은 지적인듯요 Regex가 단순 문자열같은 느낌인줄

m6z1 · 2024-08-27T08:15:48Z

app/src/main/res/layout/dialog_profile_edit_character.xml

+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_marginTop="36dp"
+            android:text="프로필 선택"


c: 스트링 추출

이거 일부러 냅뒀음요 새로 이슈파서 뜯어야할거같아서

…o feat/227 # Conflicts: # app/src/main/java/com/teamwss/websoso/data/mapper/UserMapper.kt # app/src/main/java/com/teamwss/websoso/data/repository/UserRepository.kt # app/src/main/java/com/teamwss/websoso/ui/profileEdit/ProfileEditActivity.kt

m6z1

굿이에영

…to feat/250 # Conflicts: # app/src/main/java/com/teamwss/websoso/ui/profileEdit/ProfileEditViewModel.kt # app/src/main/java/com/teamwss/websoso/ui/profileEdit/model/ProfileEditUiModel.kt # app/src/main/res/layout/activity_profile_edit.xml

feat: 캐릭터 선택 구현

…o feat/227 # Conflicts: # app/src/main/java/com/teamwss/websoso/data/mapper/UserMapper.kt # app/src/main/java/com/teamwss/websoso/data/remote/api/UserApi.kt # app/src/main/java/com/teamwss/websoso/data/repository/UserRepository.kt # app/src/main/java/com/teamwss/websoso/ui/main/myPage/MyPageFragment.kt # app/src/main/res/values/strings.xml

junseo511 added 3 commits August 27, 2024 15:18

feat: 닉네임 검증 로직 추가

cef866e

feat: 캐릭터 변경 바텀시트 추가

cc52bd8

feat: 프로필 수정 API 연결

b49aac7

junseo511 added 🍯 [FEAT] 새로운 기능을 개발합니다. 🔮 법사 준서 아카데미의 천재 마법사 labels Aug 27, 2024

junseo511 self-assigned this Aug 27, 2024

junseo511 requested a review from a team August 27, 2024 07:21

s9hn requested changes Aug 27, 2024

View reviewed changes

m6z1 reviewed Aug 27, 2024

View reviewed changes

junseo511 added 10 commits August 28, 2024 09:35

refactor: 함수명 변경

b11b3a3

refactor: 네트워크 전달 인자 변경

245771a

refactor: 트레일링 콤마 추가

078ffd2

feat: 직렬화 확장함수 추가

e8163d1

fix: 누락된 인자 추가

ae2819d

fix: 누락된 인자 추가

408564b

refactor: 함수명 변경

5747a74

feat: 비교 로직 추가

fe3357a

refactor: 함수 가독성 개선

76688ae

junseo511 requested review from m6z1 and s9hn August 28, 2024 01:39

junseo511 added 3 commits August 28, 2024 11:05

refactor: 트레일링 콤마 추가

ca9d501

feat: Parcelable 확장함수 추가

71dce58

feat: 다이알로그 디자인 추가

ec42680

m6z1 approved these changes Aug 28, 2024

View reviewed changes

junseo511 added 5 commits August 28, 2024 16:00

feat: 캐릭터 리사이클러뷰 추가

8810944

refactor: 불필요한 임포트 제거

c29b0b7

feat: 로티 애니메이션 추가

1eaa9e8

refactor: 불필요한 로직 제거

babbad2

refactor: 파일 형식 변경 및 리사이클러뷰 개선

f0d8708

junseo511 and others added 26 commits August 30, 2024 02:14

feat: 프로필 수정 유즈케이스 추가

f9c6328

refactor: 함수명 변경

d0e088a

refactor: 모델 가독성 개선

e3aac6e

refactor: 변수명 변경

8d901ba

refactor: 변수명 변경

90b8e56

refactor: 불필요한 함수 제거

f20bb55

refactor: 상수 변수화

d3113b4

refactor: 닉네임 검증 가독성 및 성능 개선

777f81c

refactor: 프로필 저장 롤백

48d3d5c

refactor: 프로필 저장 롤백

0ba15cd

fix: 머지 컨플릭트 반영

371b0ad

refactor: 불필요한 함수 제거

64ea15b

refactor: 옵저빙 변수 사용

4a42b49

refactor: 옵저빙 변수 사용

f7ec9d5

refactor: 애니메이션 성능 개선

4827e03

refactor: 뷰홀더 성능 개선

c1cacb0

refactor: 뷰모델 가독성 개선

d80a8df

fix: xml 오류 해결

364acaa

refactor: 분기처리 수정

e2656ea

refactor: 분기처리 수정

a24bdb9

feat: 완료 분기처리 추가

77f8cd2

Merge pull request #255 from Team-WSS/feat/250

26a2c20

feat: 캐릭터 선택 구현

refactor: DTO 이름 변경

9e104f2

fix: 머지 컨플릭트 해결

113cab4

librarywon approved these changes Aug 30, 2024

View reviewed changes

feat: 뷰모델스코프 추가

e3464e0

junseo511 merged commit 16e9cd9 into develop Aug 30, 2024

librarywon deleted the feat/227 branch September 11, 2024 12:05

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: 프로필 수정 API 연결 #230

feat: 프로필 수정 API 연결 #230

junseo511 commented Aug 27, 2024

s9hn Aug 27, 2024

junseo511 Aug 28, 2024

s9hn Aug 27, 2024

junseo511 Aug 28, 2024

s9hn Aug 27, 2024

junseo511 Aug 28, 2024

s9hn Aug 27, 2024

junseo511 Aug 28, 2024

s9hn Aug 27, 2024

junseo511 Aug 28, 2024 •

edited

Loading

m6z1 left a comment

m6z1 Aug 27, 2024

junseo511 Aug 28, 2024

m6z1 Aug 27, 2024

junseo511 Aug 28, 2024

m6z1 Aug 27, 2024

junseo511 Aug 28, 2024

m6z1 Aug 27, 2024

m6z1 Aug 27, 2024

junseo511 Aug 28, 2024

m6z1 Aug 27, 2024

m6z1 Aug 27, 2024

junseo511 Aug 28, 2024

m6z1 Aug 27, 2024

junseo511 Aug 28, 2024

m6z1 Aug 27, 2024

junseo511 Aug 28, 2024

m6z1 left a comment

feat: 프로필 수정 API 연결 #230

feat: 프로필 수정 API 연결 #230

Conversation

junseo511 commented Aug 27, 2024

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

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

junseo511 Aug 28, 2024 • edited Loading

Choose a reason for hiding this comment

m6z1 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

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

m6z1 left a comment

Choose a reason for hiding this comment

junseo511 Aug 28, 2024 •

edited

Loading