Java中判断错误回调是否可用：用null比较还是instanceof？

方案1（null比较）：更推荐的标准写法 #

这是Java里处理可空回调参数的常规且正确的做法。
因为构造方法参数onError的编译时类型已经明确是Consumer<Throwable>，只要它不是null，就一定是合法的Consumer实例，可以安全调用accept方法。这种写法逻辑清晰，所有Java开发者都能一眼看懂，完全符合代码可读性和维护性的要求，长期来看不会有歧义或冗余问题。

方案2（instanceof判断）：冗余且不合理 #

这个方案存在明显的逻辑冗余：既然参数类型已经声明为Consumer<Throwable>，那么只要参数不为null，它必然是Consumer的实例，instanceof判断完全是多余的。
虽然null instanceof 任何类型都会返回false，看起来能实现“null时跳过”的效果，但这种写法会让后续维护的开发者产生困惑——为什么要对一个已经明确类型的参数做类型检查？是不是有什么特殊的隐式逻辑？反而增加了理解成本，不利于长期维护。

其他优化方案：用Optional简化代码 #

如果你的项目使用Java 8及以上版本，可以用Optional来简化null判断逻辑，代码更简洁优雅：

Optional.ofNullable(onError).ifPresent(c -> c.accept(err));

Optional.ofNullable会把null包装成空的Optional，ifPresent方法只有在存在非null值时才会执行传入的lambda表达式，完美实现“非null时调用回调”的需求，同时代码更简洁，符合现代Java的编程风格。

内容的提问来源于stack exchange，提问作者carrj-nm