检查默认响应时避免代码重复

Question

我有一个调用外部API的Java程序（RealApi在下面的代码中），有时我想避免调用此API，而是返回预先构造的响应（由FakeApi生成）。

所以，我在大多数方法中都复制了这种构造：

public Type1 m1(String s) {
    try {
        Type1 r = FakeApi.m1(s);
        if (r != null) {
            return r;
        }
    } catch (Exception e) {
        // log error
    }

    return RealApi.m1(s);
}

有哪些选项可以避免在任何地方重复这个try/catch块？重要的是，如果FakeApi抛出异常或返回null，则必须调用RealApi。

Answer 1

这不像Thomas Preiúler的答案那么简单，但它会帮助你根本不重复任何方法。所以如果你扩展接口，你只需要修改具体的类，而不是描述你想要的实际行为的链接器。

创建一个包含RealApi所有方法的接口：

interface Api {
  Type1 m1(String s);
}

然后是一个执行实际调用的类：

class ConcreteApi implements Api {
  public Type1 m1(String s) {
    return RealApi.m1(s);
  }
}

然后创建你的FakeApi：

class TotallyFakeApi implements Api {
  public Type1 m1(String s) {
    return FakeApi.m1(s);
  }
}

现在，要避免重蹈覆辙，需要注意的一点是：

private static Object callImplementation(Api api, Method method, Object[] methodArgs) throws Exception {
  Method actualMethod = api.getClass().getMethod(actualMethod.getName(), actualMethod.getParameterTypes());
  return actualMethod.invoke(api, methodArgs);
}
Api fakeOrReal(Api fakeApi, Api realApi) {
  return (Api) Proxy.newProxyInstance(
      FakeApi.class.getClassLoader(),
      new Class[]{Api.class},
      (proxy, method, methodArgs) -> {
        try {
          Object r = callImplementation(fakeApi, method, methodArgs);
          if (r != null) {
            return r;
          }
        } catch (Exception e) {
          // logError(e);
        }
        return callImplementation(realApi, method, methodArgs);
      }
    );
  
}

获得如下实际实现：

Api apiToUse = fakeOrReal(new TotallyFakeApi(), new ConcreteApi());

Answer 2

一个选项是将错误检查行为封装到自己的方法中：

public <T> T fakeOrReal(Supplier<T> fake, Supplier<T> real) {
  try {
    T r = fake.get();
    if (r != null) {
      return r;
    }
  }
  catch (Exception e) {
    // log error
  }

  return real.get();
}

然后你就可以用

public Type1 m1(String s) {
  return fakeOrReal(() -> FakeApi.m1(s), () -> RealApi.m1(s));
}