#work 本週的大寶貝。完蛋了大寶貝事蹟已經多到要週更
本週大寶貝寫的小系統終於要準備上線,老闆指示Merge request指給我,要刁刁他不好的寫code習慣
https://images.plurk.com/709ZzNUwThIC691ABmep9h.png 於是他推了一個Request給我,還告訴我說老闆希望他明天中午之前要上線所以希望我明天中午之前要看完
latest #6
掰噗~
7 months ago
(brokenheart)
推request給我之後也有在工作群組上tag我,所以我老闆來關心了一下「 takochan110 啊,啊你看得完嗎?」
我故意一臉驚嚇「我...會盡力...不過...有...點點點多...」
老闆走過來一看看到這個code量就開始罵大寶貝XD
然後大寶貝的code不看則已一看驚人
舉例來說,(偷網路上解釋call stack的圖來用)程式的執行是用stack的結構在跑https://images.plurk.com/UIugsq5elCM8K7opXUFiY.png
像圖裏頭那樣,如果函式3執行時raise error,函式3本身沒有catch這個error,就會往外拋給函式2,甚至是函式1
立即下載
反正就是一路往外拋拋拋拋到有人處理不然就是程式中斷。
大寶貝的code在使用os.write的時候完全沒有try-catch,反正一路拋到網頁上也沒接住
所以我當時就給了review comment,告訴他說file相關的操作以及網路連線這種行為最好要進行錯誤處理
https://images.plurk.com/2FVPMPUeq2zsQZ1YovN1uW.png 反正我給了comment覺得是足夠詳細了啦
結果他還走過來問我說
請問要做什麼錯誤處理?
明明有寫入檔案的行為,還跟我說,我這邊不是只有讀取檔案嗎?這樣會有什麼問題?
但code看來就不是只有讀取,有寫入檔案啊...
真的是chatGPT工程師欸
back to top