Code Review 有感

關於最近一次Code Review 的一些想法@TOC

前言

團隊內部會經常進行code review ,以前並沒review 自己的代碼,一直作爲一個旁觀者的角度去看問題。這次review 自己的代碼之後,讓自己有意思到很多的不足。

Code Review 有感

  • 關於服務的邊界?
    • 之前自己從來沒有思考過這個問題,都是從以滿足業務爲基本要求去寫代碼。舉一個簡單的例子,之前如果要提供一個修改狀態的接口,我可能只會對外暴露一個updateStatus 的接口,他們可以把狀態(例如:INIT,FAIL,ERROR,SUCCESS)作爲一個參數傳進來,調用者想要更新爲什麼狀態,都可以(當然,不是所有的狀態修改都會成功,在服務內可以做一狀態的校驗)。
    • 在code review的時候,他們認爲應該對外提供多個接口,比如updateStatus2SUC/updateStatus2Fail,在自己服務內部的時候,可以重用大部分的代碼。這樣只給接口調用者最小的權限。
  • 關於接口中參數的定義?
    • 之前自己定義接口中的dto 的時候,屬性基本上和domain是一樣的。所以像updateTime,createTime 這樣的屬性都是由接口調用者設置好之後傳到服務端,然後設置/保存到數據庫的。其實自己也知道,想updateTime 這種字段是不需要傳的,可以直接使用數據的時間。這次code review的時候,被同事明確指出來,以後這種字段能不傳就不傳,一些可以確定的字段不需要傳,比如創建時間,初始化狀態是INIT……

希望自己在下次code review的時候有更好的表現_

發表評論
所有評論
還沒有人評論,想成為第一個評論的人麼? 請在上方評論欄輸入並且點擊發布.
相關文章