findbugs代码检查常见问题及处理.doc

代码检查常见问题及解决方法注：以下解决方法为参考办法，如果有更好的解决办法欢迎提出，我们共同改进。序号问题英文描述问题分析解决办法1Comparison of String objects using = or !=比较字符串使用了双等号使用String的equels()方法2Call to equals() comparing different typesEquals方法比较了两个不同的数据类型转换类型后比较3Call to method of static java.text.DateFormat同一个format 多次调用会导致线性不安全， privatestaticfinalSimpleDateFormatformat=newSimpleDateFormat(dd/MM/yyyy);方法体内调用：format.format(newDate();在方法体内使用，尽量避免定义全局的对象方法体内定义：SimpleDateFormat format=newSimpleDateFormat(dd/MM/yyyy);format.format(newDate();4Bad attempt to compute absolute value of signed 32-bit random integerString nums = Math.abs(r.nextInt() + ;修改为String nums = Math.abs(r.nextInt(Integer.MAX_VALUE) + ;5The class name com.aisino.fpcxbd.action.printAction doesnt start with an upper case letter类名首字母大写类名首字母大写6Dead store to rePacket不被使用的变量或常量看业务是否会使用，如果不使用，请删除，如果有使用，初始值不要直接new 一个对象，初始值可以置为null7Exception is caught when Exception is not thrown没有抛出异常时，异常被捕获结合业务进行处理8Load of known null value填充了空值如果没有的空值，请删除9Method invokes inefficient new String(String) constructor方法中调用了低效的new String()构造方法，如context.put(name,new String(Velocity) );如果内容本身为字符串，则直接赋值即可，如context.put(name,Velocity );10Method may fail to close stream方法可能未关闭stream，方法产生了一个IO流，却未关闭，将会导致文件描绘符的泄漏。建议使用finally block来确保io stream被关闭。如果无异常，请使用完毕后关闭IO流。11Method might ignore exception捕捉了异常，但是没有进行处理需要在catch体中对异常进行处理。或者打印相关的描述。12Method names should start with a lower case letter方法首字母没有小写， 没有遵循Java命名规范请将方法首字母小写13Method uses the same code for two branches例如：fpkjmx.setSyl(fpkjmx.getSyl();请确认该写法是否有意义。14Non-transient non-serializable instance field in serializable class在可序列化的类中存在不能序列化或者不能暂存的数据将属性对象实现可实例化15Null pointer dereference会出现空指针，如if(null != nsrxx)elsensrxx.setLoginZt(3);这种逻辑有明显的错误，请根据业务修改16Nullcheck of value previously dereferenced会出现空指针异常先做非空判断，再进行业务逻辑的处理，避免空指针异常17Possible null pointer dereference可能会出现空指针异常请根据业务处理，如有必要先做非空判断18Possible null pointer dereference in method on exception path在异常部分放弃null值检查，可能会导致后面的代码出现空指针异常请根据业务进行处理19Redundant nullcheck of value known to be non-null该对象已不为空，没有必要再做非空判断没有必要的判断去掉即可，（根据实际业务）20Redundant nullcheck of value known to be non-null已知该对象为空，没有必要再做非空判断已知该对象为空，没有必要再做非空判断21Repeated conditional tests重复条件测试重复的判断，请去掉一个22Self assignment of local variable自赋值的局部变量请检查代码，是否可以去掉自赋值23Should be a static inner class应该定义为静态内部类如果出现该提示，请定义为静态内部类24Store of no