作者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/cn.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