Java 8 Streams：在流中简化 o1 -> Objects.equals(o1.getSome().getSomeOther(), o2.getSome().getSomeOther())

Question

给定以下代码：

stream.filter(o1 -> Objects.equals(o1.getSome().getSomeOther(),
                                   o2.getSome().getSomeOther())

这怎么可能简化？

是否有一些 equals-utility 可以让您首先提取密钥，就像 Comparator.comparing 接受密钥提取器功能一样？

请注意，代码本身 (getSome().getSomeOther()) 实际上是从模式生成的。

Answer 1

编辑：（与同事讨论并重新访问后：Is there a convenience method to create a Predicate that tests if a field equals a given value?）

我们现在来到了以下可重用的功能接口：

@FunctionalInterface
public interface Property<T, P> {

  P extract(T object);

  default Predicate<T> like(T example) {
     Predicate<P> equality = Predicate.isEqual(extract(example));
     return (value) -> equality.test(extract(value));
  }
}

以及以下静态便捷方法：

static <T, P> Property<T, P> property(Property<T, P> property) {
  return property;
}

过滤现在看起来像：

stream.filter(property(t -> t.getSome().getSomeOther()).like(o2))

相对于之前的解决方案，我喜欢这个解决方案：它清楚地将属性的提取和Predicate 本身的创建分开，并且更清楚地说明了正在发生的事情。

以前的解决方案：

<T, U> Predicate<T> isEqual(T other, Function<T, U> keyExtractFunction) {
  U otherKey = keyExtractFunction.apply(other);
  return t -> Objects.equals(keyExtractFunction.apply(t), otherKey);
}

导致以下用法：

stream.filter(isEqual(o2, t -> t.getSome().getSomeOther())

但如果有人有更好的解决方案，我会更高兴。

Answer 2

我认为您的问题的方法比您的答案更具可读性。而且我也认为使用内联 lambda 是可以的，只要 lambda 简单而简短。

但是，出于维护、可读性、调试和可测试性的原因，我总是将我在 lambda（谓词或函数）中使用的逻辑转移到一个或多个方法中。在你的情况下，我会这样做：

class YourObject {

    private Some some;

    public boolean matchesSomeOther(YourObject o2) {
        return this.getSome().matchesSomeOther(o2.getSome());
    }
}

class Some {

    private SomeOther someOther;

    public boolean matchesSomeOther(Some some2) {
        return Objects.isEqual(this.getSomeOther(), some2.getSomeOther());
    }
}

有了这些方法，你的谓词现在变得微不足道了：

YourClass o2 = ...;

stream.filter(o2::matchesSomeOther)