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

看板Tech_Job作者時間7年前 (2018/11/07 15:01), 7年前編輯推噓35(36推 1噓 67→)
留言104則, 41人參與, 7年前最新討論串1/2 (看更多)
我是軟體業小白 以前也不是讀相關科系的 第一份工作也沒上過甚麼資策會 就是莫名其妙就開始寫Code了 大多數的東西都是自學而來 不管是上班還是下班 現在都在寫.net Mvc 然後前端搭一點Jquery和簡單的vue 不知道為啥寫著寫著挺有興趣的 可是現在蠻想離職的 主要是覺得成長的速度變慢了 東西都靠自己撞牆撞出來，沒人帶 都是說你做這個，偶爾加個關鍵字這樣 然後就讓我自己去撞牆了 有時候也怕自己撞牆撞出來的結論是錯的 雖然東西是有出來 印象中聽鄉民說有所謂的Code Review 正常一般公司真的會有CodeReview嗎？ 我是指以軟體為主要商品的公司 然後這個時期找工作是不是死定了 可是最近離職的念頭好大 一直被當神燈精靈 累 然後問一下.net MVC要到怎樣的水準才能在台北領個45k? -- ※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 118.165.226.252 ※ 文章網址: https://www.ptt.cc/bbs/Tech_Job/M.1541574109.A.8F7.html ※ 編輯: milktea0629 (118.165.226.252), 11/07/2018 15:02:32 ※ 編輯: milktea0629 (118.165.226.252), 11/07/2018 15:05:58
推
11/07 15:12, 7年前 , 1F
不一定，但如果是 team work 通常會有
11/07 15:12, 1F
→
11/07 15:13, 7年前 , 2F
這類公司新人進去就會先發 code 準則，像國軍準則一樣要
11/07 15:13, 2F
→
11/07 15:14, 7年前 , 3F
背要看要記牢，變數怎麼命名、換行時機、縮排要領
11/07 15:14, 3F
→
11/07 15:15, 7年前 , 4F
一些固定基本的東西跟單兵戰鬥詞一樣要背清楚
11/07 15:15, 4F
→
11/07 15:16, 7年前 , 5F
然後結構要跟單兵戰鬥一樣演練，function scope 掩護obj
11/07 15:16, 5F
→
11/07 15:16, 7年前 , 6F
每週開會檢討 git 上的 commit 問其他人有什麼想法
11/07 15:16, 6F
→
11/07 15:18, 7年前 , 7F
麻煩一點你想用內插法解的也被大家打槍要用積分函數做
11/07 15:18, 7F
這樣一講感覺Code Review是很可怕的東西 不過這類公司感覺做事很分明啊 我對於我現在一個PG要幹SD甚至SA的事覺得很疲憊 ※ 編輯: milktea0629 (118.165.226.252), 11/07/2018 15:20:12
→
11/07 15:19, 7年前 , 8F
做到最後都要經神分裂了，但 Leader 會覺得這叫有品質
11/07 15:19, 8F
推
11/07 15:21, 7年前 , 9F
沒有review肯定是不行的，但是review也分很多種
11/07 15:21, 9F
就是說有單兵戰鬥準則式的Review 也有國軍飲水宣導式的Review嗎 ※ 編輯: milktea0629 (223.140.115.151), 11/07/2018 15:30:51
推
11/07 15:30, 7年前 , 10F
絕對不分明啊XDD這事是所有不相干的人都要跳下來大亂鬥
11/07 15:30, 10F
推
11/07 15:30, 7年前 , 11F
有code review才是對的，一堆人在改又不做review 才可怕
11/07 15:30, 11F
不就是程式碼風格很多變而已嗎？其實我是比較care這個過程能不能讓人進步啦哈哈 我現在翻寫webform就看到很多變的風格 還看到寫著"我不知道這段是幹嘛的"這樣的註解 哈哈哈哈哈...嗚嗚嗚嗚
推
11/07 15:30, 7年前 , 12F
我們家會，不過通常都是主管摳而已，只有大功能多叫一兩
11/07 15:30, 12F
→
11/07 15:30, 7年前 , 13F
個人一起看
11/07 15:30, 13F
※ 編輯: milktea0629 (223.140.115.151), 11/07/2018 15:33:48
推
11/07 15:33, 7年前 , 14F
你負責寫通訊協定，做UI的跟做資料庫的都可能來 review
11/07 15:33, 14F
→
11/07 15:34, 7年前 , 15F
資料庫的說你協定有缺陷，大家同意，你就得改
11/07 15:34, 15F
→
11/07 15:34, 7年前 , 16F
code review 的目的是要同事之間不要犯相同的錯誤
11/07 15:34, 16F
→
11/07 15:35, 7年前 , 17F
然後你認為你那邊是完美的，就要提出報告說明設計原理
11/07 15:35, 17F
→
11/07 15:35, 7年前 , 18F
最後會花很多時間，但可以事先把隱藏風險找出來
11/07 15:35, 18F
噓
11/07 15:35, 7年前 , 19F
廢話 去廢物小公司就是要兼 不然叫她付你3個人薪
11/07 15:35, 19F
→
11/07 15:36, 7年前 , 20F
就如同你說，你的東西都是自己google來的，你也不曉得好
11/07 15:36, 20F
→
11/07 15:36, 7年前 , 21F
不好，就是因為這樣才要做code review
11/07 15:36, 21F
→
11/07 15:36, 7年前 , 22F
同事之間才可以互相學習而不是互相拖累
11/07 15:36, 22F
→
11/07 15:36, 7年前 , 23F
比如你的協定可能忽略 timeout 等例外處理或處理不夠好
11/07 15:36, 23F
→
11/07 15:37, 7年前 , 24F
而做資料庫的因為常提供 API 所以知道 timeout 有啥問題
11/07 15:37, 24F
→
11/07 15:38, 7年前 , 25F
review的過程才會讓像你這樣沒經驗的小嫩嫩成長
11/07 15:38, 25F
→
11/07 15:38, 7年前 , 26F
但你可能一開始就設計可忽略 response 的單向通訊協定
11/07 15:38, 26F
→
11/07 15:39, 7年前 , 27F
結果跟做資料庫的爭執不下，其他同事覺得他有道理就挺他
11/07 15:39, 27F
→
11/07 15:39, 7年前 , 28F
如果你不覺得別人的code會成長或是檢討自己的code會讓自
11/07 15:39, 28F
→
11/07 15:39, 7年前 , 29F
己成長，那你就真的是太自以為是了
11/07 15:39, 29F
那當然要Code Review一下啊 我們現在就是沒Code Review 我常常不知道我寫的東西跟業界有沒有脫鉤 會不會有那些東西大家其實有固定的默契，但是我卻自己在土砲 尤其在寫物件的時候更是如此..
→
11/07 15:40, 7年前 , 30F
即便你對自己設計有信心也會因為 review 被迫改掉
11/07 15:40, 30F
※ 編輯: milktea0629 (223.140.115.151), 11/07/2018 15:43:20
→
11/07 15:41, 7年前 , 31F
Code review是一種維持品質但不見得有開創性的事
11/07 15:41, 31F
→
11/07 15:42, 7年前 , 32F
最後的結果是你妥協了用公司一脈相傳的通訊協定方法
11/07 15:42, 32F
→
11/07 15:43, 7年前 , 33F
但你開創性設計不會被產品所用，產品品質穩定卻無突破
11/07 15:43, 33F
還有 31 則推文
→
11/07 20:24, 7年前 , 65F
版命名風格 編譯警告破500 魔術數字沒有說明 非常痛苦
11/07 20:24, 65F
→
11/07 20:25, 7年前 , 66F
尤其我們是一人維護一個機種 所以問人不一定知道
11/07 20:25, 66F
推
11/07 20:33, 7年前 , 67F
一般一部分整個公司只有一個人管吧 到底怎review
11/07 20:33, 67F
→
11/07 20:33, 7年前 , 68F
不懂的人來看還不是找話講而已
11/07 20:33, 68F
→
11/07 20:37, 7年前 , 69F
上一個沒有按照code rule的已經被學長肛到不要不要
11/07 20:37, 69F
→
11/07 20:38, 7年前 , 70F
聽起來像軟體代工 而不是純軟的公司
11/07 20:38, 70F
推
11/07 20:43, 7年前 , 71F
沒辦法想像沒有code review
11/07 20:43, 71F
推
11/07 20:45, 7年前 , 72F
整個公司只有一個人code review，小公司嗎？
11/07 20:45, 72F
→
11/07 20:51, 7年前 , 73F
不是 是一個部分只有你維護 那別人怎看得懂?
11/07 20:51, 73F
→
11/07 20:52, 7年前 , 74F
前因後果都不懂 不就只能挑格式而已了
11/07 20:52, 74F
推
11/07 21:21, 7年前 , 75F
縮排不是有自動檢查
11/07 21:21, 75F
推
11/07 21:58, 7年前 , 76F
為了Code Review而Code Review，不如沒有
11/07 21:58, 76F
→
11/07 22:06, 7年前 , 77F
每個人的程式邏輯都不太一樣，有時後code review只是一
11/07 22:06, 77F
→
11/07 22:06, 7年前 , 78F
堆人給一堆意見，結果上線還是一堆問題，因為需求過程那
11/07 22:06, 78F
→
11/07 22:06, 7年前 , 79F
些人也沒去談，純粹把程式複雜化
11/07 22:06, 79F
推
11/07 22:15, 7年前 , 80F
code review不會cc不懂的人吧....
11/07 22:15, 80F
→
11/07 22:16, 7年前 , 81F
會找不懂的人，本身程序就有問題，不是review的問題
11/07 22:16, 81F
推
11/07 22:20, 7年前 , 82F
老實說我覺得code review的功能比較像大家一起看你這樣
11/07 22:20, 82F
→
11/07 22:21, 7年前 , 83F
寫有沒有不完全吻合原本大架構設計或是跟其他function
11/07 22:21, 83F
→
11/07 22:23, 7年前 , 84F
會互相衝突之類的 說要成長甚麼的還真的蠻有限....
11/07 22:23, 84F
→
11/07 22:24, 7年前 , 85F
會share的人你平時跟他多交流他也會講五四三
11/07 22:24, 85F
→
11/07 22:24, 7年前 , 86F
不share的人你code review N次 他也是呵呵笑笑帶過去
11/07 22:24, 86F
→
11/07 22:25, 7年前 , 87F
想要成長甚麼的 還是靠自己吧
11/07 22:25, 87F
推
11/07 22:46, 7年前 , 88F
理想上會希望每一次的Commit最後都要有人進行 pull requ
11/07 22:46, 88F
→
11/07 22:46, 7年前 , 89F
est的審核 但實際上為了加速開發 常常跳過審核 只好要求
11/07 22:46, 89F
→
11/07 22:46, 7年前 , 90F
Testing要寫足 透過Daily Build/Test去盡早發現問題 但T
11/07 22:46, 90F
→
11/07 22:46, 7年前 , 91F
est能過跟Code的品質不是正相關 所以能有review當然是
11/07 22:46, 91F
→
11/07 22:46, 7年前 , 92F
最好了
11/07 22:46, 92F
推
11/07 22:55, 7年前 , 93F
我覺得可以請有做Review的人分享一下 有做Review的案子
11/07 22:55, 93F
→
11/07 22:55, 7年前 , 94F
大概有多少人共同開發 而不是看公司大小
11/07 22:55, 94F
→
11/08 00:38, 7年前 , 95F
不要聽一樓唬爛
11/08 00:38, 95F
→
11/08 00:44, 7年前 , 96F
聽過Python code被review，HW也跳下來，然後狂問
11/08 00:44, 96F
→
11/08 00:45, 7年前 , 97F
因為他們不懂為什麼有些function可以retry..
11/08 00:45, 97F
推
11/08 00:54, 7年前 , 98F
想想QA救救QA，不做review就不斷一直像富奸一樣死了又活
11/08 00:54, 98F
→
11/08 08:50, 7年前 , 99F
我是時常是重構別人的爛code
11/08 08:50, 99F
→
11/08 08:52, 7年前 , 100F
code review 不做很悲慘 人是很粗心的
11/08 08:52, 100F
推
11/08 09:03, 7年前 , 101F
自己review 啊，我寫完隔天就忘了 當別人寫的一樣
11/08 09:03, 101F
→
11/08 13:49, 7年前 , 102F
自己要做啊，你寫考卷不用驗算嗎？review就是你自己啊
11/08 13:49, 102F
推
11/08 20:51, 7年前 , 103F
看你寫的東西是什麼 如果只是能動就好 那就沒
11/08 20:51, 103F
推
11/13 10:02, 7年前 , 104F
系統直接寫成沒有reviewboard沒有shipit不能commit就好
11/13 10:02, 104F
‣ 更多 milktea0629 的文章
文章代碼(AID): #1RuetTZt (Tech_Job)
‣ 更多 milktea0629 的文章
文章代碼(AID): #1RuetTZt (Tech_Job)