一般公司真的會有CodeReview這件事情嗎？ - 工程師

原文吃光,代po

大叔我跟你一樣自學 .NET MVC，
也是非資工系畢業，
也曾在一人公司寫程式寫到覺得人生無望看不到盡頭，
每天就這樣敲鍵盤，好像寫程式就 "只是這樣" 爾爾，

直到三、四年前無意間碰觸到了以下...
Agile software development,
Test Drive Development,
Code Principle,
Design Pattern,
Clean code,
Clean Architecture

感謝 Bob 大叔，讚嘆 Bob 大叔。

才了解到，原來從初心者到我 "以為老手" 的這段期間
程式碼都在亂寫。

相信寫程式的人都曾經出現過，
幹，這段誰寫的看不懂，
看了git 才發現是自己上個月加的...

要怎麼避免這種問題，
一來自我要求，二來做 code review 讓大家要求你。

回到原文的問題，一般公司有沒有做 code review?
就我待過的公司，只有我現在的公司有在做，
而且是我主動要求部門開始做的。

那... Code review 到底可以 review 哪些東西?

提供平常在幫同事看程式的幾個方向
1. Clean code，命名有沒有正確，表達有沒有符合意義，類別函式數量是否最小化，註
解有沒有亂寫
2. Code principle，單一職責，開放封閉，依賴反轉，李式替換，介面隔離都做到了?
3. 相似的程式碼是否有抽象化，是否套用 Design pattern
4. 不能有重覆的程式碼 (很重要)
5. 測試程式涵蓋率是否完善 (非常重要)

其實啊，
講這麼多，
最重要的是，
公司要有 "程式人員" 懂這些東西，
才有辦法做這種 code review，

若公司沒有這樣的人，
那就 "自學" 讓自己變成更好的 Programer，
不要只是當個 Coder。

--