CodeReview是什么呢？
什么是codereview？字面意思是審查您的代碼，Talkis便宜，showme代碼是它的意思。越來越多的公司要求研發(fā)團(tuán)隊(duì)在代碼開發(fā)過程中進(jìn)行CODEREVIEW（CR）。在確保代碼質(zhì)量的同時，它促進(jìn)了團(tuán)隊(duì)成員之間的溝通，并提交了代碼級別。然而，近年來，CR文化的促進(jìn)只開始了。像亞馬遜和谷歌這樣的外國公司需要在代碼合并到主分支機(jī)構(gòu)時完成CR。國內(nèi)互聯(lián)網(wǎng)比國外延遲，加上一個略微復(fù)雜的漢語特征（我對自己的其他人寫的壞代碼不太尷尬......）。早期Cr更多的形式。現(xiàn)在隨著互聯(lián)網(wǎng)流量的迅速增加，必須保證代碼質(zhì)量，以確保業(yè)務(wù)的高穩(wěn)定性和高可用性，因此CR開始成為企業(yè)發(fā)展的必備現(xiàn)象。簡單地說，Cr就像吃早餐。如果你不吃它，你的身體似乎似乎并不糟糕。因此，由于忙碌的工作或睡覺后，每個人都能忽視早餐或只是吃飯和吃東西來應(yīng)對。然而，由于身體在高負(fù)荷和高壓下逐漸變得有點(diǎn)不同，因此每個人都開始意識到“身體是革命的首都”。必須吃早餐，必須始終如一地食用。這對身體最有利。更進(jìn)一步，以力量更好地做到。但為什么你吃早餐，當(dāng)你吃它時，你是如何吃它的，你吃的東西也很優(yōu)雅。就像當(dāng)Cr會這樣做時，它將如何做到，應(yīng)該如何注重，也很細(xì)膩。
為什么？ - 你為什么需要審查？ CR是代碼標(biāo)準(zhǔn)化的保證，帶來知識傳播和團(tuán)隊(duì)建設(shè)。有些人可能認(rèn)為代碼審查是找到錯誤，并且不需要代碼審查。事實(shí)上，這不是這種情況。從編碼人的角度來看，他們每天都忙于編碼，交貨時間即將推出。為了快速交付，要求減少，未寫入單元測試用例，并且在編碼時的性能和安全性的角度下不考慮更好的實(shí)施。但是，雖然代碼按時提交，但它不是一種高質(zhì)量的代碼。在運(yùn)營中可能存在問題，并且人們難以接管。如果有CR，認(rèn)為您的代碼將由您的同事和領(lǐng)導(dǎo)審查，您還會降低要求嗎？我必須加班完成以質(zhì)量和數(shù)量的代碼寫作。此外，在CR的過程中，有高級老年人解釋您的代碼設(shè)計思想和算法。這絕對比降低你的頭腦更快，思考進(jìn)步。 CR.判斷的原因就像和家人和朋友一起吃早餐。當(dāng)你孤獨(dú)時，你會醒來醒來或匆忙吃東西。如果您的家人或朋友與您同住，您會考慮您的家庭的健康和您的家人對您的擔(dān)憂。 ，你還不會早起吃早餐甚至喝早餐？什么時候？ - 審查是什么？每個代碼合并（PullRequest / Merge Request）是最好的時間。 PullRequest意味著您無權(quán)提交您寫入特定倉庫或分支的代碼。您可以請授權(quán)人員將您從源倉庫的源分支中提交的代碼合并到目標(biāo)倉庫的目標(biāo)分支中。每個要求更改應(yīng)通過提交合并請求盡快合并到主分支機(jī)構(gòu)中，因此可以盡早找到代碼寫作中的問題。我們現(xiàn)在所倡導(dǎo)的持續(xù)整合也是相同的想法。不要等到所有要求的開發(fā)和合并。一次提交大量代碼也會為審稿人帶來巨大的負(fù)擔(dān)。修改一次并提交一次。當(dāng)然，這份提交也是一個質(zhì)量提交，至少在提交之前，我已經(jīng)完全審查并通過了單位測試。 Cr評論的時間就像做早餐一樣。你必須嘗試用正確的原料和煮熟，甜蜜和咸的東西吃它，然后讓別人一起吃飯。如何？ - 怎么判斷？選擇合適的工具，與合適的開發(fā)過程合作，選擇正確的形式非常重要。對于工具，許多代碼托管工具，如Github，Gitlab，Alibaba云云和騰訊工作人員Bee都有自己的CR工具，而開發(fā)團(tuán)隊(duì)可以根據(jù)自己的情況選擇。對于開發(fā)過程，當(dāng)前流行的Gitflow，Trunk開發(fā)模型和Fork開發(fā)模型所有支持PullRequest / MergeRequest當(dāng)代碼合并到主分支時。
出于適當(dāng)?shù)谋砀瘢ㄔ诰€審查，離線審查和特殊處理，為輕量級CR（如小型功能模塊的開發(fā)，小于500行代碼），您可以直接邀請代碼托管工具人員中的同一組高級人士審查代碼并與反饋結(jié)合進(jìn)行更改;為了開發(fā)大型功能模塊或架構(gòu)的變化，團(tuán)隊(duì)成員可以組織用于離線評論。開發(fā)人員談?wù)撟约旱脑O(shè)計邏輯，審稿人提供意見，代碼審核行按行;對于一些緊急情況，例如需要迫切地推出的線路上的緊急錯誤，但沒有人在那里，可以在此時進(jìn)行緊急合并，但之后仍然需要添加CR。 CR.評估方法就像吃早餐。如果你是一個人，它可以更簡單，牛奶面包可以補(bǔ)充必要的蛋白質(zhì);如果是一個家庭在一起，那么它肯定會更富裕。饅頭，粥，大豆牛奶，油炸面團(tuán)棒和泡菜都會有一點(diǎn)，所有谷物都會補(bǔ)充;如果是緊急火車或飛行，你可以跳過吃它并等到你上火車或飛機(jī)來彌補(bǔ)它。什么？ - 審查是什么？ CR評論什么？在CR中，我們審查了標(biāo)準(zhǔn)化，一致性，編碼樣式，代碼的安全問題，代碼冗余和代碼的功能性能設(shè)計。關(guān)于標(biāo)準(zhǔn)化，在Java中，我們將檢查后臺線程是否具有對主線程的同步訪問，無論是功能和變量命名是否準(zhǔn)確，是否組件分層是合理的，無論是合理提取的常用邏輯是否合理，是文件組織是否是合理的邏輯合理，無論函數(shù)評論是清晰全面的，代碼的可讀性都很好，無論是更優(yōu)雅的寫作方式，程序設(shè)計是否滿足單個原理，開放和封閉的原理。為了完整性，我們檢查代碼是否完全實(shí)現(xiàn)了設(shè)計文檔中提出的功能要求，無論是否已創(chuàng)建所需的數(shù)據(jù)庫，以及它是否包含正確的初始化數(shù)據(jù)。對于正確性，我們檢查是否正確定義并使用了所有變量，無論是否使用未定義的變量，是否存在明顯或潛在的邏輯錯誤，無論它們都意外落入無限循環(huán)，是否已經(jīng)避免了無限遞歸。對于穩(wěn)健性，我們檢查代碼是否與陣列折疊或內(nèi)存溢出的例外處理。為了重新使用，我們檢查組件是可重復(fù)使用的以及是否有重復(fù)代碼。為了可伸縮性，我們檢查功能組件是否易于展開以及是否可以重用代碼。為了安全，我們檢查是否執(zhí)行身份驗(yàn)證，授權(quán)，輸入數(shù)據(jù)驗(yàn)證，避免安全威脅，如SQL注入和跨站點(diǎn)腳本（XSS），加密敏感數(shù)據(jù)（密碼，信用卡信息等）以及是否導(dǎo)入依賴項(xiàng)是安全的，將成熟，公共組件和工具函數(shù)的變化會影響其他業(yè)務(wù)嗎？可以看出CR不是一個簡單的事情。一個好的代碼和良好的工程師必須被鍛煉。 CR的評論內(nèi)容就像早餐一樣。我們將注意哪些成分在營養(yǎng)方面是否均衡，無論烹飪是合適的，無論是含量是否足夠，無論是成分都是安全的，無論是清潔，無論是清潔的，無論是清潔的，衛(wèi)生，無論價格合適， 等等。不要說經(jīng)營迭代太多，需求太多，發(fā)射時間緊張。沒有時間做Cr。不要為您的丑陋代碼制作華麗的借口。如果您沒有時間做好準(zhǔn)備，您將有很多時間來處理錯誤和投訴。 。就像不要說你沒有時間吃早餐，工作太忙或困倦，你現(xiàn)在的拯救時間將被未來的各種疾病令人作嘔。因此，如果您的團(tuán)隊(duì)沒有完成CR，并且CR并不好好，您必須推動您的領(lǐng)導(dǎo)者找到根本原因，并實(shí)施CR。對于質(zhì)量代碼，一切都值得。就像那樣，如果你還沒有享用美味的早餐，那么從現(xiàn)在吃早餐時，就會為健康的身體值得一切值得。
?
