处理 Objects.requireNonNull() 抛出的 NullPointerException 的最佳方法是什么？ [复制]

Question

这种方法看起来非常棒，但是，当将它与生产代码一起使用并将它用于可能为空的对象时，它看起来很笨拙。考虑我的例子：

public class ShowDialogTask extends AsyncTask<Void, Void, Void> {
    private WeakReference<Context> contextReference;

    public ShowDialogTask(Context context) {
        contextReference = new WeakReference<>(context);
    }

    @Override
    protected Void doInBackground(Void... voids) {
        ...do a long running task...
        return null;
    }

    @Override
    protected void onPostExecute(Void void) {
        super.onPostExecute(void);
        Context ctx = Objects.requireNonNull(contextReference.get(), "Context became null");
        AlertDialog.Builder builder = new AlertDialog.Builder(ctx);
        builder.setTitle("Dialog title");
        builder.setCancelable(true);
        builder.create().show();
    }

在 onPostExecute() 函数中，我使用 Objects.requireNonNull 方法设置了一个局部变量对象 ctx。我的问题是，让 contextReference.get() 等于 null 对我来说有点难以重现，而且它绝对有可能在生产中发生。

我想知道将此功能用于生产目的的最佳方式。

我的第一个想法是将代码包装在 try-catch 中，但在任何地方都这样做似乎是糟糕的编程：

try {
    Context ctx = Objects.requireNonNull(contextReference.get(), "Context became null");
    AlertDialog.Builder builder = new AlertDialog.Builder(ctx);
    builder.setTitle("Dialog title");
    builder.setCancelable(true);
    builder.create().show();
} catch (Throwable t) {
    Log.d(TAG, t.getMessage());
}

Answer 1

最好的处理方法是什么？一开始就不要扔。

捕获NullPointerException 是您几乎不应该做的事情，除非您正在与编写不佳的 API 进行交互。 NullPointerException 是一个未经检查的异常，意思是它表示一个编程错误：最好修复这个编程错误。

使用条件检查事物是否为空，如果是，则执行不同的操作。

Context ctx = contextReference.get();
if (ctx != null) {
  // Use ctx.
} else {
  Log.d(TAG, "Context became null");
}

---

总的来说，捕捉Throwable几乎永远不会是正确的做法，因为你可能会捕捉和处理你不打算做的事情，导致你吞咽问题你不知道你必须处理。