※ 引述《alihue (wanda wanda)》之銘言: : 轉自推特 : https://twitter.com/BenLesh/status/1372562839475470336?s=20 : Add comments about WHY code exists, not what it does. : The code is right there, we know what it does. : 註解應該用來解釋這段 code 的背景需求/含意, : 而不是把 code 表面上的意思再講一次 上週在重構某段程式碼時,其中一位同事在 code review 中建議把某個註解刪掉,另一 個同事看到這個評論時,在下面留了言說他認為不應該刪掉,於是我們就拉了一個小討論 ,聊在程式碼中寫註解這件事。 因為這個經驗,我回去重翻史丹佛電腦科學教授 John Ousterhout 寫的《A Philosophy of Software Design》一書,並整理了筆記。該教授的觀點是認為程式碼寫註解有很多好 處,但不是任何地方都該寫註解。 在版上找到這篇之前有版大發的文,基本上跟 John Ousterhout 教授的觀點一樣, 就是註解要解釋背後的「為什麼」,而不是把程式碼做的事重複說一次。 ------ (以下是我讀完該章節後整理的筆記) ------ ## 網誌好讀版: http://bit.ly/3WdTo1b 每當提到在程式中寫註解 (comments),你大概會在網路上看到兩派人馬,有人覺得應該 要寫註解;又有另一群人覺得程式碼應該要寫得夠清楚,如果有註解就代表寫不夠清楚, 應該要重構而不是加註解。當然除了這極端的兩派人馬外,多數人都是在中間,部分的程 式碼寫註解,但不會全部都寫。 關於寫註解這件事,在 《A Philosophy of Software Design》 書當中也有談及。John Ousterhout 教授的觀點是,如果註解寫得好,將有效改善整體的系統設計。假如你是反 註解派的人,或許可以一起來讀讀他為什麼這麼認為。 ## 程式本身寫得夠清楚,就不用註解嗎? 「程式本寫得夠清楚,就不用註解」是很多人認為不該寫註解的第一大理由。然而程式本 身或許可以清楚表達該程式碼做的事,但不能解釋「為什麼」要這樣寫,也不能解釋在什 麼情境可能比較適合某個方法。對於這些相對後設 (meta) 的內容,註解就是很好的幫手 。 更進一步說,好的軟體設計,應該要把複雜度藏起來,要藏起複雜度,我們需要抽象化, 讓其他使用該段程式碼的開發者,可以不用去管背後的細節。因此,如果一個開發者,還 要去讀程式的細節才能懂如何用該函式或方法,那就失去抽象化的意義。寫註解可以讓其 他開發者,不用去讀程式裡頭的細節也知道如何用該函式或方法,這才能達到抽象化的意 義。因此不論程式本身寫的清楚與否,好的抽象化搭配好的註解,都是對整個程式庫的維 護有幫助的。 ## 別說你沒時間寫註解 在開發時,很多開發者會認為寫註解的優先順序比較低,因此會想把時間花在寫新功能, 而不是為已經開發好的功能寫註解。然而在軟體開發中,永遠有寫不完的功能,如過說因 為要開發新功能而沒時間寫註解,就永遠不會有註解。從長遠的角度來看,這樣的代價是 程式庫的可維護性會降低。如果從效益與成本的角度來看,寫註解的時間,會遠比其他開 發者花在弄懂沒註解的程式所花的時間要少很多。 ## 不要找其他不寫註解的理由 很多開發者也會找其他理由不寫註解。例如認為註解如果沒隨著程式更新,反而會誤導人 ;或者是因為過去讀過別人寫的註解覺得沒用,就認為註解沒有用。這些都是外部因素, 無法證成註解本身沒有效益。此外這些都是可以透過好的流程而解決的,所以作者認為不 要找這些理由不寫註解,而是要去打造更好的流程來改善這些問題。 ## 寫註解的好處 上面談了這麼多,大概可以看出 John Ousterhout 教授有強烈的觀點認為要寫註解。不 過回過頭來說,寫註解有什麼好處呢? 他認為最大的好處在於註解可以捕捉到程式碼沒辦 法傳達的資訊,這能夠讓未來要維護這段程式碼的開發者,能夠更快速地上手。特別對於 團隊中加入的新成員,程式碼中的註解可以大幅降低讀程式的認知負擔 (cognitive load),以及減少未知。 如果沒有記錄下這些脈絡與原因,未來的開發者就不會知道為什麼當初是這樣寫,這變得 只能猜測原作者的意圖。這不僅更耗時,也可能會因為理解錯原本的意圖,導致在維護該 段程式碼時出錯。很多時候,未來要維護程式碼的是自己,許多人在寫完某段程式碼幾個 月後,可能忘了自己當初為什麼這樣寫,所以寫註解不僅是幫助其他開發者,也很可能是 在幫助未來的自己。 ## 註解該寫什麼? 雖說 John Ousterhout 教授的觀點是認為寫註解對軟體設計有幫助,但他也同意不是什 麼都該寫成註解。他認為只有在程式碼中不顯而易見的才該寫註解 (comments should describe things that aren’t obvious from the code)。 在軟體設計中最重要的抽象化,即是提供一個更簡易的思考方式,讓開發者可以不用深入 過於細節的部分。即使開發者可以透過細讀程式碼來了解其如何運作,但這樣做太花時間 。John Ousterhout 教授的觀點認為開發者應該要不讀程式碼內容,即可理解模組提供的 抽象化,而註解是能夠協助做到這樣的方法。 ## 不要重複程式碼本身 前面提到,註解應該要講「為什麼」這樣寫,而不是講程式碼在做什麼。畢竟程式碼本身 就代表程式碼在做的事,如果註解在寫一次,會是多此一舉。下面是書中提到的負面教材 ,基本上註解就是把程式碼做的事情。 ptr_copy = get_copy(obj) # Get pointer copy if is_unlocked(ptr_copy): # Is obj free? return obj # return current obj if is_copy(ptr_copy): # Already a copy? return obj # return obj thread_id = get_thread_id(ptr_copy) if thread_id == ctx.thread_id: # Locked by current ctx return ptr_copy # Return copy 要怎麼判斷你寫的這段註解是不是跟程式碼本身一樣? 書中提到,在你寫完註解後可以問 問自己「如果某個之前沒讀過這段程式碼的人,能不能看著這段註解寫出程式碼?」如果 可以的話,就代表註解只是在描述程式碼,而不是在解釋為什麼這樣寫該段程式碼。因此 ,在寫完註解後,可以再比對一下程式碼與註解,如果重複就真的不推薦寫。 ## 從讀者的角度出發 最後 John Ousterhout 教授提到,寫註解要從程式碼讀者的角度出發,要思考有哪些關 鍵資訊是讀者不知道的。特別是在程式碼審查 (code review) 時,如果你寫的某段程式 碼讓別人說很難懂,或者某個資訊讓別人說並不顯而易見,這時候不要去跟對方爭,而是 去想為什麼對方會有困惑,以及你可以如何靠註解讓程式碼更好被理解。 (文末備註:這個章節本身有很多範例,但為了避免筆記變太長,就僅有放其中一個。有 興趣透過案例更具體了解作者的觀點的話,推薦直接買這本書來讀)。 -- 南漂一不小心漂到了新加坡 https://www.explainthis.io/zh-hant/singapore -- ※ 發信站: 批踢踢實業坊(pttsite.org.tw), 來自: 116.86.64.199 (新加坡) ※ 文章網址: https://pttsite.org.tw/Soft_Job/M.1673785027.A.36D
FukadaKyoko: 推~感謝分享 01/15 20:44
S2067030: 推 謝謝大大分享 01/15 21:19
ohmylove347: 好文推推 01/15 21:25
prag222: 沒時間看完整篇文章,這就跟一萬小時變大師的觀點呵呵 01/15 21:57
iamOsaka: 推 01/15 22:43
smilejin: 有人被洋鬼子包養過嗎 01/15 22:43
derekjj: 推 我註解都看心情 然後畫恐龍註解圖 嘻嘻 01/15 23:12
holebro: 講的好 01/15 23:59
viper9709: 推分享~註解本來就該寫這段在做啥,為啥這樣做 01/16 00:03
drajan: 推~上次看這本書好久以前了 看完心得整理好像又讀一遍了 01/16 01:18
wei115: 我想要知道函式的功能 和為什麼要用無符號整數來存指標QQ 01/16 02:04
qusekii: 到底要多有錢才會想包養 01/16 02:04
zrna0515: 推,meta data那段講的不錯 01/16 08:04
aaa1234136: 分享推 01/16 08:45
lou01: 感謝分享~ 01/16 09:06
aidansky0989: 那需求一直變,後續也要持續維護這段註解嗎? 01/16 09:11
vi000246: 我記憶力很差 我的註解都是為了讓以後維護的我看得懂這 01/16 09:50
Y949731: 閨蜜上包養網還推薦我... 01/16 09:50
vi000246: 段code在寫啥 01/16 09:50
leolarrel: 樓上的狀況我也有,只是我都用git取代註解就是了 01/16 10:30
hobnob: 回 aid:我自己會註解加說是什麼時候的需求、修改的新邏 01/16 10:30
hobnob: 輯是什麼,程式註解我都會維護 01/16 10:30
t64141: 註解是需要維護的,但註解只是輔助工具,理論上不會多到 01/16 10:57
T730733: 包養? 01/16 10:57
t64141: 造成負擔,如果會,可能是程式太亂需要大量註解,或是有 01/16 10:57
t64141: 太多沒必要的註解 01/16 10:57
chatnoir: 我註解只會寫特殊的商業邏輯來輔助程式碼的不足~ 01/16 11:21
knives: 推樓上 01/16 11:53
fiiox3: 當文件的一環吧 01/16 11:55
FireStation: 現在包養網都這麼直接嗎 01/16 11:55
fiiox3: 文件要維護,註解當然也要 01/16 11:55
yuinami: 優文推一個,謝謝分享 01/16 12:19
wulouise: 註解當然要維護啊,但至少沒維護歷史看得出來 01/16 12:28
aidansky0989: 感謝回覆,之前看clean code提到很多寫註解的壞處 01/16 13:44
aidansky0989: ,像是接手的人不一定會維護註解、註解缺少維護可 01/16 13:44
Thobel: 歐美包養真的很平常嗎? 01/16 13:44
aidansky0989: 能會過時、誤導等壞處,後來只有TODO我才會寫註解 01/16 13:44
aidansky0989: ,寧可後人多跟PM確認新的現狀 01/16 13:44
aidansky0989: 不然註解寫出來,也不太有人敢修改、刪除 01/16 13:45
aidansky0989: 可能也是個問題 01/16 13:45
foreverk: 沒寫註解基本上也不應該隨意修改刪除啊,要做的事情是 01/16 15:15
Reji: 男友上包養網 該放生嗎 01/16 15:15
foreverk: 補測試或是找人釐清需求,再去做變動,註解就是輔助工 01/16 15:15
foreverk: 程師更好做到這些事 01/16 15:15
foreverk: 很多人不寫註解或是不喜歡文件以及測試的理由都是要多 01/16 15:21
foreverk: 維護的工,但往往省略掉這些以後,是花更多的時間開會 01/16 15:21
foreverk: ,或是慢慢變技術債 01/16 15:21
Hathael: 是這個包養平台嗎 01/16 15:21
fiiox3: “不一定會維護“這為啥要算註解的缺點.... 01/16 18:24
fiiox3: 整個軟體開發一堆東西都有人不維護= = 01/16 18:25
foreverk: 另外需求一直變並不是不寫註解的理由,不寫就是用人腦 01/16 19:09
foreverk: 記憶,沒紀錄還可能記錯,基本上就是賭這個未爆彈不會 01/16 19:09
foreverk: 炸到你而已 01/16 19:09
yovroc: 交男友跟包養有什麼差別 01/16 19:09
aidansky0989: 修改刪除是指註解,因為不及時更新註解就會隨需求 01/16 19:33
aidansky0989: 變動過時,可能造成誤導而非指引 01/16 19:33
gobears5566: 這本書作者的論點跟 foreverk 大說的蠻相近的 01/16 19:46
gobears5566: 這位史丹佛教授在書中最開始就提到,他不認同蠻多 01/16 19:46
gobears5566: Clean Code 裡面提到的觀點,但他也不覺得自己的就 01/16 19:47
helgalie: 包養網到底在紅什麼? 01/16 19:47
gobears5566: 適合在所有脈絡。所以他的書名是 a philosophy 01/16 19:47
gobears5566: 而不是 the philosophy 01/16 19:47
foreverk: 如果指得是不敢修改或刪除註解,那就代表改code的人並 01/16 20:38
foreverk: 沒有完全掌握這項功能,改善方式應該是增加掌握度,而 01/16 20:38
foreverk: 不是捨棄註解本身 01/16 20:38
OREOMZA: 有人被包養 01/16 20:38
jay123peter: 推 01/16 21:54
lovdkkkk: 假如有好好寫 commit 訊息,再寫個簡單 script 可以過濾 01/16 21:55
lovdkkkk: 時間/檔案/行數/關鍵字/作者 找出相關 commit 也不錯 01/16 21:56
moom50302: 好的程式不需要註解,你說得都是文件應該要做到的 01/17 00:18
gggaaammm: 推 01/17 00:48
punjab: 求包養...管飽就好XD 01/17 00:48
foreverk: 不寫註解很常伴隨著不寫文件,最常見的第二個理由就是 01/17 02:09
foreverk: 認為好程式不需要註解/文件,可是一些特殊商業邏輯所產 01/17 02:09
foreverk: 生的程式,他不好但他是因應當時時空背景而生,這類情 01/17 02:09
foreverk: 況還不寫,那就是上面我說的未爆彈 01/17 02:09
foreverk: 樓上說的commit也是應該要好好寫的一點,甚至應該pr也 01/17 02:17
sashare: 阿姨!我不想努力了(求包養) 01/17 02:17
foreverk: 好好寫,不想維護一整份文件的話,更應該從這些細節去 01/17 02:17
foreverk: 提升程式可維護性 01/17 02:17
leolarrel: 寫註解這種事就是誰當老闆就聽誰的... 01/17 10:12
leolarrel: 現實世界就是階級,書再怎麼賣學校再有名都沒用 01/17 10:14
vi000246: 如果有些重構會搬移程式碼到不同的專案或檔案 01/17 14:12
sijiex: 有沒有富二代要包養 01/17 14:12
vi000246: 這時git的紀錄也很難追踨 01/17 14:12
lovdkkkk: 一個方式是搬家時舊的留著標 deprecated, see also... 01/17 20:43
lovdkkkk: 留個線索給人查 01/17 20:44
superpandal: 不認同 為什麼不是精簡化實現而是把有複雜度的東西藏 01/17 22:18
superpandal: 起來 這團code怎麼亂七八糟都沒差反正有註解? 現在 01/17 22:19
VLADINA: 身邊有朋友被包養 01/17 22:19
superpandal: 一堆東西坑很多就是這樣來的 包含框架 01/17 22:21
superpandal: 甚至語言底層本身如果你不去仔細研究也會被坑到 01/17 22:21
superpandal: 寫程式有兩種方式 一種是把程式寫的很簡單 明顯不容 01/17 22:22
superpandal: 易有缺陷 一種是寫的很複雜 讓別人看不出有明顯的缺 01/17 22:22
superpandal: 陷 01/17 22:23
odemagus: 亞洲最大包養平台上線了 01/17 22:23
superpandal: 依照這樣搞你邊寫沒寫可以模糊焦點的註解 還可以邊藏 01/17 22:24
superpandal: bug和洞 01/17 22:25
superpandal: s/沒寫// 01/17 22:26
kyoe: 有沒有可能一段程式複雜到需要註解的時候就是應該再拆分它然 01/18 00:48
kyoe: 後把功能單一化的時候? 01/18 00:48
yes500: 這個包養網正妹好多 是真的嗎 01/18 00:48
foreverk: 我的經驗是,通常不是程式複雜到需要註解,而是因應特 01/18 06:42
foreverk: 殊需求而出現的程式需要,他可能寫死很多地方,程式風 01/18 06:42
foreverk: 格明顯不符,或根本是糞code,但暫時需要 01/18 06:42
strlen: 我們非英語系國家 不然解法就是在method名稱上取一個好一 01/19 08:28
strlen: 點的名稱讓大家所見即所得 然後一個method盡量只做一件事 01/19 08:28
alexantiy: 真的有這麼多人在找包養 01/19 08:28
strlen: 但一堆人英文不好ABC狗嘎D method名稱亂取一通 01/19 08:29
htury: 好文,推註解是要說明為什麼,而不是幹什麼 01/19 14:05
loadingN: 有一種狀況就是我不知道為什麼,所以我寫了它幹什麼... 01/19 15:09
silence0925: // [MOLYXXXXXX] 01/19 17:28
superpandal: 複雜和不複雜還是不複雜好多了 複雜本身也會掩蓋程式 01/19 20:02
sowrey: 有人可以分析一下包養平台的差異嗎 01/19 20:02
superpandal: 真實的目的 學著用本身是種痛苦不說 學到的東西也是 01/19 20:04
superpandal: 皮毛 也沒什麼高深的哲學用來提升自己 就是硬塑造的 01/19 20:06
superpandal: 技術壁壘 01/19 20:08
superpandal: 不複雜就容易追蹤程式碼 再差也差不過一堆你不爬文就 01/19 20:11
superpandal: 不知道怎麼回事的東西 01/19 20:11
cw758: 那個包養網人最多XD 01/19 20:11
wulouise: 寫code一樣註解新手滿容易犯,Review別人code有助進步 01/19 23:23
timofEE: 推 01/22 18:55
RickyWdrUm: 感謝分享~ 01/23 07:35
friends29: 推 01/24 09:14
layer0930: 註解不寫…,那其實等於沒寫過什麼重要的東西。所以才 01/24 14:43
ludi: 我妹上包養網被我發現= = 01/24 14:43
layer0930: 養成不寫註解 01/24 14:43
xluds24805: 當你覺得這段 code 未來會有人來問你為什麼這樣寫時, 01/27 12:10
xluds24805: 就代表該留點註解了 01/27 12:10
ph90119: 推 01/27 16:53
viper9709: 推樓樓上 01/27 17:47
peernut: 隔壁桌的人竟然在討論包養... 01/27 17:47
superpandal: 重要的東西照樣不寫 要寫其實文檔好過註解 01/28 05:18
superpandal: 就現況來看清晰好了解的註解根本沒有 因為人人 01/28 05:19
superpandal: 都不愛寫 都是叫別人寫頤指氣使 01/28 05:20
superpandal: 自己要寫堆三阻四 給人製造障礙 01/28 05:20
superpandal: 要看那堆垃圾註解 不如程式本身簡單質量好 01/28 05:25
xikimi: 樓上是不是被包養 01/28 05:25
superpandal: 看到最後還是自己分析程式比較快 01/28 05:40
t64141: 註解優於文件的點在於修改出出程式時可以一併集中修改, 01/28 22:35
t64141: 不需要大老遠去找文件出來改。另一方面,在不願寫註解的 01/28 22:36
t64141: 情境下,更不可能願意去翻找並維護離程式碼更遙遠的文件 01/28 22:36
t64141: 。 01/28 22:36
Avero: 未看先猜這包養 01/28 22:36
t64141: 除非在包含系統整體設計與脈絡時,另外管理的文件才會展 01/28 22:38
t64141: 現出優勢 01/28 22:38
superpandal: 註解是分散的 你改一個以為相關的不用改? 01/29 23:17
superpandal: 文檔具有統一性更好些 大專案重要的是整體架構 01/29 23:18
superpandal: 註解表達的是片面的 只是講述區塊的程式 01/29 23:18
ejoz: 一定又是這包養 01/29 23:18
superpandal: 你也不可能逐字講解 本身表達力就不足 01/29 23:19
superpandal: 至於找文件...一個quick open 01/29 23:21
superpandal: 快捷鍵就打開了好嗎 還可以內文檢索 01/29 23:22
superpandal: 如果會vi/vim那就更強了 01/29 23:24
foreverk: 我想兩個都有是比較理想的情況,雖然實際情況很常兩個 01/30 15:00
FishRoom: 包養平台不意外 01/30 15:00
foreverk: 都沒有 01/30 15:00
rxforever: 沒什麼好說的 從來沒有刪掉註解的 02/16 23:22
rxforever: 除非註解寫錯了 02/16 23:22