Week4 essential #9

1971123-seongmin · 2024-11-13T09:04:27Z

Related issue 🛠

closed [Week4] 4주차 필수과제 #8

Work Description ✏️

서버 주소 숨기기 숨기기
서버 통신 세팅 (DI, Clean Architecture)
Token Interceptor (DataStore)
유저 등록 API 구현
- 회원가입 뷰 구조 변경(이메일, 비밀번호 -> username, password, hobby 입력하는 구조로 변경)
- username, password, hobby가 조건을 만족하지 않는 경우(8자 초과인 경우) 토스트 메시지 표시
로그인 API 구현
- 로그인 뷰 구조 변경(이메일, 비밀번호 -> username, password 받는 구조로 변경)
- (8자 초과인 경우) 토스트 메시지 표시
- 로그인 성공 시 응답으로 받는 token값 저장 (헤더에 인터셉터로 넣음)
- 로그인 실패, 성공 안내 토스트 메시지 표시
내 취미 조회 API 구현
- MY 뷰 구조 변경(이메일 -> 취미 표시)

Screenshot 📸

Toast 메시지	전체 실행 영상

Uncompleted Tasks 😅

MVI 구조 적용

To Reviewers 📢

로그인, 회원가입 페이지 MVI 적용
- 나머지 화면도 추후 적용하겠습니다.
현재 네비게이션 백스택 관리가 안되서 이 부분 수정할 예정입니다.

greedy0110

네트워크 연결하는 작업, DI로 의존성을 전달하는 과정에서 학습할 것이 많으셨을 텐데 고생하셨습니다 성민님

greedy0110 · 2024-11-16T08:22:33Z

app/src/main/java/org/sopt/and/GlobalApplication.kt

 class GlobalApplication: Application() {
-    private lateinit var dataStore: InfoDataStore
+    private lateinit var dataStore: TokenManager


dataStore는 범용적인 이름이지만, 실제 타입은 token을 관리하는 TokenManager로 비교적 구체적 타입이네요. tokenManager로 가는건 어떨까 싶습니다.

TokenManager가 더 적절한 이름이네요. 감사합니다

greedy0110 · 2024-11-16T08:28:57Z

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

+    @Provides
+    @Singleton
+    @Token
+    fun provideTokenInterceptor(tokenManager: TokenManager): AccessTokenInterceptor {
+        return AccessTokenInterceptor(tokenManager)
+    }
+
+    @Provides
+    @Singleton
+    fun provideOKHttpClient(
+        httpLoggingInterceptor: HttpLoggingInterceptor,
+        accessTokenInterceptor: AccessTokenInterceptor
+    ): OkHttpClient =


token qualifier 지정이 무의미 한 상황 같습니다.
실제로 지금도 AccessTokenInterceptor 객체는 딱 하나만 제공하고 있으므로, Token qualifier 없이 잘 의존성이 들어가고 있는 걸 볼 수 있네요.

greedy0110 · 2024-11-16T08:29:58Z

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

+            connectTimeout(30, TimeUnit.SECONDS)
+            readTimeout(30, TimeUnit.SECONDS)
+            writeTimeout(30, TimeUnit.SECONDS)
+            addInterceptor(httpLoggingInterceptor)


실제 릴리즈 코드에 포함될 이유는 없으므로 디버그 빌드에만 추가하도록 지정함은 어떨지요?

Suggested change

addInterceptor(httpLoggingInterceptor)

if (BuildConfig.DEBUG) addInterceptor(httpLoggingInterceptor)

디버그 빌드에 따로 지정가능한지 몰랐는데 감사합니다.

kamja0510

성민씨 점점 따라잡기 어려워지네요 ㅎ
그냥 과제만 한게 아니라 mvi 적용도 하고 노력하신게 보이네요..
근데 중간에 줄 공백 불편한거 좀 있어서 지은, 혜음한테 혼나실듯하네요 ㅎ

kamja0510 · 2024-11-17T07:16:28Z

app/src/main/java/org/sopt/and/base/BaseResponse.kt

+
+@Serializable
+data class BaseResponse<T>(
+    @SerialName("code")


어짜피 변수명과 json의 변수명이 같은데 SerialName 어노테이션으로 명시하는게 나은 것인가요?

변수명이 동일하면 생략해도 되긴한데 저는 똑같아도 습관적으로 쓰는 편이에요.

kamja0510 · 2024-11-17T07:18:29Z

app/src/main/java/org/sopt/and/base/BaseViewModel.kt

@@ -0,0 +1,63 @@
+package org.sopt.and.base


혹시 base라는 패키지는 어떤것을 의미하나요?
mvi를 적용하시면서 생겨난 것인가요?

base 패키지는 제가 주로 BaseViewModel, BaseResponse / BaseActivity, BaseFragment, BaseAdapter (이 세개는 xml에서) 등 base로 상속받아 사용하는 기반 클래스들을 모아둘 때 쓰고있어요

kamja0510 · 2024-11-17T07:23:49Z

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

+
+    @Qualifier
+    @Retention(AnnotationRetention.BINARY)
+    annotation class MainServer


신기한 문법.. 어떤 의도인지도 슬슬 모르겠네요..

의존성 주입을 사용하다 보면 구현체들이 여러개 생기는데 이것들을 구분하기 위한 태그, 이름이라고 생각하시면 되요.
ex) 스프링서버 / 파이썬 서버를 구분할 때 (얘네들 쓸 때 리포지토리, 데이터소스 등 클래스, 인터페이스 등 파일이 다를 텐데 이걸 따로 분리하지 않고 사용하면 복잡해요)

kamja0510 · 2024-11-17T07:25:32Z

app/src/main/java/org/sopt/and/data/interceptor/AccessTokenInterceptor.kt

+    }
+
+    companion object {
+        const val HEADER_AUTHORIZATION = "token"


세심하네요..

kamja0510 · 2024-11-17T07:30:04Z

app/src/main/java/org/sopt/and/data/model/response/UserLoginResponse.kt

+import kotlinx.serialization.Serializable
+
+@Serializable
+data class UserLoginResponse(


아하 성공에 대한 Response만 다룰 DTO를 만드시고 제너럴 타입으로 주시는 것 같은데 한 DTO에 성공 실패 둘다 들어갈 수 있게 하는 거 보다 유지보수에서 장점이 있어 이렇게 하신건가요?

질문을 이해 못했는데 좀 더 풀어서 설명해주실 수 있으실까요? ㅠㅠ

kamja0510 · 2024-11-17T07:35:23Z

app/src/main/java/org/sopt/and/enum/Movie.kt

-    예능,
-    애니,
-    해외시리즈
+enum class Movie(val description: String) {


오오오 깔끔

kamja0510 · 2024-11-17T07:39:13Z

app/src/main/java/org/sopt/and/presentation/ui/signin/SignInRoute.kt

+import org.sopt.and.utils.showToastMsg
+
+@Composable
+fun SignInRoute (


라우트(?)를 도입하셨군요

네비게이션 파일에서 직접 이걸 정의하면 너무 길어지고 복잡해져서 분리했어요.

kamja0510 · 2024-11-17T07:42:20Z

app/src/main/java/org/sopt/and/presentation/utils/contract/SignInContract.kt

+import org.sopt.and.presentation.utils.UiEvent
+import org.sopt.and.presentation.utils.UiState
+
+class SignInContract {


이런식으로 모아놓군요!

MVI 쓸 때 이런식으로 모아두고 쓰더라구요.
저는 MVI 공부할 때 https://velog.io/@kkmtk00314/Android-Compose-MVI-%EC%A0%81%EC%9A%A9%ED%95%B4%EB%B3%B4%EA%B8%B0 여기랑 다른 깃허브같이 참고해서 했어요.

kamja0510 · 2024-11-17T07:43:55Z

app/src/main/java/org/sopt/and/utils/TokenManager.kt

+
+    companion object {
+        private val Context.infoDataStore: DataStore<Preferences> by preferencesDataStore(name = "info")
+        private val ACCESS_TOKEN_KEY = stringPreferencesKey("access_token")


보통 유저의 토큰으로 쓰이면 access_token이라고 네이밍을 하나요?

저는 평소에 JWT 토큰 사용할 때 주로 사용해서 그것 때문에 네이밍을 저렇게 한 것 같아요.

kamja0510 · 2024-11-17T07:46:16Z

app/src/main/java/org/sopt/and/utils/showToastMsg.kt

+import android.content.Context
+import android.widget.Toast
+
+fun Context.showToastMsg(message: String, duration: Int = Toast.LENGTH_SHORT) {


와 이렇게 만들면 context를 안받게 하군요

hyeeum

어느덧 4주차까지 왔네요!! 유즈케이스 사용하신 거 보고.. 깜놀.. 저도 알려주세요 저 유즈케이스 몰라요 서버통신이 은근 고려해야할 게 많은데 고생하셨습니다 : - )

hyeeum · 2024-11-17T08:35:56Z

app/src/main/java/org/sopt/and/data/di/UseCaseModule.kt

성민님도 유즈케이스를 사용하셨군요.. 다들 왤케 아는게 많아요?

인정 ㅠ 머임 왤케 잘하세요

hyeeum · 2024-11-17T08:37:52Z

app/src/main/java/org/sopt/and/data/service/UserApi.kt

+
+    @GET("/user/{no}/my-hobby")
+    suspend fun getOthersHobby(
+


공백 발생..!

hyeeum · 2024-11-17T08:45:45Z

app/src/main/java/org/sopt/and/presentation/viewmodel/SignInViewModel.kt

+                                setState(currentState.copy(loginStatus = SignInContract.SignInStatus.FAILURE))
+                                setEffect(SignInContract.Effect.ShowToast("로그인 실패: ${exception.message}"))


실패의 경우까지 세심하게 신경써주셨네요 좋아요 !!

hyeeum · 2024-11-17T08:47:35Z

app/src/main/java/org/sopt/and/utils/showToastMsg.kt

+import android.content.Context
+import android.widget.Toast
+
+fun Context.showToastMsg(message: String, duration: Int = Toast.LENGTH_SHORT) {


확장함수까지 넘 좋아요! 그리고 함수명/변수명에서 축약어를 사용하는 건 지양해보아요!
물론 msg 라는 단어는 우리가 흔하게 사용하므로 이 의미를 유추할 수 있긴 하나, 협업을 할 때 축약어를 사용하게 되며 가독성이 떨어질 수 있습니다 :)

HAJIEUN02

MVI 적용까지..ㄷㄷ
너무 늘었어요 또 이 짧은 시간에ㅠㅜ!!
왜이렇게 우리 코리조가 천재야

HAJIEUN02 · 2024-11-17T11:18:11Z

app/build.gradle.kts

-dependencies {
+    dependencies {
+
+        implementation(libs.androidx.core.ktx)


androidx 계열 친구들을 bundles로 묶으면 한 줄로 이 많은 친구들을 한번에 추가할 수 있서요! 버전 카탈로그의 bundles에 대해 알아보시면 좋을 것 같아요!

몰랐던 내용인데 공유해 주셔서 감사합니다.

HAJIEUN02 · 2024-11-17T11:44:58Z

app/src/main/java/org/sopt/and/data/di/RepositoryModule.kt

+
+    @Provides
+    @Singleton
+    fun provideTokenManager(@ApplicationContext context: Context): TokenManager {


레포지토리 모듈에 들어가는 내용이러면 토큰매니저 네이밍을 변경하는 것도 좋을 것 같아요! 어떤 역할인지 직관적으로 와닿지 않는 것 같아용

원래 토큰 모듈로 따로 저장해두는데 리포지토리 모듈에 들어가있네요.. 다음에 수정하겠습니다

HAJIEUN02 · 2024-11-17T11:45:14Z

app/src/main/java/org/sopt/and/data/di/UseCaseModule.kt

인정 ㅠ 머임 왤케 잘하세요

HAJIEUN02 · 2024-11-17T11:46:46Z

app/src/main/java/org/sopt/and/data/remote/source/auth/AuthDataSourceImpl.kt

+import org.sopt.and.data.service.AuthApi
+import javax.inject.Inject
+
+class AuthDataSourceImpl @Inject constructor(


@kamja0510 이 부분은 MVI보단 클린아키텍처와 힐트에 대해 공부해보시면 이해될 거에요!

HAJIEUN02 · 2024-11-17T11:49:51Z

app/src/main/java/org/sopt/and/data/remote/source/user/UserRepositoryImpl.kt

+    override suspend fun getMyHobby(): Result<MyHobbyEntity> =
+        runCatching {
+            val response = userDataSource.getMyHobby()
+            response.result.mapperToMyHobbyEntity()


이 부분은 꼭 response 변수를 만들지 않고 한줄로 해도 괜찮을 것 같아요! 크게 사용되지 않는 변수를 굳이 정의할 필요는 없으니까용

HAJIEUN02 · 2024-11-17T11:50:19Z

app/src/main/java/org/sopt/and/data/service/AuthApi.kt

+
+interface AuthApi {
+
+    @POST("/user")


취향에 따라 uri를 상수화하는 경우도 있답니다!
저도 원래는 사실 하는데 이번엔 못했어요..

HAJIEUN02 · 2024-11-17T11:51:02Z

app/src/main/java/org/sopt/and/data/service/UserApi.kt

+
+    @PUT("/user")
+    suspend fun editUserInfo(
+


공백 잡아내기 쑈

HAJIEUN02 · 2024-11-17T11:51:27Z

app/src/main/java/org/sopt/and/domain/mapper/AuthMapper.kt

+import org.sopt.and.domain.model.auth.UserIdEntity
+import org.sopt.and.domain.model.auth.UserTokenEntity
+
+object AuthMapper {


매퍼를 이렇게 한군데에 모아두셨군요
이것두 좋은 방법이다 짱이당

HAJIEUN02 · 2024-11-17T11:52:05Z

app/src/main/java/org/sopt/and/enum/Movie.kt

+    ANIMATION("애니"),
+    FOREIGN_SERIES("해외시리즈");
+
+    fun getGenre(): String {


와ㅜ 짱이다 코리 반영 야무지게 해주셨네요 히히

HAJIEUN02 · 2024-11-17T11:53:00Z

app/src/main/java/org/sopt/and/presentation/component/BoxOverlayImage.kt

@@ -28,7 +28,7 @@ fun BoxOverlayImage(
 ) {
    Box(
        modifier = modifier
-            .width(180.dp)  // 비율로 수정
+            .width(180.dp)


비율로 수정 안하셨는데 왜 투두 지우심!!
기기대응을 위해 LocalCinfiguration을 사용해서 수정해보셔요

어떻게 할지 모르겠어서 일단 뒀는데 알려주신대로 해볼게요. 감사합니다.

jihyunniiii

수고하셨습니다!
합세도 파이띵 ~

jihyunniiii · 2024-11-19T16:21:33Z

app/src/main/java/org/sopt/and/base/BaseResponse.kt

+import kotlinx.serialization.Serializable
+
+@Serializable
+data class BaseResponse<T>(


BaseResponse를 이런 형태로 가져가신 이유가 있나 궁금합니다!

jihyunniiii · 2024-11-19T16:28:16Z

app/src/main/java/org/sopt/and/domain/mapper/AuthMapper.kt

+import org.sopt.and.domain.model.auth.UserIdEntity
+import org.sopt.and.domain.model.auth.UserTokenEntity
+
+object AuthMapper {


해당 mapper를 domain에 두신 이유가 있나요?
이 mapper는 data 레이어에 있는 model을 참조하기 때문에 이렇게 하면 domain 레이어가 data 레이어에 의존하게 됩니다.
이렇게 되면 클린 아키텍처를 따르지 않게 되겠죠?
어떤 식으로 수정하면 좋을지 고민해봅시다!

jihyunniiii · 2024-11-19T16:33:54Z

app/src/main/java/org/sopt/and/domain/model/auth/LoginUserEntity.kt

@@ -0,0 +1,6 @@
+package org.sopt.and.domain.model.auth
+
+data class LoginUserEntity(


domain의 모델을 정의할 때 -Entity라는 네이밍을 사용하시는 이유가 있나요?

jihyunniiii · 2024-11-19T16:40:13Z

app/src/main/java/org/sopt/and/presentation/viewmodel/MyPageViewModel.kt

+    init {
+        geyMyHobby()
+    }


init 블럭은 어떻게 동작할까요?
그리고 어떤 로직들을 담는 것이 좋을까요?

jihyunniiii · 2024-11-19T16:41:21Z

app/src/main/java/org/sopt/and/utils/TokenManager.kt

+class TokenManager @Inject constructor(
+    @ApplicationContext private val context: Context
+) {
+


companion object는 최하단에 적어주시는 것이 컨벤션입니다.

1971123-seongmin added 30 commits November 6, 2024 16:56

[feat] : #8 Network gradle 추가 및 주소 BuildConfig

b03fe4b

[feat] : #8 Network permission 추가

d24e02a

[mod] : #8 Movie enum 수정

28f2744

[chore] : #8 주석 제거

8679a8f

[feat] : #8 SocialLoginRow list remember 추가

30aadd5

[feat] : #8 coli gradle 추가

5537e13

[feat] : #8 component preview 함수 추가

0a3bc75

[feat] : #8 AccessTokenInterceptor 추가

8d2e980

[feat] : #8 InfoDataStore -> TokenManger 이름 변경 및 토큰 추가

798734f

[feat] : #8 NetworkModule 추가

e7674da

[chore] : #8 GlobalApplication TokenManager 이름 변경

7f49c7c

[feat] : #8 strings 추가

b218cc7

[feat] : #8 BaseResponse 추가

2ddf6de

[feat] : #8 임시파일 제거

3094bb3

[feat] : #8 회원가입 Dto, Entity 추가

fc097b8

[feat] : #8 회원가입 Mapper 함수 추가

d6a890d

[mod] : #8 회원가입 ValidationSignup 수정

74282e2

[feat] : #8 MVI 적용을 위한 파일 추가

c64b3f9

[feat] : #8 MainActivity @androidentrypoint 추가

fe734bc

[feat] : #8 Toast 확장 함수 추가

70e1ae5

[feat] : #8 @serializable 추가

0fbbe40

[feat] : #8 @HiltAndroidApp 추가

36adaad

[feat] : #8 hilt-navigation-compose gradle 추가

3ce5ea7

[feat] : #8 Di Module 추가

fb09a4c

[feat] : #8 회원가입 API 관련 파일 추가

193b5f1

[feat] : #8 BaseResponse -> NullableBaseResponse로 변경

be65c71

[feat] : #8 SignUpViewModel MVI, API 연결

1159e91

[feat] : #8 SignUpScreen MVI 적용

75304bf

[feat] : #8 data class 삭제

c6ada3b

[feat] : #8 SignInContract 생성

4ad46ca

1971123-seongmin added 19 commits November 13, 2024 04:32

[feat] : #8 로그인 api 관련 코드 추가

7543db2

[feat] : #8 strings 추가

80076c4

[chore] : #8 SignUpViewModel 로그 삭제, toast 메시지 문구 변경

069c647

[feat] : #8 UseCaseModule에 LoginUseCase 추가

034d34f

[feat] : #8 로그인 MVI 구조로 변경 및 API 연결

f6be2c0

[feat] : #8 로그인 응답으로 받는 토큰 DataStore에 저장

b60260f

[feat] : #8 AGP 버전 변경

cb58a1f

[chore] : #8 mapper object 이름 변경

ace6aa2

[chore] : #8 패키지 구조 변경

dca7009

[chore] : #8 AuthApi 미사용 api 삭제

d7b5bd6

[chore] : #8 패키지 구조 변경

973a346

[feat] : #8 내 취미 조회 dto, entity 생성

9536fec

[feat] : #8 내 취미 조회 api 관련 파일 추가

b02cc39

[feat] : #8 User 서버통신 모듈 추가

40f06bc

[feat] : #8 operator 키워드 추가

b32abe5

[chore] : #8 줄바꿈 제거

34a6dcb

[chore] : #8 MyPage API 연결

b4d30d0

[feat] : #8 PreView 함수 추가

d24712b

[chore] : #8 미사용 부분 삭제

0ad776f

1971123-seongmin self-assigned this Nov 13, 2024

1971123-seongmin requested review from hyeeum, kamja0510 and HAJIEUN02 November 13, 2024 09:04

[chore] : #8 미사용 부분 삭제

8313924

greedy0110 reviewed Nov 16, 2024

View reviewed changes

kamja0510 approved these changes Nov 17, 2024

View reviewed changes

hyeeum approved these changes Nov 17, 2024

View reviewed changes

HAJIEUN02 approved these changes Nov 17, 2024

View reviewed changes

1971123-seongmin requested a review from jihyunniiii November 19, 2024 09:15

jihyunniiii reviewed Nov 19, 2024

View reviewed changes

	addInterceptor(httpLoggingInterceptor)
	if (BuildConfig.DEBUG) addInterceptor(httpLoggingInterceptor)

		setState(currentState.copy(loginStatus = SignInContract.SignInStatus.FAILURE))
		setEffect(SignInContract.Effect.ShowToast("로그인 실패: ${exception.message}"))

		@@ -0,0 +1,6 @@
		package org.sopt.and.domain.model.auth

		data class LoginUserEntity(

Week4 essential #9

Are you sure you want to change the base?

Week4 essential #9

Conversation

1971123-seongmin commented Nov 13, 2024 • edited Loading

Related issue 🛠

Work Description ✏️

Screenshot 📸

Toast 메시지

전체 실행 영상

Uncompleted Tasks 😅

To Reviewers 📢

greedy0110 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

kamja0510 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

Choose a reason for hiding this comment

hyeeum 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

HAJIEUN02 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

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

Choose a reason for hiding this comment

1971123-seongmin commented Nov 13, 2024 •

edited

Loading