[心得] 如何進行code review
1.善用自動排版
2.自動排版後再手動檢查重點,最近常遇到的有PSR格式的問題
a. 標頭<?php 下面無條件空行
b. use namespace 獨立一行
c. if 上面空行,以及括號空行
d. array() 全部改成 $array = ["a" => $b]; 有換行要對齊
e. 奇怪的註解或dump() 要清乾淨
f. 沒用的區塊要砍掉
g. 每支API註解寫清楚功能
h. 等號兩邊空格,逗號位置,類別變數檔名路徑命名方式...,同功能的扣放同一塊
3. 善用 git diff 加上diff color 來檢查更動紀錄,以及有沒有多餘空白
4. 可以在vim 中加入行尾空白提示
5.測試覆蓋率
2.自動排版後再手動檢查重點,最近常遇到的有PSR格式的問題
a. 標頭<?php 下面無條件空行
b. use namespace 獨立一行
c. if 上面空行,以及括號空行
d. array() 全部改成 $array = ["a" => $b]; 有換行要對齊
e. 奇怪的註解或dump() 要清乾淨
f. 沒用的區塊要砍掉
g. 每支API註解寫清楚功能
h. 等號兩邊空格,逗號位置,類別變數檔名路徑命名方式...,同功能的扣放同一塊
3. 善用 git diff 加上diff color 來檢查更動紀錄,以及有沒有多餘空白
4. 可以在vim 中加入行尾空白提示
5.測試覆蓋率
留言
張貼留言