[Feature/#106] : Ban 기능 구현 + etc

[sohyun127]

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

• merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕)
• 리뷰가 필요한 경우 리뷰어를 지정해 주세요
• P1 단계의 리뷰는 필수로 반영합니다.
• Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다.

📌 𝗜𝘀𝘀𝘂𝗲𝘀

• closed [FEAT] : 벤하기, 레벨 태그, 로딩뷰 빈도 #106

📎𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻

• Ban 기능 구현했습니다. (home feed, home detail feed, profile comment, profile feed)

  • home detail의 comment의 경우 대댓글 완료된 후 구현하겠습니다.

• 로딩 뷰 빈도 조정했습니다.

  • AS-IS : 마이페이지, 소식, 알림 -> 홈
  • TO-BE : 마이페이지 -> 홈

• 레벨 태그 이미지 변경했습니다.

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

밴(블라인드)

default.mp4

로딩뷰 조정

default.mp4

💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀

• 밴 기능 구현을 위해 로그인 API response에 추가된 isAdmin 값 datastore에 저장 되도록 구현했습니다. 커밋 확인해주세요! 272abd9
• 한 가지 고민 사항이 생겼습니다! ViewModel에서 재사용 되는 좋아요나 user type 판단.. 등의 비즈니스 로직들이 꽤 있는데, usecase로 캡슐화 하는 방향 어떠신가요? 리팩하게 되면 domain 모듈 하나 만들어서 usecase를 관리 할 예정입니다. 괜찮으면 이번 스프린트 끝나고 리팩토링 하겠습니다! 자세한 설계는 아래 노션 링크 참고해주세요.
  참고용 노션 링크

[chanubc]

domain분리 좋습니다1!
고생 많으셨어요 코리 달게 없네유

[chanubc]

viewmodel에서 init블록을 사용하면 좋지 않다는 의견이 있어 달아 봅니다!

ViewModel 생성과 강하게 커플링됨
데이터를 가져오는 타이밍이 ViewModel 생성시에 있기 때문에, 새로고침 등 사용자 상호작용이나 기타 이벤트에 따라 데이터 로딩시점을 제어하기 어렵게 만든다.
-> 근데 여긴 작은 데이터라 굳이 상관 없어 보이긴해요

https://www.youtube.com/watch?v=mNKQ9dc1knI
https://proandroiddev.com/mastering-android-viewmodels-essential-dos-and-donts-part-1-%EF%B8%8F-bdf05287bca9

[sohyun127]

오 좋은 글 공유 감사합니다 👍

[chanubc]

그리고 만약 추후에 view단에서 sateFlow로 필요 하다면

  val isPermissionState: StateFlow<Boolean> = userInfoRepository.getIsPushAlarmAllowed()
        .catch {
            _sideEffect.emit(ProfileSideEffect.ShowSnackBar(it))
        }.stateIn(
            scope = viewModelScope,
            started = SharingStarted.WhileSubscribed(5_000),
            initialValue = false,
        )

이런식으로 stateIn을 통해 cold stream인 flow를 hot stream으로 직접 변경하는 방법도 있습니다!

[chanubc]

이 부분의 경우 onEach를 사용해서 이렇게 쓸 수 도 있을거 ㅅ같은데 해보진 않았어요 ㅎㅎ

   val isAdminState: StateFlow<Boolean> = userInfoRepository.getIsPushAlarmAllowed()
        .catch {
            _sideEffect.emit(ProfileSideEffect.ShowSnackBar(it))
        }.onEach {
            fetchUserType(it)
        }
        .stateIn(
            scope = viewModelScope,
            started = SharingStarted.WhileSubscribed(5_000),
            initialValue = false,
        )

view단에서 따로 수집할 필요가 없으면 지금 방식이 더 효율적일것 같긴 합니다!

[chanubc]

위 유투브에 나온대로 init을 안쓰고 onStart()를 쓴다면

  val isAdminState: StateFlow<Boolean> = userInfoRepository.getIsPushAlarmAllowed()
        .catch {
            _sideEffect.emit(ProfileSideEffect.ShowSnackBar(it))
        }.onStart {
            fetchUserType()
        }
        .stateIn(
            scope = viewModelScope,
            started = SharingStarted.WhileSubscribed(5_000),
            initialValue = false,
        )

    fun fetchUserType(): ProfileUserType = when (isPermissionState.value) {
        true -> ProfileUserType.ADMIN
        false -> ProfileUserType.MEMBER
    }

이런느낌이 아닐까 싶네요

[sohyun127]

admin 값은 고정된 값이라 계속 수집할 필요는 없어보이고, 푸시 알림 여부 확인 로직 쪽에 반영해볼게요!
최고👍