年度程式碼滾動更新站點的前端程式碼審查問題摘要



阿里梅指南。

* 回顧在技術團隊的工程師文化建設是很有意義的，這是形成統一團隊作風的最有效方法，筆者自己團隊在一年的CR中常見的小問題做了一些梳理，希望能對大家有所幫助。 一、引言。

團隊已經開展了一年多的線下週評，作為主評委，我原本以為給出**建議會花費大量的時間和精力，一直以為對被評審者改進**設計問題會有很大的幫助，但實際上評審發現大部分學生還存在一些基本的**質量問題， 而改進設計或發現業務bug以達到目標的審查很少出現，所以我覺得團隊對CR還是應該有足夠的耐心，把基本問題當成乙個bug來解釋，在通往優秀工程的路上積累起來。同時，我也能深深感受到，每個學生對**的態度都不一樣，這就產生了兩個差異，有的人進步很快，有的人會不斷重複複習過的問題。我希望它對每個人都能幫助不大。二、翻車場景（CR中的常見問題）。

2.1 **規範類

2.1.1 使用法力值展期指數：描述：Mana 表示乙個未在 ** 中宣告並直接使用的值，這在我們的 ** 複習問題中很普遍。

bad case

const now = date.now();const lastvisittime = localstorage.getitem('last_visit_time');

if (lastvisittime &&parseint(lastvisittime, 10) >24 * 60 * 60 * 1000)

localstorage.setitem('last_visit_time', now.tostring())

const last_visit_time_cache_key = 'last_visit_time';const day_in_ms = 24 * 60 * 60 * 1000;

const now = date.now();const lastvisittime = localstorage.getitem(last_visit_time_cache_key);

if (lastvisittime &&parseint(lastvisittime, 10) >day_in_ms)

localstorage.setitem(last_visit_time_cache_key, now.tostring())

2.1.3 使用 Ghost 依賴項展期指數：描述：幽靈依賴項（也稱為隱式依賴項或隱式依賴項）是不在專案模組中的包，但不在該模組中直接在 JSON 檔案中宣告的依賴項。

2.1.4 不遵循“不歸還”原則展期指數：描述：no-else-return 是乙個質量規則，強調簡潔性和可讀性，用於指示當 if 塊已包含 return 語句時無需使用 else 塊。 因為 if 塊中的 return 語句會在執行 if 塊中的 return 語句後立即退出當前函式，所以 else 是多餘的。 ❌ you can skip the else block

function hello(condition: boolean) else

function hello(condition: boolean)

return '