Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SearchViewModel 구현 및 연관된 기능 구현 #78

Merged
merged 4 commits into from
Mar 4, 2024
Merged

Conversation

eung7
Copy link
Contributor

@eung7 eung7 commented Mar 3, 2024

고민, 과정, 근거 💬

이번 PR에서는 ViewModel 및 관련된 Dependency를 구현했습니다.
이번 작업도 Home이랑 ProductInfo와 비슷한 작업입니다.
그외 ViewModel의 비즈니스 로직이 가장 다른 점이고 UI를 수정하면서 변경이 될 수 있습니다.

나중에 개선하면 좋은 점

  • 코드로 구현해도 좋지만, Coordinator에 대해 익숙해져 있다보니 View에 화면 전환 코드를 작성하는 것이 조금 거슬리네요.
    나중에 SwiftUI 관련 Coordinator Pattern 적용을 고려하는 것도 좋은 생각일 것 같습니다.
  • 아무래도 현재 ViewModel이나 다른 코드가 너무 겹치는 것이 많아서.. 나중에 따로 조치를 취하는 게 좋을 것 같습니다 😭

  • Closed: #(issue-here)

@eung7 eung7 added 🛜 Network Networking 🔍 Search Search View labels Mar 3, 2024
@eung7 eung7 added this to the v2.0.0 milestone Mar 3, 2024
@eung7 eung7 requested a review from a team March 3, 2024 15:56
@eung7 eung7 self-assigned this Mar 3, 2024
Copy link
Member

@WhiteHyun WhiteHyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

우선 HomeView에서 SearchView 생성할 때 같이 주입하는 방식으로 구현해두고 나중에 아이디어를 서로 공유해봐야할 것 같아요.
(의존성 주입을 위해선 결국 AppRootComponent가 모든 걸 알아야할 수 밖에 없는데, 각 뷰마다 어떻게 의존성 주입 코드를 만들 것인지 고민해봐요 ☺️)

@eung7 eung7 merged commit 146eeb4 into main Mar 4, 2024
6 checks passed
@eung7 eung7 deleted the feature/search/77 branch March 4, 2024 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛜 Network Networking 🔍 Search Search View
Projects
Development

Successfully merging this pull request may close these issues.

2 participants