Step7 Feedback - Githubissues

somedd commented 6 years ago

피드백 모음

somedd commented 6 years ago

1차 피드백

IBAction 핸들러 이름과 내부 메소드 이름이 같네요. 이렇게 결정한 이유는 뭔가요? 같은게 좋은가요 같지 않은게 좋은가요? 고민해보고 댓글로 설명해보세요.
함수명을 오해할 수 있을 것 같습니다. updateDetailInfo 는 DetailInfo를 새로운 값으로 업데이트하는 것처럼 보입니다. 무엇을 하는 것일까요?
drawThumbnailsView() 함수에서 다운로드까지 직접 하고 있으면 이 코드는 MVC 역할 중에서 어떤 역할을 하는건가요? 역할에 따라 코드를 분리해보세요.

somedd commented 6 years ago

1차 피드백 반영

일부 메소드 이름 수정
drawThumbNailsView()의 다운로드 기능 분리
thumnail 이미지수와 detailSection 이미지수가 일정하지 않음을 확인하여 각각의 이미지 수에 맞게 scrollView들의 contenSize가 정해지도록 추가 수정하였습니다.

somedd commented 6 years ago

View 에서 이벤트를 받아서 모델을 변경하는 코드와모델이 바뀌어서 View를 바꾸는 코드를 항상 분리해서 생각하세요.

somedd / swift-StoreApp