作者polola6212 (Polo)
看板Soft_Job
標題Re: [討論] 寫三元判斷式code review被打槍
時間Thu Dec 29 16:34:51 2022
review code 的時候, 對code style 有意見的人真很呵呵,
真的要對code style 有意見,麻煩就寫進code style checker,
如果在commit 之前的 code style checking 都過了,
就不要在這上面花時間,不然會沒完沒了。
因為有更多更重要的事要討論,
如何抽象化、file strcture, time complexity, memoery complexity,
db query cost, lock...
一個人的時間有限,請妥善利用...
--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 219.68.121.149 (臺灣)
※ 文章網址: https://webptt.com/m.aspx?n=bbs/Soft_Job/M.1672302894.A.825.html
1F:→ loadingN: LGTM 12/29 16:43
2F:→ leolarrel: 現在不就是在吵coding style checker 要發落誰的嗎? 12/29 16:46
3F:→ polola6212: 挑一個合理的就好啊,重點這東西不該在code review 12/29 16:48
4F:→ polola6212: 的時候拿出來討論,依經驗來說很浪費時間 12/29 16:48
5F:→ polola6212: 就大頭們討論完寫進去checker就好 12/29 16:48
6F:推 internetms52: 推 12/29 18:54
7F:推 ohohoh: 那光有coding style卻沒有checker的公司怎麼辦 12/29 19:36
8F:→ labbat: 人蠢是沒有極限的,一大堆魚目混珠的寫法列舉不完的 12/29 19:52
9F:推 xuzx: 會要求code style的公司就會有checker吧 12/29 20:32
10F:推 ku72: 真的 看完整串後我也有這麼想 除非寫要在有限資源下榨效能的 12/29 21:00
11F:→ ku72: 程式 不然追究這個沒啥意思 我自己code review 比較大還是注 12/29 21:00
12F:→ ku72: 意整體架構 思路 邏輯關係 12/29 21:00
13F:推 k798976869: LGTM 12/30 08:59