鎮樓小姐姐
36份一線互聯網Java面試電子書
84個Java稀缺面試題視頻
今天在瀏覽網站的時候，看到別人發的這麼一個帖子，剛剛入職一個新公司，代碼審查的時候，leader 對他的代碼進行了一些修改，而這個程序員感覺很多地方沒有必要，你們看完上面這個帖子什麼感覺？
看法
我看的看法是：
一是，遇到這樣的領導真的很好，咱先不討論領導這樣的修改，有些地方是否有沒有必要，光看領導這麼事無鉅細的在這些小地方都幫你 code review 進行一些修改，就說明領導非常負責，領導的這些修改和你的哪個更規範？這個不好說，但是領導的修改我個人認為確實很規範，最起碼沒錯。
二是，我認為確實領導的一些修改沒有必要。就比如：上面的那個我畫紅框的地方，把 setVisible 換成了 show ，其實沒必要，但是我認為領導的那個更容易讓人看懂和辨識。還收上面的那個常量的命名，領導也給修改，其實確實也是沒必要的地方。
還有一個地方比如：a.do1() a.do2() ，領導給修改成 a.do1.do2()，或許沒必要，但是領導的這個修改可以讓代碼更簡潔，看起來更方便，在維護代碼和更新迭代上來講，確實讓你一眼就懂，很清楚，方便整個團隊工作的管理和交接。
想法
其實，作為一個團隊來講，首先看看整個團隊有沒有代碼規約和規範，裡面是怎麼規定這個變量，常量，方法函數的命名的，如果這個團隊裡有代碼規約就是這麼制定的命名規則，我們還是應該按照這個規則來命名。
你想想一下：
一個團隊的 leader 下面十幾個人，你是想讓領導適應十幾個人的風格，還是讓十幾個人統一到領導的風格？
代碼風格和規範統一了，才利於整個團隊代碼的維護和交接，有利於代碼的管理和升級。這就要求團隊必須有一個代碼規範。
比如：上述程序員，不滿意領導的修改，你先看看團隊裡有沒有代碼規範，代碼規範是對於命名是怎麼規定呢？如果有，是你沒有按照規範來使用，那就是你的問題，如果沒有規範，那你可以找 leader 談一談，團隊應該制定一個規則，能否出個規則，以後我按照這個規約來寫，也可以減輕領導 code review 的工作量。
代碼評審
為什麼要進行代碼評審？
1、提高質量
2、及早發現潛在缺陷與 BUG，降低事故成本。
3、促進團隊內部知識共享，提高團隊整體水平
4、評審過程對於評審人員來說，也是一種思路重構的過程。幫助更多的人理解系統。
另外，整個團隊必須要有一個明確的代碼規範和規約的好處是，code review 應該是做重要的事，而不是花在這些不規則的命名上，命名的事，讓規約來約束大家，code review 最重要的是提高代碼的質量，發現潛在缺陷與 BUG，尋找項目模塊中不合理的地方，比如：系統關鍵模塊，業務較複雜的模塊，缺陷率較高的模塊等。
最後，我想說，截圖上的那個領導，確實水平很高，光從命名上來講，確實很規範，雖然可能有點較真和過了，但是確實值得學習。
閱讀更多 李紅 的文章
