屎山代码赏析23.09.12

327 阅读1分钟

赏屎,看屎,不写屎,共筑良好的代码环境

屎香浓郁

等级:严重令人呕吐

类型:代码风格

image.png

含屎分析

作者意图分析

分析代码得知,作者是想将多个表的查询结果进行返回

屎点1

作者的思路是将主表的返回结果用map表示,其他表的结果,put到这个map中去。

🤮

1. 用map作为数据载体非常灵活,所以map作为方法返回值,可读性极差,易用极差,可维护性极差
2. map作为返回值不易生成文档

屎点2

作者将实体类转为map的方式,竟然是先转成json字符串,再转成map

🤮

  1. 来回转换效率问题
  2. 类似日期这种字段处理不好,容易丢失信息

屎点3

大量魔法值

铲屎建议

  1. 代码反工
  2. 作者枪毙