4주차 과제 #4

ThirFir · 2024-11-14T07:14:15Z

Work Description ✏️

회원가입, 로그인, 취미 검색, 프로필 수정 API 연동
Base url -> local.properties
토큰 저장 (EncryptedSharedPreferences)
API Response 파싱 (Retrofit Converter, Interceptor)
에러 분기
자동 로그인
검색 디바운싱 및 로딩

Screenshot 📸

default.mp4

To Reviewers 📢

UI 직접 구성하는 건 너무 어렵네요 ㅠ
과제 요구사항에 대한 최소한으로만 구성했습니다.
Request 헤더는 현재 수동으로 넣어주고 있는데, 추후에 Interceptor에 chaining 해보겠습니다.

0se0

수고많으셨습니다~!
합세 파이팅 🍀

0se0 · 2024-11-17T07:33:54Z

data/src/main/java/com/sopt/data/datasource/local/TokenLocalDataSource.kt

+class TokenLocalDataSource @Inject constructor(
+    @TokenEncryptedSharedPref private val tokenEncryptedSharedPref: SharedPreferences
+) {
+
+    fun saveToken(token: String) {
+        tokenEncryptedSharedPref.edit()
+            .putString(TOKEN, token)
+            .apply()
+    }
+
+    fun getToken(): String {
+        return tokenEncryptedSharedPref.getString(TOKEN, "").orEmpty()
+    }
+
+    companion object {
+        private const val TOKEN = "token"
+    }
+}


최고십니다.

최고짱짱맨이십니다

네?
감사합니다.

0se0 · 2024-11-17T07:34:44Z

data/src/main/java/com/sopt/data/repository/UserRepositoryImpl.kt

+    private val userLocalDataSource: UserLocalDataSource,
+    private val userRemoteDataSource: UserRemoteDataSource,
+    private val tokenLocalDataSource: TokenLocalDataSource,
+    @IODispatcher private val ioDispatcher: CoroutineDispatcher


분리를 기가맥히게 잘하셧네요

0se0 · 2024-11-17T07:35:20Z

data/src/main/java/com/sopt/data/request/SignInRequest.kt

+@Serializable
+data class SignInRequest(
+    @SerialName("username") val username: String,
+    @SerialName("password") val password: String
+)


원래 이렇게 한줄로 구현하시는게 취향이신가요?(정말 궁금해서..)

넵! 저는 변수에 대한 어노테이션은 강경 한줄파입니다 ! (너무 긴 거 아니면..)

Roel4990

고생하셨습니다!

Roel4990 · 2024-11-17T08:04:50Z

app/build.gradle.kts

 plugins {
    alias(libs.plugins.android.application)
    alias(libs.plugins.kotlin.android)
    alias(libs.plugins.kotlin.compose)
    alias(libs.plugins.kotlin.kapt)
    alias(libs.plugins.hilt)
+    kotlin("plugin.serialization") version "2.0.20"


이렇게 추가할 수도 있군요!
위 코드 처럼 alias(libs.plugins.kotlin.serialization) 이런식으로 추가해주는게 어떨까요..?!

수정했습니다 !

Roel4990 · 2024-11-17T08:06:57Z

app/build.gradle.kts

@@ -18,6 +25,7 @@ android {
        versionName = "1.0"

        testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
+        buildConfigField("String", "BASE_URL", "String.valueOf(\"${localProperties["base_url"]}\")")


"String.valueOf("${localProperties["base_url"]}")" => localProperties["base_url"].toString() 로 작성하는건 어떨까요?!

local.properties에 키 작성할 때 base_url="qwerasdf1234" 이런식으로 따옴표를 붙일 수 있지만, 저는 따옴표를 붙이지않고 base_url=qwerasdf1234 이렇게 쓰고 싶었습니다 !
그런데 따옴표를 붙이면 toString()으로 되는데, 따옴표를 붙이지 않으면 toString으로 되진 않고 저 방식으로만 해야 했습니다 !!

따옴표 붙이지 않는 이유가 있나요? 궁금해요!

manifestPlaceholders로 Manifest에 변수 넣을 때는 따옴표를 넣으면 안되더라구여. 그래서 저는 통일성을 위해 전부 따옴표 없이 사용합니당
(지금은 상관없긴하지만..!)

Roel4990 · 2024-11-17T08:12:50Z

app/src/main/java/org/sopt/and/di/LocalApiModule.kt

+    fun provideUserSharedPreferences(
        @ApplicationContext context: Context
    ): SharedPreferences {
        return context.getSharedPreferences("user.pref", Context.MODE_PRIVATE)
    }
+
+    @TokenEncryptedSharedPref
+    @Provides
+    @Singleton
+    fun provideTokenEncryptedSharedPreferences(
+        @ApplicationContext context: Context
+    ): SharedPreferences {
+        val masterKeyAlias = MasterKey
+            .Builder(context, MasterKey.DEFAULT_MASTER_KEY_ALIAS)
+            .setKeyScheme(MasterKey.KeyScheme.AES256_GCM)
+            .build()
+
+        return EncryptedSharedPreferences.create(
+            context,
+            "token.pref.enc",
+            masterKeyAlias,
+            EncryptedSharedPreferences.PrefKeyEncryptionScheme.AES256_SIV,
+            EncryptedSharedPreferences.PrefValueEncryptionScheme.AES256_GCM
+        )
+    }


오 좋네요. 암호화된 SharedPreferences 저도 다음에 한번 써봐야겠군요..

Roel4990 · 2024-11-17T08:16:45Z

app/src/main/java/org/sopt/and/di/NetworkModule.kt

+    @Singleton
+    @Provides
+    fun provideRetrofit(
+        client: OkHttpClient
+    ): Retrofit {
+        val json = Json { ignoreUnknownKeys = true }
+        return Retrofit.Builder()
+            .baseUrl(BuildConfig.BASE_URL)
+            .client(client)
+            .addConverterFactory(ResultConverterFactory())
+            .addConverterFactory(json.asConverterFactory("application/json".toMediaType()))
+            .build()
+    }
+}


val json = Json { ignoreUnknownKeys = true } 이거 어떤이유로 사용한건지 여쭤봐도 될까요?

데이터 클래스로 지정해두지 않은 필드가 API 통신을 통해 들어왔을 때, 그 필드를 무시하는 속성입니다 !
이번 과제에선 큰 의미는 없는데, 그냥 써봤습니다 ㅎㅎ

Roel4990 · 2024-11-17T08:19:14Z

data/src/main/java/com/sopt/data/datasource/local/TokenLocalDataSource.kt

+class TokenLocalDataSource @Inject constructor(
+    @TokenEncryptedSharedPref private val tokenEncryptedSharedPref: SharedPreferences
+) {
+
+    fun saveToken(token: String) {
+        tokenEncryptedSharedPref.edit()
+            .putString(TOKEN, token)
+            .apply()
+    }
+
+    fun getToken(): String {
+        return tokenEncryptedSharedPref.getString(TOKEN, "").orEmpty()
+    }
+
+    companion object {
+        private const val TOKEN = "token"
+    }
+}


최고십니다.

Roel4990 · 2024-11-17T08:22:21Z

domain/src/main/java/com/sopt/domain/exception/ExceptionExtensions.kt

+import kotlin.coroutines.cancellation.CancellationException
+
+suspend fun <T, R> T.runCatchingByCode(
+    vararg exceptions: Pair<Int, Throwable>,


vararg 는 처음보네요.. 공부하겠습니다.

Roel4990 · 2024-11-17T08:23:06Z

domain/src/main/java/com/sopt/domain/exception/NetworkError.kt

+data class NetworkError(
+    val statusCode: Int,
+    val errorCode: Int,
+    override val message: String?
+) : IOException()
+
+sealed class SignInError : Throwable() {
+    class NotExistUsername() : SignInError()
+    class PasswordNotMatchingWithUsername() : SignInError()
+    class UsernameInputEmpty() : SignInError()
+    class PasswordInputEmpty() : SignInError()
+}
+
+sealed class SignUpError : Throwable() {
+    class InvalidUsername() : SignUpError()
+    class InvalidPassword() : SignUpError()
+    class InvalidHobby() : SignUpError()
+    class UsernameInputEmpty() : SignUpError()
+    class PasswordInputEmpty() : SignUpError()
+    class HobbyInputEmpty() : SignUpError()
+    class AlreadyExistUsername() : SignUpError()
+}
+
+sealed class SearchHobbyError : Throwable() {
+    class InputEmpty : SearchHobbyError()
+    class InputNotNumber : SearchHobbyError()
+    class NotExistUserNo : SearchHobbyError()
+}
+
+sealed class UpdateProfileError : Throwable() {
+    class InvalidPassword() : UpdateProfileError()
+    class InvalidHobby() : UpdateProfileError()
+    class PasswordInputEmpty() : UpdateProfileError()
+    class HobbyInputEmpty() : UpdateProfileError()
+}


깔끔하네요..!

KkamSonLee

고생하셨습니다! 공통화를 정말 열심히 하시려고 노력하신 것 같네요 ㅎㅎ 이전에 언급드린 부분도 수정된 것 확인했습니다!

KkamSonLee · 2024-11-17T07:55:27Z

app/src/main/java/org/sopt/and/adapter/ResponseConverter.kt

+        annotations: Array<out Annotation>,
+        retrofit: Retrofit
+    ): Converter<ResponseBody, *> {
+        val json = Json { ignoreUnknownKeys = true }


coerceInputValues, encodeDefaults, isLenient 등등 option 값들이 정말 많습니다! 해당 옵션들이 정말 필요한 경우가 많기 때문에 한 번 알아보시면 좋을 것 같아요!

직렬화가 공부할 게 참 많은 것 같습니다..!
더 학습해볼게요 !!

KkamSonLee · 2024-11-17T07:57:31Z

app/src/main/java/org/sopt/and/di/NetworkModule.kt

+            .readTimeout(30, TimeUnit.SECONDS)
+            .writeTimeout(15, TimeUnit.SECONDS)
+            .addInterceptor(HttpLoggingInterceptor().apply {
+                level = HttpLoggingInterceptor.Level.BODY


level 은 항상 BODY 를 출력하는게 좋진 않을 것 같아요! 개발을 하는 동안 DEBUG 단계에서만 출력 되어도 좋을 것 같습니다!

BuildConfig로 구분하겠습니다 !

KkamSonLee · 2024-11-17T07:59:02Z

app/src/main/java/org/sopt/and/di/NetworkModule.kt

+            .connectTimeout(10, TimeUnit.SECONDS)
+            .readTimeout(30, TimeUnit.SECONDS)
+            .writeTimeout(15, TimeUnit.SECONDS)


해당 �timeout 이 각각 10, 30, 15인 이유가 있을까요?? 지금은 상관 없지만 파일 같은걸 업로드 하는 데에 writeTimeout 이 15초로 부족할 수도 있긴합니다! (물론 그 때는 업로드용 OkhttpClient를 timeOut이 널널하게 만들면 좋겠죠!)

딱히 큰 의미를 둔 건 아닙니다 ㅎㅎ..
이것도 적정 시간 같은 게 있는지 더 알아봐야 할 것 같습니다 🫠

KkamSonLee · 2024-11-17T08:01:30Z

data/src/main/java/com/sopt/data/dto/user/SignUpDto.kt

+
+@Serializable
+data class SignUpDto(
+    @SerialName("no") val no: Int


다른 Dto 들은 nullable 하게 설계하신 것 같은데 해당 필드는 nullable 하지 않은 이유가 있나요??

아앗.. 미스입니다 !
지정해두겠습니다

KkamSonLee · 2024-11-17T08:08:26Z

data/src/main/java/com/sopt/data/repository/UserRepositoryImpl.kt

+    private val myHobby: StateFlow<String> = flow {
+        val token = tokenLocalDataSource.getToken()
+        _myHobby.value = userRemoteDataSource.fetchMyHobby(token).hobby ?: ""
+
+        emitAll(_myHobby)
+    }.stateIn(
+        scope = CoroutineScope(ioDispatcher),
+        started = SharingStarted.Lazily,
+        initialValue = ""
+    )


fetchMyHobby() 시에 사용되도록 만드신 것 같은데 해당 코드를 fetchMyHobby() 내부에 넣어도 되지않나용?? 함수로써 쓰이는 것 같아서요! _myHobby 가 업데이트 된다고 해당 필드가 업데이트가 되는 것도 아닌 것 같아서요!
updateProfile이 되었을 때 자동적으로 fetchMyHobby() 를 collect 하고 있고, stateFlow 로 들고 있는 영역들에게 업데이트 collect가 호출 되었으면 좋겠어요!

우선 위 코드 목적은 :
myHobby의 값이 여러 스크린에서 쓰이므로, 하나의 객체를 공유하는 것으로 api호출을 줄이고자 함입니다.

myHobby라는 변수로 관리한 이유는, 저 flow 블럭을 fetchMyHobby()함수 내부로 옮기면, fetchMyHobby를 호출하는 쪽은 호출할 때마다 서로 다른 새로운 flow를 제공받을 것이므로 결국 매 호출마다 api호출이 일어날 것으로 예상했습니다.
또, flow 내부에서는 emitAll에 의한, _myHobby의 업데이트 시 myHobby의 변경을 유도했습니다 !

updateProfile 시에는 현재 flow 로 변경 사항을 받고 있는 뷰모델에서는 변화를 감지하지 못할 것 같아서요! 만일 hobby�가 계속 보여지는 화면에서 updateProfile 을 했을 때 알아서 hobby가 쓰여지고 있는 뷰가 업데이트 되어야할 때에는 fetchUserHobby 를 호출하게 될 것 같아서요! updateProfile 후에 fetchMyHobby() 함수를 호출하거나 myHobby 가 변화를 감지할 수 있었으면 좋겠습니다!!

사용측은 사실상 myHobby를 구독하는 것과 마찬가지이므로, updateProfile 호출 후에는 현재 모든 사용측에서 자동으로 변화를 감지하고 있습니다! (stateIn으로 repository로부터 값을 전달받고 있습니다)

물론, 구독은 뷰모델에서 이루어지므로 뷰모델이 소멸된 곳에서는 다시 재구독을 행하겠지만, Repository에서 사실상 싱글톤과 같은 객체로 myHobby를 관리하고 있기 때문에, 재구독이 되더라도 모든 업데이트가 반영된 최신의 값으로 받게 됩니다!!

KkamSonLee · 2024-11-17T08:34:35Z

presentation/src/main/java/com/sopt/presentation/ui/screen/search/viewmodel/SearchViewModel.kt

+        query.findLast { it.isDigit().not() }?.let { return }
+
        savedStateHandle[SEARCH_QUERY] = query
    }
-    
+


검색어 업데이트에 해당 조건으로 return 을 하는 정책이 생긴게 있나요??

TextField에 keyboardType을 Number로 주어도 복사붙여넣기로 인한 입력은 막아지지 않는 것 같아서... 복사붙여넣기로 숫자가 아닌 것을 입력하는 것에 대한 방지입니다..!

그렇다면 입력된 값을 onValueChange 에서 filtering 하는 것도 좋을 것 같아요! 아예 입력이 되지 않게요!

// ex currentTextValue.copy(text = currentTextValue.text.filter { it.isDigit() })

아하 반영하겠습니다!

KkamSonLee · 2024-11-17T08:43:27Z

presentation/src/main/java/com/sopt/presentation/ui/screen/search/composable/SearchScreen.kt

+                when {
+                    searchedHobbyUiState is SearchResultUiState.Success -> {
+                        val hobby = (searchedHobbyUiState as SearchResultUiState.Success).hobby
+                        PrimaryText(text = "너의 취미는 $hobby",)
+                    }
+                    searchedHobbyUiState is SearchResultUiState.NotExistUser -> {
+                        PrimaryText(text = "존재하지 않는 사용자입니다.",)
+                    }
+                    searchedHobbyUiState is SearchResultUiState.Loading -> {
+                        CircularProgressIndicator()
+                    }
+                }


UiState 가 5개인데 3가지만 처리되어 있는 것 같아요! 나머지도 처리 되면 좋을 것 같습니다!

Suggested change

when {

searchedHobbyUiState is SearchResultUiState.Success -> {

val hobby = (searchedHobbyUiState as SearchResultUiState.Success).hobby

PrimaryText(text = "너의 취미는 $hobby",)

}

searchedHobbyUiState is SearchResultUiState.NotExistUser -> {

PrimaryText(text = "존재하지 않는 사용자입니다.",)

}

searchedHobbyUiState is SearchResultUiState.Loading -> {

CircularProgressIndicator()

}

}

when(searchedHobbyUiState) {

is SearchResultUiState.Success -> {

val hobby = (searchedHobbyUiState as SearchResultUiState.Success).hobby

PrimaryText(text = "너의 취미는 $hobby",)

}

is SearchResultUiState.NotExistUser -> {

PrimaryText(text = "존재하지 않는 사용자입니다.",)

}

is SearchResultUiState.Loading -> {

CircularProgressIndicator()

}

// TODO other

}

감사합니다 !

KkamSonLee · 2024-11-17T08:47:30Z

domain/src/main/java/com/sopt/domain/exception/ExceptionExtensions.kt

+import kotlin.coroutines.cancellation.CancellationException
+
+suspend fun <T, R> T.runCatchingByCode(
+    vararg exceptions: Pair<Int, Throwable>,


exceptions 만을 나타내는 필드는 아닌 것 같아요! data class 로 감싸거나 이름을 바꾸면 좋을 것 같습니다! 아니면 보내는 Throwable �커스텀 클래스에 code 를 넣어두거나요!

어떤 도메인에 관한 에러들을 하나의 sealed class의 자식으로 관리를 하다 보니, 일일이 생성자에 code: Int를 작성해주어야 하는 번거로움때문에 커스텀 Throwable 클래스 프로퍼티에 code를 넣어두기가 어려웠습니다😭
Code랑 Throwable을 가지는 클래스를 만드는 것도 고려해보았는데, 파라미터 넘길 때마다 새 객체를 생성해주는 행위가 사용할 때 번거로울 것 같았습니다...
그래서 나름대로 최대한으로 생각해 본 게 infix function to 를 활용할 수 있는 Pair 타입의 사용이었습니다. 사용할 때 편한 게 중요하다고 생각했습니다...!!

그런데 말씀해주신 내용에서 exceptions만을 나타내는 필드가 아닌 것 같다는 부분을 잘 이해를 못해서 정확히 어떤 의미신지 알 수 있을까요 ?!

code는 생성자에서 입력 되는 방식이면 안 될 것 같구 고유하게 필드로 들고 있으면 될 것 같아요! 예를 들면 아래와 같이요!!

sealed class SignInError : Throwable() { abstract val code: Int class NotExistUsername : SignInError() { override val code: Int get() = 1 } class PasswordNotMatchingWithUsername : SignInError() { override val code: Int get() = 2 } class UsernameInputEmpty : SignInError() { override val code: Int get() = 3 } class PasswordInputEmpty : SignInError() { override val code: Int get() = 4 } }

Pair<Int, Throwable> 에서 Int, Throwable 만으로 exception 이라는 이름의 필드가 되어야하는데 그렇게 보이지는 않아서요 사용하는 방식도 NetworkError 에만 특정 동작을 행하고 있기 때문에 함수 이름이나 파라미터명 수정이 필요해보입니다!

"catch를 하는데, NetworkError 라는 Exception이 발생했을 때 param으로 보낸 exceptions 중 NetworkError 의 code와 Pair.first()에 맞는 Pair.second()를 반환한다." 라는 동작이 코드량에 비해서 설명이 길어지는 편인데 pair 라서 조금 더 이해하기 어려울 것 같아서요!

code가 뭘 의미하는지, 왜 first()의 0, 1 일 때 second() 의 Throwable 을 반환하는지 잘 모르겠어서요 그렇다면 관련 정책은 보내는 위치에서 주석으로 적어놓던지 해야할텐데 그렇게 된다면 정말 많은 파편화가 될 것 같아요! data class 로 감싸서 code 의 필드에 대해서 내용을 기재하거나 NetworkError.kt 에서 위에 보여드린 code 필드 같은 것을 추가해서 "이 에러는 무슨 코드의 에러로 반환이 온다" 라는 것을 암시할 수 있으니까요!

"Code랑 Throwable을 가지는 클래스를 만드는 것도 고려해보았는데, 파라미터 넘길 때마다 새 객체를 생성해주는 행위가 사용할 때 번거로울 것 같았습니다..." => 기존에도 Pair 를 생성해주고 있어서 비슷할 것 같아요!

말씀해주신대로 커스텀 예외 선언과 동시에 code를 지정하는 것이 옳다고 생각합니다!

abstract class CommonError : Throwable() { open val code: Int = -1 } sealed class SignInError : CommonError() { class NotExistUsername : SignInError() { override val code: Int = 2 } class PasswordNotMatchingWithUsername : SignInError() { override val code: Int = 1 } class UsernameInputEmpty : SignInError() class PasswordInputEmpty : SignInError() }

요런 느낌으로 바꿔봤습니다!

KkamSonLee · 2024-11-17T08:50:10Z

data/src/main/java/com/sopt/data/repository/UserRepositoryImpl.kt

+        return runCatchingByCode {
+            val token = tokenLocalDataSource.getToken()
+            userRemoteDataSource.updateProfile(token, UpdateProfileRequest(password, hobby)).also {
+                _myHobby.value = hobby
+            }
+        }


공통화도 좋지만 vararg exceptions: Pair<Int, Throwable> 를 보내지 않는다면 runCatchingByCode 함수를 안 쓰는게 더 커스텀 error 를 반환 하는 함수를 찾기 더 쉬울 것 같아요!

요기서는 runCatchingByCode를 CancellableException을 다시 던지기 위해 사용했습니다 !

그리고 제가 말씀해주신 것 이해를 못해서... 커스텀 error 반환 함수를 찾기 쉽다는 것이 혹시 어떤 의미인지 자세히 알 수 있을까요....?!

함수가 여러 목적으로 사용 되는 것이 좋을 수도 있지만 해당 함수는 exceptions 로 보낸 Pair 에 대해서 �first 와 일치하는 second를 반환한다는 의미가 더 큰 것 같아서요! CancellableException 를 throw 해야한다면 함수를 2개로 나눠서 network 용과, 단순하게 CancellableException 를 throw 하는 용(network 용에서도 사용)으로 만드는게 어떨까요??

커스텀 error 반환 함수를 찾기 쉽다
=> NetworkErrkr.kt 에 있는 클래스들이 data layer 단에서 쓰이는 곳을 runCatchingByCode 함수에서만 "command + b" 로 find 해도 찾을 수 있으니까요 !

KkamSonLee · 2024-11-17T08:51:59Z

data/src/main/java/com/sopt/data/repository/UserRepositoryImpl.kt

+    override suspend fun saveAccount(account: Account): Result<Unit> {
+        return runCatchingByCode {
+            userLocalDataSource.saveAccount(account)
+        }
    }


해당 함수는 Local 에 있는 데이터를 접근하기 때문에 runCatchingByCode 함수로 넣을 수 있는건 Result 로 감싸져서 오는 동작만 수행 할 것 같아요(NetworkError는 올 수가 없는 구조) runCatchingByCode 는 api 호출, 즉 네트워크 호출에 필요한 함수로 보이는데 RemoteDataSource 단에서 호출하는게 어떨까요??

예리한 지적 감사합니다 !
현재는 ErrorCode가 네트워크 API 에서만 오고 있지만, 호옥시 "로컬 API에 대한 에러 코드를 만들어 사용할 수도 있지 않을까?" 라는 생각으로 위와 같이 사용했었습니다 !
그런데 생각해보니, 그러면 NetworkError라는 이름으로 지었으면 안됐을 것 같긴하네요...ㅎㅎ!

네 그래서 함수를 나누는게 좋을 것 같아요!

chrin05

과제하시느라 수고많으셨습니다!
코드 읽으면서 많이 배워갑니다!!!

chrin05 · 2024-11-17T13:44:12Z

data/src/main/java/com/sopt/data/api/remote/UserApi.kt

+interface UserApi {
+
+    @POST("user")
+    suspend fun signUp(@Body signUpRequest: SignUpRequest)


아예 여기서 suspend를 사용할 수도 있군요

suspend 붙이면 Retrofit Call객체를 사용하지 않아도 돼서 편해요!!

chrin05 · 2024-11-17T13:51:56Z

data/src/main/java/com/sopt/data/datasource/local/TokenLocalDataSource.kt

+class TokenLocalDataSource @Inject constructor(
+    @TokenEncryptedSharedPref private val tokenEncryptedSharedPref: SharedPreferences
+) {
+
+    fun saveToken(token: String) {
+        tokenEncryptedSharedPref.edit()
+            .putString(TOKEN, token)
+            .apply()
+    }
+
+    fun getToken(): String {
+        return tokenEncryptedSharedPref.getString(TOKEN, "").orEmpty()
+    }
+
+    companion object {
+        private const val TOKEN = "token"
+    }
+}


최고짱짱맨이십니다

chrin05 · 2024-11-17T14:03:37Z

app/src/main/java/org/sopt/and/di/ApiModule.kt

저는 모듈 구현을 거의 안해봐서 처음 보는 코드들이 많은데 많이 배워갑니다..!

저도 사실 과제를 통해서 처음해보는 것들이 대부분입니다...!

새로운 코드에 너무 부담 안느끼고 막 다 해보는 게 좋은 것 같아요 !!

chrin05 · 2024-11-17T14:04:19Z

data/src/main/java/com/sopt/data/repository/UserRepositoryImpl.kt

+        return runCatching {
+            userRemoteDataSource.signUp(SignUpRequest(email, password, hobby))
+        }


runCatching 몰랐는데 공부해봐야겠네요!

코틀린이 참 다양하게 뭐가 많은것같아요 😎

chrin05 · 2024-11-17T14:06:16Z

presentation/src/main/java/com/sopt/presentation/ui/screen/signup/viewmodel/SignUpViewModel.kt

+            ).onSuccess {
+                signUpUiState.emit(SignUpUiState.Success)


state 관리하는 거 잘 이해가 안갔는데 서버 연동하면서 필요성을 많이 느낍니다

확실히 연동이 진행될수록 구조화가 많이 중요해지는 것 같아요..!!

jihyunniiii

에러 처리 정말 야무지게 잘 해주셨네요! 배우고 갑니다 ㅋㅅㅋ 합동 세미나도 파이띵!!!

jihyunniiii · 2024-11-20T11:22:10Z

app/build.gradle.kts

@@ -18,6 +25,7 @@ android {
        versionName = "1.0"

        testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
+        buildConfigField("String", "BASE_URL", "String.valueOf(\"${localProperties["base_url"]}\")")


따옴표 붙이지 않는 이유가 있나요? 궁금해요!

ThirFir added 30 commits November 12, 2024 19:26

add: Retrofit dependency

45f5c41

add: 회원가입 API 연동

9bb8c16

add: hobby 입력 필드 추가

9c4c8e6

fix: email -> username

6d30f25

fix: Password, hobby validation 검사 변경

570fc44

add: 로그인 API 연동

456cb7f

add: result unwrap call adapter factory

166364c

fix: suspend

c38fd5f

add: EncryptedSharedPref dependency

6cf6f0e

add: 회원가입, 로그인 응답 객체 설정 및 토큰 저장

df71590

refactor: User -> TokenLocalDataSource 변경 및 token 헤더 추가

1bdaa01

fix: 인터셉터 제거, 내 취미 api 연동

3f79f7a

add: 내 취미 api 호출 로직

62c4df2

add: 내 취미 뷰에 추가

49e7937

add: Success Response Converter

15caab8

add: Error response parsing

90e1ea6

fix: runCatching 잘못 사용

d787e18

add: runCatching 확장함수

1279cf8

refactor: runCatching -> 커스텀 확장함수

d099448

add: 취미 입력

834c5ab

refactor: 회원가입 메시지 변경

392f586

fix: runCatching 중복

19f90f1

add: 로그인 API 결과

0c1c9a6

add: 로그인 성공 시 키보드 hide

05f69b1

chore: get -> fetch

743bb86

add: 다른사람 취미 조회 API

c37bf31

add: 다른사람 취미 조회 로직 및 뷰

c25a680

add: 내 설정 route

0aff239

add: 네비게이션 연결

dc2a5e5

add: domain coroutine dependency

9550255

ThirFir added 11 commits November 12, 2024 19:26

add: 프로필 수정 API

a583091

add: 디스패처 DI

243cc60

add: EditProfileScreen

f2d7cbd

refactor: 취미 fetching 로직

ffd39f2

fix: navigation 애니메이션 제거(임시)

c6f0a26

refactor: Debounce search로 변경

8afb1dc

add: 자동 로그인

2ccae19

chore: 변수명 명료화

902a06e

chore: 리뷰 반영

d2ec5b0

add: 검색 로딩 뷰

29c9025

chore: 로깅 제거

53a1f6e

ThirFir self-assigned this Nov 14, 2024

ThirFir added the 과제 label Nov 14, 2024

ThirFir requested review from chrin05, Roel4990, jihyunniiii and 0se0 November 14, 2024 07:14

0se0 reviewed Nov 17, 2024

View reviewed changes

Roel4990 reviewed Nov 17, 2024

View reviewed changes

KkamSonLee reviewed Nov 17, 2024

View reviewed changes

chrin05 approved these changes Nov 17, 2024

View reviewed changes

jihyunniiii reviewed Nov 20, 2024

View reviewed changes

4주차 과제 #4

Are you sure you want to change the base?

4주차 과제 #4

Conversation

ThirFir commented Nov 14, 2024 • edited Loading

Work Description ✏️

Screenshot 📸

To Reviewers 📢

0se0 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

ThirFir Nov 17, 2024 • 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

Roel4990 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

KkamSonLee 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

ThirFir Nov 17, 2024 • 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

Choose a reason for hiding this comment

KkamSonLee Nov 18, 2024 • 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

Choose a reason for hiding this comment

Choose a reason for hiding this comment

chrin05 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

jihyunniiii left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ThirFir commented Nov 14, 2024 •

edited

Loading

ThirFir Nov 17, 2024 •

edited

Loading

ThirFir Nov 17, 2024 •

edited

Loading

KkamSonLee Nov 18, 2024 •

edited

Loading