代码之家  ›  专栏  ›  技术社区  ›  James Thompson

在类中寻找隐式通信

  •  2
  • James Thompson  · 技术社区  · 14 年前

    我现在正在重构C++中一个非常有用但设计不好的类,并且我遇到了一个设计问题:使用参数的方法来传递数据,通过在类中设置私有状态变量来传递数据。这使得我很难描绘出数据是如何在函数中移动的。我在周末的任务是尽可能多地删除这种传递数据的方式,这使得程序很难仅仅从方法签名中理解,因为签名只讲述了故事的一部分。我已经决定了

    我目前测试方法是否使用私有类级变量进行通信的方法如下:

    1. 编辑该方法并使其成为函数而不是方法,这将删除对类中状态变量的访问。
    2. 编辑对方法的所有调用,以便它们调用函数而不是方法。
    3. 编译,看看是否有任何中断。列出要添加到原始类中的访问器。
    4. 运行单元测试,看看我是否以非常微妙的方式破坏了任何东西。

    有没有更好的方法可以做到这一点,也许一个可以很容易地自动化?如果我向其他人展示重构技术,我可以引用它吗?

    到目前为止,我唯一提到这个问题的是 Coders at Work 通过 Object-oriented programming Wikipedia entry :

    “面向对象语言的问题是,它们拥有所有这些隐含的环境。你想要一根香蕉,但你得到的是一只大猩猩,手里拿着香蕉和整个丛林

    编辑以回答来自的好问题 Oli Charlesworth :

    我知道OOP的目的是有时通过类的状态变量进行通信。我当前案例的困难在于,该类中目前有78个不同的数据成员,其中许多是其他数据类型的字符串的键值对,并且对它们需要初始化的顺序存在未记录的隐式依赖关系。有可能给一个足够聪明的程序员使用这个类是很容易的,但目前这对我来说是非常困难的。我认为这些数据类型中的一些可以抽象到它们自己的类中,但是在我这样做之前,我需要更清楚地了解数据成员之间是如何交互的。

    2 回复  |  直到 7 年前
        1
  •  2
  •   Len Holgate    14 年前

    考虑到问题的澄清,我的“你确定这不仅仅是你不喜欢另一个程序员的风格”comment dies a death;)

    就我个人而言,我只是正常重构。也就是说,有78个数据成员和许多相关的位,但不在它们自己的类中,我将首先对相关的数据进行分组并提取对其起作用的功能。没有必要,IMHO,通过一个阶段,您明确地将数据传递到现有类中的函数。只需选择一组相关的数据项,给出一个像样的名称,提取它们,并找出它们的使用位置以及如何将功能移到新类中。

    理想情况下,我会开始为主类和新的分解类编写单元测试,然后。。。

        2
  •  1
  •   Carl Manaster    14 年前

    与其让方法的所有调用方都调用函数,一个较小的中间更改是让方法为所有调用方保留在适当的位置,并通过调用函数简单地进行委托。稍后可以内联方法调用,以便所有调用方都直接调用函数。

    另外,从您的描述来看,您似乎是通过手动测试来实现这一点的。有了全面的单元测试,您将获得更好的成功(更容易重构,减少错误风险),当然,您描述的代码很难进行单元测试。尽管如此,还是要努力实现更多的测试自动化。