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

根据本人多年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闪退。

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

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

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

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

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

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

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

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

更多相关文章

随机推荐