我怎样才能更好地抽象？

Question

给定一个实用程序 class，其中包含两个相似的方法，仅在 setter/getter 领域不同：

public static void method_A(Dbo dbo) {
    if (dbo instanceof Zdbo) {
      ((Zdbo)dbo)
          .getSome()
          .forEach(z -> z.setFieldX(dbo.getFieldX()));
    }
  }

public static void method_B(Dbo dbo) {
    if (dbo instanceof Zdbo) {
      ((Zdbo)dbo)
          .getSome()
          .forEach(z -> z.setFieldZ(dbo.getFieldZ()));
    }
  }

我的问题是：我怎样才能去掉重复的代码？

我的方法是实现这样的目标：

private static void xxx(Dbo dbo, Consumer c) {
    if (dbo instanceof Zdbo) {
      ((Zdbo)dbo).getSome().forEach(c);
    }
  }

Answer 1

public static Stream<Zdbo> getSome(Dbo dbo) {
    return dbo instanceof Zdbo ? ((Zdbo)dbo).getSome() : Stream.empty();
}

public static Optional<Zdbo> asZdbo(Dbo dbo) {
    return dbo instanceof Zdbo ? Optional.of((Zdbo)dbo) : Optional.empty();
}

public static void method_A(Dbo dbo) {
    getSome(dbo).forEach(z -> z.setFieldX(dbo.getFieldX()));
}

public static void method_B(Dbo dbo) {
    getSome(dbo).forEach(z -> z.setFieldZ(dbo.getFieldZ()));
}

我会将其保留在 Stream 或 Optional 级别。上面的 method_A 和 method_B 可以简单地用它们的内容替换，而不需要传递 setter 和 getter。而且它更通用，不会产生代码开销。

请注意，对于 instanceof+cast a next java 可能有更好的解决方案。

Answer 2

我会使用 Function<Dbo, T> 提取字段，并使用 BiConsumer<Some, T> 将字段设置为每个 Some 对象：

private static <T> void setField(
        Dbo dbo, 
        Function<? super Dbo, ? extends T> extractor, 
        BiConsumer<? super Some, ? super T> setter) {

    Consumer<Some> c = some -> setter.accept(some, extractor.apply(dbo));

    if (dbo instanceof Zdbo) ((Zdbo) dbo).getSome().forEach(c);
}

This adapts the BiConsumer<Some, T> setter to a Consumer<Some> c, by binding the 2nd argument of setter 到 extractor 函数返回的值。

然后，您可以按如下方式调用它：

public static void method_A(Dbo dbo) {
    setField(dbo, Some::getFieldX, Some::setFieldX);
}

public static void method_B(Dbo dbo) {
    setField(dbo, Some::getFieldY, Some::setFieldY);
}

我怎样才能更好地抽象？

How can I abstract better?

java

simplify

java-stream