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

[리펙토링] BaseViewController 생성 #323

Merged
merged 10 commits into from
Nov 30, 2021
Merged

[리펙토링] BaseViewController 생성 #323

merged 10 commits into from
Nov 30, 2021

Conversation

sujeong000
Copy link
Member

작업 내용

  • Foreground의 모든 ViewController가 BaseViewControllerType을 채택하도록 리팩토링

시연 방법

기타 (고민과 해결, 리뷰 포인트 등)

  • 모든 뷰 관련 프로퍼티를 [각씬이름]View() 내부로 집어넣음.
  • busRouteType을 받아 view의 컬러를 바꾸는 메소드 configureColor(routeType:)를 ViewController -> View 쪽으로 이동시킴. 관련 뷰를 ViewController->View로 이동시켰으므로 해당 메소드도 뷰로 이동되어야 한다고 판단.
  • Refresh 버튼과 CustomNavigationBar는 추후 프로토콜화 되어 각 뷰컨트롤러에서 구현할 필요없으므로 addSubview()는 하지 않아 화면에 보이지 않는 상태임.

@sujeong000 sujeong000 added 🧱 refactor 리팩토링, 기능적인 부분에 수정이 없이 전체적인 코드의 구조를 개선 👍 review 리뷰 요청시 사용하는 라벨 labels Nov 30, 2021
@Modyhoon Modyhoon merged commit f10070c into develop Nov 30, 2021
@Modyhoon Modyhoon deleted the issue/#312 branch November 30, 2021 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧱 refactor 리팩토링, 기능적인 부분에 수정이 없이 전체적인 코드의 구조를 개선 👍 review 리뷰 요청시 사용하는 라벨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants