工作中,少不了要定義各種接口,系統集成要定義接口,前後臺掉調用也要定義接口。接口定義一定程度上能反應程序員的編程功底。列舉一下工作中我發現大家容易出現的問題:
1. 返回格式不統一
同一個接口,有時候返回數組,有時候返回單個;成功的時候返回對象,失敗的時候返回錯誤信息字符串。工作中有個系統集成就是這樣定義的接口,真是辣眼睛。這個對應代碼上,返回的類型是map,json,object,都是不應該的。實際工作中,我們會定義一個統一的格式,就是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. 沒有考慮失敗情況
一開始只考慮成功場景,等後面測試發現有錯誤情況,怎麼辦,改接口唄,前後臺都改,勞民傷財無用功。
錯誤範例:
//不返回任何數據,沒有考慮失敗場景,容易返工
@PostMapping("/update")
public void update(long id, xxx) {
}
3. 出現和業務無關的輸入參數
如lang語言,當前用戶信息 都不應該出現參數裏面,應該從當前會話裏面獲取。後面講ThreadLocal會說到怎麼樣去掉。除了代碼可讀性不好問題外,尤其是參數出現當前用戶信息的,這是個嚴重問題。
錯誤範例:
// (當前用戶刪除數據)參數出現lang和userid,尤其是userid,大忌
@PostMapping("/delete")
public Map<String, Object> delete(long id, String lang, String userId) {
}
4. 出現複雜的輸入參數
一般情況下,不允許出現例如json字符串這樣的參數,這種參數可讀性極差。應該定義對應的bean。
錯誤範例:
// 參數出現json格式,可讀性不好,代碼也難看
@PostMapping("/update")
public Map<String, Object> update(long id, String jsonStr) {
}
5. 沒有返回應該返回的數據
例如,新增接口一般情況下應該返回新對象的id標識,這需要編程經驗。新手定義的時候因爲前臺沒有用就不返回數據或者只返回true,這都是不恰當的。別人要不要是別人的事情,你該返回的還是應該返回。
錯誤範例:
// 約定俗成,新建應該返回新對象的信息,只返回boolean容易導致返工
@PostMapping("/add")
public boolean add(xxx) {
//xxx
return configService.add();
}
很多人看了我的這篇文章 程序員你爲什麼這麼累?,都覺得裏面的技術也很簡單,沒有什麼特別的地方,但是,實現這個代碼框架之前,就是要你的接口的統一的格式ResultBean,aop纔好做。有些人誤解了,我那篇文章說的都不是技術,重點說的是編碼習慣工作方式,如果你重點還是放在什麼技術上,那我也幫不了你了。同樣,如果我後面的關於習慣和規範的帖子,你重點還是放在技術上的話,那是丟了西瓜撿芝麻,有很多貼還是沒有任何技術點呢。
附上ResultBean,沒有任何技術含量:
@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 ;
}
}
統一的接口規範,能幫忙規避很多無用的返工修改和可能出現的問題。能使代碼可讀性更加好,利於進行aop和自動化測試這些額外工作。大家一定要重視。
本文作者:曉風輕
原文鏈接:https://zhuanlan.zhihu.com/p/28708259
版權歸作者所有,轉載請註明出處