代碼評審方法6(代碼評審該如何做)
1.代碼評審該如何做
評審尤其是同行評審，是提前發(fā)現(xiàn)缺陷，降低開發(fā)成本的有效方法；但是很多企業(yè)并不重視，或者效果不佳。
原因有很多：沒有獲得合適的專家、組織不合理、方法不當(dāng)?shù)取ａ槍Υa評審，我想簡單地談幾點(diǎn)（其實(shí)其它工程文檔的評審存在同樣問題）：1、評審要計(jì)劃。
因?yàn)榇蠹叶己苊?，各領(lǐng)域的專家也都有各自的工作，評審結(jié)果的好壞對他們工作績效沒有影響，所以專家投入多少精力來用于評審不受控，完全靠他們自己的職業(yè)精神。 所以，這就要求項(xiàng)目經(jīng)理比較有影響力，能夠爭取到足夠的、合適的專家來參與評審。
專家最好要提前協(xié)調(diào)，當(dāng)然項(xiàng)目經(jīng)理首先要知道要請哪些專家（如系統(tǒng)專家、領(lǐng)域?qū)＜?、測試人員、QA等），這些專家在哪里？能否獲得專家資源的支持，是項(xiàng)目經(jīng)理的項(xiàng)目管理能力一個(gè)重要體現(xiàn)。評審時(shí)間的投入比例，一般是第一次版本準(zhǔn)備時(shí)間的3/5以上。
2、評審要分層次和重點(diǎn)。作為項(xiàng)目成員，在代碼編寫完成后，首先是自檢，這時(shí)發(fā)現(xiàn)的缺陷不計(jì)算；然后是項(xiàng)目組內(nèi)的評審，這時(shí)發(fā)現(xiàn)的缺陷應(yīng)計(jì)算；最后才是外部評審。
應(yīng)提前把相關(guān)文檔和代碼交給評審專家，讓他們各自評審；根據(jù)項(xiàng)目和評審結(jié)果的情況，決定是否召開評審會議，評審會議以確認(rèn)問題為主，而不是討論解決方案。對于重要的核心的代碼，要投入重點(diǎn)評審力量；另外，對于能力不足的成員所完成的代碼，也要重點(diǎn)評審。
每個(gè)角色也應(yīng)明白各自評審重點(diǎn)，如QA重點(diǎn)放在編程規(guī)范、測試人員側(cè)重在可測性、系統(tǒng)專家側(cè)重在從整體來考慮（如對其它功能的影響、性能等）。3、問題的確認(rèn)與跟蹤。
評審專家發(fā)現(xiàn)問題后，首先與該代碼的完成人進(jìn)行確認(rèn)，如果大家達(dá)成一致認(rèn)可這個(gè)問題，然后由代碼完成人提出解決方案，這個(gè)解決方案要得到問題發(fā)現(xiàn)者的同意，然后編程人員編碼實(shí)現(xiàn)該方案，并進(jìn)行測試和驗(yàn)證，并將驗(yàn)證結(jié)果提交問題發(fā)現(xiàn)人，問題發(fā)現(xiàn)人確認(rèn)無誤后，該問題就可關(guān)閉。所以，評審專家發(fā)現(xiàn)問題后并沒有結(jié)束，而是要負(fù)責(zé)跟蹤該問題。
總之，評審活動的組織其實(shí)比較復(fù)雜，也是項(xiàng)目經(jīng)理項(xiàng)目管理能力的一個(gè)重要體現(xiàn)，項(xiàng)目經(jīng)理對評審活動負(fù)有重要責(zé)任。
2.代碼評審該如何做
代碼review主要就是事前的準(zhǔn)備，執(zhí)行，和后期的跟蹤事前的準(zhǔn)備也叫做review的輸入，這個(gè)輸入是在什么條件下可以做為輸入，可以是個(gè)人檢查完了，也可以是相互交叉的檢查，這里有個(gè)檢查的標(biāo)準(zhǔn)，如有個(gè)檢查單什么的，等規(guī)定的輸入完成之后，進(jìn)行review的執(zhí)行在review中，主要就是首先保證什么人參加，這是最主要的，如果找?guī)讉€(gè)沒有什么經(jīng)驗(yàn)的，那review的效果肯定是打折扣的，review的多少時(shí)間跟蹤，主要是問題的跟蹤和后期的數(shù)據(jù)的度量你上面說的相互評審，主要是項(xiàng)目內(nèi)部的review,review分好幾種，有內(nèi)部的，專家的，客戶的review代碼時(shí)主要看是否符合編碼規(guī)范，特別是是否符合設(shè)計(jì)書（需求）的要求，邏輯是否正確，主要是專家級！。
3.評標(biāo)方法有哪幾種
常用的評標(biāo)方法有 1單項(xiàng)評議法 2綜合評標(biāo)法 3合理單價(jià)評標(biāo)法 常用的評標(biāo)方法介紹 1單項(xiàng)評議法 單項(xiàng)評議法，又稱單因素評議法、低標(biāo)價(jià)法，是一種只對投標(biāo)人的投標(biāo)報(bào)價(jià)進(jìn)行評議從而確定中標(biāo)人的評標(biāo)定標(biāo)方法，主要適用于小型工程。
單項(xiàng)評議法的主要特點(diǎn)是僅對價(jià)格因素進(jìn)行評議，不考慮其他因素，報(bào)價(jià)低的投標(biāo)人中標(biāo)。當(dāng)然，這里未考慮的其他因素，實(shí)際上在資格審查時(shí)已獲通過，只不過不作為評標(biāo)定標(biāo)時(shí)的考慮因素，因而也不是投標(biāo)人競爭成敗的決定性因素。
采用單項(xiàng)評議法評標(biāo)定標(biāo)，決定成敗的唯一因素是標(biāo)價(jià)的高低。但不能簡單的認(rèn)為，標(biāo)價(jià)越低越能中標(biāo)。
一般的做法是，通過對投標(biāo)書進(jìn)行分析、比較，經(jīng)初審后，篩選出低標(biāo)價(jià)，通過進(jìn)一步的澄清和答辯，經(jīng)終審證明該低標(biāo)價(jià)確實(shí)是切實(shí)可行、措施得當(dāng)?shù)暮侠淼蜆?biāo)價(jià)的，則確定該合理低標(biāo)價(jià)中標(biāo)。合理低標(biāo)價(jià)不一定是最低投標(biāo)價(jià)。
所以，單項(xiàng)評議法可以是最低投標(biāo)價(jià)中標(biāo)，但并不保證最低投標(biāo)價(jià)必然中標(biāo)。 采用單項(xiàng)評議法對投標(biāo)報(bào)價(jià)進(jìn)行評議的方法多種多樣，主要有以下三類具有代表性的模式： （1）將投標(biāo)報(bào)價(jià)與標(biāo)底價(jià)相比較的評議方法 這種方法是將各投標(biāo)人的投標(biāo)報(bào)價(jià)直接與經(jīng)招標(biāo)投標(biāo)管理機(jī)構(gòu)審定后的標(biāo)底價(jià)相比較，以標(biāo)底價(jià)為基礎(chǔ)來判斷投標(biāo)報(bào)價(jià)的優(yōu)劣，經(jīng)評標(biāo)被確認(rèn)為合理低標(biāo)價(jià)的投標(biāo)報(bào)價(jià)即能中標(biāo)。
（2）將各投標(biāo)報(bào)價(jià)相互進(jìn)行比較的評議方法 從純粹擇優(yōu)的角度看，可以對投標(biāo)人的投標(biāo)報(bào)價(jià)不做任何限制、不附加任何條件，只將各投標(biāo)人的投標(biāo)報(bào)價(jià)相互進(jìn)行比較，而不與標(biāo)底相比，經(jīng)評標(biāo)確認(rèn)投標(biāo)報(bào)價(jià)屬最低價(jià)或次低價(jià)的（即為合理低標(biāo)價(jià)的），即可中標(biāo)。 這種對投標(biāo)報(bào)價(jià)的評議方法，優(yōu)點(diǎn)是給了投標(biāo)人充分自主報(bào)價(jià)的自由，標(biāo)底的保密性不成問題，評標(biāo)工作也比較簡單。
不足之處是，招標(biāo)人無需編制標(biāo)底，或雖有標(biāo)底但形同虛設(shè)，不起什么作用，因而導(dǎo)致投標(biāo)人對投標(biāo)報(bào)價(jià)的預(yù)期和認(rèn)同心中無數(shù)，事實(shí)上處于一種盲目狀態(tài)，很難說清楚是否科學(xué)、合理。而投標(biāo)人為了中標(biāo)常常會進(jìn)行竟相壓價(jià)的惡性競爭，也極易形成串通投標(biāo)。
（3）將投標(biāo)報(bào)價(jià)與標(biāo)底價(jià)結(jié)合投標(biāo)人報(bào)價(jià)因素進(jìn)行比較的評議方法 這種方法的特點(diǎn)，是要借助于一個(gè)可以作為評標(biāo)定標(biāo)參照物的價(jià)格。這個(gè)在評標(biāo)定標(biāo)中作為參照物的價(jià)格，是指投標(biāo)報(bào)價(jià)最接近于該價(jià)時(shí)便能中標(biāo)的價(jià)格，我們稱之為“最佳評標(biāo)價(jià)”。
2綜合評標(biāo)法 綜合評議法，是對價(jià)格、施工組織設(shè)計(jì)（或施工方案）、項(xiàng)目經(jīng)理的資歷和業(yè)績、質(zhì)量、工期、信譽(yù)和業(yè)績等因素進(jìn)行綜合評價(jià)從而確定中標(biāo)人的評標(biāo)定標(biāo)方法。它是適用最廣泛的評標(biāo)定標(biāo)方法，各地通常都采用這種方法。
綜合評議法按具體分析方式的不同，可分為定性綜合評議法和定量綜合評議法。 （1）定性綜合評議法（評議法） 定性綜合評議法，又稱評議法。
通常的做法是，由評標(biāo)組織對工程報(bào)價(jià)、工期、質(zhì)量、施工組織設(shè)計(jì)、主要材料消耗、安全保障措施、業(yè)績、信譽(yù)等評審指標(biāo)，分項(xiàng)進(jìn)行定性比較分析，綜合考慮，經(jīng)評議后，選出其中被大多數(shù)評標(biāo)組織成員認(rèn)為各項(xiàng)條件都比較優(yōu)良的投標(biāo)人為中標(biāo)人，也可用記名或無記名投票表決的方式確定中標(biāo)人。定性評議法的特點(diǎn)是不量化各項(xiàng)評審指標(biāo)。
它是一種定性的優(yōu)選法。采用定性綜合評議法，一般要按從優(yōu)到劣的順序，對各投標(biāo)人排列名次，排序第一名的既為中標(biāo)人。
但當(dāng)投標(biāo)人超過一定數(shù)量（如在5家以上）時(shí)，可以選擇排序第二名的投標(biāo)人為中標(biāo)人。 采用定性綜合評議法，有利于評標(biāo)組織成員之間的直接對話和交流，能充分反映不同意見，在廣泛深入地開展討論、分析的基礎(chǔ)上，集中大多數(shù)人的意見，一般也比較簡單易行。
但這種方法，評議標(biāo)準(zhǔn)彈性較大，衡量的尺度不具體，各人的理解可能會相去甚遠(yuǎn)，造成評標(biāo)意見懸殊過大，會使評標(biāo)決策左右為難，不能讓人信服。 （2）定量綜合評議法（打分法、百分法） 定量綜合評議法，又稱打分法、百分制計(jì)分評議法（百分法）。
通常的做法是，事先在招標(biāo)文件或評標(biāo)定標(biāo)辦法中將評標(biāo)的內(nèi)容進(jìn)行分類，形成若干評價(jià)因素，并確定各項(xiàng)評價(jià)因素在百分之內(nèi)所占的比例和評分標(biāo)準(zhǔn)，開標(biāo)后由評標(biāo)組織中的每位成員按照評分規(guī)則，采用無記名方式打分，最后統(tǒng)計(jì)投標(biāo)人的得分，得分最高者（排序第一名）或次高者（排序第二名）為中標(biāo)人。 采用定量綜合評議法，原則上實(shí)行得分最高的投標(biāo)人為中標(biāo)人。
但當(dāng)招標(biāo)工程在一定限額（如1000萬元）以上，最高得分者和次高得分者的總得分差距不大（如差距僅在2分之內(nèi)），且次高得分者的報(bào)價(jià)比最高得分者的報(bào)價(jià)低到一定數(shù)額（如低2%以上）的，可以選擇次高得分者為中標(biāo)人。對此，在制定評標(biāo)定標(biāo)辦法時(shí)，應(yīng)作出詳盡說明。
定量綜合評議法的主要特點(diǎn)是要量化各評審因素。對各評審因素的量化是一個(gè)比較復(fù)雜的問題，各地的做法不盡相同。
從理論上講，評標(biāo)因素指標(biāo)的設(shè)置和評分標(biāo)準(zhǔn)分值的分配，應(yīng)充分體現(xiàn)企業(yè)的整體素質(zhì)和綜合實(shí)力，準(zhǔn)確反映公開、公平、公正的竟標(biāo)法則，使質(zhì)量好、信譽(yù)高、價(jià)格合理、技術(shù)強(qiáng)、方案優(yōu)的企業(yè)能中標(biāo)。 3合理單價(jià)評標(biāo)法 “合理低價(jià)評標(biāo)法”應(yīng)該是“最低價(jià)中標(biāo)”評標(biāo)法的一次揚(yáng)棄，存其優(yōu)化競爭的一面，棄。
4.代碼走查的代碼走查與代碼審查
代碼走查（code walkthrough）和代碼審查（code inspection）是兩種不同的代碼評審方法，
代碼審查是一種正式的評審活動，而代碼走查的討論過程是非正式的。
最近對項(xiàng)目組進(jìn)行代碼評審，發(fā)覺需要對代碼評審中找到的問題進(jìn)行一下分類，大概可以分成以下幾類問題：
1. Comment
注釋沒寫，或者格式不對，或者毫無意義
2. Coding Standard
沒遵守代碼規(guī)范
3. Existing Wheel
重復(fù)現(xiàn)成的代碼，或者是開源項(xiàng)目，或者公司已有代碼
4. Better practice
Java或者開源項(xiàng)目，有更好的寫法
5. Performance bottle and Improvement
性能瓶頸和提高
6. Code Logic Error
代碼邏輯錯(cuò)誤
7. Business Logic Error
業(yè)務(wù)邏輯錯(cuò)誤
代碼審查列出問題的類型，并有解決情況報(bào)告
5.評標(biāo)辦法中一共有幾種方法
1、根據(jù)《評標(biāo)委員會和評標(biāo)方法暫行規(guī)定（國家發(fā)展計(jì)劃委員會、國家經(jīng)濟(jì)貿(mào)易委員會、建設(shè)部、鐵道部、交通部、信息產(chǎn)業(yè)部、中水利部令第12號）》規(guī)定：
“第二十九條 評標(biāo)方法包括經(jīng)評審的最低投標(biāo)價(jià)法、綜合評估法或者法律、行政法規(guī)允許的其他評標(biāo)方法?！?/p>
2、根據(jù)上文，評標(biāo)方法分為最低投標(biāo)價(jià)法、綜合評估法兩大類。實(shí)際應(yīng)用的各種評標(biāo)方法多種多樣，基本屬于以上兩類。其中最低投標(biāo)價(jià)法定義簡單、明確。而綜合評估法設(shè)定方法多種多樣，內(nèi)容繁多，體現(xiàn)形式也各有不同。
3、抽簽法等不能體現(xiàn)競爭、擇優(yōu)評標(biāo)方法，不屬于法律認(rèn)可范圍。
6.代碼走查的代碼走查與代碼審查
代碼走查（code walkthrough）和代碼審查（code inspection）是兩種不同的代碼評審方法，
代碼審查是一種正式的評審活動，而代碼走查的討論過程是非正式的。
最近對項(xiàng)目組進(jìn)行代碼評審，發(fā)覺需要對代碼評審中找到的問題進(jìn)行一下分類，大概可以分成以下幾類問題：
1. Comment
注釋沒寫，或者格式不對，或者毫無意義
2. Coding Standard
沒遵守代碼規(guī)范
3. Existing Wheel
重復(fù)現(xiàn)成的代碼，或者是開源項(xiàng)目，或者公司已有代碼
4. Better practice
Java或者開源項(xiàng)目，有更好的寫法
5. Performance bottle and Improvement
性能瓶頸和提高
6. Code Logic Error
代碼邏輯錯(cuò)誤
7. Business Logic Error
業(yè)務(wù)邏輯錯(cuò)誤
代碼審查列出問題的類型，并有解決情況報(bào)告
7.怎樣進(jìn)行代碼的review
什么是Code Review?
Code Review代碼評審是指在軟件開發(fā)過程中，通過對源代碼進(jìn)行系統(tǒng)性檢查的過程。通常的目的是查找各種缺陷，包括代碼缺陷、功能實(shí)現(xiàn)問題、編碼合理性、性能優(yōu)化等；保證軟件總體質(zhì)量和提高開發(fā)者自身水平。 Code Review是輕量級代碼評審，相對于正式代碼評審，輕量級代碼評審所需要的各種成本要明顯低得多，如果流程正確，它可以起到更加積極的效果。正因如此，輕量級代碼評審經(jīng)常性地被引入到軟件開發(fā)過程中。
為什么Code Review?
1. 提高代碼質(zhì)量。
2. 及早發(fā)現(xiàn)潛在缺陷，降低修改/彌補(bǔ)缺陷的成本。
3. 促進(jìn)團(tuán)隊(duì)內(nèi)部知識共享，提高團(tuán)隊(duì)整體水平。
4. 評審過程對于評審人員來說，也是一種思路重構(gòu)的過程。幫助更多的人理解系統(tǒng)。
5. 是一個(gè)傳遞知識的手段，可以讓其它并不熟悉代碼的人知道作者的意圖和想法，從而可以在以后輕松維護(hù)代碼。
6. 鼓勵(lì)程序員們相互學(xué)習(xí)對方的長處和優(yōu)點(diǎn)。
7. 可以被用來確認(rèn)自己的設(shè)計(jì)和實(shí)現(xiàn)是一個(gè)清楚和簡單的。
8.代碼審查能夠?qū)δ男﹩栴}提出建議
評審尤其是同行評審，是提前發(fā)現(xiàn)缺陷，降低開發(fā)成本的有效方法；但是很多企業(yè)并不重視，或者效果不佳。
原因有很多：沒有獲得合適的專家、組織不合理、方法不當(dāng)?shù)?。針對代碼評審，我想簡單地談幾點(diǎn)（其實(shí)其它工程文檔的評審存在同樣問題）：1、評審要計(jì)劃。
因?yàn)榇蠹叶己苊Γ黝I(lǐng)域的專家也都有各自的工作，評審結(jié)果的好壞對他們工作績效沒有影響，所以專家投入多少精力來用于評審不受控，完全靠他們自己的職業(yè)精神。 所以，這就要求項(xiàng)目經(jīng)理比較有影響力，能夠爭取到足夠的、合適的專家來參與評審。
專家最好要提前協(xié)調(diào)，當(dāng)然項(xiàng)目經(jīng)理首先要知道要請哪些專家（如系統(tǒng)專家、領(lǐng)域?qū)＜?、測試人員、QA等），這些專家在哪里？能否獲得專家資源的支持，是項(xiàng)目經(jīng)理的項(xiàng)目管理能力一個(gè)重要體現(xiàn)。評審時(shí)間的投入比例，一般是第一次版本準(zhǔn)備時(shí)間的3/5以上。
2、評審要分層次和重點(diǎn)。作為項(xiàng)目成員，在代碼編寫完成后，首先是自檢，這時(shí)發(fā)現(xiàn)的缺陷不計(jì)算；然后是項(xiàng)目組內(nèi)的評審，這時(shí)發(fā)現(xiàn)的缺陷應(yīng)計(jì)算；最后才是外部評審。
應(yīng)提前把相關(guān)文檔和代碼交給評審專家，讓他們各自評審；根據(jù)項(xiàng)目和評審結(jié)果的情況，決定是否召開評審會議，評審會議以確認(rèn)問題為主，而不是討論解決方案。對于重要的核心的代碼，要投入重點(diǎn)評審力量；另外，對于能力不足的成員所完成的代碼，也要重點(diǎn)評審。
每個(gè)角色也應(yīng)明白各自評審重點(diǎn)，如QA重點(diǎn)放在編程規(guī)范、測試人員側(cè)重在可測性、系統(tǒng)專家側(cè)重在從整體來考慮（如對其它功能的影響、性能等）。3、問題的確認(rèn)與跟蹤。
評審專家發(fā)現(xiàn)問題后，首先與該代碼的完成人進(jìn)行確認(rèn)，如果大家達(dá)成一致認(rèn)可這個(gè)問題，然后由代碼完成人提出解決方案，這個(gè)解決方案要得到問題發(fā)現(xiàn)者的同意，然后編程人員編碼實(shí)現(xiàn)該方案，并進(jìn)行測試和驗(yàn)證，并將驗(yàn)證結(jié)果提交問題發(fā)現(xiàn)人，問題發(fā)現(xiàn)人確認(rèn)無誤后，該問題就可關(guān)閉。所以，評審專家發(fā)現(xiàn)問題后并沒有結(jié)束，而是要負(fù)責(zé)跟蹤該問題。
總之，評審活動的組織其實(shí)比較復(fù)雜，也是項(xiàng)目經(jīng)理項(xiàng)目管理能力的一個(gè)重要體現(xiàn)，項(xiàng)目經(jīng)理對評審活動負(fù)有重要責(zé)任。
9.如何進(jìn)行代碼審查
首先，我們先來看看Code Reivew的用處：
Code reviews 中，可以通過大家的建議增進(jìn)代碼的質(zhì)量。
Code reviews 是一個(gè)傳遞知識的手段，可以讓其它并不熟悉代碼的人知道作者的意圖和想法，從而可以在以后輕松維護(hù)代碼。
Code reviews 也鼓勵(lì)程序員們相互學(xué)習(xí)對方的長處和優(yōu)點(diǎn)。
Code reviews 也可以被用來確認(rèn)自己的設(shè)計(jì)和實(shí)現(xiàn)是一個(gè)清楚和簡單的。
你也許注意到了在上面的Code Reivew中的諸多用處中，我們沒有提到可以幫助找到程序的bug和保證代碼風(fēng)格和編碼標(biāo)準(zhǔn)。這是因?yàn)槲覀冋J(rèn)為：
Code reviews 不應(yīng)該承擔(dān)發(fā)現(xiàn)代碼錯(cuò)誤的職責(zé)。Code Review主要是審核代碼的質(zhì)量，如可讀性，可維護(hù)性，以及程序的邏輯和對需求和設(shè)計(jì)的實(shí)現(xiàn)。代碼中的bug和錯(cuò)誤應(yīng)該由單元測試，功能測試，性能測試，回歸測試來保證的（其中主要是單元測試，因?yàn)槟鞘亲罱咏麭ug，也是Bug沒有擴(kuò)散的地方）
Code reviews 不應(yīng)該成為保證代碼風(fēng)格和編碼標(biāo)準(zhǔn)的手段。編碼風(fēng)格和代碼規(guī)范都屬于死的東西，每個(gè)程序員在把自己的代碼提交團(tuán)隊(duì)Review的時(shí)候，代碼就應(yīng)該是符合規(guī)范的，這是默認(rèn)值，屬于每個(gè)人自己的事情，不應(yīng)該交由團(tuán)隊(duì)來完成，否則只會浪費(fèi)大家本來就不夠的時(shí)間。我個(gè)人認(rèn)為“meeting”是奢侈的，因?yàn)槟切枰蠹以谕粫r(shí)刻都擠出時(shí)間，所以應(yīng)該用在最需要的地方。代碼規(guī)范比起程序的邏輯和對需求設(shè)計(jì)的實(shí)現(xiàn)來說，太不值得讓大家都來了。
10年前，上面這兩件事會是理所當(dāng)然的（10年前的中國的軟件開發(fā)還沒有Code Reivew呢），今天，在中國的很多公司上面這兩件事依然被認(rèn)為是Code Reivew最重要的事，所以，我能夠看到很多開發(fā)Team抱怨Code Review就是一個(gè)形式，費(fèi)時(shí)費(fèi)力不說，發(fā)現(xiàn)的問題還不如測試，而評審者們除了在代碼風(fēng)格上有些見術(shù)，別的也就沒什么用了，長而久之，大家都會開始厭煩這個(gè)事了。
所以，在今天，請不要把上面的那兩件事分散了Code Review的注意力，取而代之的是，對于Bug，程序的作者要在Review前提交自己的單元測試報(bào)告（如：XUnit的測試結(jié)果），對于代碼規(guī)范，這是程序作者自己需要保證的，而且，有一些工具是可以幫你來檢查代碼規(guī)范的。
