特殊的反 for-if 編程模式
本文是從 Introducing the for-if anti-pattern 這篇文章翻譯而來。 |
這些年來,我看到過大量的反編程模式。我感覺應(yīng)該向大家分享一些。
今天,我要介紹的是被我稱作反for-if編程模式的反模式,也就是人們所說的”我們賣給你整個座位,但你需要的只是一個邊。”
這是一個特殊的反for-case模式,其中所有的情況中只有一次會是null。
- for (int i = 0; i < 100; i++) {
- if (i == 42) { do_something(i); }
- }
這種情況可以簡單的寫成
- do_something(42);
這個反for-if模式可以表現(xiàn)成各種各樣的形式。比如:
- foreach (string filename in Directory.GetFiles("."))
- {
- if (filename.Equals("desktop.ini", StringComparison.OrdinalIgnoreCase))
- {
- return new StreamReader(filename);
- }
- }
它是在一個目錄里遍歷查找一個指定文件,如果找到了,就返回文件的數(shù)據(jù)流。這段代碼的一種不是那么折騰的寫法是
- if (File.Exists("desktop.ini"))
- {
- return new StreamReader("desktop.ini");
- }
請注意,兩個版本的代碼片段具有相同的競爭條件:如果這個desktop.ini本來是存在的,但在你創(chuàng)建StreamReader之前被刪掉了,你就會得到一個FileNotFoundException錯誤。
再舉一個例子:
- foreach (object o in hashtable.Keys)
- {
- if (o == "target") return hashtable["target"];
- }
等同于
- return hashtable["target"];
我猜測這些家伙不喜歡在圖書館里通過書名找一本書,因為他們的做法是如此的繁瑣:
他們來到圖書館里員面前說,“把你所有的書都給我,”然后他們拿著裝滿了上千本書的籃子,坐到墻角里自言自語:
“不是,這本書的書名不對”,
“不是,這本也不是”,
“標(biāo)題還是不對。”
“這本書呢?”
”不是,也不是這本。“
”老天,我要這樣一本一本翻到什么時候…“
原文鏈接:http://www.aqee.net/introducing-the-for-if-anti-pattern/
【編輯推薦】