天天看點

如何code review?

一.目标(按重要程度排序)

    1.結構和設計(方法、類、包、檔案等)

        --1、代碼需要清晰明了

        --2、代碼類、方法、注解、注釋設計規範

        --3、是否備援、代碼語義是否含糊不清

    2.代碼邏輯(正确性、bug)

        --1、執行前的檢查是否完善

        --2、不同條件下是否和期望的結果相同

    3.安全設計(判空、異常處理、記憶體)

        --1、連接配接用完關閉

        --2、第三方調用傳回結果校驗,正确處理異常

        --3、字段類型、值校驗

        --4、異常捕獲、處理規範

        --5、輸入輸出的校驗以及特殊值校驗(如SQL/URL等)

    4.運維設計(日志、名額)

        --1、日志必須齊全、級别設定規範、位置規範

        --2、有監控位置規範、名額齊全。

    5.性能設計(記憶體、資源、性能)

        --1、大資源不能直接加載到記憶體中

        --2、避免嵌套循環

    6.代碼風格

        --1、代碼行間距、列間距規範、縮進規範等

        --2、變量命名以及類等規範

二.PR原則

    1.改動盡量小

    2.對自己代碼負責(非常重要)

三.review原則

    1.review其他同僚是幫助其他同僚成長

    2.review優質代碼是幫助自己成長