Question

给出一个具有两个类似方法的util类，它们的区别仅在于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，下一个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);
}

通过将BiConsumer<Some, T> setter的第二个参数绑定到由Consumer<Some> c返回的值，将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); }。 def alta_baja(val): color = 'black' if (val <= 10.0): color = 'green' elif (val >= 19.0): color = 'red' return 'color: %s' % color df.style.applymap(alta_baja, subset=['MEDIA'])功能。

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

{{1}}

我怎样才能更好地抽象？

2 个答案: