当实现 Predicate 而不是 boolean

Question

我正在阅读一个不相关的线程，当我读到一条评论时：每当我发现自己需要一个多行 lambda 时，我都会将这些行移动到一个私有方法并传递方法引用而不是 lambda。我在问：实现这种行为的正确方法是什么？使用评论中发布的布尔方法还是谓词？示例：假设我想检查 aTable是否可用，其中可用意味着isClean, isEmpty, hasChair。class Table{    public boolean hasChair(){...}    public boolean isClean(){...}    public boolean isEmpty(){...}}我可以List<Table> tablesList = Arrays.asList(table1,table2,table3,table4);通过两种方式为我的列表实现过滤测试：第一种是布尔值：public boolean isUsable(){    return hasChair() && isClean() && isEmpty();}并使用它 tablesList.stream().filter(Table::isUsable)第二种方式是使用谓词：public Predicate<Table> isUsable(){    return table -> table.isEmpty() && table.isClean() && table.hasChair();}可与 tablesList.stream().filter(isUsable())哪个是正确的实现？为什么选择一个而不是另一个？有什么大的区别吗？

Answer 1

我想你的意思是第二个例子

public static Predicate<Table> isUsable(){

    return table -> table.isEmpty() && table.isClean() && table.hasChair();

}

这可能已经表明这种形式可能会使读者感到困惑。没有static你可以写table.isUsable()，Table::isUsable但它不会按照你的想法去做。

哪个是正确的实现？

我更喜欢 ，Table::isUsable因为它也可以用作table.isUsable实例。

为什么选择一个而不是另一个？

我觉得第一个例子更自然，更容易混淆。

第二种形式对于操作谓词更有用，例如 Predicate.or(Predicate)

有什么大的区别吗？

在这种情况下，使用 Stream 可能会更慢，但更重要的是，更容易混淆。

返回 Predicate 的方法的一个优点是它可以添加到任何类中，例如由于某种原因您不能更改 Table。

Answer 2

有Predicate<Table> isUsable()假定你总是需要在需要的地方使用的是逻辑Predicate<Table>实例，这是一个限制。

另一方面， haveboolean isUsable()使您可以灵活地Table::isUsable在Predicate<Table>需要a的地方使用，或Table::isUsable用作其他一些功能接口的实现（与该方法的签名匹配）或直接调用t.isUsable()特定Table实例。因此我发现这个替代方案更有用。

Answer 3

static List<Predicate<Table>> predicateList = Arrays.asList(Table::hasChair, Table::isClean);

static boolean isUsable(Table table) {

    return predicateList.stream().allMatch(p -> p.test(table));

}

使用 isUsable：

List<Table> tablesList = ...

Stream<Table> usableTables = tablesList.stream().filter(Table::isUsable);