作者：晓风轻

来源：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 data = new HashMap(); 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 delete(long id, String lang) { Map data = new HashMap(); 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 delete(long id) { return new ResultBean(configService.delete(id));}

用到的技术就是AOP，也不是什么高深技术。怎么样？代码量就一行，特性一个都没有丢。这就是我们项目组现在的controller的样子！（如果恰好有我带过的项目组的人，看到ResultBean<>应该很熟悉应该知道我是谁了）

所以说技术无所谓高低，看你怎么样用。上面的代码简单说一下问题，第一，lang和业务没有什么关系，我后面的代码框架去掉了（不是说我后面的代码没有这个功能，是把他隐藏起来对开发人员透明了，使用的技术就是ThreadLocal）。第二，前面那个代码，实际上干活的就只有一行，其他都和业务代码没有一毛钱关系，我的代码框架里面完全看不到了。

使用的技术真的很简单，但是编码效果非常好，因为大家不要因为使用的技术初级就觉得不重要！！使用这套框架后，大家再也不需要大部分时间都写一些无聊的代码，可以有更加多时间学习其他技术。说实话，在我项目组的开发人员都是比较幸运的，觉得能学到东西，不是像其他项目组，写了几年都是一样的CRUD代码，虽然我比较严厉，但是还是愿意待在我项目组，毕竟加班比其他项目组少啊。

这就是我说的工作中，编码习惯（或者说编码风格）比技术更加重要。我工作了也有很长时间了，我觉得我个人价值最大的地方就是这些，技术上其实我懂的也和大家差不多，但编码上我还是觉得可以超过大部分人的。后面我会把我们这些业务系统中大家编码的问题一个一个写出来，并把我的解决办法分享出来。