吐槽我见过的最烂的Java代码
吐槽我见过的最烂的Java代码
只针对spring框架，贴一份我见过最烂的代码（没有之一），我就想知道，有没有其他人能写出这样的代码？还有没有更加烂的编码方式？
controller代码：
/**
* 增加配置
*/
@PostMapping("/add")
public void addConfig(HttpServletRequest request, HttpServletResponse response)
throws IOException {
response.setContentType("application/json;charset=UTF-8");
//返回了一个json格式的字符串。。
String result = configService.add(request);
response.getOutputStream().write(result.getBytes());
response.getOutputStream().flush();
}services代码：
/**
* ！！！错误的示例
*
* @param request
* @return
*/
public String add(HttpServletRequest request) {
Map<String, Object> data = new HashMap<String, Object>();
try {
String name = (String) request.getParameter("name");
String value = (String) request.getParameter("value");
//示例代码
long newID = add(name, value);
data.put("code", 0);
data.put("newID", newID);
} catch (CheckException e) {
// 参数等校验出错，已知异常，不需要打印堆栈，返回码为-1
data.put("code", -1);
data.put("msg", e.getMessage());
} catch (Exception e) {
// 其他未知异常，需要打印堆栈分析用，返回码为99
logger.error("add config error", e);
data.put("code", 99);
data.put("msg", e.toString());
}
return JSONObject.toJSONString(data);
}看到这样的代码，我第一个反应是震惊，然后是愤怒，最后是悲哀。问了一下开发人员，为什么这样写，他说一过来项目组里面的代码就是这样的，继续追下去原来是一个工作了多年的老开发人员定的代码框架，我就说嘛，开发新手也写不出这么烂的代码啊。这个写法还不如在servlet里面写呢。
所以说，simple is not easy。很多人就是因为觉得简单，所以功能完成自己测试ok了就算了，没有思考有没有更加好的方式。归根到底是因为编码习惯太糟糕，写的代码太烂，导致无法定位频繁修改频繁出问题。
其实，对于个人来说，技术很重要，但是对于工作来说，编码的习惯比技术更加重要。工作中你面试的大部分技术用到的很少。但是，因为你的编码习惯不好，写的代码质量差，代码冗余重复多，很多无关的代码和业务代码搅在一起，导致了你疲于奔命应付各种问题。
所以我作为SE，不管接手任何项目组，第一步就是制定代码框架，制定项目组的开发规范，把代码量减下去。事实上证明，这一步之后，大家的代码量能下去最少1/3，后台的问题数下降比较明显，大家的加班会比之前少。
再给大家看一个直观的例子。下面是controller的一个删除数据的接口，我来之前大家写的这个样子的（其实一开始比这个还差很多），功能很简单，输入一个对象id执行删除返回是否删除成功。大家有没有觉得有什么问题？
＠PostMapping("/delete")
public Map<String, Object> delete(long id, String lang) {
Map<String, Object> data = new HashMap<String, Object>();
boolean result = false;
try {
// 语言（中英文提示不同）
Locale local = "zh".equalsIgnoreCase(lang) ? Locale.CHINESE : Locale.ENGLISH;
result = configService.delete(id, local);
data.put("code", 0);
} catch (CheckException e) {
// 参数等校验出错，这类异常属于已知异常，不需要打印堆栈，返回码为-1
data.put("code", -1);
data.put("msg", e.getMessage());
} catch (Exception e) {
// 其他未知异常，需要打印堆栈分析用，返回码为99
log.error(e);
data.put("code", 99);
data.put("msg", e.toString());
}
data.put("result", result);
return data;
}其实上面的代码也没有大问题。而我接手之后，我会开发自己的代码框架，最后制定代码框架交付的代码如下（这是controller的部分）:
＠PostMapping("/delete")
public ResultBean<Boolean> delete(long id) {
return new ResultBean<Boolean>(configService.delete(id));
}用到的技术就是AOP，也不是什么高深技术。怎么样？代码量就一行，特性一个都没有丢。这就是我们项目组现在的controller的样子！（如果恰好有我带过的项目组的人，看到ResultBean<>应该很熟悉应该知道我是谁了）
所以说技术无所谓高低，看你怎么样用。上面的代码简单说一下问题，第一，lang和业务没有什么关系，我后面的代码框架去掉了（不是说我后面的代码没有这个功能，是把他隐藏起来对开发人员透明了，使用的技术就是ThreadLocal）。第二，前面那个代码，实际上干活的就只有一行，其他都和业务代码没有一毛钱关系，我的代码框架里面完全看不到了。
使用的技术真的很简单，但是编码效果非常好，因为大家不要因为使用的技术初级就觉得不重要！！使用这套框架后，大家再也不需要大部分时间都写一些无聊的代码，可以有更加多时间学习其他技术。说实话，在我项目组的开发人员都是比较幸运的，觉得能学到东西，不是像其他项目组，写了几年都是一样的CRUD代码，虽然我比较严厉，但是还是愿意待在我项目组，毕竟加班比其他项目组少啊。
这就是我说的工作中，编码习惯（或者说编码风格）比技术更加重要。我工作了也有很长时间了，我觉得我个人价值最大的地方就是这些，技术上其实我懂的也和大家差不多，但编码上我还是觉得可以超过大部分人的。后面我会把我们这些业务系统中大家编码的问题一个一个写出来，并把我的解决办法分享出来。