Android Lint扫描优化(一)-阿里云开发者社区

开发者社区> kylinarm> 正文

Android Lint扫描优化(一)

简介: 一.使用Lint 工具栏 -> Analyze -> Inspect Code 弹出窗的Custom scope可以选择对哪一指定部分进行扫描,whole project是整个项目,不太建议每次都扫描整个项目,因为太耗时了。
+关注继续查看

一.使用Lint

工具栏 -> Analyze -> Inspect Code

img_003413e42a8470804980f12b46a839ea.png

img_6ea33aee83af0b76ed425efe1c05f992.png

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

二.观察Lint的结果

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


img_8e3a65507813904c1ceaa98f2db4ffbb.png

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

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

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


img_d0d4a6998a8db0d295ca5f4ee84c706f.png

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


img_f6246e00a2c17edbb4d19d27af321581.png

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

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

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

2. Correctness 这是正确性的意思

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


img_121d9e443c7c60b6b94b5818a8f0e6e6.png

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

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


img_97852654f4f1adcdcd99381a9d6a6b40.png

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

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

3.Internationalization 意思是国际化

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


img_171a741e229680d75e902b153595aa83.png

我这里有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 性能的意思

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


img_c98b70aa51337284b58f3604b9a1bd70.png

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


img_88166850d7a1168cf7917e265050e111.png

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

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

lineViews = new ArrayList();

改成

lineViews.clear();

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

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


img_527a62a694eb4d08694f4c31bce0307b.png

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

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


img_7bc11094998636a974c1d17b0cd88cba.png

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


img_8c08fdc7723f6b431b6e00e6b78b7cd8.png

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


img_25f0d2c58921aaebed03d9d927fac87a.png

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

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

5.Security 安全的意思
img_292bb0ddfa627536bf731ae612ce04a3.png

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

6.Usability 复用/适用 的意思
img_401aea2316935674095bd2e5d4710d8e.png

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


img_9f7a89622384750668203b2b61b6776e.png

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

7.issues

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

8.Imports

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

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


img_34e886e466921320c8fffed6ff98982d.png
9.Declaration redundancy

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

10.Spelling

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

11.probable bugs

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

img_35ba8dd20655117cad5b73f53c87efaf.png

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

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

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

版权声明:本文内容由阿里云实名注册用户自发贡献,版权归原作者所有,阿里云开发者社区不拥有其著作权,亦不承担相应法律责任。具体规则请查看《阿里云开发者社区用户服务协议》和《阿里云开发者社区知识产权保护指引》。如果您发现本社区中有涉嫌抄袭的内容,填写侵权投诉表单进行举报,一经查实,本社区将立刻删除涉嫌侵权内容。

相关文章
WIN10的WIFI不能自动链接?
WIN10的WIFI不能自动链接?
4 0
云原生时代的"应用级"多云管理
当前云计算有多种形态公有云、私有云、边缘云、虚拟机等,如何高效管理多云是当前面临的问题,在云原生时代,又该如何利用云原生技术实现多云管理?本文将讲解通过 Rainbond 实现“应用级”多云管理。
5 0
美国禁止绑定应用商店内购支付:苹果App Store世纪审判,Epic赢了
继韩国之后,美国也表明了自己的态度。以后在 iPhone 端玩游戏,内购项目可以走其他支付渠道了。
3 0
抢先报名丨2021阿里云弹性计算年度峰会将于12月21日在上海举办,五大看点不容错过
结合企业上云大势和云计算的技术红利浪潮,本次峰会将在前沿技术、产品生态、上云实践等方面带来最新的业态解读与创新实践分享,弹性计算五大新品亮相、“一云多态”最佳实践展示、计算巢最佳实践分享、CloudOps运维新趋势解读等看点值得期待。
4 0
搭建一个tomcat的JSP服务器环境
搭建一个tomcat的JSP服务器环境
3 0
tomcat启动时报错ports are invalid,默认端口不要为-1
tomcat启动时报错ports are invalid,默认端口不要为-1
8 0
eclipse多余的Workspace如何删除
eclipse多余的Workspace如何删除
6 0
eclipse中默认编码设置为UTF-8
eclipse中默认编码设置为UTF-8
5 0
参数量仅为原来1%,北邮等利用超分算法提出高性能视频传输方法
来自北京邮电大学和英特尔中国研究院的研究团队创新性地利用超分辩率算法定义了网络视频传输任务,减小了网络视频传输的带宽压力。
3 0
华人学生团队获国际神经网络验证大赛佳绩:总分第一,五大单项第一
由来自卡内基梅隆大学、美国东北大学、哥伦比亚大学、加州大学洛杉矶分校的成员共同开发的工具α,β-CROWN 获得了第二届国际神经网络验证大赛总分第一,以及 5 个单项第一!其中该团队的学生作者均为华人。
4 0
+关注
kylinarm
搬砖于造轮
43
文章
2
问答
文章排行榜
最热
最新
相关电子书
更多
文娱运维技术
立即下载
《SaaS模式云原生数据仓库应用场景实践》
立即下载
《看见新力量:二》电子书
立即下载