[TIL] 9월 13일
·
TIL
오늘 계산기과제의 대한 피드백을 받았다. 메서드 네이밍과 클래스 책임관련에 대한 피드백이였는데, 기존에 인풋 값이 올바른지 체크하고, 올바르다면 해당하는 operator를 변환해서 리턴하도록하는 두가지의 일을 수행하는 메서드를 단순히 setUpOperation이라고 지었었는데 이름만 보았을 땐 메서드에서 어떤 동작을 수행하는지 알기 어려우며, 이를 두개의 메서드로 분리하거나 이름을 바꾸는게 좋다는 피드백을 받았고 객체지향에서 메서드는 를 표현하므로 동사형으로 이름을 짓는게 좋다는 피드백을 받았다.네이밍 관련해서는 평소에도 어떤 이름으로 지을지 고민했었는데 앞으로도 좀 더 고민해봐야겠다. 우선 `Reg` enum은 `CalculatorApp` 내에 있는 게 아니라 외부 패키지로 분리하는 것이 필요해보..