【Android】Android开发者最容易犯的几个错误

骑猪看日落 2021-09-27 15:08 382阅读 0赞

根据本人多年Android开发及代码审查经验，发现Android开发者极易犯下面这些错误。

#### 一、Activity回退，没有取消异步任务，比如网络请求。 ####

> 后果：异步任务回调后因找不到容器app闪退。

#### 二、注册组件没有反注册。比如广播、EventBus等。 ####

> 后果：Context被引用，组件无法回收。

#### 三、for循环满足条件后循环没有中断。 ####

举个例子，看下面一段代码：

for(int i=0;i<array.length;i++){ 
        if(array[i]==100){ 
        //找到了，循环还会继续进行
        //正确的做法是加上:break
        }
    }

> 后果：由于已经找到了元素，循环就可以取消了。但是上面这个循环每次都会走完。这样就会增大cpu开销，降低app性能。

#### 四、使用不确定的对象没有判空。比如http接口返回的实体bean、回调listener等 ####

> 后果：由于http接口返回数据存在较大的不确定性，可能大多数时候接口都是正常的，但是不排除接口数据返回了null，那么解析实体bean就是null。使用这个bean就会出现空指针异常，app闪退。

#### 五、if条件嵌套层级太多不易理解，没有事先进行过滤。 ####

举个简单例子，看一段伪代码：

public void purchase(){ 
       if(有网络){ 
            if(用户已经登录){ 
                 if(用户是会员){ 
                       if(用户余额足够){ 
                         //业务逻辑
                       }else{ 
                       ...
                       }
                 }else{ 
                  ...
                 }
              }else{ 
            ...
            }
        }
    }

其实这样写，太难阅读了。层级太多。推荐的写法是：

public void purchase(){ 
       if(没有网络){ 
         return;
       }
       if(用户没有登录){ 
         return;
       }
       if(用户不是会员){ 
         retrun;
       }
       if(用户余额不足){ 
         return;
       }
       //业务逻辑
    }

> 后果：代码不易理解，逻辑容易出错。

#### 六、RecyclerView数据适配器Adapter的onBindView函数中只有If没有else的处理。 ####

> 后果：由于Adapter存在复用机制，当逻辑不满足if条件进入else分支时，没有处理就会出错。

#### 七、数据库操作没有开启事务。操作失败没有回滚。 ####

> 后果：失败没有回滚，数据出错，后果很严重。

#### 八、字符串判断把变量放在了前面，容易出现空指针。 ####

举个简单的例子：

if(str.equals("hello")){ 
       ...
    }

这样写str如果是null程序就会闪退。正确的写法是：

if("hello".equals(str)){ 
       ...
    }

> 后果：空指针异常，app闪退。