作为移动应用开发者,我们总希望发布的 apk 文件越小越好,不希望资源文件没有用到的图片资源也被打包进 apk,不希望应用中使用了高于 minSdk 的 api,也不希望 AndroidManifest 文件存在异常,lint 就能解决我们的这些问题。Android lint 是在 ADT 16 提供的新工具,它是一个代码扫描工具,能够帮助我们识别代码结构存在的问题,主要包括:
1)布局性能(以前是 layoutopt 工具,可以解决无用布局、嵌套太多、布局太多) 2)未使用到资源 3)不一致的数组大小 4)国际化问题(硬编码) 5)图标的问题(重复的图标,错误的大小) 6)可用性问题(如不指定的文本字段的输入型) 7)manifest 文件的错误
开始使用Android Lint 的工作过程比较简单,一个基础的 Lint 过程由 Lint Tool(检测工具),Source Files(项目源文件) 和 lint.xml(配置文件) 三个部分组成,Lint Tool 读取 Source Files,根据 lint.xml 配置的规则(issue)输出结果(如下图)。
步骤:将鼠标放在代码区右键->Analyze->Inspect Code–>界面选择你要检测的模块->点击确认开始检测,如下图:
之后可以双击进行代码修改、优化啦!很方便。
Android Lint 对检查的结果进行了分类,同一个规则(issue)下的问题会聚合,其中针对 Android 的规则类别会在分类前说明是 Android 相关的,主要是六类:
- Accessibility 无障碍,例如 ImageView 缺少 contentDescription 描述,String 编码字符串等问题。
- Correctness 正确性,例如 xml 中使用了不正确的属性值,Java 代码中直接使用了超过最低 SDK 要求的 API 等。
- Internationalization 国际化,如字符缺少翻译等问题。
- Performance 性能,例如在
onMeasure
、onDraw
中执行new
,内存泄露,产生了冗余的资源,xml
结构冗余等。 - Security 安全性,例如没有使用 HTTPS 连接 Gradle,AndroidManifest 中的权限问题等。
- Usability 易用性,例如缺少某些倍数的切图,重复图标等。
我们在使用 Android Lint 对项目进行检查后,整理了一些问题及解决方法,下面列举较为常见的场景:
-
ScrollView size validation 在 ScrollView 的第一层子元素中设置了高度为
match_parent
,这是错误的写法,实际上在 measure 时这里必定会被当作wrap_content
去处理,因此按照 Lint 的建议,直接改为wrap_content
即可。 -
Handler reference leaks Handler 引用的内存泄露问题,例如下面的例子:
protected static final int STOP = 0x10000;
protected static final int NEXT = 0x10001;
@BindView(R.id.rectProgressBar) QMUIProgressBar mRectProgressBar;
@BindView(R.id.circleProgressBar) QMUIProgressBar mCircleProgressBar;
int count;
private Handler myHandler = new Handler() {
@Override
public void handleMessage(Message msg) {
super.handleMessage(msg);
switch (msg.what) {
case STOP:
break;
case NEXT:
if (!Thread.currentThread().isInterrupted()) {
mRectProgressBar.setProgress(count);
mCircleProgressBar.setProgress(count);
}
}
}
};
首先非静态的内部类或者匿名类会隐式的持有其外部类的引用,内部类使用了外部类的方法/成员变量也会导致其持有外部类引用,因此上面这种情况会导致 handler 持有了外部类,外部类同时持有 handler,handler 是异步的,当 handler 的消息发送出去后,外部类因 hanlder 的持有而无法销毁,最终导致内存泄露。
解决办法则是把该内部类改为 static,内部类中使用的外部类方法/成员变量改为弱引用,具体如下:
@BindView(R.id.rectProgressBar) QMUIProgressBar mRectProgressBar;
@BindView(R.id.circleProgressBar) QMUIProgressBar mCircleProgressBar;
int count;
private ProgressHandler myHandler = new ProgressHandler();
@Override
protected View onCreateView() {
myHandler.setProgressBar(mRectProgressBar, mCircleProgressBar);
}
private static class ProgressHandler extends Handler {
private WeakReference weakRectProgressBar;
private WeakReference weakCircleProgressBar;
public void setProgressBar(QMUIProgressBar rectProgressBar, QMUIProgressBar circleProgressBar) {
weakRectProgressBar = new WeakReference(rectProgressBar);
weakCircleProgressBar = new WeakReference(circleProgressBar);
}
@Override
public void handleMessage(Message msg) {
super.handleMessage(msg);
switch (msg.what) {
case STOP:
break;
case NEXT:
if (!Thread.currentThread().isInterrupted()) {
if (weakRectProgressBar.get() != null && weakCircleProgressBar.get() != null) {
weakRectProgressBar.get().setProgress(msg.arg1);
weakCircleProgressBar.get().setProgress(msg.arg1);
}
}
}
}
}
-
Memory allocations within drawing code onMeasure、onDraw 都是被频繁调用的方法,因此 Lint 不建议在其中执行 new 操作,可以在 onCreateView 等非频繁调用的时机进行 new 操作,并用成员变量保存,再在 onMeasure 中使用成员变量。
-
‘private’ method declared ‘final’
private static final void addLinkMovementMethod(TextView t) {
MovementMethod m = t.getMovementMethod();
// ...
}
如上面的示例代码,会产生 ‘private’ method declared ‘final’ 的警告,因为私有方法是不会被 override 的,因此完全没有必要声明 final
。