我是軟體業小白
以前也不是讀相關科系的
第一份工作也沒上過甚麼資策會
就是莫名其妙就開始寫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就好