如何重构链式方法？

Question

从这段代码开始：

 new Person("ET").WithAge(88)

如何将其重构为：

 new Person("ET", 88)

需要执行什么重构顺序才能完成转换？

为什么？ 因为可能有数百个，我不想通过手动执行来引入错误。

您是否认为流畅界面的缺点是它们不能轻松重构？

注意：我想自动执行此操作，而无需手动输入代码。

Answer 1

也许重构它的最简单方法是将名称“WithAge”更改为“InitAge”，将 InitAge 设为私有，然后从构造函数中调用它。 然后更新 new Person(string).WithAge(int) 的所有引用以使用新的构造函数。

如果 WithAge 是单行代码，您只需将代码移动到新的构造函数中，并完全取消 InitAge ，除非使用附加方法提供额外的可读性。

良好的单元测试将隔离引入错误的位置（如果有）。

Answer 2

方法怎么样

Person(string name, int age)
{
    this.name = name;
    this.WithAge(age);
}

假设 WithAge 是 Person 上的一个返回 Person 的方法，那么像或更一般化的

Person(SomeType originalParameter, FluentParamType fluentParameter)
{
    //Original constructor stuff
    this.FluentMethod(fluentParameter);
}

：然后，如果您不想要它，请将 FluentMethod 设为私有，或者如果您想允许这两种方式，则将其保持为公开。

Answer 3

如果这是 C#（理想情况下您会用语言标记问题），则 Person 类需要此构造函数：

public Person(string name, int age)
    : this(name) { WithAge(age); }

然后，要更改所有客户端代码以在适当的情况下调用此新构造函数，您需要找到该模式的所有出现位置：

new Person(x1).WithAge(x2)

where x1 x2 是表达式，将其替换为：

new Person(x1, x2)

如果除了 WithAge 之外还有其他修饰符方法，则可能会变得更复杂。 例如：

new Person(x1).WithHair(x2).WithAge(x3)

也许您希望它变成：

new Person(x1, x3).WithHair(x2)

这完全取决于您是否有一个 IDE 可以让您定义类似的语言感知搜索/替换模式。 通过简单的文本搜索和替换，结合重播一系列按键的宏，您可以找到很长的解决方案。

你能说一下流利的缺点吗？
接口是他们不容易
重构了吗？

并不是特别重要 - IDE 中的重构功能要么设计得足够灵活，可以让您创造性地发明新的重构，要么针对某些常见情况进行硬编码。 我更喜欢将常见案例定义为我可以改变以发明新案例的示例。

Answer 4

我对此类事情没有任何实际经验，但如果我处于您的情况，我会去寻找的地方 自定义 Eclipse 重构（或者 Refactor! Pro for .Net 中的同等内容，如果您是这样的话）使用）。

基本上你想要的是匹配和替换，除了你的正则表达式应该匹配抽象语法树而不是纯文本。 这就是自动重构。

这种重构的一个风险是目标版本不如原始版本精确。 考虑一下：

class Person {
  public Person(String name, int age);
  public Person(String name, int numberOfChildren);
}

没有办法知道对 Person.WithAge 的链式调用应该替换为这些构造函数中的哪一个。

因此，对此的自动支持必须在允许您继续之前检查此类歧义。 如果已经存在带有目标参数的构造函数，则中止重构。

除此之外，它看起来非常简单。 为新的构造函数提供以下内容：

public Person(String name, int age) {
  this(name);
  withAge(age);
}

然后您可以安全地用新的调用替换原始调用。

（存在一个微妙的额外风险，即在构造函数内（即在部分构造的对象上）调用 withAge 与在构造函数之后调用它并不完全相同。如果您有继承链和<，则差异很重要/em> 如果 withAge 做了一些不平凡的事情，但这就是你的单元测试的目的......）

Answer 5

1. 为旧代码编写单元测试。

2. 重构直到测试再次通过。