4

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

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)
4

3 回答 3

7

注释创建一个合同。@NonNull表示此方法不接受null并传递它可能会使程序崩溃（这正是发生的情况）。Android Studio 将警告该方法的任何使用，它可以推断null可能通过。但它不会阻止通过null.

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

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

于 2016-08-12T09:38:46.507 回答
1

如果您注释@NonNull您声明，则输入将不是null. 它不会阻止愚蠢的开发人员在那里传递一个空值。

我会考虑添加guavacheckNonNull 代替：

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

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

在您的情况下，它看起来像您的accountsis null。调查那个！

于 2016-08-12T09:09:48.067 回答
1

考虑将Lombok 库与Android Studio 插件一起使用以实现对以下两者的支持：方法开始时的警告和运行时异常，只需使用带有@NonNulllombok 注释的注释参数。

它看起来像 

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();
    }
}
于 2016-11-09T10:15:58.620 回答