Android开发中必备的代码Review清单

总结Android开发中必备的代码Review清单

前言

本文收集了我自己工作以来提交代码前的所有检查点。事实证明，这样能有效提高自己的代码质量和功能的稳定性。所以推荐大家以后每次提交代码前，都可以看下这份Review清单哈。

此外，可能还有些检查点我并没有发现，欢迎大家踊跃在评论区补充哈～

清理操作

  1. 页面退出时，是否完成必要的清理操作

    1. 是否调用Handler的removeCallbacksAndMessages(null)来清空Handler里的消息；
    2. 是否取消了还没完成的请求；
    3. 在页面里注册的监听，是否反注册；
    4. 假如自己用到观察者模式，是否反注册；
    5. 假如用了RxJava的话，是否解除订阅；

  2. 数据库的游标是否已经关闭
    这个点一般人都知道，出问题一般在于，没有考虑到多线程并发时的情况下，Cursor没有被释放。
    所以数据库的操作需要加上同步代码块
    详细可参考：http://www.2cto.com/kf/201408/329574.html

  3. 打开过的文件流是否关闭

  4. Android 3.0以下的版本，使用完的Bitmap是否调用recycle()，否则会一直占用内存
    而Android 3.0及以上的版本不需要调用recycle()，因为这些版本的Bitmap全部放到虚拟机的堆内存中，让GC自动回收。

  5. WebView使用完是否调用了其destory()函数

是否能进一步优化自己的代码

  1. 保存在内存中的图片，是否做过压缩处理再保存在内存里
    否则可能由于图片质量太高，导致OOM

  2. Intent传递的数据太大，会导致页面跳转过慢。太大的数据可以通过持久化的形式传递，例如读写文件

  3. 频繁地操作同一个文件或者执行同一个数据库操作，是否考虑把它用静态变量或者局部变量的形式缓存在内存里。用空间换时间

  4. 放在主页面的控件，是否可以考虑用ViewStub来优化启动速度

要小心第三方包

  1. build.gradle远程依赖第三方包时，版本号建议写死，不要使用+号
    避免由于新版本的第三方包引入了新的问题

  2. 导入第三方工程时，记得把编码转换成自己工程当前是用的编码

  3. 调用第三方的包或者JDK的方法时，要跳进他们的源码，看要不要加 try-catch
    否则可能会导致自己应用的崩溃

  4. 使用第三方包时，是否加上其混淆规则
    若漏掉加上第三方包的混淆规则，会导致第三方包不该混淆的代码被混淆。在Debug版本没有发现问题，但是Release版本就会出现问题

  5. 系统应用添加so时，是否在固件对应的Android.mk文件上加入新增的so，否则系统可能编译不过

@lib/armeabi/libcommon.so \
@lib/armeabi/libabcdefg.so \

注意要成对出现的地方

  1. 系统的、自己写的，注册和反注册的方法，是否成对出现

  2. 在生命周期的回调里，创建和销毁的代码是否对应起来
    比如：onCreate()里面创建了Adapter，那么对应Adapter的退出处理操作(比如清空Image缓存)，一般就要写在onDestory()，而不能写在onDestoryView()。

类似的生命周期对应的代码有：
onStart()、onStop();
onCreate()、onDestory();
onResume()、onPause();
onCreateView()、onDestoryView()

  1. 若ListView的item复用了，对Item里View的操作是否成对出现
    比如：
switch (type) {case ArticleListItem.TYPE_AD:......mTitleView.setText(tencentAdBean.title);mGreenLabelView.setVisibility(VISIBLE);mRedLabelView.setText("");mRedLabelView.setVisibility(GONE);break;case ArticleListItem.TYPE_ARTICLE:......mTitleView.setText(mzAdBean.adData.getTitle());mGreenLabelView.setVisibility(GONE);mRedLabelView.setText("ABC");mRedLabelView.setVisibility(VISIBLE);break;
}

比如以上对mTitleView、mGreenLabelView和mRedLabelView的操作，都是成对出现。否则ListView可能会由于Item复用，导致Item显示错乱问题

防内存泄漏

  1. 内部类，比如Handler、Listener、Callback是否是成static class
    因为非静态内部类会持有外部类的引用。

  2. 假如子线程持有了Activity，要用弱引用来持有
    比如Request的Activity就应该用弱引用的形式，防止内存泄漏。

  3. 要求传入Activity作为参数的函数，是否可以改用getApplicationContext()来作为参数

Handler相关

  1. 使用View.post()是否会有问题
    因为在View处于detached状态期间，post()里面的Runnable是不会被执行的。只有在此View处于attached状态时才会被执行。

如果想改Runnable每次肯定会被执行，那么应该是用Handler.post来替代

  1. 假如程序可能多次在同一个Handler里post同一个Runnable，每次post之前都应该先清空这个Handler中还没执行的该Runnable
    如：
if (mCloudRun != null) {mHandler.removeCallbacks(mCloudRun);mCloudRun = null;
}
mCloudRun = new Runnable() {@Overridepublic void run() {CloudAccelerateSwitchRequest request = new CloudAccelerateSwitchRequest();request.setPriority(RequestTask.PRIORITY_LOW);RequestQueue.getInstance().addRequest(request);}
};
mHandler.post(mCloudRun);

其他

  1. 多思考某些情况下，某变量是否会为空
    而且在函数体内，处理参数前，必须加上判空语句

  2. 回调函数是否处理好
    回调函数很容易出问题。比如网络请求的回调，需要判断此时的Aciivity等是否还存在，再进行调用。因为异步操作回来，Activity可能就消失不存在了。
    而且还要对一些可能被回收的变量进行判空。

  3. 修改数据库后，是否把数据库的版本号+1

  4. 启动第三方的Activity时，是否判断了该Intent能否被解析

Intent sendIntent = new Intent(mContext, Demo.class);
// 这种方式判断是否存在
if (sendIntent.resolveActivity(getPackageManager()) != null) {startActivity(sendIntent);
}

若Activity不存在，会出现ActivityNotFoundException的异常
5. 新注册的Activity、Service或Provider，若AndroidManifest.xml中exported属性为true，要考虑是否会引发安全性问题

<activity android:name="com.inkenka.DemoActivity"android:exported="true"/>

因为exported属性为true时，外部应用就可以直接调用起该Activity。
可能导致的问题：

  • 若外部应用直接启动详情页，从而让某些验证页面直接被绕过
  • 若外部应用给该Activity传递乱七八糟的Intent，可能让该应用崩溃。也就是Android中的拒绝服务漏洞

  1. 除数是否做了非0判断

  2. 不要在Activity的onCreate里调用PopupWindow的showAsLoaction方法，由于Activity还没被加载完，会报错

功能完成后，自测时的检查点

  1. 思考某些情况下，某个变量是否会造成空指针问题
  2. 把手机横屏，检查布局是否有Bug
  3. 在不同分辨率的机型上，检查布局是否有Bug
  4. 切换到英文等外文字体下，检查外文是否能完整显示
  5. 从低版本升级上来，会不会有问题,比如可能会出现数据库不兼容的问题
  6. 按下Home再返回是否正常
  7. 熄灭屏幕再打开是否正常
  8. 切换成其它应用再切换回来会怎样
  9. 利用手机的开发者选项中的 “调试GPU过度绘制” ，“GPU呈现模式分析” 和 “显示FPS和功耗” 功能，看自己的新功能是否会导致过度绘制、是否会掉帧
  10. 测试看是否影响启动速度adb shell am start -W 包名/Activity
  11. 对比看APK大小是否有增大
  12. 跑1小时Monkey，测试其稳定性

转自:
良心推荐：总结Android开发中必备的代码Review清单

补充:总结工作中的Android内存泄漏问题

简单判断是否有内存泄漏

判断内存泄漏的定位的大单位是Activity。

可以通过反复进入退出一个Activity，然后用adb shell dumpsys meminfo + 包名 查看虚拟机的堆是否有不断地增长

定位内存泄漏

1.使用Leak Canary

在代码上加入Leak Canary，然后不断跑Monkey或者手动反复进出不同页面。若出现内存泄漏问题，会自动导出来，生成以下页面。

2.使用DDMS导出hprof，并用MAT工具进行分析
  • 强烈建议先跑30分钟Monkey测试
  • 使用eclipse的ddms找到对应的进程，触发一次gc后，dump出里面的内存快照hprof文件以分析当前应用内存的堆有什么东西
  • 使用Android SDK 里的platform-tools文件夹的 hprof-conv工具，对刚才 hprof 文件进行转换，以至于 后面MAT工具能正常打开
  • 使用MAT打开hprof文件，进入Histogram。输入自己猜测可能泄漏的Activity（项目中Activity不多时，可每个Activity都重复以下3、4、5步骤）
  • 键该其中一项，打开菜单选择list objects ->with incoming refs将列出该类的实例
  • 右健Path to GC Roots–>exclue all phantom/weak/soft etc. reference，找出这个实例GC后，还会存在什么对象的引用关系。

常见导致内存泄漏的几个点

生命周期的原因

比如：Activity中关联了一个生命周期超过Activity的Thread，这个Thread 若持有该Activity的引用，就会导致内存泄漏。

内部类的原因

因为内部类会隐式地持有外部类的引用，若内部类不被释放，外部类也是无法释放。常见的有内部的Listener、Callback、Handler等导致。

情景1：若外部类应该释放的时候，内部类还在执行里面的函数，会导致外部类无法释放。

情景2：若一个异步操作，会回调内部类的Listener、Callback、Handler。当外部类应该释放的时候，但是这个异步操作还存在，而这个异步操作类又持有了Listener、Callback、Handler，导致外部类无法被释放。PS：这个原因也属于生命周期的原因。

静态变量的原因

单例类里包含Activity

静态变量的类里引用到Activity

注册与反注册、打开与关闭没成对出现的原因

比如：注册广播接收器、注册观察者（典型的譬如数据库的监听）等。或者自己写的跟Activity引用有关的clear()函数没有成对出现

解决方法

解决内部类的问题（以Handler作为例子）
  1. onDestroy时候remove所有msgActivity finish后未处理的msg是问题根源，所以清空所有未被执行的msg
mHandler.removeCallbacksAndMessages(null);

PS：比如Listener、Callback等其他内部类的问题，页面退出的时候，应该完成必要的清理操作，比如Cancel 请求

  1. 使用静态内部类 + weakReference
    静态内部类不会保留对外部类的引用，如果一定要引用外部类，使用weakReference
static class MyHandler extends Handler {WeakReference<Activity > mActivityReference;MyHandler(Activity activity) {mActivityReference= new WeakReference<Activity>(activity);}@Overridepublic void handleMessage(Message msg) {final Activity activity = mActivityReference.get();if (activity != null) {mImageView.setImageBitmap(mBitmap);}}}

PS:比如Listener、Callback等其他内部类的问题，也可以通过这个方法来解决

单例类里面尽量不要传入Activity，最好穿入ApplicationContext。假如传入了Activity，持有的时长也不能大于Activity的生命周期
对象的注册与反注册要成对出现
不使用WebView对象时，应该调用它的destory()函数来销毁它，并释放其占用的内存
因为View会持有Context，所以注意不要异步引用View，不要让静态对象持有View，不要在集合框架中存储View

本文来自互联网用户投稿，该文观点仅代表作者本人，不代表本站立场。本站仅提供信息存储空间服务，不拥有所有权，不承担相关法律责任。如若转载，请注明出处：http://www.rhkb.cn/news/428504.html

如若内容造成侵权/违法违规/事实不符，请联系长河编程网进行投诉反馈email:809451989@qq.com，一经查实，立即删除！

相关文章

设计模式 组合模式（Composite Pattern）

设计模式 组合模式（Composite Pattern）

组合模式简绍 组合模式&#xff08;Composite Pattern&#xff09;是一种结构型设计模式&#xff0c;它允许你将对象组合成树形结构来表示“部分-整体”的层次结构。组合模式使得客户端可以用一致的方式处理单个对象和组合对象。这样&#xff0c;可以在不知道对象具体类型的条…
阅读更多...
7--SpringBoot-后端开发、原理详解（面试高频提问点）

7--SpringBoot-后端开发、原理详解（面试高频提问点）

目录 SpringBoot原理 起步依赖 自动配置 配置优先级 Bean设置 获取Bean 第三方Bean SpringBoot原理 内容偏向于底层的原理分析 基于Spring框架进行项目的开发有两个不足的地方&#xff1a; 在pom.xml中依赖配置比较繁琐&#xff0c;在项目开发时&#xff0c;需要自己去找…
阅读更多...
手写Spring

手写Spring

简单实现Spring基于注解配置 ComponentScan Target(ElementType.TYPE) Retention(RetentionPolicy.RUNTIME) public interface ComponentScan {String value() default ""; } 相当于component-scan HspSpringConfig ComponentScan(value "spring.write.com…
阅读更多...
C#自定义曲线绘图面板

C#自定义曲线绘图面板

一、实现功能 1、显示面板绘制。 2、拖动面板&#xff0c;X轴、Y轴都可以拖动。 3、显示面板缩放&#xff0c;放大或者缩小。 4、鼠标在面板中对应的XY轴数值。 5、自动生成的数据数组&#xff0c;曲线显示。 6、鼠标是否在曲线上检测。 二、界面 拖动面板 鼠标在曲线上…
阅读更多...
【随手笔记】使用J-LINK读写芯片内存数据

【随手笔记】使用J-LINK读写芯片内存数据

第一种使用JLINK.exe 1. 打开j-link.exe 2.输入【usb】 3. 连接芯片 输入【connect】输入芯片型号【STM32L071RB】输入连接方式 【S】 使用SWD连接方式输入连接速率 【4000】连接成功 4. 输入【&#xff1f;】查看指令提示 5. 读写指令 Mem Mem [<Zone>…
阅读更多...
DataFrame生成excel后为什么多了一行数字

DataFrame生成excel后为什么多了一行数字

问题描述 python查询数据生成excel文件&#xff0c;生成的excel多了第一行数字索引&#xff0c;1,2,3,4,5...... 代码&#xff1a; df pd.DataFrame(data)df.to_excel(filename, sheet_name用户信息表, indexFalse) 解决&#xff1a; 原理也很简单&#xff0c;就是设置个参…
阅读更多...
【python设计模式7】行为型模式2

【python设计模式7】行为型模式2

目录 策略模式 模板方法模式 策略模式 定义一个个算法&#xff0c;把它们封装起来&#xff0c;并且使它们可以相互替换。本模式使得算法可独立于使用它的客户而变化。角色有&#xff1a;抽象策略、具体策略和上下文。 from abc import abstractmethod, ABCMeta from datetim…
阅读更多...
前端vue-ref与document.querySelector的对比

前端vue-ref与document.querySelector的对比

ref只在本组件中查找&#xff0c;而document.querySelector是在整个页面查找
阅读更多...
Golang | Leetcode Golang题解之第414题第三大的数

Golang | Leetcode Golang题解之第414题第三大的数

题目&#xff1a; 题解&#xff1a; func thirdMax(nums []int) int {var a, b, c *intfor _, num : range nums {num : numif a nil || num > *a {a, b, c &num, a, b} else if *a > num && (b nil || num > *b) {b, c &num, b} else if b ! ni…
阅读更多...
SQL Server性能优化之读写分离

SQL Server性能优化之读写分离

理论部分: 数据库读写分离&#xff1a; 主库&#xff1a;负责数据库操作增删改 20% 多个从库&#xff1a;负责数据库查询操作 80% 读写分离的四种模式 1.快照发布&#xff1a;发布服务器按照预定的时间间隔向订阅服务器发送已发布的数据快照 2.事务发布[比较主流常见]&#xf…
阅读更多...
Ceph 基本架构（一）

Ceph 基本架构（一）

Ceph架构图 Ceph整体组成 Ceph 是一个开源的分布式存储系统&#xff0c;设计用于提供优秀的性能、可靠性和可扩展性。Ceph 的架构主要由几个核心组件构成&#xff0c;每个组件都有特定的功能&#xff0c;共同协作以实现高可用性和数据的一致性。 以下是 Ceph 的整体架构及其…
阅读更多...
2024 “华为杯” 中国研究生数学建模竞赛（C题）深度剖析|数据驱动下磁性元件的磁芯损耗建模|数学建模完整代码+建模过程全解全析

2024 “华为杯” 中国研究生数学建模竞赛（C题）深度剖析|数据驱动下磁性元件的磁芯损耗建模|数学建模完整代码+建模过程全解全析

当大家面临着复杂的数学建模问题时&#xff0c;你是否曾经感到茫然无措&#xff1f;作为2022年美国大学生数学建模比赛的O奖得主&#xff0c;我为大家提供了一套优秀的解题思路&#xff0c;让你轻松应对各种难题&#xff01; CS团队倾注了大量时间和心血&#xff0c;深入挖掘解…
阅读更多...
使用LangGPT提示词让大模型比较浮点数

使用LangGPT提示词让大模型比较浮点数

使用LangGPT提示词让大模型比较浮点数 背景介绍环境准备创建虚拟环境安装一些必要的库安装其他依赖部署大模型启动图形交互服务设置提示词与测试 LangGPT结构化提示词 背景介绍 LLM在对比浮点数字时表现不佳&#xff0c;经验证&#xff0c;internlm2-chat-1.8b (internlm2-cha…
阅读更多...
MySQL聚合统计和内置函数

MySQL聚合统计和内置函数

【数据库】MySQL聚合统计 王笃笃-CSDN博客https://blog.csdn.net/wangduduniubi?typeblog显示平均工资低于2000的部门和它的平均工资 mysql> select deptno,avg(sal) deptavg from emp group by deptno; --------------------- | deptno | deptavg | --------------…
阅读更多...
【第33章】Spring Cloud之SkyWalking服务链路追踪

【第33章】Spring Cloud之SkyWalking服务链路追踪

文章目录 前言一、介绍1. 架构图2. SkyWalking APM 二、服务端和控制台1. 下载2. 解压3. 初始化数据库4. 增加驱动5. 修改后端配置6. 启动7. 访问控制台8. 数据库表 三、客户端1. 下载2. 设置java代理3. idea配置3.1 环境变量3.2 JVM参数3.3 启动日志 4. 启用网关插件 四、链路…
阅读更多...
C++和OpenGL实现3D游戏编程【目录】

C++和OpenGL实现3D游戏编程【目录】

欢迎来到zhooyu的专栏。 个人主页&#xff1a;【zhooyu】 文章专栏&#xff1a;【OpenGL实现3D游戏编程】 贝塞尔曲面演示&#xff1a; 贝塞尔曲面演示zhooyu 本专栏内容&#xff1a; 我们从游戏的角度出发&#xff0c;用C去了解一下游戏中的功能都是怎么实现的。这一切还是要…
阅读更多...
电脑ip会因为换了网络改变吗

电脑ip会因为换了网络改变吗

在当今数字化时代&#xff0c;IP地址作为网络世界中的“门牌号”&#xff0c;扮演着至关重要的角色。它不仅是设备在网络中的唯一标识&#xff0c;也是数据交换和信息传递的基础。然而&#xff0c;对于普通用户而言&#xff0c;一个常见的问题便是&#xff1a;当电脑连接到不同…
阅读更多...
Visual Studio 2022 - QT 环境中文字符乱码问题

Visual Studio 2022 - QT 环境中文字符乱码问题

Visual Studio 2022 - QT 环境中文字符乱码问题 一、Visual Studio 2022 - Qt 环境 在 QT 中使用中文字符串常会出现乱码现象&#xff0c;如下&#xff1a;以下提供了几个解决方法&#xff0c;仅供参考 QString str "百香果真是一直可爱的小猫咪"; qDebug() <…
阅读更多...
SAP HCM 组织增量解决方案

SAP HCM 组织增量解决方案

增量&#xff1a;今天遇到一个比较麻烦的问题&#xff0c;就是客户搭建中台&#xff0c;表结构和SAP的表结构一致&#xff0c;因为中台没有SAP那么多校验的逻辑&#xff0c;导致现在两边的主数据有差异&#xff0c;现在需要做个增量方案&#xff0c;SAP修改后增量传输给中台&am…
阅读更多...
实施项目，“流程重组”你是躲不开的

实施项目，“流程重组”你是躲不开的

文/杨长春 作者简介&#xff1a;某IT公司项目总监&#xff0c;资深IT博主&#xff0c;专注于IT项目知识分享&#xff0c;著有《实战需求分析》、《软件需求分析实战》、《数字化管理软件实施》。 甲方跟本项目相关的领域&#xff0c;一定运行着一套管理体系&#xff0c;各个岗…
阅读更多...
最新文章
推荐文章

Copyright @ 2022~2023

友情链接： 我的编程经验分享 一条长河网 尧图网站开发