Android Code Review 注意点记录（不断更新~）

许进进

已于 2023-01-12 12:28:12 修改

阅读量1.8k

点赞数 1

文章标签： android 代码复审 java

于 2022-10-17 00:23:32 首次发布

本文链接：https://blog.csdn.net/LucasXu01/article/details/127355815

版权

本文列举了代码审查的重要节点，包括架构合理性、性能优化、异常处理、算法效率等，并提供了实操建议，如避免循环内remove操作、合理使用成卫语句、注意循环与异常处理的关系、优化数据结构等。同时强调了使用静态分析工具和遵循代码样式指南的重要性，以及在代码审查中保持积极和尊重的态度。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

一 Review的关键节点指导

架构是否合理：虽然是在Review代码，其实很多时候我们都是在评判架构是否合理。比如数据的同步机制是否合理等等；
跨网络请求：跨网络请求一般是性能占比最高的地方，是否在循环中调用，是否在子线程、是否反注册；
异常处理：结合业务查看异常处理流程是否正确合理，异常是否需要精细化，是否在catch中捕获了全部异常；
算法效率：算法是否过于复杂；
内存泄漏、OOM；

二工作中常见的代码Review点实操

1、for循环中的remove操作（return、break除外）；

2、super非阻断性的放最前；

3、自己的代码注释修改；

4、try catch 注意需不需要Exception 包囊全部；

5、横竖屏在8.0上的问题；

if (Build.VERSION.SDK_INT != Build.VERSION_CODES.O && Build.VERSION.SDK_INT != Build.VERSION_CODES.O_MR1) {
            setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_PORTRAIT);
        }

6、注册反注册、网络请求unsubscribe

7、注意循环内try，不影响下一个item的处理；如果循环外try，一旦报错整个循环就结束了；看场景确定内try还是外try；

8、合理使用成卫语句，也不要硬写成卫语句，如下为硬写：

if(){
		...
		return 结果；
}
return 结果；

9、map和循环：Map比for遍历效率更高。

10、数据结构和程序有时候是可以互相转换的，比如如下程序

        if (str.equals("a")) {
            return "1";
        } else if (str.equals("b")) {
            return "2";
        } else if (str.equals("c")) {
            return "3";
        } else {
            throw new UnsupportedOperationException();
        }

可以用数据结构表示：

        Map<String, String> map = new HashMap<>();
        map.put("a", "1");
        map.put("b", "2");
        map.put("c", "3");

使用的时候：

        if (map.containsKey(str)) {
            return map.get(str);
        } else {
            throw new UnsupportedOperationException();
        }