歡迎來到合肥浪訊網絡科技有限公司官網
咨詢服務熱線：400-099-8848
搜索引擎一直不收錄網站怎么辦 |
| 發布時間：2016-10-07 文章來源： 瀏覽次數：2126 |
| 代碼審查(Code Review)是軟件開發中常用的手段，和QA測試比擬，它更輕易發現和架構以及時序相關等較難發現的題目，還可以匡助團隊成員進步編程技能，同一編程風格等。 1. 代碼審查要求團隊有良好的文化 團隊需要熟悉到代碼審查是為了進步整個團隊的能力，而不是針對個體設置的檢查“關卡”。 “A的代碼有個bug被B發現，所以A能力不行，B能力更好”，這一類的陷阱很輕易被擴散從而影響團隊內部的協作，因此需要避免。 另外，代碼審查本身可以進步開發者的能力，讓其從自身犯過的錯誤中學習，從他人的思路中學習。假如開發者對這個流程有抵觸或者反感，這個目的就達不到。 2. 謹嚴的使用審查中題目的發現率作為考評尺度 高效代碼審查的十個經驗 在代碼審查中假如發現題目，對于題目的發現者來說這是好事，應該予以鼓勵。但對于被發現者，我們不主張使用這個方式予以懲罰。軟件開發中bug在所難免，過度苛求本身有悖常理。更糟的是，假如造成介入者怕承擔責任，不愿意在審查中指出題目，代碼審查就沒有任何的價值和意義。 3. 控制每次審查的代碼數目 根據smartbear在思科所作的調查，每次審查200行-400行的代碼效果最好。每次試圖審查的代碼過多，發現題目的能力就會下降. 高效代碼審查的十個經驗 我們在實踐中發現，跟著開發平臺和開發語言的不同，最優的代碼審查量有所不同。但是限制每次審查的數目確實非常必要，由于這個過程是高強度的腦力密集型流動。時間一長，代碼在審查者眼里只是字母，無任何邏輯聯系，天然不會有太多的產出。 4. 帶著題目去進行審查 我們在每次代碼審查中，要求審查者利用自身的經驗先思索可能會遇到的題目，然后通過審查工作驗證這些題目是否已經解決。一個竅門是，從用戶可見的功能出發，假設一個比較復雜的使用場景，在代碼閱讀中驗證這個使用場景是否能夠準確工作。 使用這個技巧，可以讓審查者有代入感，真正的陶醉入代碼中，進步效率。大家都知道看武俠小說不輕易瞌睡兒，而看專業書輕易瞌睡兒，原因就是武俠小說更輕易產生代入感。 有的研究建議每次樹立目標，控制單位時間內審核的代碼數目。這個方法在我們的實踐中顯得很機械和流程化，不如上面的方法效果好。 5. 所有的題目和修改，必需由原作者進行確認 假如在審查中發現題目，務必由原作者進行確認。 這樣做有兩個目的： (1)確認題目確實存在，保證題目被解決 (2)讓原作者了解題目和不足，匡助其成長 有些時候為了追求效率，有經驗的審查者更傾向于直接修改代碼乃至重構所有代碼，但這樣不利于進步團隊效率，并且會增加由于重構引入新bug的幾率，通常情況下我們不予鼓勵。 6.利用代碼審查激活個體“能動性" 即使項目進度比較緊張，無法完全的進行代碼審查，至少也要進行部門代碼的審查，此時隨即抽取一些樞紐部門是個不錯的辦法。 背后的邏輯是，軟件開發長短常有創造性的工作，開發者都有強烈的自我驅動性和自我實現的要求。閃開發者知道他寫的任何代碼都可能被其他人閱讀和審察，可以促使開發者集中留意力，尤其是避免將質量糟糕，乃至有初級錯誤的代碼提交給同伴審查。開源軟件也很好的利用了這種心態來進步代碼質量。 7.在非正式，輕松的環境下進行代碼審查 如前所述，代碼審查是一個腦力密集型的工作。介入者需要在比較輕松的環境下進行該工作。因此，我們以為像某些實踐中建議的那樣，以會議的形式進行代碼審查效果并不好，不僅由于長時間的會議輕易讓效率低下，更由于會議上可能泛起的爭議和思索不利于進行如斯復雜的工作。 8.提交代碼前自我審查，添加對代碼的說明 所有團隊成員在提交代碼給其他成員審查前，必需提高前輩行一次審查。這次自我修正形式的審查除了檢查代碼的準確性以外，還可以完成如下的工作： (1)對代碼添加注釋，說明本次修改背后的原因，利便其他人進行審查。 (2)修正編碼風格，尤其是一些樞紐數據結構和方法的命名，進步代碼的可讀性。 (3)從全局審閱設計，是否完整的考慮了所有情景。在實現之前做的設計假如存在考慮不周的情況，這個階段可以很好的進行補救。 我們在實踐中發現，即使只有原作者進行代碼審查，仍舊可以很好的進步代碼質量。 9.實現中記實筆記可以很好的進步題目發現率 成員在編碼的時候應做隨手記實，包括在代碼頂用注釋的方式表示，或者記實簡樸的個人文檔，這樣做有幾個好處： (1)避免漏掉。在編碼時將考慮到的任何題目都記實下來，在審查階段再次檢查這些題目都確認解決。 (2)根據研究，每個人都習慣犯一些重復性的錯誤。這類題目在編碼是記實下來，可以在審查的時候用作檢查的依據。 (3)在反復記實筆記并在審查中發現類似的題目后，該類題目泛起率會明顯下降 10. 使用好的工具進行輕量級的代碼審查 “工欲善其事，必先利其器”。我們使用的是bitbucket提供的代碼托管服務。 每個團隊成員獨立開發功能，然后利用Pull Request的形式將代碼提交給審查者。復審者可以很利便在網頁上閱讀代碼，添加評論等，然后原作者會自動收到郵件提醒，對審視的意見進行討論。 即使團隊成員分布在天南海北，利用bitbucket提供的工具也能很好的進行代碼審查。 |
|
