跳到主要內容

續-成員變數宣告為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...

用 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 狀況下,以下三件事更重要: 回到上一版,縮小差異範圍。 釐清真正問題來...

借助 AI 除錯:解決登出時的解密錯誤

  前言 最近遇到一個有趣的 bug,趁這次機會試試看用 AI 來協助除錯。整個過程蠻有收穫的,分享給大家參考。 問題發現 Bug 描述 在測試登出功能時,發現 app 會發生 crash。追蹤後發現是在登出時,API 會呼叫底層的解密函式,但解密用的 key 明顯有問題,導致解密失敗而引發錯誤。 由於這段流程是另一位同事負責開發的,程式碼邏輯比較複雜,短時間內看不出根本原因。於是先採取防禦性做法: 如果 key 為空值,就將解密失敗的錯誤吃掉不處理 。判斷後認為這樣不會影響 UI 操作,因為登出後 key 會重新取得。測試後確認沒問題,就先把這個 bug 暫時壓下來了。 深入追查 第一回合:初步分析 趁著有空檔,決定詳細釐清這個問題。想試試 AI 的能耐,於是下了第一個 prompt: 分析 SessionManager 的 processEncryptedData 方法中,為何會發生 bytesToTrim > currentLength 的錯誤?從呼叫該方法的地方分析可能發生的原因。 AI 給出了幾個可能的原因: 加密數據損壞或截斷 網路傳輸過程中數據包丟失或損壞 API 回應中的加密字串不完整 解密後的最後一個字節值異常大 錯誤的加密算法或密鑰 使用了錯誤的解密密鑰或 IV Session key 交換失敗,但程式繼續使用無效密鑰解密 解密產生亂碼 Hex 字串轉換錯誤 包含非法字符、長度不是偶數、部分缺失等問題 AES CBC 解密失敗但沒有檢查錯誤 Whitebox 密鑰準備失敗 IV 不匹配 數據長度不符合 AES 塊大小要求 PKCS7 Padding 格式錯誤 加密時使用了錯誤的 padding 方式 最後字節不是有效的 padding 長度值 (1-16) 多重解密處理錯誤 數據被重複解密或解密順序錯誤 記憶體緩衝區問題 緩衝區大小分配錯誤或溢出 AI 也分析了呼叫鏈: API 回應 → DataTransformer → CryptoHandler → decryptSessionKey → processEncryptedData 評估: 這些分析大多是從相關函式內部找錯誤,但沒有太大用處,...