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

小编典典

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

java

最近，我与队友讨论了Guava Optional在方法中使用可选参数的问题。

假设方法是

List<Book> getBooks(String catalogId, Optional<String> categoryId) {
     Validate.notNull(catalogId);
     Validate.notNull(categoryId); // Point of conflict. Is this required?

接受一个catalogId和 可选参数 categoryId，它返回该目录中列出的书籍，如果还传递了类别，则仅返回该类别中的书籍。

冲突点是，验证Optional<String> categoryId是否为空检查。我认为不应对此进行空检查，因为它是可选参数。函数的调用者可以通过null或，Optional.<String>absent()并且getBooks函数应if(categoryId==null && categoryId.isPresent())在实现中处理两种情况。

我的意见是，Optional对于可选参数，只会使方法的约定更加明确。只需查看方法签名即可知道此参数是可选的，不需要读取javadocs。但是，Optional.absent()当他不想使用该可选参数时，不应强迫他通过。

我的队友有不同的看法。他想对此进行空检查，因此迫使呼叫者始终通过Optional.<String>absent()。他的观点是，为什么我们要传递一个null可选。而且getBooks("catalog123", Optional.absent())看起来比更具可读性getBooks("catalog123", null)。

此方法在我们的一个库软件包中，并且由我们拥有的多个软件包使用。

您Optional对此方案的用法有何建议？

谢谢

阅读 219

2020-10-25

共1个答案

小编典典

您对此方案的用法有何建议？

避免。避免。避免。

虽然Optional是一个 很好的替代 来null，在Java中，你会总是与它是一个结束了 糟糕的除了
代替。正如Seelenvirtuose所写，在三种可能性中，您只需要两种。就像JB
Nizet所写的那样，最好将它用作返回值，以使调用者想起所需的检查。作为方法参数，它没有任何帮助。

理想情况下，可选参数应该像

getBooks(String catalogId, String categoryId = null)

这不是有效的Java。AFAIK C ++编译器将其转换为两种方法

getBooks(String catalogId, String categoryId)
getBooks(String catalogId)

您必须自己用Java编写。省略参数是明确表明它是可选的最清晰的方法。将可选参数标记@Nullable为几乎是一样的。使用可空性检查工具可以帮助您避免使用NPE（这是的主要参数Optional）。

重要的是一致性。您的班级之间的一致性就在您手中。与JDK的一致性意味着null至少在可预见的将来（Optional可能有一天JDK 8
会使用）使用。

2020-10-25