PTT動漫區 / C_Chat (希洽)

Re: [閒聊] 寫code不加註解真的很顧人怨嗎

看板C_Chat (希洽)作者 (old cat)時間13小時前 (2024/12/25 22:55), 12小時前編輯推噓21(21推 0噓 27→)
留言48則, 26人參與, 4小時前最新討論串5/5 (看更多)
※ 引述《SkankHunt42 (凱子爸)》之銘言： : ※ 引述《ianlin1216 (伊恩可可)》之銘言： : : 餓死抬頭 : : https://i.imgur.com/3QcIsVN.jpeg
: : 本魯不是資工系的啦 : : 所以不知道寫程式不加註解會有多嚴重 : : 想請問相關從業的鄉民 : : 實務上遇到這種情況真的很賭爛嗎 : : 乾五西恰 : 大家好 我月薪28k軟體工程師啦 : 我的觀察齁 程式設計師有好幾種類型 : 1. 無口型 : 做了很奇怪的事也不註解，commit也找不到原因 : 等到哪個有重構強迫症的改壞程式以後才發現原來看似很奇 : 怪很醜的寫法才是最佳解 但因為沒註解所以大家還是繞了一圈 : 無口型有高手也有智障 : 2. 廢話型 : if(IsMatched()) { : // IsMatched 回傳true時會進來 : doSomething(); : } : 典型的話癆，可能以為自己在寫教科書 : 3. 宅宅型 : if(!Initialized()) { : assert(false && "you shall not pass!!!!!"); : } : 喜歡在註解裡面埋梗，不懂梗的同事會很痛苦 : 4. 版本控制是什麼型 : // 2024/12/25 by SkankHunt42 : 不知道版本控制為何物，在註解特地註明自己改了這段程式碼 : 5. 上班不知道在想什麼型 : // ( )( ) : 可能在想LISP : 6. 連環形 : // 把這則comment複製到另外3個檔案，7天後之後，你就會升職加薪。 : // 如果不複製，績效就會被打C : 7. 影片說明型 : // The video in this link explains this piece of code. : // https://www.youtube.com/watch?v=dQw4w9WgXcQ
真的有這麼奇葩的註解嗎 上 code 前不是都會做 code review 嗎 現在很多工具除了會做 style check，commit message 之外， 甚至有些工具會檢查註解是否符合格式吧 這種亂上 code 真的不會被幹到起飛嗎 把自己名字打上去也太好笑，直接讓人抓出是誰寫出這坨糞是吧 雖然 git blame 一下，也是會被抓到就是 我們在上 code 到 server 還要自動驗證是否可以編譯過，是否通過冒煙測試 還要經過資深同事 code review 過，都沒有問題才能上 code 到 server -- ※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 111.240.74.137 (臺灣) ※ 文章網址: https://www.ptt.cc/bbs/C_Chat/M.1735138512.A.F86.html
推
12/25 22:57, 13小時前 , 1F
有餘力做code review的公司不多，畢竟每個人手上的事情都一
12/25 22:57, 1F
→
12/25 22:57, 13小時前 , 2F
沒有人會想再去幫別人看code，就自己的code自己負責，很多系
12/25 22:57, 2F
→
12/25 22:58, 13小時前 , 3F
統爆炸也是這個原因 XD
12/25 22:58, 3F
推
12/25 22:58, 13小時前 , 4F
因為工具越來越強 整體軟體業都朝向人力精簡配置方向
12/25 22:58, 4F
推
12/25 22:58, 13小時前 , 5F
就沒人專門負責統合 上司有空看就看 沒空可以work就好
12/25 22:58, 5F
→
12/25 22:59, 13小時前 , 6F
這樣的團隊(尤其是新創)會更不在乎團隊合作
12/25 22:59, 6F
→
12/25 22:59, 13小時前 , 7F
帕魯的伺服器就號稱一個人搞定
12/25 22:59, 7F
確實，之前在新創公司待過，遇過有同事把dram 相關的設定關掉，讓整個硬體部門陪他 解一個禮拜的bug 。我就知道code review 多重要了 ※ 編輯: k12s35h813g (101.10.4.54 臺灣), 12/25/2024 23:01:05
推
12/25 22:59, 13小時前 , 8F
有些公司的code只要能動就好，什麼工具都當不存在，review
12/25 22:59, 8F
→
12/25 22:59, 13小時前 , 9F
一個人是要做甚麼code review
12/25 22:59, 9F
→
12/25 23:00, 13小時前 , 10F
註解是不是用英文寫的commit message是不是只寫fix都可能
12/25 23:00, 10F
推
12/25 23:00, 13小時前 , 11F
小軟體公司很多步驟都是一個人幹，驗證測試時間也不一
12/25 23:00, 11F
→
12/25 23:00, 13小時前 , 12F
定完善，所以有錯誤挺正常的
12/25 23:00, 12F
→
12/25 23:01, 13小時前 , 13F
更是找不到有空的前輩看，所以薪水越爛的鬼故事越多。
12/25 23:01, 13F
→
12/25 23:01, 13小時前 , 14F
銀行/支付/自駕這種就是另一種文化畢竟金錢上下生死攸關
12/25 23:01, 14F
推
12/25 23:02, 13小時前 , 15F
commit message: hot fix <--- 這種嗎XD
12/25 23:02, 15F
這種我覺得還好，有些bug 真的很微小的，我們有的時候也會上 Fix typo 的訊息 ※ 編輯: k12s35h813g (101.10.4.54 臺灣), 12/25/2024 23:04:04
→
12/25 23:03, 13小時前 , 16F
我自己的side project還蠻常這樣的畢竟沒人會看啊
12/25 23:03, 16F
推
12/25 23:08, 13小時前 , 17F
老公司也會啊 以前不review沒事就一直不會改
12/25 23:08, 17F
推
12/25 23:08, 13小時前 , 18F
真好qq
12/25 23:08, 18F
→
12/25 23:19, 12小時前 , 19F
不會 你要先說服老闆花錢多請一個人來互相review
12/25 23:19, 19F
→
12/25 23:20, 12小時前 , 20F
然後你老闆還要再跟上面的老闆說服
12/25 23:20, 20F
→
12/25 23:20, 12小時前 , 21F
說我們單位每年花百萬請工程師的工作 output
12/25 23:20, 21F
→
12/25 23:20, 12小時前 , 22F
有一部分時間花在做 code review
12/25 23:20, 22F
→
12/25 23:21, 12小時前 , 23F
然後大老闆就會說 這個工作沒有必要 不用做
12/25 23:21, 23F
→
12/25 23:21, 12小時前 , 24F
然後部門文化就定型了
12/25 23:21, 24F
我遇到的狀況是，開發是你的責任，幫同事code review 也是你的責任，自己想辦法在時 程內完成QQ
推
12/25 23:22, 12小時前 , 25F
一堆公司都馬能動就好 這可是慘業
12/25 23:22, 25F
※ 編輯: k12s35h813g (101.10.4.54 臺灣), 12/25/2024 23:25:51
→
12/25 23:24, 12小時前 , 26F
案子很忙的時候REVIEW都馬隨便看看就好
12/25 23:24, 26F
→
12/25 23:28, 12小時前 , 27F
忙得要死 review看個3分鐘意思一下
12/25 23:28, 27F
推
12/25 23:29, 12小時前 , 28F
code review、code style 不少公司都沒在做吧
12/25 23:29, 28F
→
12/25 23:30, 12小時前 , 29F
做clean code比較像在為了未來的自己好維護
12/25 23:30, 29F
→
12/25 23:39, 12小時前 , 30F
最怕人都離職了才爆炸的code，一直講要review又不給人，
12/25 23:39, 30F
→
12/25 23:39, 12小時前 , 31F
是要叫誰去review啦
12/25 23:39, 31F
推
12/25 23:43, 12小時前 , 32F
通常做新專案客戶有要求就會統一做code style 的整合
12/25 23:43, 32F
→
12/25 23:43, 12小時前 , 33F
然後現在文件也直接用vscode插件直接gen出來
12/25 23:43, 33F
推
12/25 23:55, 12小時前 , 34F
對，良好的編程習慣是對自己好，否則會有惡性循環，之
12/25 23:55, 34F
→
12/25 23:55, 12小時前 , 35F
前寫太爛導致後面難以維護所以寫更爛
12/25 23:55, 35F
推
12/25 23:59, 12小時前 , 36F
更多的是ZIP檔版控的公司…
12/25 23:59, 36F
推
12/26 00:58, 11小時前 , 37F
當leader就是那個亂搞的人，一切都不再會是問題了~
12/26 00:58, 37F
推
12/26 01:18, 10小時前 , 38F
有啊 有驗證跟review才能上code,不過從頭到尾所有事
12/26 01:18, 38F
→
12/26 01:18, 10小時前 , 39F
情都是同一個人在做
12/26 01:18, 39F
推
12/26 04:42, 7小時前 , 40F
2不是廢話，是最有價值的資訊。當你實務上遇到改來改去的
12/26 04:42, 40F
→
12/26 04:43, 7小時前 , 41F
變化，非A也非B的情況下又要滿足另一種"情境"，就懂了。
12/26 04:43, 41F
→
12/26 04:44, 7小時前 , 42F
複合型情境超過10種以上，可見複雜度高的實務需求。
12/26 04:44, 42F
推
12/26 07:02, 5小時前 , 43F
一個人搞定...那他辭職不就直接爆開
12/26 07:02, 43F
推
12/26 07:26, 4小時前 , 44F
說不定現在可以直接丟給AI寫註解
12/26 07:26, 44F
推
12/26 07:44, 4小時前 , 45F
現在有AI就還好了 比較麻煩的是多個檔案的那種
12/26 07:44, 45F
推
12/26 07:46, 4小時前 , 46F
沒好處賺沒人喜歡CR 檔人快速拉屎deliver升級加薪跑路
12/26 07:46, 46F
→
12/26 08:06, 4小時前 , 47F
不是每間公司都有那麼多人力做review啦
12/26 08:06, 47F
推
12/26 08:11, 4小時前 , 48F
你公司是已經很大的了 多的是根本不review 所以容易踩坑
12/26 08:11, 48F
‣ 更多 k12s35h813g 的文章
文章代碼(AID): #1dR1pG-6 (C_Chat)
C_Chat 近期熱門文章
更多 近期熱門文章 >>
PTT動漫區 即時熱門文章
更多 即時熱門文章 >>
‣ 更多 k12s35h813g 的文章
文章代碼(AID): #1dR1pG-6 (C_Chat)