[Refactor] Week6 과제 by gitsuhyun · Pull Request #11 · AND-SOPT-ANDROID/suhyeon-kim

gitsuhyun · 2024-12-04T09:02:11Z

Related issue 🛠

closed [REFACTOR] week6 필수 과제 #10

Work Description ✏️

필수 과제
- clean architecture 적용
- 레이어 분리
심화 과제
- Coroutine을 활용한 api 재연결
도전 과제
- hilt 적용

Screenshot 📸

Screen_recording_20241204_175459.mp4

Uncompleted Tasks 😅

To Reviewers 📢

합세에서 배운걸 바탕으로 과제 진행해봤습니닷

yskim6772

클린 아키텍처 적용도 고생 많으셨습니다 ~
MVI도 화이팅 !!

yskim6772 · 2024-12-08T11:53:27Z

app/src/main/java/org/sopt/and/data/mapper/todata/UserLoginModelMapper.kt

+fun UserLoginModel.toData() : UserLoginRequestDto = UserLoginRequestDto(
+    username = this.username,
+    password = this.password
+)


mapper 함수 네이밍이 조금 더 자세하면 좋을 것 같아요 ~ 저는 주로 toRequestLoginDto 같은 형태를 사용한답니다
그치만 정답은 없어요 !

app/src/main/java/org/sopt/and/data/mapper/todomain/UserSignUpResponseDtoMapper.kt

yskim6772 · 2024-12-08T11:54:04Z

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

+@Module
+@InstallIn(SingletonComponent::class)
+abstract class DataSourceModule {
+    @Binds
+    @Singleton
+    abstract fun bindUserDataRemoteSource(
+        userDataRemoteSourceImpl: UserDataRemoteSourceImpl
+    ): UserDataRemoteSource
+}


저와는 다르게 data 폴더와 di 폴더를 아예 구분하셨군요 ! 굿굿 좋아요 ~

yskim6772 · 2024-12-08T11:54:33Z

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

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


이 부분은 혹시 어디서 쓰일까요 ?!

동일한 타입의 retrofit 객체가 여러개여서 이를 구분하려는 용도로 사용했어요! 그런데 지금 생각해보니 과제에서는 불필요한 것 같네용

yskim6772 · 2024-12-08T11:55:07Z

app/src/main/java/org/sopt/and/domain/repository/UserRepository.kt

+import org.sopt.and.domain.model.request.UserLoginModel
+import org.sopt.and.domain.model.request.UserSignUpModel
+import org.sopt.and.domain.model.response.Hobby
+import org.sopt.and.domain.model.response.Token
+import org.sopt.and.domain.model.response.UserNumber
+
+interface UserRepository {
+    suspend fun postUserSignUp(userSignUpModel: UserSignUpModel): Result<UserNumber>
+    suspend fun postUserLogin(userLoginModel: UserLoginModel): Result<Token>
+    suspend fun getUserHobby(token: String): Result<Hobby>
+}


의존성 분리 잘하셨네요 ~~ 절대 domain에서는 dto가 아닌, model을 생성해서 사용해야 한다 ! 구웃 ~

yskim6772 · 2024-12-08T11:55:28Z

app/src/main/java/org/sopt/and/domain/type/HomeTabType.kt

+enum class HomeTabType(
+    val text: String,
+) {


enum 좋네요 ~~ 나중에 entity가 많아진다면, domain 폴더 안에 entity 폴더를 생성해서, 그 안에 type 폴더를 만들어서 넣어놔도 좋을 것 같아요 ~

app/src/main/java/org/sopt/and/presentation/home/HomeViewModel.kt

yskim6772 · 2024-12-08T11:56:47Z

app/src/main/java/org/sopt/and/presentation/login/LoginState.kt

+import org.sopt.and.domain.model.response.Token
+
+sealed class LoginState {
+    data object Idle: LoginState()


Idle은 혹시 어떤 state를 나타내는 걸까요 ?

Idle은 아직 아무런 동작도 수행되지 않은 state를 나타냅니다!

app/src/main/java/org/sopt/and/presentation/login/LoginViewModel.kt

yskim6772 · 2024-12-08T11:58:23Z

app/src/main/java/org/sopt/and/presentation/mypage/MyViewModel.kt

+        _myState.value = MyState.Loading
+        viewModelScope.launch {
+            val result = userRepository.getUserHobby(token = token)
+            _myState.value = result.fold(


fold 함수를 사용하신 이유가 궁금합니다 ~!

onSuccess와 onFailure에서 각 스크린에서 정의한 같은 형태의 state를 반환해주기 위해 사용했습니당

yihwanggeun

전반적으로 계층분리와 Hilt적용까지 잘 해주신 것 같습니다~! MVI도 화이팅입니당!

yihwanggeun · 2024-12-08T14:15:21Z

app/src/main/java/org/sopt/and/data/mapper/todomain/UserTokenResponseDtoMapper.kt

+
+fun UserTokenResponseDto.toDomain(): Token = Token(
+    token = this.token
+)


제가 Mapper를 사용하지 않아서 데이터 변환에 많은 고생을 했는데 Mapper를 꼭 써야겠다는 생각이 드네요

yihwanggeun · 2024-12-08T14:16:53Z

app/src/main/java/org/sopt/and/domain/model/request/UserSignUpModel.kt

@@ -0,0 +1,7 @@
+package org.sopt.and.domain.model.request


파일명에 Request 용으로 사용할건지 Data Class 용도로 사용할건지 좀 더 직관적으로 적어주시면 좋을 것 같아요!

app/src/main/java/org/sopt/and/domain/model/response/UserNumber.kt

app/src/main/java/org/sopt/and/presentation/login/LoginViewModel.kt

jihyunniiii

고생하셨습니다 ~

jihyunniiii · 2024-12-16T17:06:49Z

app/build.gradle.kts

+    hilt {
+        enableAggregatingTask = false
+    }


이거 관련해서 합세 리뷰에 적어두었으니 확인해주시면 좋을 것 같네요 ~

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

jihyunniiii · 2024-12-16T17:15:27Z

app/src/main/java/org/sopt/and/domain/type/HomeTabType.kt

@@ -0,0 +1,24 @@
+package org.sopt.and.domain.type
+
+enum class HomeTabType(


domain에 위치시키신 이유가 궁금해요!

app/src/main/java/org/sopt/and/presentation/search/SearchViewModel.kt

gitsuhyun added 9 commits December 3, 2024 20:32

#10 [refactor] : clean architecture 구조 세팅

b02e0f9

#10 [feat] : api 연결 세팅

b808a91

#10 [add] : hilt dependencies 추가

9c4ccf4

#10 [feat] : hilt 세팅

9baff1b

#10 [feat] : module 추가 및 LoginViewModel 수정

2289982

#10 [feat] : SignUpApi 연결

2ef0449

#10 [feat] : Hobby 조회 연결

6753a6f

#10 [feat] : HomeTabType 추가

577186a

#10 [del] : ApiFactory 삭제

3275b4e

gitsuhyun added the enhancement New feature or request label Dec 4, 2024

gitsuhyun requested review from jihyunniiii, yihwanggeun and yskim6772 December 4, 2024 09:02

gitsuhyun self-assigned this Dec 4, 2024

yskim6772 reviewed Dec 8, 2024

View reviewed changes

yihwanggeun approved these changes Dec 8, 2024

View reviewed changes

gitsuhyun added 5 commits December 9, 2024 12:54

#10 [chore] : signup response 변수명 수정

d379fb7

#10 [del] : 불필요한 Qualifier 삭제

76bab7a

#10 [chore] : 전체 스크린 hiltViewModel 적용

930e917

#10 [del] : log 삭제

388764e

#10 [feat] : ErrorHandler 함수 추가

e8f5d4d

gitsuhyun requested a review from yskim6772 December 9, 2024 04:51

jihyunniiii reviewed Dec 16, 2024

View reviewed changes

		@@ -0,0 +1,24 @@
		package org.sopt.and.domain.type

		enum class HomeTabType(

Conversation

gitsuhyun commented Dec 4, 2024

Related issue 🛠

Work Description ✏️

Screenshot 📸

Uncompleted Tasks 😅

To Reviewers 📢

Uh oh!

yskim6772 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

yihwanggeun left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jihyunniiii left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants