Java可选的方法参数？

Question

我读了一篇关于使用选项作为方法参数的文章，大家一致认为永远不要使用选项作为方法参数。

Guava可选作为可选参数的方法参数

这种使用optional作为方法参数的情况合法吗？如果不是，我如何使这是可列举的？

private static enum DbConnectionSupplier {
    ORACLE(optArg -> DbManager.getOracleConnection()),
    MYSQL(optArg -> DbManager.getMySQLConnection()),
    TERADATA(optArg -> DbManager.getTeradataConnection()),
    SQLITE(optArg -> DbManager.getSQLiteConnection(optArg.get()));

    private final Function<Optional<String>, DatabaseConnection> supplier;
    private DbConnectionSupplier(Function<Optional<String>, DatabaseConnection> supplier) { 
        this.supplier = supplier;
    }

}

Answer 1

首先，如果您使用的是lambda表达式，那么您将使用Java8进行编译，并且可以使用Java.util.optional，而不是Guava的。

我想你有以下几种方法：

DatabaseConnection getConnection(String arg) {
   return supplier.apply(Optional.of(arg));
}

DatabaseConnection getConnection() {
   return supplier.apply(Optional.empty());
}

现在，它并不能保护您免受某人调用Teradata.getConnection（“bar”）的影响。另一方面，sqlite.getConnection()将在optarg.get()处以NosuchelementException(Java8)或IllegalStateException(Guava)失败，因为可选项为空。从这个例外情况来看，并不清楚GetConnection中是否需要一个参数（也许很清楚，但调用Function.Apply会“污染”stacktrace）

下面的实现避免了optional的使用，并提供了明确的指示，说明参数是应该的还是不应该的。

TERADATA(()-> DbManager.getTeradataConnection()),
SQLITE(arg -> DbManager.getSQLiteConnection(arg)),
FOO(()-> DbManager.getFooConnection(),
    arg->DbManager.getFooConnection(arg));
//Foo supports both styles: with and without a parameter

private final Supplier<DatabaseConnection> supplier;
private final Function<String, DatabaseConnection> function;

private DbConnectionSupplier(Supplier<DatabaseConnection> supplier) { 
    this.supplier = supplier;
    this.function = null;
}    

private DbConnectionSupplier(Function<String, DatabaseConnection> function) { 
    this.function = function;
    this.supplier = null;
}    `

private DbConnectionSupplier(
    Supplier<DatabaseConnection> supplier,
    Function<String, DatabaseConnection> function) { 
    this.function = function;
    this.supplier = supplier;
}    

public DatabaseConnection getConnection(String arg) {
   if (function==null) throw ...
   return function.apply(arg);
}

public DatabaseConnection getConnection() {
   if (supplier==null) throw ...
   return supplier.get();
}