하나의 책임으로 멤버 함수 중 하나를 위해 노력해야합니까, 아니면 멤버 변수의 수를 최소화해야합니까?

Question

많은 멤버 함수가있는 SolarPosition 클래스가 있습니다. 현재 리팩토링을위한 주요 후보자 인 CalculateSolarAzimuthAndAltitude이라는 멤버 함수가 있습니다.

나는 추가로 열 개 멤버 변수를 추가하는 것이 었습니다 일부 예비 계산이 두 기능에 의해 사용되는 것과 유일한 방법은, CalculateSolarAzimuth 및 CalculateSolarAltitude로 리팩토링하는 것을 발견했다.

두 가지 책임이있는 멤버 함수에 만족해야합니까, 아니면 여러 개의 과도 멤버 변수를 분할하여 합리적인 희생을 제공해야합니까?

Answer 1

내 제안은 책임을 두 개의 멤버 함수로 나눠주는 것입니다. 10 개의 추가 임시 변수를 도입하면,이를 함께 그룹화하고 SolarPosition 클래스에서 해당 구조체의 멤버를 정의하는 구조체를 정의 할 수 있습니다. 가독성도 향상됩니다.

여전히 클래스 선언에서 내부 데이터 변수를 숨겨서 SolarPosition 클래스를 훨씬 깨끗하게 유지하려면 pimpl-idiom을 참조하십시오. 자세한 내용은 다음 웹 사이트를 방문하십시오 : Is the pImpl idiom really used in practice?

Answer 2

추가 할 변수가 더 많으면 SolarPosition 클래스에 추가 클래스가 숨어있는 것 같습니다. 그 수업을 추출하고 거기서 계속 나아가 야한다고 생각하십시오.