java - 可以使用Optional作为捕获ConversionException的替代方法

Question

如何重构以下读取属性文件的代码，使其根据读取的值返回 int、double 或 String？

    public static <T> T readFromConfig(String keyName) {
    PropertiesConfiguration config = new PropertiesConfiguration();
    String propertiesFilePath = "src/main/resources/application.properties";
    try {
        config.load(propertiesFilePath);
        try {
            Integer value = config.getInt(keyName);
            return (T) value;
        } catch (ConversionException notInteger) {
            try {
                Double value = config.getDouble(keyName);
                return (T) value;
            } catch (ConversionException notDouble) {
                return (T) config.getString(keyName);
            }
        }
    } catch (ConfigurationException e) {
        logger.warn("Could not parse " + propertiesFilePath);
        return (T) "";
    }
}

Answer 1

正如作者自己所想:Optional<>在这里不是一个选项，因为，正如另一个答案所示:它将导致返回 Optional<Object>这提供了更少的类型信息。

但老实说，从干净的代码角度来看，即使是

public static <T> T readFromConfig(String keyName) {

有点有缺陷。这个方法能买到什么？ 什么都没有。因为调用者说:我希望返回一个 Integer，但你推回了一个 Double 甚至 String。你看，编译器被告知“该方法应该返回 Integer，或 Double，...”，然后它看到:“是的，可能”。但这与运行时发生的事情完全脱钩。

如果你去:

Integer intVal = readFromConfig("keyPointingToDoubleValue");

编译器不会提示。因为它看到:您想要一个 Integer；嘿，该方法可以返回一个整数。

运行时？当检索到的值不是 Integer 时，将返回 Double 或 String。不知道这里会发生什么(类转换异常，或者可能是一些堆栈冲突)。但它不应该在运行时工作。

所以，真正的解决方案是这样的:

要么你有多种方法，例如:

public static Integer readIntegerFromConfig(String keyName) throws SomeException ...
public static Integer readIntegerFromConfig(String keyName, Integer Default) throws SomeException ...

或者也许:

public static Object readFromConfig(String keyName) {

或

public static <T> T readFromConfig(String keyName, T default)

换句话说:您需要一个 API，允许用户真正说出他们想要的内容，并始终为他们提供他们想要的内容。或者您完全避免该级别上的不同类型，并返回字符串，并让客户端代码进行转换。

如前所述，您当前的方法:不会为您带来任何好处，但代价是误导性的、复杂的 API。