跳到主要內容

續-成員變數宣告為private

我贊同小麟的說法,最有彈性的作法並不一定是最好的作法,開發時程是很重要的考量,我提到這個建議,並不是說要把現在所有的class的成員變數改為private,而是建議,今後撰寫任何的class,其成員變數最好都宣告成private,甚至繼承一個沒有被其他類別繼承的類別,亦建議把原類別所有的成員變數改為private,再繼承,除了享有封裝的好處,也可以避免copy/paste的後遺症,改寫繼承的變數可以半強迫讓編寫者再仔細看一次抄過來的code,了解所有copy過來的code是不是真的都需要。實際上重點是,這並沒有花太多的時間,以我的經驗,500行的程式碼超過10個成員變數,利用輔助程式替換,含測試不會超過30分鐘。這還是半熟練運用工具的狀態下,相信熟練的話會更快。花一點小時間,為自己留一點後路不是很好嗎。

但是我覺得程式的彈性真的很重要,一個程式的壽命與成長性取決於程式的彈性,因此我建議在開發時,第一輪先把程式完成,完成後先做簡單的重構,更名、萃取函式、封裝變數這種不浪費時間的小工作(善用工具的話,這真的是小工作)。在單元測試完後或α中後期,探討相關可能變動,對相關的架構做適當的調整。這段時間做大步重構,有三個原因,

  • 第一、這時期功能已經開發完成,測試也進入中後期,經由多番測試大約了解變動的因子,再進行code review,知道程式何處無法因應將來可能的變化,也可避免所謂的過度設計。
  • 第二、這時也對相關功能,可能需做的測試案例,已經相當的完備了,利用完整的測試案例,重構的品質更有保證。
  • 第三、這是我認為進行重構最好的時刻,開發者這段時間對程式最熟練,經過一個專案週期後,會幾乎忘記上次為啥寫這樣的程式碼,將來要變動程式時,所花費的成本將逐漸遞增。開發者熟練重構的話,可以完全不改變功能,將架構改的更好。且此時進行重構已經不影響時程。

當然這些假設都是,開發時間來得及,解bug夠快以不影響時程來規劃。所以第二階段重構其實也是非必要做的,只是有做會比較好,但是我建議簡單的重構一定要做,我覺得一個容易理解的程式碼比完整的文件更重要,尤其當你面對一個函式,竟然要拉動三頁才能看完,或者一個神秘的判斷式,完全不能了解他在判斷什麼時,就更能體會了。

ps.蘇哥用他的防毒軟體測我新抓的蕃茄有毒,可是我已經用五種以上的防毒軟體測過了,都沒掃到,norton最新的病毒定義碼也裝了,就是沒毒啊!真是奇怪?可能我的電腦天生對此病毒免疫,所以新版蕃茄還是不要散發出去好了,還是有勇者願意嘗試。

我真的是瘋了,又打那麼多。

留言

這個網誌中的熱門文章

解決 CI Trust Issue:Target Must Be Enabled Before It Can Be Used

📱 iOS開發 | 🔧 CI/CD | 💻 Xcode | 🐛 除錯筆記 🔴 問題描述 這兩天在跑 CI 時突然出現錯誤訊息: Package@swift-6.0.swift:PACKAGE-TARGET:CasePathsMacros: error: Target 'CasePathsMacros' must be enabled before it can be used 🤔 嘗試過的解法 💬 Claude 的建議 首先詢問了 Claude,得到以下步驟: 先更新 swift-case-paths 到最新版本 確保使用 "Up to Next Major Version" 執行 File → Packages → Reset Package Caches Clean Build Folder (Cmd + Shift + K) 重新 Build 結果: 一看就知道沒用 😅 🤖 ChatGPT 的建議 接著試了 ChatGPT 的解法,主要是降低引用到的 package 版本。繞了一圈,還是沒用。 ✅ 最終解決方案 最後還是回到 Google,找到了真正有效的解法。針對這個 macro fingerprint validation 問題,有三種解決方式: 📌 方法一:本機開發用(Terminal 指令) defaults write com.apple.dt.Xcode IDESkipMacroFingerprintValidation -bool YES 📌 方法二:xcodebuild 參數 在執行 xcodebuild 指令時,加上 -skipMacroValidation 參數 📚 參考連結: https://vocus.cc/article/690779ebfd89780001859b14 📌 方法三:CI 正統做法 ⭐️(推薦) 步驟 1: 在專案根目錄建立資料夾 ci_scripts 步驟 2: 在此資料夾中建立腳本 ci_post_clone.sh ,內容如下: #!/bin/zsh mkdir -p ~/Library/org.swift.swiftpm/security/ cp macros.js...

萬年季

這個活動上上個禮拜六看到新聞,我以為只有一天,禮拜日就沒去,感覺有點可惜。沒想到是九天的活動,看到阿蓮他們也要去。那就剛好約一起去囉!前一天剛騎完充滿自我鍛練的阿里山之旅,又摔車,左手臂形同殘廢,但是既然約好了還是要去,捨命陪美女。那天阿蓮一直說他可能遲到,沒想到當天因為跑去看醫生,結果遲到的人是我。拍謝啦!當天中午吃飯,虹君還問我明明知道馥菁沒來,為什麼還要來,ㄟ!太明顯囉!我是這種人嗎?我會讓Jim獨享兩位美女嗎?喔!不!我會拋棄Jim嗎?約了就要來啊!虹君後來說,他原本拔智齒也不太想來,臉太腫了。不會啊!我看不出來,都一樣圓!ㄜ!不!一樣可愛!笑起來,頗令人心動(有圓回來嗎..) 所有照片http://picasaweb.google.com/cccmail/JVqzQG# 從中午就一直繞著蓮池潭逛,上次來的時候還在清潭,封閉中,這次來看果然漂亮很多,和小時候的印象完全不一樣。大家好像不喜歡照相,身為高雄人在蓮池潭照相,有種詭異的感覺。 照這種人形立牌,是我很喜歡做的事,看得出來誰是誰嗎? 過七關,有意思的祈福關卡,但是太多人了沒去排。 阿蓮與蓮花。蓮花太小了。 偷照擺Pose。 當天在廟口前的表演,人太多,只能從狹縫中看到一點,兩位女生,完全看不到.. 看了一陣子,他們終於受不了,拉我們走了。順便去吃晚餐,Jim也受不了,他要找個位置坐下來,他說那天好像行軍,走的腳超酸。結果途中遇到馥菁帶著學妹來逛,78年次,喔!這個補!喔!不!真年輕。哥哥可以裝年輕嗎?很多人看我不是還是學生就是剛畢業!看樣子我可以講我75年次,這有人相信嗎? 歷經20多年再走一次龍虎塔與春秋閣,之前的18層地獄好像換掉了。我覺得是很棒的特色ㄟ,可惜了童年的回憶。 最後主舞台也是人山人海,只能看大螢幕。 失去舞台的舞獅,在人群中自己表演起來了。 主辦單位邀請各廟宇,舞獅或跳人偶的來表演。但是大家好像都表演得很盡興,無法控制長度,主持人一直在趕人,"XX國小,時間咖控制機勒,節目已經Delay了",國台英語交雜,頗有喜感。表演隊伍也很有意思,原本以為只是表演各廟宇慶典活動或民俗技藝,但沒想到還有原住民舞...

用 AI Debug 的迷思:當建議越改越糟時

現在許多開發者習慣用 AI 來協助 debug,但在實務上常遇到一種情況: 依照 AI 建議改了兩三輪後,錯誤仍然存在,甚至越改越複雜。 這種狀況其實有幾個常見的盲點,值得特別注意。 1. 先回到「上一個正常版本」 當你已經按照 AI 的方向修了好幾次但問題仍未解決時,最有效的第一步是: 回到上一個正常工作的版本,縮小問題來源。 許多 bug 並不是你正在看的那段程式碼造成的,而可能是: 同事剛好修改了某個底層模組 某個 shared component 產生 side effect Auto Layout 層級重新 layout 時觸發 crash 如果只是盯著眼前的 function 修,反而容易被誤導。 2. AI 沒有看到你的整個專案 AI 通常只能根據你貼出的片段判斷問題,這代表它不知道: 你的 view hierarchy 裡是否有其他 constraint 影響 layout 某些 model 是否被 extension 修改過 父層或子層邏輯是否干擾目前的行為 整個專案採用的 concurrency 模型是什麼 因此,AI 可能會朝著完全錯誤的方向修,導致反覆修改卻無法解決。 3. Swift 6 例子:錯誤真正原因常不在你修改的那一行 例如開發者常遇到的錯誤: passing closure as a 'sending' parameter risks causing data races 許多人(包含 AI)會開始從 function 內部調整,但這類錯誤真正的關鍵通常是: 傳進去的物件沒有實作 Sendable。 也就是說,你不是要改 function,而是要回頭檢查: 傳入的 model / struct / class 裡面是否有 non-Sendable 成員 是否需要標註 @unchecked Sendable 如果 AI 沒看到相關檔案,自然很難找到正確方向。 結語:AI 是工具,不是預言機 AI 很適合用來: 解釋概念 協助產生測試程式 提供重構建議 釐清你已懷疑的方向 但在 debug 狀況下,以下三件事更重要: 回到上一版,縮小差異範圍。 釐清真正問題來...