TeamNADA / NADA-iOS-ForRelease

릴리즈를 목표로 하는 민재이준현규의 으라차차🦾 대소동🧨 iOS❤️‍🔥 TeamNADA
37 stars 5 forks source link

feat: 명함 만들기 뷰의 ViewModel 구현 (#376) #377

Closed hyun99999 closed 1 year ago

hyun99999 commented 1 year ago

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

🚨 참고사항

📸 스크린샷

기능 스크린샷
명함 만들기 뷰

📮 관련 이슈

hyun99999 commented 1 year ago

@dlwns33 어떻게 생각하시나여?

RxGesture 를 사용해서 리펙토링을 하고자 했는데요.

        basicBackgroundView.rx.tapGesture()
            .when(.recognized)
            .bind { _ in
               // 뷰의 배경색 변경과 화면전환.
            }

이를 사용해서 bind 안에 이벤트 동작을 넣거나 bind 하지 않고 tapGestue() 의 결과인 Observable 을 사용하는건데 UI 에 특화된 ControlEvent trait 에서 굳이 변환할 필요가 없다고 판단했습니당 어떻게 생각하시나요?

dlwns33 commented 1 year ago

@dlwns33 어떻게 생각하시나여?

RxGesture 를 사용해서 리펙토링을 하고자 했는데요.

        basicBackgroundView.rx.tapGesture()
            .when(.recognized)
            .bind { _ in
               // 뷰의 배경색 변경과 화면전환.
            }

이를 사용해서 bind 안에 이벤트 동작을 넣거나 bind 하지 않고 tapGestue() 의 결과인 Observable 을 사용하는건데 UI 에 특화된 ControlEvent trait 에서 굳이 변환할 필요가 없다고 판단했습니당 어떻게 생각하시나요?

오호 그렇다면 RxGesture을 사용해서 리팩토링 하지 않고 지금 있는 상태로 둬도 될 것 같아요!

저는 RxGesture을 사용하면 gesture 인스턴스를 따로 만들지 않아도 되서 코드가 조금 더 간결해져서 사용했습니다! 그리고 (나다에서는 아니겠지만) 혹시 다른 종류의 gesture을 사용하게 된다면 바로 바꿔주기에도 좋을 것 같아서! image