Guava可选作为可选参数的方法参数
问题内容:
最近,我与队友讨论了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
对此方案的用法有何建议?
谢谢
问题答案:
您对此方案的用法有何建议?
避免。避免。避免。
虽然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
会使用)使用。