這是我見過寫得最爛的Controller層代碼，沒有之一！
一. 接口定義
工作中，少不了要定義各種接口，系統(tǒng)集成要定義接口，前后臺掉調(diào)用也要定義接口。接口定義一定程度上能反應(yīng)程序員的編程功底。列舉一下工作中我發(fā)現(xiàn)大家容易出現(xiàn)的問題：
1. 返回格式不統(tǒng)一
同一個接口，有時候返回數(shù)組，有時候返回單個；成功的時候返回對象，失敗的時候返回錯誤信息字符串。工作中有個系統(tǒng)集成就是這樣定義的接口，真是辣眼睛。這個對應(yīng)代碼上，返回的類型是map，json，object，都是不應(yīng)該的。實際工作中，我們會定義一個統(tǒng)一的格式，就是ResultBean，分頁的有另外一個PageResultBean。
錯誤范例：
//返回map可讀性不好，盡量不要
＠PostMapping("/delete")
public Map<String, Object> delete(long id, String lang) {
}
// 成功返回boolean，失敗返回string，大忌
＠PostMapping("/delete")
public Object delete(long id, String lang) {
try {
boolean result = configService.delete(id, local);
return result;
} catch (Exception e) {
log.error(e);
return e.toString();
}
}2. 沒有考慮失敗情況
一開始只考慮成功場景，等后面測試發(fā)現(xiàn)有錯誤情況，怎么辦，改接口唄，前后臺都改，勞民傷財無用功。
錯誤范例：
//不返回任何數(shù)據(jù)，沒有考慮失敗場景，容易返工
＠PostMapping("/update")
public void update(long id, xxx) {
}3. 出現(xiàn)和業(yè)務(wù)無關(guān)的輸入?yún)?shù)
如lang語言，當(dāng)前用戶信息 都不應(yīng)該出現(xiàn)參數(shù)里面，應(yīng)該從當(dāng)前會話里面獲取。后面講ThreadLocal會說到怎么樣去掉。除了代碼可讀性不好問題外，尤其是參數(shù)出現(xiàn)當(dāng)前用戶信息的，這是個嚴(yán)重問題。
錯誤范例：
// （當(dāng)前用戶刪除數(shù)據(jù)）參數(shù)出現(xiàn)lang和userid，尤其是userid，大忌
＠PostMapping("/delete")
public Map<String, Object> delete(long id, String lang, String userId) {
}4. 出現(xiàn)復(fù)雜的輸入?yún)?shù)
一般情況下，不允許出現(xiàn)例如json字符串這樣的參數(shù)，這種參數(shù)可讀性極差。應(yīng)該定義對應(yīng)的bean。
錯誤范例：
// 參數(shù)出現(xiàn)json格式，可讀性不好，代碼也難看
＠PostMapping("/update")
public Map<String, Object> update(long id, String jsonStr) {
}5. 沒有返回應(yīng)該返回的數(shù)據(jù)
例如，新增接口一般情況下應(yīng)該返回新對象的id標(biāo)識，這需要編程經(jīng)驗。新手定義的時候因為前臺沒有用就不返回數(shù)據(jù)或者只返回true，這都是不恰當(dāng)?shù)摹e人要不要是別人的事情，你該返回的還是應(yīng)該返回。
錯誤范例：
// 約定俗成，新建應(yīng)該返回新對象的信息，只返回boolean容易導(dǎo)致返工
＠PostMapping("/add")
public boolean add(xxx) {
//xxx
return configService.add();
}很多人都覺得技術(shù)也很簡單，沒有什么特別的地方，但是，實現(xiàn)這個代碼框架之前，就是要你的接口的統(tǒng)一的格式ResultBean，aop才好做。有些人誤解了，上周末那篇文章說的都不是技術(shù)，重點說的是編碼習(xí)慣工作方式，如果你重點還是放在什么技術(shù)上，那我也幫不了你了。同樣，如果我后面的關(guān)于習(xí)慣和規(guī)范的帖子，你重點還是放在技術(shù)上的話，那是丟了西瓜撿芝麻，有很多貼還是沒有任何技術(shù)點呢。
附上ResultBean，沒有任何技術(shù)含量：
＠Data
public class ResultBean<T> implements Serializable {
private static final long serialVersionUID = 1L;
public static final int SUCCESS = 0;
public static final int FAIL = 1;
public static final int NO_PERMISSION = 2;
private String msg = "success";
private int code = SUCCESS;
private T data;
public ResultBean() {
super();
}
public ResultBean(T data) {
super();
this.data = data;
}
public ResultBean(Throwable e) {
super();
this.msg = e.toString();
this.code = FAIL ;
}
}統(tǒng)一的接口規(guī)范，能幫忙規(guī)避很多無用的返工修改和可能出現(xiàn)的問題。能使代碼可讀性更加好，利于進行aop和自動化測試這些額外工作。大家一定要重視。
二. Controller規(guī)范
上面2段代碼，第一個是原生態(tài)的，第2段是我指定了接口定義規(guī)范，使用AOP技術(shù)之后最終交付的代碼，從15行到1行，自己感受一下。接下來說說大家關(guān)注的AOP如何實現(xiàn)。
先說說Controller規(guī)范，主要的內(nèi)容是就是接口定義里面的內(nèi)容，你只要遵循里面的規(guī)范，controller就問題不大，除了這些，還有另外的幾點：
1.所有函數(shù)返回統(tǒng)一的ResultBean/PageResultBean格式
原因見我的接口定義這個貼。沒有統(tǒng)一格式，AOP無法玩。
2.ResultBean/PageResultBean是controller專用的，不允許往后傳！
3.Controller做參數(shù)格式的轉(zhuǎn)換，不允許把json，map這類對象傳到services去，也不允許services返回json、map。
一般情況下！寫過代碼都知道，map，json這種格式靈活，但是可讀性差，如果放業(yè)務(wù)數(shù)據(jù)，每次閱讀起來都比較困難。定義一個bean看著工作量多了，但代碼清晰多了。
4.參數(shù)中一般情況不允許出現(xiàn)Request，Response這些對象
主要是可讀性問題。一般情況下。
5.不需要打印日志
日志在AOP里面會打印，而且我的建議是大部分日志在Services這層打印。
規(guī)范里面大部分是 不要做的項多，要做的比較少，落地比較容易。
ResultBean定義帶泛型，使用了lombok。
@Data
public class ResultBean<T> implements Serializable {
private static final long serialVersionUID = 1L;
public static final int NO_LOGIN = -1;
public static final int SUCCESS = 0;
public static final int FAIL = 1;
public static final int NO_PERMISSION = 2;
private String msg = "success";
private int code = SUCCESS;
private T data;
public ResultBean() {
super();
}
public ResultBean(T data) {
super();
this.data = data;
}
public ResultBean(Throwable e) {
super();
this.msg = e.toString();
this.code = FAIL;
}
}AOP代碼，主要就是打印日志和捕獲異常，異常要區(qū)分已知異常和未知異常，其中未知的異常是我們重點關(guān)注的，可以做一些郵件通知啥的，已知異常可以再細分一下，可以不同的異常返回不同的返回碼：
/**
* 處理和包裝異常
*/
public class ControllerAOP {
private static final Logger logger = LoggerFactory.getLogger(ControllerAOP.class);
public Object handlerControllerMethod(ProceedingJoinPoint pjp) {
long startTime = System.currentTimeMillis();
ResultBean<?> result;
try {
result = (ResultBean<?>) pjp.proceed();
logger.info(pjp.getSignature() + "use time:" + (System.currentTimeMillis() - startTime));
} catch (Throwable e) {
result = handlerException(pjp, e);
}
return result;
}
private ResultBean<?> handlerException(ProceedingJoinPoint pjp, Throwable e) {
ResultBean<?> result = new ResultBean();
// 已知異常
if (e instanceof CheckException) {
result.setMsg(e.getLocalizedMessage());
result.setCode(ResultBean.FAIL);
} else if (e instanceof UnloginException) {
result.setMsg("Unlogin");
result.setCode(ResultBean.NO_LOGIN);
} else {
logger.error(pjp.getSignature() + " error ", e);
//TODO 未知的異常，應(yīng)該格外注意，可以發(fā)送郵件通知等
result.setMsg(e.toString());
result.setCode(ResultBean.FAIL);
}
return result;
}
}AOP配置：(關(guān)于用java代碼還是xml配置，這里我傾向于xml配置，因為這個會不定期改動)
<!-- aop -->
<aop:aspectj-autoproxy />
<beans:bean id="controllerAop" class="xxx.common.aop.ControllerAOP" />
<aop:config>
<aop:aspect id="myAop" ref="controllerAop">
<aop:pointcut id="target"
expression="execution(public xxx.common.beans.ResultBean *(..))" />
<aop:around method="handlerControllerMethod" pointcut-ref="target" />
</aop:aspect>
</aop:config>現(xiàn)在知道為什么要返回統(tǒng)一的一個ResultBean了：1.為了統(tǒng)一格式 ；2.為了應(yīng)用AOP ；3.為了包裝異常信息。
分頁的PageResultBean大同小異，大家自己依葫蘆畫瓢自己完成就好了。
貼一個簡單的controller（左邊的箭頭表示AOP攔截了）。請對比 吐槽我見過的最爛的java代碼里面原來的代碼查看，沒有對比就沒有傷害。
最后說一句，先有統(tǒng)一的接口定義規(guī)范，然后有AOP實現(xiàn)。先有思想再有技術(shù)。技術(shù)不是關(guān)鍵，AOP技術(shù)也很簡單，這個帖子的關(guān)鍵點不是技術(shù)，而是習(xí)慣和思想，不要撿了芝麻丟了西瓜。網(wǎng)絡(luò)上講技術(shù)的貼多，講習(xí)慣、風(fēng)格的少，這些都是我工作多年的行之有效的經(jīng)驗之談。