MOD/#19: 사용자저장소 관련 코드 수정

NOW-SOPT-ANDROID · Jun 7, 2024 · 812a7db · 812a7db
1 parent fb985fd
commit 812a7db
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 16 deletions.
diff --git a/app/src/main/java/com/sopt/now/compose/container/SoptAppContainer.kt b/app/src/main/java/com/sopt/now/compose/container/SoptAppContainer.kt
@@ -59,10 +59,7 @@ class SoptAppContainer(context: Context): AppContainer {
     }
 
     private val okHttpClientWithInterceptor: OkHttpClient.Builder by lazy {
-        val memberId = context
-            .getSharedPreferences(PREFERENCE_ID, Context.MODE_PRIVATE)
-            .getString(USER_ID_KEY, "")
-            .orEmpty()
+        val memberId = userRepository.getUserId()
 
         OkHttpClient.Builder().apply {
             addInterceptor(Interceptor { chain ->
@@ -78,7 +75,5 @@ class SoptAppContainer(context: Context): AppContainer {
     companion object{
         const val BASE_URL = BuildConfig.AUTH_BASE_URL
         const val PREFERENCE_ID = "SOPT"
-
-        private const val TAG = "SoptAppContainer"
     }
 }
diff --git a/app/src/main/java/com/sopt/now/compose/container/impl/UserRepositoryImpl.kt b/app/src/main/java/com/sopt/now/compose/container/impl/UserRepositoryImpl.kt
@@ -7,7 +7,7 @@ import com.sopt.now.compose.models.User
 class UserRepositoryImpl(
     private val sharedPreferences: SharedPreferences
 ) : UserRepository {
-    override suspend fun getUserProfile(): User {
+    override fun getUserProfile(): User {
         sharedPreferences.run {
             val id = getString(ID_KEY, "") ?: ""
             val pw = getString(PW_KEY, "") ?: ""
@@ -17,7 +17,7 @@ class UserRepositoryImpl(
         }
     }
 
-    override suspend fun setUserProfile(user: User) {
+    override fun setUserProfile(user: User) {
         val edit = sharedPreferences.edit()
         edit.run {
             putString(ID_KEY, user.id)
@@ -28,11 +28,11 @@ class UserRepositoryImpl(
         }
     }
 
-    override suspend fun getUserId(): String {
+    override fun getUserId(): String {
         return sharedPreferences.getString(USER_ID_KEY, "").orEmpty()
     }
 
-    override suspend fun setUserId(userId: String) {
+    override fun setUserId(userId: String) {
             val edit = sharedPreferences.edit()
             edit.putString(USER_ID_KEY, userId).apply()
 

diff --git a/app/src/main/java/com/sopt/now/compose/container/repository/UserRepository.kt b/app/src/main/java/com/sopt/now/compose/container/repository/UserRepository.kt
@@ -5,10 +5,9 @@ import android.util.Log
 import com.sopt.now.compose.models.User
 
 interface UserRepository {
-    suspend fun getUserProfile(): User?
-    suspend fun setUserProfile(user: User)
-
-    suspend fun getUserId(): String
-    suspend fun setUserId(userId: String)
+    fun getUserProfile(): User?
+    fun setUserProfile(user: User)
+    fun getUserId(): String
+    fun setUserId(userId: String)
 }
 
diff --git a/app/src/main/java/com/sopt/now/compose/ui/screens/login/LoginViewModel.kt b/app/src/main/java/com/sopt/now/compose/ui/screens/login/LoginViewModel.kt
@@ -80,7 +80,7 @@ class LoginViewModel(
         password =  _uiState.value.pw
     )
 
-    private fun setUserIdInPreference(userId: String) = viewModelScope.launch {
+    private fun setUserIdInPreference(userId: String) {
         userRepository.setUserId(userId)
     }