一份合格的代码不应只满足于实现功能, 更应该遵循良好的规范. 遵循良好的代码规范有利于:
提升程序稳定性, 减少代码隐患, 降低故障率;
增强可扩展性, 大幅提高维护效率;
统一标准, 提升多人协作效率;
方便新人快速上手, 在项目组人员发生变动时保证项目进度;
这里梳理一下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导出应用数据;