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