代码评审出现的简单问题

1.红框所示
红框中的内容可以直接使用systemName.trim(),不必再通过systemName = systemName.trim().

2.判空操作
判空操作将判断条件中的null 写在前面更健壮一些，而不是interfaceCode != null
,然后是””.equals(interfaceCode) 这样写容易出现问题，在interfacesCode为null时会报错，判断长度更好一些，例如：0 != interfaceCode.length()
最好使用系统中提供的公用判断工具方法，更加规范容易管理，不容易出现隐蔽的bug。
3.字符串命名规范
命名一般使用驼峰命名法，而下图红框中的字符串标识符中使用了下划线。

命名要求java类文件名等使用大驼峰标识，方法名、变量名等使用小驼峰标识，
规范命名可使代码更清晰，容易辨识，减少错误的产生，也更容易发现错误。
4.代码格式
下图红框中的if所属的花括号没有和if上下对齐

要求Java类文件、jsp文件、XML文件以及js和css文件都要求使用格式化工具进行统一格式，统一规范的格式可以使代码的整体结构更清晰明朗，更容易看清代码中的逻辑，也便于代码的调试工作。
5.自动生成的代码没有和自己写的区分开

Mybatis Mapper文件自动生成的代码和自己写的代码区分开，红框中的代码是自动生成的，两遍的是后来写的，不应该让自动生成的代码和自己写的代码交叉出现。
自动生成的为*Mapper.xml,自己写的sql放在*XMapper.xml，防止数据库修改后重新生成覆盖自己写的部分。