2012-01-05 114 views
2

只是在這裏出去。正在做一個代碼審查,通過抽樣幾個類和代碼單元由開發人員足以判斷他提供輸入做錯了什麼?這是爲了防止時間緊迫,無法檢查他們寫的每一行。我正在假設開發人員可能在各處複製相同類型的錯誤。這是一種有效的方法嗎?或部分有效,或根本沒有效?代碼審查方法

還有哪些其他方法更好?與每位編寫代碼行的開發人員坐在一起,詢問他們爲什麼以特定方式做到這一點?或者自己做,然後通過電子郵件發送你的發現摘要?

當您必須在典型的15天衝刺中查看所有代碼庫時,您如何處理由初級開發人員/同行編寫的代碼審查代碼?

回答

3

我不認爲挑選幾個類和單位就足以知道代碼是可以接受的。您可以設法選擇完美編寫的代碼中的幾個部分,並忽略大問題。

我認爲如果時間有限,那麼你需要考慮優化你的審查方式。首先,您可以運行一些工具(StyleCop,FxCop和許多其他工具)來自動查看常見錯誤或編碼標準偏差。

此後我想你需要弄清楚重要功能是什麼,並逐行檢查它們。如果需要發送大綱,然後與開發人員坐下來似乎是一個明智的做法。

這可能只是第一組需要全面檢查的代碼,如果你可以找時間做這件事以後看看發生了什麼變化通常不會太糟糕,但這一切都取決於你的團隊規模。

1

作爲一名初級開發人員,我發現如果你有時間,坐下是最好的。當有些事情不清楚或者是錯誤的時候,更容易解釋我想要親自去做的事情。

查看編寫的每一行代碼都是過度的 - QA存在是有原因的。你正在尋求確保我們遵循公司的代碼規範,並防止我們沒有足夠經驗認識的愚蠢東西是愚蠢的。