Step8 FeedBack

[somedd]

피드백 모음

[somedd]

1차 피드백

• pieGraphView 처럼 UIView를 상속해서 만든 객체는 draw() 함수를 직접 호출할 필요가 없습니다. setNeedsDisplay() 와 draw() 함수와의 관계를 정리해보세요.
• View 내부 속성을 외부에서 접근할 게 아니면 private으로 항상 선언하세요.
• 새로 추가한 generateListOfHistory(), generateListOfProductName() 메소드보다는 Segment를 위해서 [Beverage:Int] 정도로 음료수당 갯수를 넘겨주는 사전 타입을 만드는 게 낫지 않은가요? 배열과 배열을 만드는건 순서도 매칭해야 하고 오히려 코드가 더 복잡해지지 않나요?

[somedd]

1차 피드백 반영

• segments private 선언
• setPieGraph() 메서드 수정 : Beverage가 직접 전달 안되도록 [String : Int]로 타입을 지정해주었고, 관련 메서드를 VendingMachine 내부에 추가해주었습니다.

[somedd]

수고하셨습니다. 이번 미션에서 배운점을 정리하는 시간을 가져보세요.