一份合格的代码不应只满足于实现功能, 更应该遵循良好的规范. 遵循良好的代码规范有利于:

提升程序稳定性, 减少代码隐患, 降低故障率;

增强可扩展性, 大幅提高维护效率;

统一标准, 提升多人协作效率;

方便新人快速上手, 在项目组人员发生变动时保证项目进度;

这里梳理一下Android开发过程中需要注意的一些地方, 包括多个部分, 另外根据约束力强弱分为两类:

强制: 如果不遵守会导致代码严重混乱, 后期维护复杂, 甚至会出现严重bug;

推荐: 如果不遵守可能会导致代码描述不清, 理解困难, 导致功能越多维护越难的问题;

系统设计

强制：

不允许出现两段相同的逻辑块, 必须抽出为公共方法, 差异性使用参数控制, 避免修改时多处修改导致遗漏;

不允许出现两段相同的处于同一逻辑组的复杂布局, 必须抽为单独的include/merge;

不允许父类中出现子类具体方法, 如果需要的话可以父类定义抽象方法, 交由子类实现;

不允许Activity内多Fragment之间的直接沟通, 必须通过Activity中转;

推荐：

推荐使用MVP或者MVVM架构;

推荐使用Kotlin语言;

采用模块分类方式替代文件类别方式, 方便快速查找模块相关内容, 例: LoginActivity/LoginPreenter/LoginHttpRequest/LoginBean/LoginAdapter等所属同一登录模块的文件放入一个文件夹, 而不是所有activity放入一个文件夹, 所有adapter放入一个文件夹.

命名方式

强制：

不允许出现中文命名方式;

java/kotlin文件使用大驼峰方式, 例: LoginActivity.kt, NewsAdapter.kt, NewsBean.java;

layout/drawable/anim/style等resource文件使用小写+下划线的方式, 例: login_activity.xml, login_logo.png;

类定义使用大驼峰方式, 例: class LoginPresenter {}, class NewsBean {};

对象使用小驼峰方式, 例: LoginPresenter loginPresenter, NewsBean newsBean;

静态常量使用全大写+下划线的方式, 例: public static final boolean IS_RELESAE = true;

Kotlin使用的布局中的控件id必须使用小驼峰方式, 例: android:id=”@+id/tvLogin”

推荐：

文件/资源命名时采用 模块+类型 的方式, 以便迅速查找相关内容, 例如登录页面: LoginActivity.kt, login_activity.xml, login_logo.png, 网络错误, #f3f3f3

java使用的布局中的id名建议使用小驼峰方式, 并且使用控件类型缩写开头, 例: android:id=”@+id/tvLogin”, 附录常用控件缩写:

可见性

强制：

所有新定义的类/方法, 默认写成private, 只有在其他类需要引用时再看情况标为public, protected, package-private;

推荐:

java定义的父类中定义的方法如果子类重写会导致问题时, 添加final关键字;

注释相关

类/复杂或者不能从方法名字看出意图的方法必须添加注释, 当类/方法添加注释时, 必须使用此类型注释:

 

变量注释不允许使用与类/方法一致的注释形式;

方法注释中不允许出现@params, @return的参数描述错误的情况, 必须实时更新;

推荐：

一段逻辑建议使用//的方式;

方法/参数建议添加 @Nullable, @NotNull, @UiThread 等注解;

代码风格

此git目录下同时存有 AndroidCodeStyleSetting.jar 配置文件, 用于AndroidStudio导入后按照统一风格进行代码的格式化.

如果没有编写代码时随时格式化代码的习惯, 可以在AndroidStudio版本控制提交窗口右侧Before Commit中勾选Reformat code选项.

Android基本组件

强制：

Intent通信时不允许传递超过1M的数据, 可以采用外部Presenter中转或者EventBus传递的方式;

Intent隐式启动时必须检查目标是否存在, 否则会出现目标未找到崩溃: if (getPackageManager().resolveActivity(intent, PackageManager.MATCHDEFAULT ONLY) != null);

Activity/Service/BroadcastReceiver内如果有耗时操作, 必须采用多线程进行处理;

应用内部发送广播时, 只能使用LocalBroadcastManager.getInstance(this).sendBroadcast(intent), 不允许 context.sendBroadcast(intent), 避免外部应用拦截;

不允许在Application中缓存数据, 全局的共享数据可以使用某presenter存储, 或者使用SharedPreference读写;

Activity或者Fragment中动态注册BroadCastReceiver时，registerReceiver和unregisterReceiver必须要成对出现;

推荐：

Activity#onPause/onStop中结合isFinishing的判断来执行资源的释放, 必免放在执行时机较晚的Activity#onDestroy()中执行;

不要在Activity#onPause中执行耗时操作, 这样会导致界面跳转卡顿, 可以放入Activity#onStop中执行;

UI/布局

强制：

布局xml优先使用ConstraintLayout, 可以保证无嵌套的情况下完成包括部分控件同时显隐需求在内的99%的布局要求;

不允许使用ScrollView包裹ListView/GridView/ExpandableListVIew等列表View, 复杂多项式列表可以使用多ItemType进行处理;

推荐：

在Activity中显示对话框或弹出浮层时, 尽量使用DialogFragment, 而非Dialog/AlertDialog, 便于随Activity生命周期管理弹窗的生命周期;

进程/线程/消息推送

强制：

存在多进程的情况时, Application中的初始化代码要根据进程分别处理, 避免初始化不必要的业务;

新建线程时, 必须通过线程池的方式, 不允许采用new Thread()的方式;

Activity/Fragment中使用Handler时, 必须使用静态内部类+WeakReferences方式或者在onStop中调用handler.removeCallbacksAndMessages;

推荐：

多进程间共享数据使用ContentProvider替代SharedPreferences#MODE_MULTI_PROCESS;

文件/数据库

强制：

使用系统API获取文件路径, 避免手拼字符串, 例: android.os.Environment#getExternalStorageDirectory(), Context#getFilesDir(), 错误示例: File file = new File(“/mnt/sdcard/Download/Album”, name);

当使用外部存储时, 必须检查外部存储的可用性: Environment.MEDIA_MOUNTED.equals(Environment.getExternalStorageState());

数据库Cursor使用之后必须关闭, 以免内存泄漏;

推荐：

SharedPreference仅存储简单数据类型, 不要存储复杂数据, 如json数据/Bitmap编码等;

SharedPreference提交数据时, 尽量使用Editor#apply(), 而非Editor#commit();

图片/动画

强制：

加载大图时必须在子线程中处理, 否则会卡UI;

在Activity.onPause()/onStop()中关闭当前activity正在执行的动画;

推荐：

Android图片建议转化为WebP格式, 可以减少APK体积;

动画尽量不要使用AnimationDrawable, 占用非常多内存;

使用ARGB_565代替ARGB_888, 减少内存占用;

当Animation执行结束时, 调用View.clearAnimation()释放相关资源;

安全性

强制：

上线包必须混淆;

加解密的秘钥/盐不允许硬编码到代码中, 以防反编译获取;

Https处理时必须校验证书, 不允许直接接受任意证书;

使用Android的AES/DES/DESede加密算法时, 不要使用默认的加密模式ECB, 应显示指定使用CBC/CFB加密模式;

禁止把敏感信息打印到log中;

在应用发布时必须确保android:debuggable为false;

必须利用X509TrustManager子类中的checkServerTrusted函数效验服务器端证书的合法性,

必须将android:allowbackup属性设置为false, 防止adb backup导出应用数据;