让setter返回"this"是不好的做法吗?

让java中的setter返回“this”是好还是坏?

public Employee setName(String name){
   this.name = name;
   return this;
}

这个模式很有用，因为你可以像这样设置链:

list.add(new Employee().setName("Jack Sparrow").setId(1).setFoo("bacon!"));

而不是这样:

Employee e = new Employee();
e.setName("Jack Sparrow");
...and so on...
list.add(e);

.．.但这有点违背标准惯例。我认为这是值得的因为它可以让setter做一些其他有用的事情。我曾在一些地方看到过这种模式(例如JMock、JPA)，但它似乎并不常见，而且通常只用于定义良好的api，在这些api中这种模式随处可见。

更新:

我所描述的显然是有效的，但我真正想要的是一些关于这是否被普遍接受的想法，以及是否存在任何陷阱或相关的最佳实践。我知道Builder模式，但它比我所描述的要复杂一些——正如Josh Bloch所描述的，有一个用于对象创建的相关静态Builder类。

当前回答

一般来说，这是一个很好的实践，但是你可能需要使用布尔类型来确定操作是否成功完成，这也是一种方法。一般来说，没有教条说这是好的还是床上的，它当然来自于情况。

其他回答

我不懂Java，但我用c++做过。也有人说这会让文字变得很长，很难读懂，但我这样做过很多次:

list.add(new Employee()
    .setName("Jack Sparrow")
    .setId(1)
    .setFoo("bacon!"));

这个更好:

list.add(
    new Employee("Jack Sparrow")
    .Id(1)
    .foo("bacon!"));

至少我是这么认为的。但如果你愿意，欢迎你给我投反对票，称我为糟糕的程序员。我不知道Java中是否允许这样做。

我不认为这有什么特别的问题，这只是风格的问题。它在以下情况下很有用:

您需要一次设置多个字段(包括在构建时) 您知道在编写代码时需要设置哪些字段对于想要设置的字段，有许多不同的组合。

这种方法的替代方法可能是:

一个mega构造函数(缺点:你可能会传递很多空值或默认值，并且很难知道哪个值对应什么) 几个重载的构造函数(缺点:一旦你有多个构造函数，就会变得笨拙) 工厂方法/静态方法(缺点:与重载构造函数相同——一旦超过几个就会变得笨拙)

如果你一次只设置几个属性，我会说不值得返回this。如果您稍后决定返回其他内容，例如状态/成功指示符/消息，那么它肯定会崩溃。

坏习惯: setter集 getter get

如果显式地声明一个方法，为U

setPropertyFromParams(array $hashParamList) { ... }

我制作setter已经有一段时间了，唯一真正的问题是库坚持使用严格的getPropertyDescriptors来获得bean读取器/写入器bean访问器。在这些情况下，您的java“bean”将没有您所期望的写入器。

例如，我还没有测试它的确定，但我不会感到惊讶，杰克逊不会识别这些设置时，从json/maps创建java对象。我希望我在这一点上是错的(我很快就会测试它)。

事实上，我正在开发一个轻量级的以SQL为中心的ORM，我必须在getPropertyDescriptors之外添加一些代码来识别返回此的设置器。

至少在理论上，它会通过在调用之间设置错误的依赖关系而破坏JVM的优化机制。

它被认为是语法糖，但实际上会在超级智能的Java 43虚拟机中产生副作用。

这就是为什么我投反对票，不要使用它。

推荐文章