我是软件业小白
以前也不是读相关科系的
第一份工作也没上过什么资策会
就是莫名其妙就开始写Code了
大多数的东西都是自学而来
不管是上班还是下班
现在都在写.net Mvc
然后前端搭一点Jquery和简单的vue
不知道为啥写着写着挺有兴趣的
可是现在蛮想离职的
主要是觉得成长的速度变慢了
东西都靠自己撞墙撞出来，没人带
都是说你做这个，偶尔加个关键字这样
然后就让我自己去撞墙了
有时候也怕自己撞墙撞出来的结论是错的
虽然东西是有出来
印象中听乡民说有所谓的Code Review
正常一般公司真的会有CodeReview吗？
我是指以软件为主要商品的公司
然后这个时期找工作是不是死定了
可是最近离职的念头好大
一直被当神灯精灵
累
然后问一下.net MVC要到怎样的水准才能在台北领个45k?
作者:
visa9527 (高级伴读士官长)
2018-11-07 15:12:00不一定，但如果是 team work 通常会有这类公司新人进去就会先发 code 准则，像国军准则一样要背要看要记牢，变量怎么命名、换行时机、缩排要领一些固定基本的东西跟单兵战斗词一样要背清楚然后结构要跟单兵战斗一样演练，function scope 掩护obj每周开会检讨 git 上的 commit 问其他人有什么想法麻烦一点你想用内插法解的也被大家打枪要用积分函数做做到最后都要经神分裂了，但 Leader 会觉得这叫有品质
没有review肯定是不行的，但是review也分很多种
作者:
visa9527 (高级伴读士官长)
2018-11-07 15:30:00绝对不分明啊XDD这事是所有不相干的人都要跳下来大乱斗
作者:
westron (魏斯通)
2018-11-07 15:30:00有code review才是对的，一堆人在改又不做review 才可怕
我们家会，不过通常都是主管抠而已，只有大功能多叫一两个人一起看
作者:
visa9527 (高级伴读士官长)
2018-11-07 15:33:00你负责写通讯协定，做UI的跟做数据库的都可能来 review数据库的说你协定有缺陷，大家同意，你就得改
作者:
westron (魏斯通)
2018-11-07 15:34:00code review 的目的是要同事之间不要犯相同的错误
作者:
visa9527 (高级伴读士官长)
2018-11-07 15:35:00然后你认为你那边是完美的，就要提出报告说明设计原理最后会花很多时间，但可以事先把隐藏风险找出来
作者:
westron (魏斯通)
2018-11-07 15:36:00就如同你说，你的东西都是自己google来的，你也不晓得好不好，就是因为这样才要做code review同事之间才可以互相学习而不是互相拖累
作者:
visa9527 (高级伴读士官长)
2018-11-07 15:36:00比如你的协定可能忽略 timeout 等例外处理或处理不够好而做数据库的因为常提供 API 所以知道 timeout 有啥问题
作者:
westron (魏斯通)
2018-11-07 15:38:00review的过程才会让像你这样没经验的小嫩嫩成长
作者:
visa9527 (高级伴读士官长)
2018-11-07 15:38:00但你可能一开始就设计可忽略 response 的单向通讯协定结果跟做数据库的争执不下，其他同事觉得他有道理就挺他
作者:
westron (魏斯通)
2018-11-07 15:39:00如果你不觉得别人的code会成长或是检讨自己的code会让自己成长，那你就真的是太自以为是了
作者:
visa9527 (高级伴读士官长)
2018-11-07 15:40:00即便你对自己设计有信心也会因为 review 被迫改掉Code review是一种维持品质但不见得有开创性的事最后的结果是你妥协了用公司一脉相传的通讯协定方法但你开创性设计不会被产品所用，产品品质稳定却无突破实际上 Code review 是很容易跟业界脱勾的，切身之痛除非 team 每个都自主学习新知，不然就是守旧为主因为要维持程式码的品质来保持产品的稳定度除非新专案诉求新的规格，不然review就是要你跟着部队走你不可以走太快，也不可以落队，要跟着部队脚步行军y
作者:
Kimheeche (Kimheeche)
2018-11-07 16:33:00会哦 我们sw team 都有做 基本都是 gerrit +git
作者:
yamakazi (大安吴彦祖)
2018-11-07 17:25:00我公司用fisheye做review 有点规模的公司软件部门应该都会做review不过我之前在某TK韧体部门 review是找间会议室 开sourceinsight大家围着投影萤幕听作者报告后来到另家公司用fisheye做review 才知道原来有这种东西perforce+fisheye
作者:
Sex5F (HTC)
2018-11-07 17:33:00纯软没code review? 光奇怪命名就可以搞死后人了
作者:
iiiii (I take 5)
2018-11-07 17:50:00同事把review当成周报进度才是崩溃，人人都是小主管，写越多越屎
作者:
ejnfu ((-. .-)b)
2018-11-07 18:03:00有
作者: qxeacwzd (mnnm) 2018-11-07 18:11:00
另类封建制度，你有+2吗
作者: uiop019 (uiop019) 2018-11-07 18:19:00
我同事mvc 架构是在易游网练出来，随时可扩充，架构就是很完善
作者:
Jyery (æ–‡å¸)
2018-11-07 18:22:00去银行阿 都写.net而且一定有人带
作者:
z3939889 (zazaa)
2018-11-07 18:22:00没code review你家QA应该会很想宰了你
作者: A0091127 (A009) 2018-11-07 19:39:00
我们公司RD,一星期一次code review,以team为单位一起开
有没有code review，真的很可怕，我们部门刚好就有对照组
作者: Mozar (沉默的乐师) 2018-11-07 19:50:00
有code review, 本来就该有
作者: t64141 (榕树) 2018-11-07 20:07:00
没有code review很可怕，尤其是频繁转手又历史悠久的专案
作者:
chuegou (chuegou)
2018-11-07 20:24:00我希望有 因为我待的公司都没有 一份code有四五种缩排排版命名风格 编译警告破500 魔术数字没有说明 非常痛苦尤其我们是一人维护一个机种 所以问人不一定知道
一般一部分整个公司只有一个人管吧 到底怎review不懂的人来看还不是找话讲而已
作者:
loadingN (sarsaparilla)
2018-11-07 20:37:00上一个没有按照code rule的已经被学长肛到不要不要
没办法想像没有code review整个公司只有一个人code review，小公司吗？
不是 是一个部分只有你维护 那别人怎看得懂?前因后果都不懂 不就只能挑格式而已了
作者:
chocopie (好吃的巧克力派 :))
2018-11-07 21:58:00为了Code Review而Code Review，不如没有
作者: SY082022 (没有暱称) 2018-11-07 22:06:00
每个人的程式逻辑都不太一样，有时后code review只是一堆人给一堆意见，结果上线还是一堆问题，因为需求过程那些人也没去谈，纯粹把程式复杂化
code review不会cc不懂的人吧....会找不懂的人，本身程序就有问题，不是review的问题
老实说我觉得code review的功能比较像大家一起看你这样写有没有不完全吻合原本大架构设计或是跟其他function会互相冲突之类的 说要成长什么的还真的蛮有限....会share的人你平时跟他多交流他也会讲五四三不share的人你code review N次 他也是呵呵笑笑带过去想要成长什么的 还是靠自己吧
作者:
sd016808 (sd016808)
2018-11-07 22:46:00理想上会希望每一次的Commit最后都要有人进行 pull request的审核 但实际上为了加速开发 常常跳过审核 只好要求Testing要写足 透过Daily Build/Test去尽早发现问题 但Test能过跟Code的品质不是正相关 所以能有review当然是最好了我觉得可以请有做Review的人分享一下 有做Review的案子大概有多少人共同开发 而不是看公司大小
作者:
Busufu (不太舒服)
2018-11-08 00:38:00不要听一楼唬烂
作者: windlll (我要工作阿) 2018-11-08 00:44:00
听过Python code被review，HW也跳下来，然后狂问因为他们不懂为什么有些function可以retry..
作者:
SHOOTA (修塔)
2018-11-08 00:54:00想想QA救救QA，不做review就不断一直像富奸一样死了又活
作者:
WunoW (WunoW)
2018-11-08 08:50:00我是时常是重构别人的烂code
作者: papayabird (papayabird) 2018-11-08 09:03:00
自己review 啊，我写完隔天就忘了 当别人写的一样
作者:
ll1117 (Linus)
2018-11-08 13:49:00自己要做啊，你写考卷不用验算吗？review就是你自己啊
作者:
mathrew (Joey)
2018-11-08 20:51:00看你写的东西是什么 如果只是能动就好 那就没
作者:
xhung (xhung)
2018-11-13 10:02:00系统直接写成没有reviewboard没有shipit不能commit就好