這是我見過寫得最爛的Controller層代碼,沒有之一!
來源:https://zhuanlan.zhihu.com/p/28708259 超全技術(shù)棧的Java入門+進階+實戰(zhàn)!(非白嫖,點擊查看) 一. 接口定義 工作中,少不了要定義各種接口,系統(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: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)驗之談。
胖虎聯(lián)合一線大廠朋友花費8個月的時間,錄制了一份Java入門+進階視頻教程
課程特色:
總共88G,時常高達365小時,覆蓋所有主流技術(shù)棧
均為同一人錄制,不是東拼西湊的
對標(biāo)線下T0級別的培訓(xùn)課,講師大廠架構(gòu)師,多年授課經(jīng)驗,通俗易懂
內(nèi)容豐富,每一個技術(shù)點除了視頻,還有課堂源碼、筆記、PPT、圖解
五大實戰(zhàn)項目(視頻+源碼+筆記+SQL+軟件)
一次付費,持續(xù)更新,永無二次費用
點擊下方超鏈接查看詳情

(或者點擊文末閱讀原文):


