通过构造要添加到列表的项目来避免使用临时变量

Question

Java 8 有很多特性可以让您避免使用临时变量和函数，似乎必须有一种方法可以避免使用这些临时变量来创建此列表：

List<Phone> phones = new ArrayList<>();
Phone p1 = new Phone();
p1.setNumber(this.mainPhoneNumber);
p1.setExtension(this.mainPhoneExt);
p1.setType(this.mainPhoneDesc);
phones.add(p1);
Phone p2 = new Phone();
p2.setNumber(this.secondaryPhoneNumberOne);
p2.setExtension(this.secondaryPhoneExtOne);
p2.setType(this.secondaryPhoneDescOne);
phones.add(p2);
Phone p3 = new Phone();
p3.setNumber(this.secondaryPhoneNumberTwo);
p3.setExtension(this.secondaryPhoneExtTwo);
p3.setType(this.secondaryPhoneDescTwo);
phones.add(p3);

我有一堆丑陋的成员变量需要整理成一个列表。 Phone 对象没有很好的构造函数，我无法更改它。

如何在不定义三个临时对象p1, p2, p3 的情况下从成员变量中创建此列表？

Answer 1

你能做这样的事吗？

public Phone phoneMaker(int number, int ext, string description) {
    Phone p = new Phone();
    p.setNumber(number);
    p.setExtension(ext);
    p.setType(description);
    return p;
}

List<Phone> phones = new ArrayList<>();
phones.add(phoneMaker(this.mainPhoneNumber, this.mainPhoneExt, this.mainPhoneDesc));

Answer 2

如果您可以修改 Phone 以使设置器可链接，您可以针对此：

phones.add(
    new Phone()
        .setNumber(this.mainPhoneNumber)
        .setExtension(this.mainPhoneExt)
        .setType(this.mainPhoneDesc)
);

如果这不可行，您可以创建一个外部构建器类。使用它看起来像：

phones.add(
    new PhoneBuilder()
        .number(this.mainPhoneNumber)
        .extension(this.mainPhoneExt)
        .type(this.mainPhoneDesc)
    .build()
);

实现构建器类很乏味，但很简单。您将所有字段值存储在成员变量中，然后在调用build() 时创建实际的Phone 对象。你会发现自己重复了很多Phone 的方法名称。不用担心，这很正常。

Answer 3

John Kugelman 和 Teo 向您展示了一些不同的可能性。

您可以通过

避免（显式）临时变量

List<Phone> phones = new ArrayList<>();
phones.add(new Phone());  // get(0)
phones.add(new Phone());  // get(1)
phones.add(new Phone());  // get(2)

phones.get(0).setNumber(this.mainPhoneNumber);
phones.get(0).setExtension(this.mainPhoneExt);
phones.get(0).setType(this.mainPhoneDesc);

phones.get(1).setNumber(this.mainPhoneNumberOne);
phones.get(1).setExtension(this.mainPhoneExtOne);
phones.get(1).setType(this.mainPhoneDescOne);

phones.get(2).setNumber(this.mainPhoneNumberTwo);
phones.get(2).setExtension(this.mainPhoneExtTwo);
phones.get(2).setType(this.mainPhoneDescTwo);

另一种方法是使用匿名类。看起来像

List<Phone> phones = new ArrayList<>();
phones.add(new Phone() {   // class
    {                      // initializer  (no this)
         setNumber(mainPhoneNumber);
         setExtension(mainPhoneExt);
         setType(mainPhoneDesc);
    }
});
phones.add(new Phone() {
    {
         setNumber(mainPhoneNumberOne);
         setExtension(mainPhoneExtOne);
         setType(mainPhoneDescOne);
    }
});

警告：您的 this 在初始化程序中不起作用。

如您所见，有很多不同口味和优缺点的可能性。

---

顺便说一句，要实现可链接的设置器或流畅的接口，只需始终返回 this。这是一个例子

class X {
    private T1 p1;
    private T2 p2;

    public X setter1 (T1 p1) {
       this.p1 = p1;
       return this;
    }

    public X setter2 (T2 p2) {
       this.p2 = p2;
       return this;
    }
...
}