boss代码review记录.docx

BOSS代码revieiw记录 修改记录修改日期版本号修改内容修改人审核人2014-05-130.5潘洪亮目录目录4修改记录131Findbugs141.1DateFormat线程安全141.1.1错误示例141.1.2Bug描述141.1.3解决方案151.2DateFormats多线程问题151.2.1错误示例151.2.2Bug描述151.2.3解决方案161.3从未使用的本地变量161.3.1错误示例161.3.2Bug描述161.4Unused field171.4.1错误示例171.4.2Bug描述171.5参数为nulll171.5.1错误示例171.5.2Bug描述181.5.3解决方案181.6方法名，首字母小写 ，骆驼命名法181.6.1错误示例181.7可能空指针异常181.7.1错误示例191.7.2Bug描述191.7.3解决方案191.8Int to long 可能溢出191.8.1错误示例191.8.2Bug描述201.8.3解决方案201.9Should be a static inner class211.9.1错误示例211.9.2Bug描述211.9.3解决方案211.10循环中+拼接字符221.10.1错误示例221.10.2Bug描述221.10.3解决方案231.11资源释放231.11.1错误示例231.11.2Bug描述231.11.3解决方案231.12不用 new，改用.valueOf241.12.1错误示例241.12.2Bug描述241.12.3解决方案241.13bad month value of 12251.13.1错误示例251.13.2Bug描述251.13.3解决方案251.14硬编码文件路径261.14.1错误示例261.14.2Bug描述261.14.3解决方案261.15命名t_ m_ s_271.15.1错误示例271.15.2Bug描述271.15.3解决方案271.16format string %s参数不足271.16.1错误示例271.16.2Bug描述271.16.3解决方案281.17If逻辑太复杂291.17.1错误示例291.17.2Bug描述291.17.3解决方案291.18用entrySet遍历Map301.18.1错误示例301.18.2Bug描述301.18.3解决方案301.19不要吞异常311.19.1错误示例311.19.2Bug描述311.19.3解决方案311.20实例对象，修改静态变量321.20.1错误示例321.20.2Bug描述321.20.3解决方案321.21Equals Hashcode321.21.1错误示例331.21.2Bug描述331.21.3解决方案331.21.4自动生成的341.22一行代码也要有{}371.22.1错误示例371.22.2Bug描述371.22.3解决方案371.23Session381.23.1错误示例381.23.2Bug描述381.23.3解决方案381.24随机数381.24.1错误示例381.24.2Bug描述381.24.3解决方案391.25提交的代码要格式化391.25.1错误示例391.25.2Bug描述391.25.3解决方案391.26Findbugs自查401.26.1错误示例401.26.2Bug描述401.26.3解决方案401.27去掉重复前缀401.27.1错误示例411.27.2Bug描述411.27.3解决方案411.28SS_SHOULD_BE_STATIC411.28.1错误示例421.28.2Bug描述421.28.3解决方案421.29Logger写法421.29.1错误示例421.29.2Bug描述421.29.3解决方案431.30Nan431.30.1错误示例431.30.2Bug描述431.30.3解决方案431.31捕获为抛出异常441.31.1错误示例441.31.2Bug描述441.31.3解决方案441.32MS_SHOULD_BE_FINAL451.32.1错误示例451.32.2Bug描述451.32.3解决方案451.33461.33.1错误示例461.33.2Bug描述461.33.3解决方案461.34461.34.1错误示例461.34.2Bug描述461.34.3解决方案462Sql472.1sqlmapper父子关系472.1.1错误示例472.1.2Bug描述472.1.3解决方案472.2外键自动建立索引482.2.1错误示例482.2.2Bug描述482.2.3解决方案482.3DEFAULT NULL可以不写492.3.1错误示例492.3.2Bug描述492.3.3解决方案492.449