1. 缺少非空判断

Missing nullability on method `***` return [MissingNullability]

修改:

import android.annotation.NonNull;

@NonNull String parameter

实现对参数或返回值的注解

参考:

frameworks/base/services/print/java/com/android/server/print/UserState.java

public Bundle print(@NonNull String printJobName, @NonNull IPrintDocumentAdapter adapter,

@Nullable PrintAttributes attributes, @NonNull String packageName, int appId)

2.异常处理错误

Methods calling system APIs should rethrow RemoteException as RuntimeException

RemoteException is typically thrown by internal AIDL, and indicates that the system process has died, or the app is trying to send too much data. In both cases, public API should rethrow as a RuntimeException to ensure that apps don’t accidentally persist security or policy decisions.

If you know the other side of a Binder call is the system process, this simple boilerplate code is the best-practice:

try {
    ...
} catch (RemoteException e) {
    throw e.rethrowFromSystemServer();
}

以上相关,可以参照以下链接的完整说明:

https://android.googlesource.com/platform/developers/docs/+/refs/heads/master/api-guidelines/index.md

Logo

为开发者提供学习成长、分享交流、生态实践、资源工具等服务,帮助开发者快速成长。

更多推荐