Question

我使用注释来确保参数不为空，假设这会导致编译器检查。

public @Nullable ApplicationAccount accountForKey(@NonNull String key) {

    return accounts.get(key);
}

然而，通过运行此代码，我在此行上完全得到NullPointerException

java.util.concurrent.ConcurrentHashMap.get (ConcurrentHashMap.java:883)

那么注释有什么意义呢？

如果我像这样编写额外的支票，那就更加模糊了

return key!=null?accounts.get(key):null;

Android Studio警告我，检查无用！

更新：完整调用堆栈：

Caused by java.lang.NullPointerException
       at java.util.concurrent.ConcurrentHashMap.get(ConcurrentHashMap.java:883)
       at co.getcontrol.services.MerchantCenter.accountForKey(MerchantCenter.java:72)
       at co.getcontrol.model.customers.CustomersAggregator.loadCustomerDetails(CustomersAggregator.java:91)
       at co.getcontrol.model.customers.CustomerDetailsPresenter.callData(CustomerDetailsPresenter.java:39)
       at co.getcontrol.reskin.ui.customers.CustomerDetailsViewFragment.onCreateView(CustomerDetailsViewFragment.java:152)
       at android.support.v4.app.Fragment.performCreateView(Fragment.java:1974)
       at android.support.v4.app.FragmentManagerImpl.moveToState(FragmentManager.java:1067)
       at android.support.v4.app.FragmentManagerImpl.moveToState(FragmentManager.java:1252)
       at android.support.v4.app.BackStackRecord.run(BackStackRecord.java:738)
       at android.support.v4.app.FragmentManagerImpl.execPendingActions(FragmentManager.java:1617)
       at android.support.v4.app.FragmentController.execPendingActions(FragmentController.java:339)
       at android.support.v4.app.FragmentActivity.onStart(FragmentActivity.java:602)
       at co.getcontrol.ui.ControlActivity.onStart(ControlActivity.java:13)
       at android.app.Instrumentation.callActivityOnStart(Instrumentation.java:1174)
       at android.app.Activity.performStart(Activity.java:5353)
       at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2352)
       at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2441)
       at android.app.ActivityThread.access$900(ActivityThread.java:151)
       at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1354)
       at android.os.Handler.dispatchMessage(Handler.java:110)
       at android.os.Looper.loop(Looper.java:193)
       at android.app.ActivityThread.main(ActivityThread.java:5345)
       at java.lang.reflect.Method.invokeNative(Method.java)
       at java.lang.reflect.Method.invoke(Method.java:515)
       at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:828)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:644)
       at dalvik.system.NativeStart.main(NativeStart.java)

Answer 1

注释会创建合同。 @NonNull表示此方法不接受null并且传递它可能会导致程序崩溃（这正是发生的事情）。 Android Studio会警告此方法的任何用法，因为它可以推断出null可能会被传递。但它不会阻止传递null。

非常相似的契约是[]的参数，对于一个不能超出其边界（例如小于0）的数组，但仍然不会阻止开发人员传递超出边界的值。

对null的检查标记为多余，因为合同中永远不应该有null。如果您添加检查，该方法现在将知道如何处理null，因此不应将其标记为@NonNull。

Answer 2

考虑使用Lombok library与Android Studio plugin同时支持两种方法：在方法开始时使用@NonNull lombok注释注释参数时的警告和运行时异常。

看起来像

import lombok.NonNull;

public class NonNullExample extends Something {
    private String name;

    public NonNullExample(@NonNull Person person) {
        super("Hello");
        this.name = person.getName();
    }
}

与box-java实现相比：

public class NonNullExample extends Something {
   private String name;

    public NonNullExample(@NonNull Person person) {
        super("Hello");
        if (person == null) {
            throw new NullPointerException("person");
        }
        this.name = person.getName();
    }
}

Answer 3

如果您注释@NonNull，则表示输入不是null。它不会阻止愚蠢的开发人员在那里传递null。

我会考虑添加guava checkNonNull：

public @Nullable ApplicationAccount accountForKey(@NonNull String key) {
    Preconditions.checkNotNull(key, "Illegal Argument passed: key is Null.");
    return accounts.get(key);
}

这会阻止您成功将null传递给accounts#get方法。

在您的情况下，accounts似乎是null。调查一下！

Android：NullPointerException尽管@NonNull

3 个答案: