成為更優秀的程序員:退后一步看問題

jopen 10年前發布 | 4K 次閱讀 程序員

請把下面的[客戶]、[老板]或其它任何東西換成你想要的名稱。

一天,在工作中…

Bug #3890 來自客戶:

有個程序出現了錯誤,程序提示說“SpeedCalculator::compute()里出現了除零情況”。

請盡快修復!

你打開SpeedCalculator.php,發現:

public function compute() {
    return $this->distance / $this->time;
}

修復bug

簡單!是誰寫的這段代碼,他怎么不用一點腦子!

public function compute() {
    if ($this->time == 0) {
        return 0;
    }
    return $this->distance / $this->time;
}

你就這樣修改完了,這個bug只用了你2分鐘。

后來,同樣的bug又出現在了RatioCalculatorMoneyCalculator中,當把它們也修正后,團隊中的所有人都認為,再也不會有這樣的問題出現了,這是最后一次!代碼現在又是堅固無比了!

一個月后,另外一個bug出現了。這次程序沒有崩潰,但客戶在報表中發現了錯誤的計算結果,是因為那句return 0;的代碼修改。

退后一步看問題

如果,我們不是匆匆忙忙的修改,而是退后一步,以更寬廣的視野看待些問題。

為什么這種事情會發生?

因為$this->time被賦值成 0.

簡單!讓我們阻止這種事情發生。

public function setTime($time) {
    if ($time == 0) {
        throw new InvalidArgumentException("Invalid value");
    }
    $this->time = $time;
}

這樣,修改的效果不是更好嗎?你保證了數據的正確性。但這樣客戶并不是很高興,因為當他輸入0時,程序會跳到一個錯誤頁面。

那么,你應該在控制層捕捉這個錯誤,讓用一個漂亮的錯誤頁面顯示它們。

當做完了這些,你認識到,在RatioCalculatorMoneyCalculator中,你也應該這樣做,于是你拷貝/張貼,修改了它們兩個。

稍等一下,客戶更喜歡讓錯誤信息顯示成橘黃色的,而不是紅色的。于是,你再次拷貝/粘貼,再次修改一遍。

再退后一步

如果,我們不是修復這個 Bug,而是發現了一個普遍的要求?

為什么客戶會輸入 0 ?因為他犯了個錯誤。

這對我們的要求是什么?

  • 我們只需要這次防止用戶輸入 0 來避免“speedCalculationForm = 0”嗎?

  • 我們只需要讓“speedCalculationForm”里的數據合法就行了嗎?

  • 還是,我們要對所有的用戶輸入繼續校驗?

為什么不開發一個校驗工具包呢?

且慢!不要自己去開發一個!請退后一步想想,深呼吸,去使用現有的第三方校驗框架。

關于需求

我們,程序員,熱愛計算機技術。當你的客戶或老板,大聲的脫口說出他們想要的東西時,我們無法阻止,我們只能想象如何去實現它們。

但是,我們需要用更全面的眼光看問題。如果我們想對自己的工作更負責,我們應該去理解為什么他們會提出這樣的需求,而不是著急著去尋找解決方案。當然,這可能會占用你更多的精力。

是客戶真的需要“一個會躲避鼠標點擊的閃光的按鈕”嗎?還是他們需要的是另外一個功能——他們不了解的功能,需要你去幫他們定義的功能?這種事情同樣會發生在你自己身上!你真的需要用程序打開一個文件,往里面寫入一些信息嗎?還是,你真正需要的是一個日志系統?

退后一步看問題,看更大的藍圖、更完整的信息。雖然你是一個很優秀的編程高手,但編程的目的是為了解決問題。

來自:Be a better programmer: take a step back

 本文由用戶 jopen 自行上傳分享,僅供網友學習交流。所有權歸原作者,若您的權利被侵害,請聯系管理員。
 轉載本站原創文章,請注明出處,并保留原始鏈接、圖片水印。
 本站是一個以用戶分享為主的開源技術平臺,歡迎各類分享!