反 for-if 編程模式

本文是從 Introducing the for-if anti-pattern 這篇文章翻譯而來。

這些年來，我看到過大量的反編程模式。我感覺應該向大家分享一些。

今天，我要介紹的是被我稱作反for-if編程模式的反模式，也就是人們所說的”我們賣給你整個座位，但你需要的只是一個邊。”
這是一個特殊的反for-case模式，其中所有的情況中只有一次會是null。

for (int i = 0; i < 100; i++) {
  if (i == 42) { do_something(i); }
}

這種情況可以簡單的寫成

do_something(42);

這個反for-if模式可以表現成各種各樣的形式。比如：

foreach (string filename in Directory.GetFiles("."))
{
    if (filename.Equals("desktop.ini", StringComparison.OrdinalIgnoreCase))
    {
        return new StreamReader(filename);
    }
}

它是在一個目錄里遍歷查找一個指定文件，如果找到了，就返回文件的數據流。這段代碼的一種不是那么折騰的寫法是

if (File.Exists("desktop.ini"))
{
    return new StreamReader("desktop.ini");
}

請注意，兩個版本的代碼片段具有相同的競爭條件：如果這個desktop.ini本來是存在的，但在你創建Stream­Reader之前被刪掉了，你就會得到一個File­Not­Found­Exception錯誤。

再舉一個例子：

foreach (object o in hashtable.Keys)
{
    if (o == "target") return hashtable["target"];
}

等同于

return hashtable["target"];

我猜測這些家伙不喜歡在圖書館里通過書名找一本書，因為他們的做法是如此的繁瑣：
他們來到圖書館里員面前說，“把你所有的書都給我，”然后他們拿著裝滿了上千本書的籃子，坐到墻角里自言自語：
“不是，這本書的書名不對”，

“不是，這本也不是”，

“標題還是不對。”

“這本書呢？”

”不是，也不是這本。“

”老天，我要這樣一本一本翻到什么時候…“

本文轉載自: 外刊IT評論 http://www.aqee.net/