Week4 #8

sayyyho · 2024-11-15T14:59:50Z

Related issue 🛠

closed [Week4] 4주차 과제 구현 #7

Work Description ✏️

유저등록 API 연동
로그인 API 연동
취미조회 API 연동

Screenshot 📸

Uploading 화면

2024-11-15.11.58.11.mov

2024-11-15.11.58.39.mov

Uncompleted Tasks 😅

To Reviewers 📢

hyoeunjoo

4주차 서버통신 많이 힘들었을텐데 고생 많았습니다ㅠㅠ
합동세미나도 화이팅!

hyoeunjoo · 2024-11-17T08:15:39Z

app/src/main/java/org/sopt/and/api/Auth.kt

+interface UserRegistrationService {
+    @POST("/user")
+    suspend fun postUserRegistration(
+        @Body userRequest: RequestUserRegistrationData
+    ): Response<ResponseUserRegistration>
+}
+
+interface LoginService {
+    @POST("/login")
+    suspend fun postLogin(
+        @Body loginRequeset: RequestLoginData
+    ): Response<ResponseLogin>
+}
+


다른 서비스라면 다른 파일로 분리하는게 나을 것 같아요
아니면 같은 파일명 Auth에 맞춰서 AuthService로 한 뒤에 같은 서비스 내에 두개의 함수를 넣는 방법도 있겠죠?

interface AuthService { @POST("/user") suspend fun postUserRegistration( @Body userRequest: RequestUserRegistrationData ): Response<ResponseUserRegistration> @POST("/login") suspend fun postLogin( @Body loginRequeset: RequestLoginData ): Response<ResponseLogin> }

이런 식으루요!

hyoeunjoo · 2024-11-17T08:19:29Z

app/src/main/java/org/sopt/and/dto/Auth.kt

+@Serializable
+data class RequestUserRegistrationData(
+    @SerialName("username")
+    val userName: String,
+    @SerialName("password")
+    val password: String,
+    @SerialName("hobby")
+    val hobby: String
+)
+
+@Serializable
+data class ResponseUserRegistration(
+    @SerialName("result")
+    val result: ResultUserNo
+)
+
+@Serializable
+data class ResultUserNo(
+    @SerialName("no")
+    val no: Int
+)


DTO같은 경우는 이름에 Dto라고 명시해주는 것이 좋습니다
또한 Response, Request등을 보통은 다른 파일로 빼는 편입니다.

Suggested change

@Serializable

data class RequestUserRegistrationData(

@SerialName("username")

val userName: String,

@SerialName("password")

val password: String,

@SerialName("hobby")

val hobby: String

)

@Serializable

data class ResponseUserRegistration(

@SerialName("result")

val result: ResultUserNo

)

@Serializable

data class ResultUserNo(

@SerialName("no")

val no: Int

)

@Serializable

data class RequestUserRegistrationDto(

@SerialName("username")

val userName: String,

@SerialName("password")

val password: String,

@SerialName("hobby")

val hobby: String

)

@Serializable

data class ResponseUserRegistrationDto(

@SerialName("result")

val result: ResultUserNo

)

@Serializable

data class ResponseUserRegistrationDto(

@SerialName("no")

val no: Int

)

이런식으로요! 각각 Dto는 다른 파일에 넣어주시는게 조금 더 좋을 것 같습니다.
그렇게 하면 주석으로 해당 Dto들을 나눠주는 것도 필요 없겠죠?

아마 패키지명이 dto여서 dto를 붙이는 것을 추천해주신 것 같아요, 이 부분은 추 후 팀 컨벤션을 세우고 각 레이어에 알맞는 suffix를 붙여주면 좋을 것 같습니다.

hyoeunjoo · 2024-11-17T08:20:28Z

app/src/main/java/org/sopt/and/myinfo/MyScreen.kt

@@ -25,14 +34,30 @@ enum class BottomNavItem(val icon: ImageVector, val description: String) {
    PROFILE(Icons.Default.AccountCircle, "내 정보")
 }

+@SuppressLint("UnrememberedMutableState")


이건 왜 사용하신건가용?

hyoeunjoo · 2024-11-17T08:31:33Z

app/src/main/java/org/sopt/and/signin/SignInScreen.kt

+fun SignInScreen(
+    signViewModel: SignViewModel,
+    onNavigateToMain: () -> Unit,
+    onNavigateToSignUp: () -> Unit
+) {


컴포저블 함수가 뷰모델을 인자로 받게 되면 해당 스크린은 뷰모델에 강하게 결합되겠죠? 그러면 컴포저블 함수의 재사용을 제한하는 경우가 생길 수 있습니다. 또한 UI와 비지니스 로직 간의 경계가 흐려질 수도 있겠습니다!

굿! 다만 저는 Screen단은 어느정도 큰 범주의 UI라고 생각해서 나쁘지는 않다고 보는 입장이긴합니다.
즉 SignInScreen을 다른 ViewModel을 참조하여, 혹은 ViewModel 없이 재활용하는 경우는 없을 것 같습니다.

물론 컴포넌트단에서는 금물이지만요 하핳

hyoeunjoo · 2024-11-17T08:42:15Z

app/src/main/java/org/sopt/and/viewmodel/SignViewModel.kt

+    /** Validate inputs for sign-up */
+    fun validateSignUpInputs(username: String, password: String, hobby: String): Boolean {
+        return username.length <= 8 && password.length <= 8 && hobby.length <= 8
+    }


이건 서버에서 검증해주는 부분이라 여기서는 따로 필요하지 않을 것 같습니다!
서버에서 내려주는 에러코드를 통해 에러핸들링을 해보시는 것도 좋을 것 같아용

imtaejugkim

4주차 정말 고생하셨습니다..! 한주한주가 해당 주가 제일 어렵다고 하는 갱신하는 주차인 것 같네요...ㅠㅠ 세미나 때 배운 통신 바탕으로 잘 구현해주신 것 같습니다! 합세동안 저희 배운 코드 바탕으로 열심히 협업해봐요 아자아자~!

imtaejugkim · 2024-11-17T12:36:01Z

app/src/main/java/org/sopt/and/api/Hobby.kt

+interface HobbyService {
+    @GET("/user/my-hobby")
+    suspend fun getHobby(
+        @Header("token") token: String?
+    ): Response<ResponseMyHobbyData>
+}


현재 명세서에는 이렇게 Header를 직접 명시해서 보내는 것으로 나와있지만, 실제 프로젝트에서는 interceptor에 토큰을 정의하는 방법도 많이 사용하더라구요! 한번 찾아보시는 것도 좋을 것 같아요!

imtaejugkim · 2024-11-17T12:48:18Z

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

+    <string name="hobby_label">취미를 입력해주세요.</string>
+    <string name="sign_up_hobby_hint">취미를 입력해주세요.</string>


같은 값으로 사용할 수 있을 것 같아요!

imtaejugkim · 2024-11-17T12:49:33Z

app/src/main/java/org/sopt/and/viewmodel/SignViewModel.kt

+    /** Validate inputs for sign-up */
+    fun validateSignUpInputs(username: String, password: String, hobby: String): Boolean {
+        return username.length <= 8 && password.length <= 8 && hobby.length <= 8
+    }


저도 클라이언트에서 예외처리를 진행했지만.. 명세를 확인해보니 검증이 있더라구요!
오류 코드에 따라 예외처리 추가하면 좋을 것 같아요

imtaejugkim · 2024-11-17T12:51:24Z

app/src/main/java/org/sopt/and/viewmodel/SignViewModel.kt

+                val response = loginService.postLogin(request) // Hilt로 주입된 서비스 사용
+                if (response.isSuccessful) {
+                    response.body()?.let {
+                        val token = it.result.token
+                        Log.d("SignViewModel", "로그인 성공, 토큰: $token") // 성공 응답 로그
+                        preferences.edit().putString("auth_token", token).apply()
+                        onSuccess()
+                    } ?: onFailure("서버 응답이 비어있습니다.")


sharedpreference 에 토큰을 저장하는 방법이군요! 토큰을 저장하고 사용하는 코드는 추후에 반복되는 header나 표시하는 작업에서도 쓰일 가능성이 있어 객체화 시켜도 좋을 것 같아용

imtaejugkim · 2024-11-17T12:55:31Z

app/src/main/java/org/sopt/and/myinfo/MyScreen.kt

        Text(
-            text = email,
+            text = hobby.ifEmpty { "sport" }, // 초기 값 및 업데이트된 값 반영
            fontSize = 15.sp,
            color = Color.White,
            fontWeight = FontWeight.Bold,


hobby 가 비어있을 때 sport로 명시하신 이유가 있을까요?

kez-lab

안녕하세요 세호님!! ㅠㅠ 넘 늦어서 죄송합니다!
뒷 부분을 못봐서... 혹시나 추가적으로 궁금한 부분 있으면 언제든 답글 남겨주세요!

4주차도 고생많으셨습니다!

kez-lab · 2024-11-20T00:30:55Z

app/src/main/java/org/sopt/and/components/SignButton.kt

    onSuccess: () -> Unit,
-    onFailure: () -> Unit,
+    onFailure: () -> Unit, // @Composable 제거
    modifier: Modifier = Modifier,
    buttonColor: Color = Color.Blue
 ) {


이 주석을 추가하게 되신 히스토리가 궁금합니다.

kez-lab · 2024-11-20T00:32:15Z

app/src/main/java/org/sopt/and/dto/Auth.kt

+@Serializable
+data class RequestUserRegistrationData(
+    @SerialName("username")
+    val userName: String,
+    @SerialName("password")
+    val password: String,
+    @SerialName("hobby")
+    val hobby: String
+)
+
+@Serializable
+data class ResponseUserRegistration(
+    @SerialName("result")
+    val result: ResultUserNo
+)
+
+@Serializable
+data class ResultUserNo(
+    @SerialName("no")
+    val no: Int
+)


아마 패키지명이 dto여서 dto를 붙이는 것을 추천해주신 것 같아요, 이 부분은 추 후 팀 컨벤션을 세우고 각 레이어에 알맞는 suffix를 붙여주면 좋을 것 같습니다.

kez-lab · 2024-11-20T00:35:09Z

app/src/main/java/org/sopt/and/myinfo/MyScreen.kt

    ) {
-        MyHeader(email = signViewModel.email)
+        MyHeader(hobby = hobby) // 최신 hobby 값을 전달
        Spacer(modifier = Modifier.height(20.dp))


인자로 넘기는 경우 최신 값을 전달한다라기 보다는 정확히는 state인 hobby를 전달해서 항상 최신값으로 리컴포지션을 보장함 이 맞을 것 같아요

kez-lab · 2024-11-20T00:37:48Z

app/src/main/java/org/sopt/and/signin/SignInScreen.kt

+fun SignInScreen(
+    signViewModel: SignViewModel,
+    onNavigateToMain: () -> Unit,
+    onNavigateToSignUp: () -> Unit
+) {


굿! 다만 저는 Screen단은 어느정도 큰 범주의 UI라고 생각해서 나쁘지는 않다고 보는 입장이긴합니다.
즉 SignInScreen을 다른 ViewModel을 참조하여, 혹은 ViewModel 없이 재활용하는 경우는 없을 것 같습니다.

물론 컴포넌트단에서는 금물이지만요 하핳

kez-lab · 2024-11-20T00:39:32Z

app/src/main/java/org/sopt/and/signup/SignUpScreen.kt

        ) {
-            SignUpHeader(onNavigateToSignIn)
+            SignUpTobBar (onNavigateToSignIn)
            Spacer(modifier = Modifier.height(20.dp))


띄어쓰기가 있네요

kez-lab · 2024-11-20T00:43:45Z

app/src/main/java/org/sopt/and/signup/SignUpScreen.kt

+            if (usernameError) {
+                Text(
+                    text = "8자보다 크면 안됩니다.",
+                    color = Color.Red,
+                    modifier = Modifier.padding(top = 4.dp)
+                )
+            }


요 글에 다양한 확장함수가 있던데 참고해보면 좋을 것 같아요 ㅎ.ㅎ
Medium: https://proandroiddev.com/top-5-extension-functions-every-jetpack-compose-developer-should-have-cbf2c50d557c

kez-lab · 2024-11-20T00:44:15Z

app/src/main/java/org/sopt/and/signup/SignUpScreen.kt

+                            coroutineScope.launch {
+                                withContext(Dispatchers.Main) {
+                                    snackbarHostState.showSnackbar("회원가입 성공") // 메인 스레드에서 호출
+                                }
+                            }


coroutineScope.launch 는 기본적으로 어떤 스레드인가요?

kez-lab · 2024-11-20T00:44:55Z

app/src/main/java/org/sopt/and/signup/SignUpScreen.kt

+                        withContext(Dispatchers.Main) {
+                            snackbarHostState.showSnackbar("입력 값을 확인해주세요.") // 메인 스레드에서 호출
+                        }
+                    }


요기도! Context Switching은 비용이 많이 드는 작업이니 항상 유의해서 사용해보면 좋을 것 같아요, 꼭 Context Switching이 필요한 상황인지..!

jihyunniiii

고생하셨습니다 ~ 합동 세미나도 파이팅 !

jihyunniiii · 2024-11-20T01:30:53Z

app/build.gradle.kts

+    alias(libs.plugins.kotlin.serialization)
+    id("kotlin-kapt")


id와 alias의 차이점은 무엇일까요!

jihyunniiii · 2024-11-20T01:31:16Z

app/build.gradle.kts

+    //hilt
+    implementation(libs.hilt.android.v2511)
+    kapt(libs.hilt.compiler.v2511)
+    implementation(libs.androidx.hilt.navigation.compose)


힐트를 사용하셨네요! 힐트를 사용했을 때의 이점은 뭘까요?

jihyunniiii · 2024-11-20T01:31:45Z

app/src/main/java/org/sopt/and/api/Api.kt

+object ApiFactory {
+    private const val BASE_URL: String = BuildConfig.BASE_URL
+
+    @Provides


@provides와 @BINDS의 차이는 뭘까요?

jihyunniiii · 2024-11-20T01:33:19Z

app/src/main/java/org/sopt/and/dto/Auth.kt

+)
+
+@Serializable
+data class ResponseUserRegistration(


지금 보면 응답 객체에서 동일한 형태가 사용되는데요, 이 경우 Base가 되는 Response를 만들어서 사용할 수도 있습니다!

sayyyho added 7 commits November 15, 2024 20:58

[Design] - 회원가입 필드 변경

819fa9f

Merge branch 'week3' into week4

7df0781

[Feature] - BASE URL 설정

e7a058c

[Feature] - 플러그인 및 의존성 추가(초기세팅)

b6c20a6

[Feature] - 회원가입 구현

eaf0866

[Feature] - 로그인 구현

dd0fabb

[Feature] - 취미 가져오기 구현

9e716f4

sayyyho linked an issue Nov 15, 2024 that may be closed by this pull request

[Week4] 4주차 과제 구현 #7

Open

3 tasks

sayyyho requested review from imtaejugkim and hyoeunjoo and removed request for imtaejugkim November 15, 2024 15:01

hyoeunjoo requested review from imtaejugkim and jihyunniiii November 17, 2024 08:13

hyoeunjoo requested changes Nov 17, 2024

View reviewed changes

imtaejugkim approved these changes Nov 17, 2024

View reviewed changes

kez-lab reviewed Nov 20, 2024

View reviewed changes

jihyunniiii reviewed Nov 20, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Week4 #8

Week4 #8

sayyyho commented Nov 15, 2024 •

edited

Loading

hyoeunjoo left a comment

hyoeunjoo Nov 17, 2024

hyoeunjoo Nov 17, 2024

kez-lab Nov 20, 2024

hyoeunjoo Nov 17, 2024

hyoeunjoo Nov 17, 2024

kez-lab Nov 20, 2024

hyoeunjoo Nov 17, 2024

imtaejugkim left a comment

imtaejugkim Nov 17, 2024

imtaejugkim Nov 17, 2024

imtaejugkim Nov 17, 2024

imtaejugkim Nov 17, 2024

imtaejugkim Nov 17, 2024

kez-lab left a comment

kez-lab Nov 20, 2024

kez-lab Nov 20, 2024

kez-lab Nov 20, 2024

kez-lab Nov 20, 2024

kez-lab Nov 20, 2024

kez-lab Nov 20, 2024

kez-lab Nov 20, 2024

kez-lab Nov 20, 2024

jihyunniiii left a comment

jihyunniiii Nov 20, 2024

jihyunniiii Nov 20, 2024

jihyunniiii Nov 20, 2024

jihyunniiii Nov 20, 2024

		<string name="hobby_label">취미를 입력해주세요.</string>
		<string name="sign_up_hobby_hint">취미를 입력해주세요.</string>

Week4 #8

Are you sure you want to change the base?

Week4 #8

Conversation

sayyyho commented Nov 15, 2024 • edited Loading

Related issue 🛠

Work Description ✏️

Screenshot 📸

Uncompleted Tasks 😅

To Reviewers 📢

hyoeunjoo 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

imtaejugkim 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

kez-lab 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

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

sayyyho commented Nov 15, 2024 •

edited

Loading