week4: 4차 필수과제 진행(xml) #10

SYAAINN · 2024-05-03T14:12:18Z

Related issue 🛠

closed week4: 4차 필수과제 진행 #9

Work Description ✏️

회원가입 서버 통신
로그인 서버 통신
마이페이지에 로그인 정보 노출
HomeFragment에 OpenAPI 연결하기

Screenshot 📸

4._XML_.1.mp4

Uncompleted Tasks 😅

비밀번호 변경 기능

To Reviewers 📢

해냈습니다..! Open API 연결 완료했습니다! 리팩토링과 함께 비밀번호 변경 기능도 구현 시도해보겠습니다!

arinming

서버 통신 어려우셨을 텐데 열심히하신게 느껴집네다...!!!!!!!!!

arinming · 2024-05-05T01:15:19Z

app/src/main/java/com/sopt/now/activity/SignInActivity.kt

-    private lateinit var binding: ActivitySigninBinding
+
+    private val binding by lazy { ActivitySigninBinding.inflate(layoutInflater) }
+    private val viewModel by viewModels<SignInViewModel>()


by viewModels로 초기화될 수 있는 건 몰랐는데 알아갑니당 ㅎㅎ

arinming · 2024-05-05T01:17:06Z

app/src/main/java/com/sopt/now/activity/SignUpActivity.kt

함수화가 너무 깔끔해여 !

성장속도 무엇...

arinming · 2024-05-05T01:17:53Z

app/src/main/java/com/sopt/now/data/ResponseSignInDto.kt

+    @SerialName("code")
+    val status: Int,


code, status 통일이 안되어 있는데 이유가 있으실까여?

ㅋㅋㅋㅋㅋㅋㅋㅋ

arinming · 2024-05-05T01:20:28Z

app/src/main/java/com/sopt/now/myPageFragment/MyPageFragment.kt

+        memberId?.let { memberId ->
+            ServicePool.authService.getUserInfo(memberId.toInt()).enqueue(object :
+                Callback<ResponseUserInfoDto> {
+                override fun onResponse(
+                    call: Call<ResponseUserInfoDto>,
+                    response: Response<ResponseUserInfoDto>
+                ) {
+                    if (response.isSuccessful) {
+                        val userInfo = response.body()?.data
+
+                        userInfo?.let {
+                            binding.tvMyPageId.text = " ${it.authenticationId}"
+                            binding.tvMyPagePw.text = " ${it.nickname}"
+                            binding.tvMyPagePhoneNumber.text = " ${it.phone}"
+                        }
+                    } else {
+                        Toast.makeText(requireContext(), "회원 정보를 가져오는데 실패했습니다.", Toast.LENGTH_SHORT)
+                            .show()
+                    }
+                }


여기도 뷰모델로 리팩토링하시면 깔끔할 듯 합니당 !

이제 슬슬 MVVM 패턴을 익히실 때입니다...!

이제 슬슬 MVVM 패턴을 익히실 때입니다...!

왜 이런 생각을 하셨는지 궁금합니다!

arinming · 2024-05-05T01:21:35Z

app/src/main/java/com/sopt/now/viewModel/SignUpViewModel.kt

+    private fun moveToSignInActivity() {
+        _navigateToSignIn.value = true
+    }
+
+    fun doneNavigatingToSignIn() {
+        _navigateToSignIn.value = false
+    }


네비게이션을 뷰모델로 구현하셨네여!

더 좋은 방법이 있을 것 같습니다!

arinming · 2024-05-05T01:22:30Z

app/src/main/res/values/strings.xml

snake 네이밍 방식으로 하면 모든 글자가 소문자여야 하는 걸로 알고이써요!

yeonjeen

이야... 너무 잘해서 할 말을 잃었... 넘 수고많으셨어용!!!!

yeonjeen · 2024-05-05T08:24:09Z

app/src/main/java/com/sopt/now/activity/SignUpActivity.kt

성장속도 무엇...

yeonjeen · 2024-05-05T08:25:41Z

app/src/main/java/com/sopt/now/homeFragment/HomeViewModel.kt

            selfDescription = "It's DAME TIME."
        ),
        HomeViewObject.FriendProfile(
            profileImage = R.drawable.charlesbarkley_profile,
            name = "Charles Barkley",
-            place = "USA, Philadelphia",
+            phoneNumber = "USA, Philadelphia",
            selfDescription = "CHUCK CHUCK"
        )
    )


여기 mock 데이터는 없어도 되지 않을까용...?

junseo511

진짜 정말정말 항상 고생 많으십니다 ㅠㅠ 늘 파이팅!!! 포기하지 말고 같이 달려봐요

junseo511 · 2024-05-05T14:52:37Z

app/build.gradle

+
+    // Network
+    implementation 'com.squareup.retrofit2:retrofit:2.9.0'
+    implementation 'org.jetbrains.kotlinx:kotlinx-serialization-json:1.4.1'
+    implementation 'com.jakewharton.retrofit:retrofit2-kotlinx-serialization-converter:1.0.0'
+
+    // define a BOM and its version
+    implementation(platform("com.squareup.okhttp3:okhttp-bom:4.10.0"))
+
+// define any required OkHttp artifacts without version
+    implementation("com.squareup.okhttp3:okhttp")
+    implementation("com.squareup.okhttp3:logging-interceptor")


정리할때 일관성을 지켜주시면 좋을거같아용

junseo511 · 2024-05-05T14:53:01Z

app/src/main/java/com/sopt/now/activity/MainActivity.kt

-        val userName = intent.getStringExtra("Name")
-        val userPlace = intent.getStringExtra("Place")
+        val memberId = intent.getStringExtra("memberId") // null일 경우에는 0으로 처리
+        Log.e("MainActivity", "memberId: ${memberId}")


불필요한 로그는 지워주세요~~!!!!

junseo511 · 2024-05-05T14:53:29Z

app/src/main/java/com/sopt/now/activity/MainActivity.kt

        val currentFragment = supportFragmentManager.findFragmentById(binding.fcvMain.id)
        if (currentFragment == null) {
            supportFragmentManager.beginTransaction()
                .add(binding.fcvMain.id, HomeFragment())
                .commit()
        }


함수분리를 해주심이 어떨까요?

junseo511 · 2024-05-05T14:54:28Z

app/src/main/java/com/sopt/now/activity/SignInActivity.kt

-            else -> {
-                showToast(getString(R.string.toast_SignInActivity_ValidSignIn))
-                navigateToMainActivity(userId, userPw, userName, userPlace)
+    private fun initObserver() {


함수 이름을 명확하게 해주시면 좋을거같슴당

junseo511 · 2024-05-05T14:54:47Z

app/src/main/java/com/sopt/now/activity/SignUpActivity.kt


+    private fun initViews() {


음~~ 이친구도 함수명이 애매해요!!!!

junseo511 · 2024-05-05T14:55:16Z

app/src/main/java/com/sopt/now/activity/SignUpActivity.kt

+        val id = binding.etSignUpId.text.toString()
+        val password = binding.etSignUpPw.text.toString()
+        val nickname = binding.etSignUpName.text.toString()
+        val phoneNumber = binding.etSignUpPhoneNumber.text.toString()


with(binding) 어떠세요?

junseo511 · 2024-05-05T14:55:35Z

app/src/main/java/com/sopt/now/data/ResponseSignInDto.kt

+    @SerialName("code")
+    val status: Int,


ㅋㅋㅋㅋㅋㅋㅋㅋ

junseo511 · 2024-05-05T14:55:58Z

app/src/main/java/com/sopt/now/homeFragment/HomeViewModel.kt

@@ -8,69 +8,69 @@ class HomeViewModel : ViewModel() {
    val mockMyProfile = HomeViewObject.MyProfile(
        profileImage = R.drawable.sonminjae_profile,
        name = "Son minjae",
-        place = "Korea, Seoul",
+        phoneNumber = "Korea, Seoul",
        selfDescription = "i can win",
        enable = true,
    )
    val mockFriendProfileList = listOf(


이친구는 이제 날려주셔도 좋지 않을까요~~!!

junseo511 · 2024-05-05T14:56:27Z

app/src/main/java/com/sopt/now/myPageFragment/MyPageFragment.kt

+        memberId?.let { memberId ->
+            ServicePool.authService.getUserInfo(memberId.toInt()).enqueue(object :
+                Callback<ResponseUserInfoDto> {
+                override fun onResponse(
+                    call: Call<ResponseUserInfoDto>,
+                    response: Response<ResponseUserInfoDto>
+                ) {
+                    if (response.isSuccessful) {
+                        val userInfo = response.body()?.data
+
+                        userInfo?.let {
+                            binding.tvMyPageId.text = " ${it.authenticationId}"
+                            binding.tvMyPagePw.text = " ${it.nickname}"
+                            binding.tvMyPagePhoneNumber.text = " ${it.phone}"
+                        }
+                    } else {
+                        Toast.makeText(requireContext(), "회원 정보를 가져오는데 실패했습니다.", Toast.LENGTH_SHORT)
+                            .show()
+                    }
+                }


이제 슬슬 MVVM 패턴을 익히실 때입니다...!

junseo511 · 2024-05-05T14:57:07Z

app/src/main/java/com/sopt/now/viewModel/SignUpViewModel.kt

+    private fun moveToSignInActivity() {
+        _navigateToSignIn.value = true
+    }
+
+    fun doneNavigatingToSignIn() {
+        _navigateToSignIn.value = false
+    }


더 좋은 방법이 있을 것 같습니다!

briandr97

안녕하세요 민재님, 민재님의 리뷰어를 맡게된 권용민이라고 합니다!

필수 과제에 xml과 compose가 있는데요 xml은 제가, compose는 준원님께서 리뷰해주실 예정입니다.
코드에 정답이 없으며, 저희 또한 많이 부족한 사람이므로 공통 로직에서 저와 준원님의 의견이 상충될 수 있습니다. 혹시 그런 상황이 발생한다면, 민재님께서 생각하시는대로 진행하고 말씀해주시면 됩니다!

또 제가 리뷰해드린 내용이라고 해서 정답은 아닙니다. 사람마다 생각이 다를 수 있으며, 제가 틀렸을 수도 있습니다. 납득이 되지 않으면 언제든 질문주시고, 민재님의 생각을 말씀해주세요!

과제하느라 고생하셨습니다! 잘부탁드립니다!

모든 리뷰에 1~5에 해당하는 번호를 남겼습니다. 해당 번호들은 중요도를 나타냅니다.
1,2번은 changes requested,
3번은 comment,
4,5번은 추가정보 정도로 생각해주시면 좋을 것 같습니다.

briandr97 · 2024-05-06T15:32:04Z

app/src/main/java/com/sopt/now/activity/SignInActivity.kt

-                showToast(getString(R.string.toast_SignInActivity_ValidSignIn))
-                navigateToMainActivity(userId, userPw, userName, userPlace)
+    private fun initObserver() {
+        viewModel.liveData.observe(this) {


[1]
뷰모델에 존재하는 상태의 변수명이 liveData인 이유가 있을까요?
변수명은 타입을 나타내기보다 어떤 역할을 하는지 알 수 있게 해주는 것이 좋습니다.

만약 다른 사람이 혹은 몇개월 후의 민재님이 이 코드를 보았을때 어떤 정보를 관찰하고 있는지 알 수 있을까요?

briandr97 · 2024-05-06T15:37:22Z

app/src/main/java/com/sopt/now/activity/SignInActivity.kt

+            SignInState ->
+            Toast.makeText(this,SignInState.message,Toast.LENGTH_SHORT).show()
+            if(SignInState.isSuccess) {
+                val intent = Intent(this@SignInActivity,MainActivity::class.java).apply{
+                    SignInState.memberId?.let { memberId -> putExtra("memberId",memberId) }
+                }
+                startActivity(intent)
            }


[1]
변수명의 첫 글자는 소문자로 하는 것이 코틀린 공식 컨벤션입니다.
대문자를 사용하게 되면 위와 같이 SignInState.message, SignInState.memberId라고 썼을 때 다른 사람들은 SignInState가 인스턴스라고 인지하기 어렵습니다. object 혹은 companion object로 인지할 확률히 매우 높습니다.

briandr97 · 2024-05-06T15:50:34Z

app/src/main/java/com/sopt/now/data/ResponseUserInfoDto.kt

+    )
+
+    companion object {
+        val defalutUser = UserInfo()


[3]
이 친구의 용도는 뭘까요?

briandr97 · 2024-05-06T16:01:49Z

app/src/main/java/com/sopt/now/myPageFragment/MyPageFragment.kt

+    companion object {
+        fun newInstance(memberId: String?): MyPageFragment {
+            val fragment = MyPageFragment()
+            val args = Bundle().apply {
+                putString("memberId", memberId)
+            }
+            fragment.arguments = args
+            return fragment
+        }
+    }


[3]

코틀린 컨벤션에 의하면 동반객체가 제일 마지막에 옵니다!

briandr97 · 2024-05-06T16:03:28Z

app/src/main/java/com/sopt/now/myPageFragment/MyPageFragment.kt

+                            binding.tvMyPagePhoneNumber.text = " ${it.phone}"
+                        }
+                    } else {
+                        Toast.makeText(requireContext(), "회원 정보를 가져오는데 실패했습니다.", Toast.LENGTH_SHORT)


[2]
문자를 하드코딩 하는 것과 strings.xml에 넣는 것의 기준이 뭘까요?

briandr97 · 2024-05-06T16:09:15Z

app/src/main/java/com/sopt/now/viewModel/SignUpViewModel.kt

+
+class SignUpViewModel : ViewModel() {
+    private val authService by lazy { ServicePool.authService }
+    val liveData = MutableLiveData<SignUpState>()


[1]
위에서 했던 리뷰와 같습니다.

briandr97 · 2024-05-06T16:11:03Z

app/src/main/java/com/sopt/now/viewModel/SignUpViewModel.kt

+    val navigateToSignIn: LiveData<Boolean>
+        get() = _navigateToSignIn


[1]
navigateToSignIn만 봐서는 로그인으로 갈 수 있는 지를 나타내는 변수인지 알 수 없습니다.
그래서 보통 Boolean형태의 변수에는 is 혹은 can과 같은 접두사가 붙습니다.

canNavigateToSignIn과 같은 형태가 된다면 더 의미가 와닿을 것 같습니다.

briandr97 · 2024-05-06T16:19:25Z

app/src/main/java/com/sopt/now/viewModel/SignUpViewModel.kt

+    private val _navigateToSignIn = MutableLiveData<Boolean>()
+    val navigateToSignIn: LiveData<Boolean>
+        get() = _navigateToSignIn


[5]
여기서는 backing property로 잘 만들어주셨네요!
위에서 리뷰드린 내용이었었는데, 알고 계셨던 거라면 수정해주시고, 모르고 사용하셨더라면 backing property에 대해 알아보시면 좋을 것 같습니다.

briandr97 · 2024-05-06T16:19:58Z

app/src/main/java/com/sopt/now/viewModel/SignUpViewModel.kt

+                        isSuccess = true,
+                        message = "회원가입 성공! 유저의 ID는 $userId 입니둥"
+                    )
+                    Log.d("SignUp", "data: $data, userId: $userId")


[5]
개발 중에 작성하신 로그는 지워주시는게 좋습니다!

briandr97 · 2024-05-06T16:22:31Z

app/src/main/java/com/sopt/now/viewModel/SignUpViewModel.kt

+                    val error = response.message()
+                    liveData.value = SignUpState(
+                        isSuccess = false,
+                        message = "회원가입 실패 $error"


[2]
위에서 말씀드렸던 내용이지만, 민재님께서 문자를 하드 코딩하는 것과 strings.xml에 넣는 것을 구분한 기준이 무엇일까요?

SYAAINN added 6 commits April 29, 2024 10:31

feat: 4주차 서버 통신 실습

f3232c1

feat: signInDto 생성

3210a20

feat: SignInActivity 서버 통신 기능 구현

31b5839

feat: SignUp, SignIn 서버 통신 구현

ebcd32e

feat: MyPageFragment 서버 통신 구현

4404284

fix: MyPageFragment 정보 전달 수정

beb6464

SYAAINN added 📗[필수 과제] 세미나_필수과제 🍯 [FEAT] 🔮 [민재] [XML] labels May 3, 2024

SYAAINN requested review from chattymin and a team May 3, 2024 14:12

SYAAINN self-assigned this May 3, 2024

arinming reviewed May 5, 2024

View reviewed changes

yeonjeen reviewed May 5, 2024

View reviewed changes

junseo511 requested changes May 5, 2024

View reviewed changes

refactor: file repackaging

3a3c55e

briandr97 suggested changes May 6, 2024

View reviewed changes

SYAAINN added 4 commits May 15, 2024 19:18

feat: open API 서버통신

f097537

feat: loggingInterceptor 적용

fc91add

feat: C SignUp oroutine 적용

5d1d934

refactor: FriendService 수정

dc30072

junseo511 approved these changes May 24, 2024

View reviewed changes

SYAAINN merged commit a685e71 into develope-xml May 24, 2024

		val navigateToSignIn: LiveData<Boolean>
		get() = _navigateToSignIn

week4: 4차 필수과제 진행(xml) #10

week4: 4차 필수과제 진행(xml) #10

Conversation

SYAAINN commented May 3, 2024 • edited Loading

Related issue 🛠

Work Description ✏️

Screenshot 📸

Uncompleted Tasks 😅

To Reviewers 📢

arinming 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

yeonjeen left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

junseo511 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

briandr97 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

SYAAINN commented May 3, 2024 •

edited

Loading