今天在瀏覽網(wǎng)站的時(shí)候，看到別人發(fā)的這么一個(gè)帖子，剛剛?cè)肼氁粋€(gè)新公司，代碼審查的時(shí)候，leader 對(duì)他的代碼進(jìn)行了一些修改，而這個(gè)程序員感覺(jué)很多地方?jīng)]有必要，你們看完上面這個(gè)帖子什么感覺(jué)?

看法

我看的看法是：

一是，遇到這樣的領(lǐng)導(dǎo)真的很好，咱先不討論領(lǐng)導(dǎo)這樣的修改，有些地方是否有沒(méi)有必要，光看領(lǐng)導(dǎo)這么事無(wú)巨細(xì)的在這些小地方都幫你 code review 進(jìn)行一些修改，就說(shuō)明領(lǐng)導(dǎo)非常負(fù)責(zé)，領(lǐng)導(dǎo)的這些修改和你的哪個(gè)更規(guī)范?這個(gè)不好說(shuō)，但是領(lǐng)導(dǎo)的修改我個(gè)人認(rèn)為確實(shí)很規(guī)范，最起碼沒(méi)錯(cuò)。

二是，我認(rèn)為確實(shí)領(lǐng)導(dǎo)的一些修改沒(méi)有必要。就比如：上面的那個(gè)我畫(huà)紅框的地方，把 setVisible 換成了 show ，其實(shí)沒(méi)必要，但是我認(rèn)為領(lǐng)導(dǎo)的那個(gè)更容易讓人看懂和辨識(shí)。還收上面的那個(gè)常量的命名，領(lǐng)導(dǎo)也給修改，其實(shí)確實(shí)也是沒(méi)必要的地方。

還有一個(gè)地方比如：a.do1() a.do2() ，領(lǐng)導(dǎo)給修改成 a.do1.do2()，或許沒(méi)必要，但是領(lǐng)導(dǎo)的這個(gè)修改可以讓代碼更簡(jiǎn)潔，看起來(lái)更方便，在維護(hù)代碼和更新迭代上來(lái)講，確實(shí)讓你一眼就懂，很清楚，方便整個(gè)團(tuán)隊(duì)工作的管理和交接。

想法

其實(shí)，作為一個(gè)團(tuán)隊(duì)來(lái)講，首先看看整個(gè)團(tuán)隊(duì)有沒(méi)有代碼規(guī)約和規(guī)范，里面是怎么規(guī)定這個(gè)變量，常量，方法函數(shù)的命名的，如果這個(gè)團(tuán)隊(duì)里有代碼規(guī)約就是這么制定的命名規(guī)則，我們還是應(yīng)該按照這個(gè)規(guī)則來(lái)命名。

你想想一下：

一個(gè)團(tuán)隊(duì)的 leader 下面十幾個(gè)人，你是想讓領(lǐng)導(dǎo)適應(yīng)十幾個(gè)人的風(fēng)格，還是讓十幾個(gè)人統(tǒng)一到領(lǐng)導(dǎo)的風(fēng)格?

代碼風(fēng)格和規(guī)范統(tǒng)一了，才利于整個(gè)團(tuán)隊(duì)代碼的維護(hù)和交接，有利于代碼的管理和升級(jí)。這就要求團(tuán)隊(duì)必須有一個(gè)代碼規(guī)范。歡迎大家關(guān)注我的微信公眾號(hào)：非著名程序員

比如：上述程序員，不滿(mǎn)意領(lǐng)導(dǎo)的修改，你先看看團(tuán)隊(duì)里有沒(méi)有代碼規(guī)范，代碼規(guī)范是對(duì)于命名是怎么規(guī)定呢?如果有，是你沒(méi)有按照規(guī)范來(lái)使用，那就是你的問(wèn)題，如果沒(méi)有規(guī)范，那你可以找 leader 談一談，團(tuán)隊(duì)?wèi)?yīng)該制定一個(gè)規(guī)則，能否出個(gè)規(guī)則，以后我按照這個(gè)規(guī)約來(lái)寫(xiě)，也可以減輕領(lǐng)導(dǎo) code review 的工作量。(http://godcoder.me/)

代碼評(píng)審

為什么要進(jìn)行代碼評(píng)審?

1. 提高質(zhì)量
2. 及早發(fā)現(xiàn)潛在缺陷與 BUG，降低事故成本。
3. 促進(jìn)團(tuán)隊(duì)內(nèi)部知識(shí)共享，提高團(tuán)隊(duì)整體水平
4. 評(píng)審過(guò)程對(duì)于評(píng)審人員來(lái)說(shuō)，也是一種思路重構(gòu)的過(guò)程。幫助更多的人理解系統(tǒng)。

其實(shí)，我認(rèn)為代碼評(píng)審，不僅僅是領(lǐng)導(dǎo)的事，每天抽出一個(gè)小時(shí)，團(tuán)隊(duì)里每個(gè)人都對(duì)其他人的代碼進(jìn)行評(píng)審也是非常好的，不僅可以找到各自身上寫(xiě)代碼的缺陷和毛病，還可以學(xué)習(xí)別人寫(xiě)代碼的優(yōu)點(diǎn)。畢竟評(píng)審過(guò)程對(duì)于評(píng)審人員來(lái)說(shuō)，也是一種思路重構(gòu)的過(guò)程。歡迎大家關(guān)注我的微信公眾號(hào)：非著名程序員

另外，整個(gè)團(tuán)隊(duì)必須要有一個(gè)明確的代碼規(guī)范和規(guī)約的好處是，code review 應(yīng)該是做重要的事，而不是花在這些不規(guī)則的命名上，命名的事，讓規(guī)約來(lái)約束大家，code review 最重要的是提高代碼的質(zhì)量，發(fā)現(xiàn)潛在缺陷與 BUG，尋找項(xiàng)目模塊中不合理的地方，比如：系統(tǒng)關(guān)鍵模塊，業(yè)務(wù)較復(fù)雜的模塊，缺陷率較高的模塊等。

***，我想說(shuō)，截圖上的那個(gè)領(lǐng)導(dǎo)，確實(shí)水平很高，光從命名上來(lái)講，確實(shí)很規(guī)范，雖然可能有點(diǎn)較真和過(guò)了，但是確實(shí)值得學(xué)習(xí)。