Skip to content

[PR]검색바 구현 완료#72

Merged
giseungNoh merged 2 commits intodevfrom
feature/HomeView-Search-V2
Jun 15, 2025
Merged

[PR]검색바 구현 완료#72
giseungNoh merged 2 commits intodevfrom
feature/HomeView-Search-V2

Conversation

@giseungNoh
Copy link
Copy Markdown
Collaborator

@giseungNoh giseungNoh commented Jun 15, 2025

검색바 구현 기능
1.태그에 맞는 비디오 목록 불러오기
2.검색바 텍스트 필드 글자 수 30자로 제한
3.서치바에서 x버튼 누르면 원래 전체 비디오 배열로 전환
4.일치 하는 태그 없으면 "검색 결과가 없습니다." 플레이스 홀더 띄움

궁금한점

  • HomeViewController에 원래 private var 로 되어있는 부분(비디오,태그 배열)을 다른 파일(HomeViewController+searchBar)에서도 사용하기 위해 그냥 var로 바꿔 뒀습니다. 이 부분을 어떻게 해야 private를 유지하면서 파일을 분리할 수 있는지 궁금합니다.
    var videoList: [Video] = []
    var tags: [Tag] = []
    var filteredTags: [Tag] = []

Ps.
*동녕님 var부분 말고는 동녕님이 작성하신 부분 안건드렸지만 혹시 컨플릭 날 만한 부분 있으면 말씀해주세요.
*지현님 비디오 지워지면 태그도 지워질 수 있도록 변경해주시면 감사하겠습니다.

남은 할일
-새로고침

@korbit-ai
Copy link
Copy Markdown

korbit-ai Bot commented Jun 15, 2025

You've used up your 5 PR reviews for this month under the Korbit Starter Plan. You'll get 5 more reviews on June 22nd, 2025 or you can upgrade to Pro for unlimited PR reviews and enhanced features in your Korbit Console.

@giseungNoh giseungNoh changed the title 라이브러리/캐쉬? 오류 발생 [PR]검색바 구현 완료 Jun 15, 2025
@kdn0325 kdn0325 self-requested a review June 15, 2025 12:31
Copy link
Copy Markdown
Collaborator

@kdn0325 kdn0325 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

@jihyeonjjang
Copy link
Copy Markdown
Collaborator

jihyeonjjang commented Jun 15, 2025

다른 파일에서도 사용하기 위해서 할 수 있는 방법은

  1. private -> fileprivate
  2. private(set): 읽기전용
    이정도가 있는데, 실제로 이런 상황에서 두가지 방법이 올바른 방향이고, 잘 쓰이는지에 대한 부분은 저도 좀 더 공부를 해봐야 알 것 같습니다

그런데 저의 의견은 그 변수들을 viewmodel로 옮기는 것이 좋아보여요
지금 기능면에서는 문제 없으니 일단 두고, 나중에 리팩토링하는 게 좋을 것 같습니다

Copy link
Copy Markdown
Collaborator

@jihyeonjjang jihyeonjjang left a comment

Choose a reason for hiding this comment

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

확인했습니다!
태그부분도 작업해서 올려드리겠습니다

@giseungNoh giseungNoh merged commit 8bfb7d7 into dev Jun 15, 2025
@giseungNoh giseungNoh deleted the feature/HomeView-Search-V2 branch June 15, 2025 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants