我应该争取一个成员职责单一的成员功能还是最小化成员变量的数量？

Question

我有一个名为SolarPosition的类，它具有许多成员函数。 我目前有一个名为CalculateSolarAzimuthAndAltitude的成员函数，它是重构的主要候选者。

我发现将其重构为CalculateSolarAzimuth和CalculateSolarAltitude的唯一方法是添加额外的十个成员变量，因为这两个函数都使用了一些初步计算。

我应该对担负两个职责的成员函数感到满意，还是为了将其拆分而将多个临时成员变量作为合理的牺牲？

Answer 1

我的建议是将职责分为两个成员职能。 如果这样引入了十个附加瞬态变量，则可以定义一个结构以将它们分组在一起，并在SolarPosition类中定义该结构的成员。 也会提高可读性。

如果您仍然想通过隐藏类声明中的内部数据变量来使SolarPosition类更整洁，请参考pimpl-idiom。 欲了解更多信息，请访问： pImpl习惯用法是否真的在实践中使用？

Answer 2

如果要添加的变量太多，我怀疑您的SolarPosition类中隐藏了一个附加类。 考虑提取该类并从那里继续。