Android项目开发中高频问题1 | 代码审核

1.对象的创建或接口的请求在需要的时候才执行

这样的现象遇到过三次,分别是

  1. 我们应用中有用到智齿客服,客服需要调用接口初始化用户信息内容。有一版本我们要优化首页的接口,发现这个初始化用户信息的接口是放在首页的。
    参考做法:客服用户使用客服的频次很低,而且使用App期间可能会更换账号,所以应该在点击客服入口那里调用初始化用户信息的接口。
  2. 我们应用中有直播SDK,直播SDK需要配置进用户信息,发现处理的是每次进入app或者更换账号的时候配置,这个配置首次会导致卡顿。
    参考做法:因为用户还没有进入直播,且直播也不是我们的主要业务,还没用到这个信息,正确的做法是每次进直播判断是否有配置用户信息或者更换了用户,然后再进行配置操作。
  3. adapter里点击会有一个弹框,弹框里的文案会用到一个配置接口里的数据,错误的做法是进去页面就调用接口得到这个数据。
    参考做法:点击的时候拿这个数据且存为变量,下次有的话直接取。

这几个基本都不影响业务逻辑,是优化项,但是对App资源消耗有影响,算是逻辑规范问题。

2.shape的gradient属性 android:type="radial"问题

如果android:type="radial",没有设置android:gradientRadius,将会报错
https://blog.csdn.net/zjdyhant/article/details/46537647

就是说使用渐变色时,android:type="radial"android:gradientRadius需要成对出现,否则可能会产成崩溃,在友盟后台看见崩溃率还挺高的。

3.integer.parseInt(id)

如果id为空会抛错误new NumberFormatException("s == null"),注意空处理

4.可以在fragment里直接获取宿主activity里的数据

可以方便的在fragment里得到宿主activity,然后再得到里面的数据,而不用通过 RxBus 或 intent传给fragment,特别是数据量大的时候应该避免后面的传值操作。

5.合理利用变量,避免多次使用缓存

例如我们使用一个缓存数据得到存的历史搜索数据:

DataUtil.getHistoryData();

应该避免这样的处理:

if(DataUtil.getHistoryData() != null ){
    // 处理逻辑
    HistoryBean bean = DataUtil.getHistoryData();
}

而是这样处理,少一次取缓存的次数:

HistoryBean bean = DataUtil.getHistoryData();
if(bean != null ){
    // 处理逻辑
    bean.get();
    ...
}

6.避免滥用SharedPreferences

业务场景:
在进一个页面A如果勾选状态,进入页面B的时候会带上这个状态。有时候我们不能通过intent传递,例如一些三方SDK的时候,需要记录这个值。

不当做法:
点击勾选后通过SP记录勾选状态,然后进入其他页面再取,在打开App的时候重置这个值。

参考做法:
存一个全局内存变量,点击勾选后更改变量值,然后在另一个页面使用,退出App时重置这个值。

7.使用android:launchMode="singleTask"然后打开Activity问题

我们有一个需求,点击按钮要定位在首页的第二个tab下,如果配置了singleTask打开其Activity时会清空上层的Activity,然后再setCurrentItem(),这样就达到了效果。

然后发现,这样会导致Activity里面的fragment走两次onPause()方法,影响了数据统计。然后不打开直接定位setCurrentItem()就没问题,因为我们是在此Activity下两个不同的fragment定位的,所以可以这样做,如果在下级页面估计也会有这样的问题。

8.别在intent里传大量的数据

这个是个基本常识了,传大量的数据,轻则卡顿的很厉害,重则直接崩溃。

9.记得处理else的情况,特别是在adapter里

adapter处理各种逻辑时,注意else的处理
处理接口请求时,注意error和数据为空的处理

10.float值不能直接对比

工具类:

    /**
     * 得到两个float比较的结果
     *
     * @return 返回值
     * a = -1,表示 source 小于 duibi
     * a = 0,表示 source 等于 duibi;
     * a = 1,表示 source 大于 duibi;
     */
    public static int getCompareResult(float source, float duibi) {
        try {
            BigDecimal bigDecimal = new BigDecimal(Float.toString(source));
            BigDecimal duibi3 = new BigDecimal(Float.toString(duibi));
            return bigDecimal.compareTo(duibi3);
        } catch (Exception e) {
            DebugUtil.error(e.getMessage());
            int i = (int) (source * 10) - (int) (duibi * 10);
            return Integer.compare(i, 0);
        }
    }

11.在adapter里设置值然后在其他地方取不可取

adapter更新不是同步的,设置值后直接在另一个地方使用可能会有问题,不是实时数据,最好还是通过数据得到。

推荐阅读更多精彩内容