RequestMapping滥用：必须指定请求方式

Controller需要异常捕获

符号"," 等其他的常量需要抽取到Constant里，禁止魔法数字

Collections.singletonList方法的使用

这个方法主要用于只有一个元素的优化，减少内存分配，无需分配额外的内存，可以从SingletonList内部类看得出来,由于只有一个element,因此可以做到内存分配最小化，相比之下ArrayList的DEFAULT_CAPACITY=10个。

list判空使用工具类

"" 请使用StringUtils.EMPTY; lang3 里面的

无意义的写法

StringBuilder ystbs = new StringBuilder("");

直接是

StringBuilder ystbs = new StringBuilder();

定义List需要泛型

List<Object> objs = new ArrayList<>();

这种是不可取的，需要添加 具体的泛型

自欺欺人字符串拼接法

代码：

StringBuilder stringBuilder = new StringBuilder();
      return stringBuilder.append(systemConfig.getSpfw().getUrl() + "/spgzqsq.html")
      .append("?businessKey=" + bk).append("&dataId=" + dataId).append("&spType=" + spType)
      .append("&attachmentUrl=" + attachmentUrl).append("&detailUrl=").append("&sysId=" + sysId).toString();

获取文件名，获取文件后缀

public class GetExtension {
    public static void main(String[] args) {


        String fileName = "xxx.doc";
        String name = FilenameUtils.getName(fileName);
        String baseName = FilenameUtils.getBaseName(fileName);
        String extension = FilenameUtils.getExtension(fileName);


        //先前的错误做法
        String fileName11 = fileName.split("\\.")[1];


        System.out.println("文件名："+ name);
        System.out.println("文件名base："+ baseName);
        System.out.println("文件名extension："+ extension);


        String fileName2 = "1.2xxx.pdf";
        String name2 = FilenameUtils.getName(fileName2);
        String baseName2 = FilenameUtils.getBaseName(fileName2);
        String extension2 = FilenameUtils.getExtension(fileName2);
        System.out.println("文件名："+ name2);
        System.out.println("文件名base："+ baseName2);
        System.out.println("文件名extension："+ extension2);
    

常量命名规范，双音节词常量名称用下划线分隔

这里提供一个idea 快捷键 ctrl + shift +u 可以把字符变成全部写或者是小写，可以先加上下划线，然后 ctrl + shift +u

循环插入，循环打印日志，这样的习惯也不好，日志打印得不好，会打印一吨的日志

for (String baSj:baSjs) {

insertToWsxx(baSj);

log.info("baSj : {}" , baSj);

}

代码易读性

方式一

//存储路径处理，为空的过滤
if(StringUtils.isNotEmpty(filePath)){
    wsxx.setFilepath(filePath));
}else{
    return;
}

和下面的代码比较

方式二

if(StringUtils.isEmpty(filePath)){
    return;
}else{
   wsxx.setFilepath(filePath));
}

明显是 方式二 更清晰一些，美

在controller 赋予了过多的业务逻辑

这一点在很多项目里面很容易犯，不小心controller 的方法就变的不纯洁了，严重违反单一职责的原则，赋予了过多的业务逻辑，应该下沉到service里面！

mapper查询返回Map结构对象，不利于代码维护

没有泛型，不知道返回的具体的对象，不知道都有哪些属性