제출한 TodoList 앱의 피드백 (튜터님)
1 .TagSettingViewController 의 viewDidLoad 구현 코드는 작은 단위의 메서드들로 한번 리팩토링 해보셔도 좋을 것 같습니다. 그 외에도 너무 길어진 메서드가 있다면 분리를 고려해보세요!
- 스토리보드 활용이 능숙하신 것 같으니, 다음 과제에서는 100% 코드기반 UI 를 작성해보셔도 좋을 것 같습니다.
- Todo 타입을 class 로 정의하신 이유가 궁금합니다. class 의 장점과 단점은 무엇이고, 어떤 경우에 사용해야하나요?
1. 너무 길어진 viewDidLoad
- 이 부분은 나도 계속 생각을 했었던 부분이다. 다른 사람의 코드를 보면서 느끼는거는 Configure() 메서드를 만들어서, 해당 메서드로 viewDidLoad의 메서드를 리팩토링을 해서. 화면을 구성해야하는 조건들을 메서드로 만들어서 호출하는 방식으로 하는게 가독성 면에서 훨씬 깔끔해보였다.
- 다음 과제, 프로젝트할때는 최대한 해당 요소를 중요하게 생각해서, 분리해보는 걸 연습해봐야겠다는 생각이 들었다.
2. 코드기반 UI 연습
- 스토리보드 vs 코드기반 UI 3월 28일 TIL 에서 확인할 수 있다.
- 아무래도 기업마다 구현 방식은 다 다르기 떄문에, 가능한 최대한 많이 경험을 해보는 것이 좋다고 생각한다.
- 다음 과제에서는 코드 기반으로도 진행을 해보면 좋을꺼 같다.
Todo타입을 Class로 정의한 이유
- 이유의 대해서는 3월 22일 TIL 에서 설명을 하였다.
- 다시 설명을 하자면
Todo타입을 Class로 만든 이유는 원래는 태그별로 해당 태그를 사용하는 Todo를 보여주는 View를 만들려고 했다. Class는 참조타입으로. Todo의 내용이 변하더라도, Tag 안에 있는 todo : [Todo] 안에 있는 내용도 같이 변경이 되기 떄문에, 원래는 Struct로 구현을 했다가 다시 변경을 했다.
마무리
- 주신 피드백으로 코드를 조금 변경을 해봐야겠다!
개발자 (진)