Question

假设我有这样的文件：

import javax.swing.JPanel;
import java.awt.event.MouseAdapter;
public class Foo extends JPanel
{
    private int m;
    private int n;
    private int o;

    public Foo()
    {
        this.addMouseListener(new Bar());
    }

    class Bar extends MouseAdapter
    {
        // ...
        // methods here access and modify values of the private
        // instance variables.
        // ...
    }
}

显然，我可以向Foo添加简单的访问器和更改器，但这会很快并且完全破坏封装。我怎样才能重构这个内部类，同时将封装损坏降到最低？

Answer 1

如果这些类看起来太大，那么你应该拆分它们。拆分它们的第一步是停止依赖外部类的私有实例变量。正如你所说，你可以添加公共getter和setter，但更好的方法是让Foo实现Bar的公共接口，让Bar简单地与该接口对话。并用自己初始化每个Bar。

public class Bar extends MouseAdapter {
    public interface Caller {
        void thingClicked();
        ...
    }
}

public class Foo extends JPanel implements Bar.Caller {
    ...
}

所以现在在Bar，你有类似的东西：

public void mouseUp() {
   m = m + 1;
   n = 0
}

你现在有

public void mouseUp() {
   caller.thingClicked();
}

，在Foo：

public void thingClicked() {
   m = m + 1;
   n = 0
}

如果没有更多细节，很难明确这一点，但基本上你的外部类正在响应消息，而鼠标监听器只负责传递这些消息，而不是响应它们所发生的事情。在上面的示例中，它看起来比您已有的代码更多，但我怀疑您会发现以这种方式将其切片最终会导致更少的代码 - 当然代码更容易测试和重用。

Answer 2

一种方法是将类bar放在一个新文件中，并将所需的所有内容注入构造函数中。您需要将int等原始类型更改为Integer等对象。然后，您可以创建一个Builder类，用于控制Foo和Bar的构造。为简单起见，我们假设您在所有Baz和Foo个实例中都需要Bar的相同实例：

class Builder {
  // objects that both classes need are stored as member variables
  // if you need multiple instances of baz you can also store a BazBuilder here
  private Baz baz;

  public Builder(Baz baz) {
    this.baz=baz;
  }

  public Foo buildFoo() {
    Foo foo = new Foo(baz);
    return foo;
  }

  public Bar buildBar() {
    Bar bar = new Bar(baz);
    return bar;
  }
}

这只是一个简约的例子，但它很容易扩展。 Builder#buildFoo()可以使用Foo的构造函数中所需的参数，同样也可以Bar。

修改使用BazBuilder：

的示例
class BazBuilder { Baz build(int value) { return new Baz(value); } }

在Builder中你可以这样使用它：

class Builder { private BazBuilder bazBuilder; public Build(BazBuilder bazBuilder) { this.bazBuilder = bazBuilder; } public Foo buildFoo() { Baz baz = bazBuilder.build(5); Bar bar = new Bar(baz); Foo foo = new Foo(baz); foo.add(bar); return foo; } }

如何重构内部类MouseAdapter？

2 个答案: