Re: [請益]一般公司真的會有CodeReview這件事情嗎?

作者: HappyBitter (Smile)   2018-11-08 12:08:33
原文吃光,代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。
作者: kai0914 (好人)   2018-11-08 12:52:00
就只勝硬體思維製造出身的老闆不懂啊
作者: Rocker5566 (搖滾56)   2018-11-08 13:17:00
問題是很多主管自己也不懂 嘻嘻
作者: badyy (nick)   2018-11-08 13:28:00
還有聽過外包品質越差越好,不好維護是績效!能讓客戶離不開你的XD
作者: lalalalaluk (luk)   2018-11-08 13:52:00
作者: bigbite (子子孫孫永保用)   2018-11-08 14:28:00
是test "driven" XD
作者: zorogto (卡稱)   2018-11-08 16:32:00
作者: qswitch (燕子)   2018-11-09 03:43:00
客戶表示火大
作者: espirit (espirit)   2018-11-09 14:47:00
不能有重複的程式碼? 壓縮程式壓過的最精簡。這要看是否為獨立的慨念。
作者: pig2014 (Rocking Man)   2018-11-10 09:01:00
李氏替換

Links booklink

Contact Us: admin [ a t ] ucptt.com