android的Findbugs问题整理

1. 对字符串不相等的判断

• 错误

if (chapterInfo.getPayMode()
!= String.valueOf(com.shuqi.base.common.Constant.PAYMODE_BOOK) && chapterInfo.getPayMode() !=

        String.valueOf(com.shuqi.base.common.Constant.PAYMODE_CHAPTER)
&& drawType == DrawType.DRAW_COUNT_DOWN_TYPE) {

    mRenderData.setDrawType(DrawType.DRAW_LOADING_TYPE);

}

• 正确

if (!String.valueOf(com.shuqi.base.common.Constant.PAYMODE_BOOK).equals(chapterInfo.getPayMode())
&& !String.valueOf(com.shuqi.base.common.Constant.PAYMODE_CHAPTER)

        .equals(chapterInfo.getPayMode())
&& drawType == DrawType.DRAW_COUNT_DOWN_TYPE) {

    mRenderData.setDrawType(DrawType.DRAW_LOADING_TYPE);

}

或者调用 TextUtils.equals() 方法

2. 典型数组越界

3. 创建文件夹判断返回值

在创建文件时，虽然判断了文件不存在再创建，但未处理创建失败的情况，会有不可预知的错误，应该处理下创建失败的情况(但不是一味的try)，应给外部返回相应的错误码。

4. 创建单实例，保证线程安全

public static DownFileManager
getInstance() {

    if (mInstance
== null)
{

        mInstance
= new DownFileManager();

    }

    return mInstance;

}

• 正确的做法是：

public static DownFileManager
getInstance() {

    if (mInstance
== null)
{

        synchronized (DownFileManager.class)
{

            if (mInstance
== null)
{

                mInstance
= new DownFileManager();

            }

        }

    }

    return mInstance;

}

或

public synchronized static DownFileManager
getInstance() {

    if (mInstance
== null)
{

        mInstance
= new DownFileManager();

    }

    return mInstance;

}

5. 关于弱引用

使用SoftReference.get()可能会有空指针异常， 必须 对返回值进行判断

6. 分支执行代码相同（DB_DUPLICATE_BRANCHES）

这个就是if-else分支中执行的代码是一样的，一种好的编程实践就是在写if时，一定要考虑到是否需要else分支，某些团队要求有if分支，就必须要有else分支，哪怕else分支什么也不做。

7. 需要float或double类型变量，除法运算分子或分母需要一个float或double

8. switch要实现default（SF_SWITCH_NO_DEFAULT）

9. 静态变量尽量final，若需要赋值操作，降低可见性设置set/get方法

不允许直接访问一个变量，必须提供对应的方法来访问，这是一个强制性的规约。静态常量请一定要添加 final 关键字。

10. float与double类型值判断相等的问题

请使用Utility.floatEquals方法来判断

public static boolean floatEquals(float a, float b)
{

     return Math.abs(a
- b) < .0000001;

}