Android Lint扫描优化(一)

一.使用Lint

工具栏 -> Analyze -> Inspect Code

弹出窗的Custom scope可以选择对哪一指定部分进行扫描,whole project是整个项目,不太建议每次都扫描整个项目,因为太耗时了。

二.观察Lint的结果

初次扫描之后,会弹出一个显示扫描结果的窗口,比如我的扫描结果:


1万7个警告,尼玛我写的代码就这么糟糕吗,我这感觉要改到下个月都改不完啊,看到这么多的警告,先不要慌,首先要做的就是删除项目重新做,开玩笑的,先找下资料看看每一栏目录表示什么。

1. Accessibility意思是便利性的意思

具体什么意思我也不知道,然后点开发现都是这个提示



这些都是针对ImageView和ImageButton的,说我缺少contentDescription这个参数,这属性是啥我也不知道,以前一直没用过,然后百度查看看。


瞬间觉得很屌,这些都是我平时开发没有考虑过的,虽然这个警告对我来说影响不大,要用的话可以按教程来做,这里我就暂时先搞空的。

而这个也验证了“Accessibility”这个的含义,主要是要你提供一些遍历的警告。

优化优先级很低,我觉得可以直接忽略

2. Correctness 这是正确性的意思

也看看是哪里有问题才能推出这个目录具体的分类。点开后发现有很多栏



其实如果写android比较多的话,就算英语不太好也大概知道是什么意思,比如第一个就是说Class没被注册到manifest中,那这个Class基本肯定是一个Activity

我发现是我的一个基类没注册,不知道为什么这会提示这个BUG。这个先不管,看我第2条,它报这个警告



这个就有意思了,虽然目前没出错,但是按照它的说法去做肯定更安全,这也体现Lint的优化的好处,我们肯定是尽量消除警告的好,虽然有些警告并不影响运行的结果,但是消除这些警告肯定会更为安全些。

然后改了一下其它的,发现这个目录就是说你用的某个地方不太好,比如说版本旧,比如说多写了某些内容,比如说让你在设置字体大小时用sp不要用dp,它就会给你一些合理的修改方法,其实我并没有全部改完,就改了一半左右。

3.Internationalization 意思是国际化

我想它应该是一个标准化的意思。



我这里有3个,第一个其实就是我在xml中直接写中文,它建议我用@String写到资源文件中。老铁,道理我都懂,我这378个地方,我要改要画时间啊,所以先把这个优先级降低,再看看第二个。

第二个Overlapping items in RelativeLayout意思是RelativeLayout的Item重叠,我结合了一下我的代码,它的意思就是说RelativeLayout中的元素可以会相互覆盖,比如说我这样的布局

<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="match_parent"
    android:layout_height="48dp"
    android:padding="10dp"
    >
    <TextView
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_centerVertical="true"
        android:id="@+id/tv_time"
        />
    <TextView
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_centerVertical="true"
        android:id="@+id/tv_order_num"
        android:layout_alignParentRight="true"
        />
</RelativeLayout>

一个在左一个在右,但是如果右边的TextView太长,会出现覆盖左边TextView。这个确实是一个写布局时大意的地方。
要防止这个事情发生,需要在第二个TextView加上

android:layout_toRightOf="@+id/tv_time"
android:gravity="right"

第三个是TextView连接显示的问题,就是我这样写

tvMoney.setText("¥" + data.act.discountAmount);

是不建议的,而是建议使用占位符,那什么是占位符呢,就是用@String文件来做,比如我这样

<string name="fetch_again">重新获取(%1$d秒)</string>

总的来说,这个目录就是建议你使用一些国际化的规范写法。总的来说优先级也不是很高,但是有时间的话还是建议改下,毕竟按规范写准没错。

4.Performance 性能的意思

可想而知这条目录的重要性和优先级肯定高。点开目录


我有差不多300个性能问题,看着都想哭。
点开每个目录,它有会有详细的英文解释,很容易就能看懂和推出哪里问题,看不懂百度翻译。
比如说第一条



它大概是说我这个 layout_width用0dp好过用match_parent,因为我用了layout_weight属性。说真的我不知道影响多大,但这里说了是性能相关,那么最好还是改了。

再比如说第二条,是因为我在自定义View的绘制过程中创建对象,所以它建议我不要这么做,确实,因为这些方法会经常被调用,在这些方法里面做创建对象的操作会创建出来一大堆对象,于是我把

lineViews = new ArrayList();

改成

lineViews.clear();

并把创建对象的操作写在外面,这个确实是我之前没那么认真注意到的一个问题。
其它的就不一一说了,那些解释都说得很清楚,主要是想说明这个Performance 的优先级很高,最好把里边涉及到的问题都改了。

然后我就再说说一些我这边会影响性能的细节
(1)嵌套权重会影响性能


(2)设置背景时要注意overdraw,意思就是说这个的上一层背景已经设了这个背景,而这个也设相同的,就造成了overdraw

(3)注意一下这项,是关于Context内存泄漏的


(4)未使用到的资源会影响性能,不过我不建议马上删除,我觉得在发布前删除比较好。


(5)无用布局不一定要无脑删



这个是说没用到的布局,但是有个地方很奇怪,就是我用了ScrollView , 虽然当前内容够一屏幕,我是为了防止不同机型而设置的,它也提示我这是多余的布局。

我目前碰到的就这些,不要看我性能这块有300个警告,其实有200多个都是多余的资源文件,真正严重的就十几二十个。

5.Security 安全的意思

我看了一下,警告的都是我拷贝的一些网上的代码,暂时看不懂是什么问题。

6.Usability 复用/适用 的意思

第一条是说我的Button很多相同,可以用style,这个优先级并不是这么高,第二个说我input类型和提示有误,这个我倒觉得不用管,第三个不是我写的代码,我不背锅,第四个说我Text的尺寸太小了


说尺寸最好不要小于12sp,我只能说,太良心了,哥们你这都要警告我。

7.issues

有Lint标签的都看完了,发现还有很多是issues 标签的,我随便抽了一些出来看,也没看出有什么。

8.Imports

这个很容易理解,就是你在类中导入没有用到的资源,这个我觉得还是要处理一下好。

这里有个技巧,点击目录右键,选择Optimize Imports就能快速把整个项目多余的Imports给删除。


9.Declaration redundancy

这个是声明冗余的意思。可以按照它的说明去更改,优先级的话我觉得一般,不是很高。

10.Spelling

这个是拼写,比较有意思的东西,你的命名不规范的话在单词下面都会画个灰色的波浪线。然而完全没影响,除了对强迫症的来说。

11.probable bugs

这个就是说你的代码中可能会出现BUG的地方,我认为这个目录的优先级很高,做好按照它的说明进行更改,比如我这个。


它就说我调用这个方法可能会产生空指针,所以我就必须在这块地方加个判断。

就暂时看这么多吧,我觉得重点就是Performance和Correctness ,然后开发时要注意一下国际化和拼写的问题,可能之前开发没太注意加上项目功能挺多,导致现在警告好多。

但是在使用过程中也发现Lint的功能挺强大的,能反应出很多代码中的问题,有助于开发,而且也不是很难,建议可以抽点时间优化下代码。

推荐阅读更多精彩内容