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

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

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

我们应用中有用到智齿客服，客服需要调用接口初始化用户信息内容。有一版本我们要优化首页的接口，发现这个初始化用户信息的接口是放在首页的。 参考做法：客服用户使用客服的频次很低，而且使用App期间可能会更换账号，所以应该在点击客服入口那里调用初始化用户信息的接口。
我们应用中有直播SDK，直播SDK需要配置进用户信息，发现处理的是每次进入app或者更换账号的时候配置，这个配置首次会导致卡顿。 参考做法：因为用户还没有进入直播，且直播也不是我们的主要业务，还没用到这个信息，正确的做法是每次进直播判断是否有配置用户信息或者更换了用户，然后再进行配置操作。
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.合理利用变量，避免多次使用缓存

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

代码语言：javascript复制

DataUtil.getHistoryData();

应该避免这样的处理：

代码语言：javascript复制

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

而是这样处理，少一次取缓存的次数：

代码语言：javascript复制

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值不能直接对比

工具类：

代码语言：javascript复制

    /**
     * 得到两个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);
        }
    }