在方法参数中使用供应商而不是可选

Question

我有一个接口，其参数可以为空。

int count(@Nullable F filter);

为了避免空参数和空检查，通常的建议是使用方法重载:

int count();
int count(@NonNull F filter);

在某些情况下，我不喜欢这种方法，因为它迫使客户做这种工作：

F f = getFilter()
f != null ? count(f) : count()

客户端使用更方便：

count(getFilter())

初看起来，可选选项可以解决这个问题，但是“可选”不应该用于参数https://rules.sonarsource.com/java/RSPEC-3553

所以我的问题是：我可以用供应商来代替吗？

//service code:
int count(Supplier<F> filter) {
 Optional.ofNullable(filter.get()).map(f->count(f)).orElseGet(() -> count());
};
//client code:
count(() -> getFilter())

以至

int count(Supplier<Optional<F>> filter)

这个想法没有任何警告。

我个人很喜欢我之后的代码，但是这不是作弊吗？为什么它比可选的更好？

编辑:感谢评论。我同意，在我的例子中，供应商只是一个复杂的@Nullable方法。

我将返回：

int count();
int count(@NonNull F filter);
/////
getFilter().map(f->count(f)).orElseGet(() -> count());

Answer 1

如果不希望客户端代码以 null 进行拨号，则可以更改方法 getFilter（） 以返回 Optional

在这种情况下，客户端中的代码如下所示:

int count = getFilter().map(MyClass::count).orElse(MyClass.count());

使用这种方法，您的方法count（F）不需要任何额外的逻辑。

如果感觉太复杂，那么可以实现count（F），如果提供的参数为null，它将委托给count（）。

您对供应商的尝试