青青草原综合久久大伊人导航_色综合久久天天综合_日日噜噜夜夜狠狠久久丁香五月_热久久这里只有精品

隨筆-341  評論-2670  文章-0  trackbacks-0

上次有人來要求我寫一篇文章談談什么代碼才是好代碼,是誰我已經忘記了,好像是AutoHotkey還是啥的專欄的作者。撇開那些奇怪的條款不談,靠譜的 代碼有一個共同的特點,就是DRY。DRY就是Don't Repeat Yourself,其實已經被人談了好多年了,但是幾乎所有人都會忘記。

什么是DRY(Don't Repeat Yourself)

DRY 并不是指你不能復制代碼這么簡單的。不能repeat的其實是信息,不是代碼。要分析一段代碼里面的什么東西時信息,就跟給物理題做受力分析一樣,想每次 都做對其實不太容易。但是一份代碼總是要不斷的修補的,所以在這之前大家要先做好TDD,也就是Test Driven Development。這里我對自己的要求是覆蓋率要高達95%,不管用什么手段,總之95%的代碼的輸出都要受到檢驗。當有了足夠多的測試做后盾的時 候,不管你以后發生了什么,譬如說你發現你Repeat了什么東西要改,你才能放心大膽的去改。而且從長遠的角度來看,做好TDD可以將開發出相同質量的代碼的時間縮短到30%左右(這是我自己的經驗值) 。

什么是信息

信息這個詞不太好用語言下定義,不過我可以舉個例子。譬如說你要把一個配置文件里面的字符串按照分隔符分解成幾個字符串,你大概就會寫出這樣的代碼:

// name;parent;description
void ReadConfig(const wchar_t* config)
{
    auto p 
= wcschr(config, L';');                            // 1
    if(!p) throw ArgumentException(L"Illegal config string"); // 2
    DoName(wstring(config, p));                               // 3
    auto q = wcschr(p + 1, L';');                             // 4
    if(!q) throw ArgumentException(L"Illegal config string"); // 5
    DoParent(wstring(p + 1, q);                               // 6
    auto r = wcschr(q + 1, L';');                             // 7
    if(r) throw ArgumentException(L"Illegal config string");  // 8
    DoDescription(q + 1);                                     // 9
}

這段短短的代碼重復了多少信息?

  • 分隔符用的是分號(1、4、7)
  • 第二/三個片段的第一個字符位于第一/二個分號的后面(4、6、7、9)
  • 格式檢查(2、5、8)
  • 異常內容(2、5、8)

除了DRY以外還有一個問題,就是處理description的方法跟name和parent不一樣,因為他后面再也沒有分號了。

那這段代碼要怎么改呢?有些人可能會想到,那把重復的代碼抽取出一個函數就好了:

wstring Parse(const wchar_t& config, bool end)
{
    auto next 
= wcschr(config, L';');
    ArgumentException up(L
"Illegal config string");
    
if (next)
    {
        
if (end) throw up;
        wstring result(config, next);
        config 
= next + 1;
        
return result;
    }
    
else
    {
        
if (!end) throw up;
        wstring result(config);
        config 
+= result.size();
        
return result;
    }
}

// name;parent;description
void ReadConfig(const wchar_t* config)
{
    DoName(Parse(config, 
false));
    DoParent(Parse(config, 
false));
    DoDescription(Parse(config, 
true));
}

是不是看起來還很別扭,好像把代碼修改了之后只把事情搞得更亂了,而且就算config對了我們也會創建那個up變量,就僅僅是為了不 重復代碼。而且這份代碼還散發出了一些不好的味道,因為對于Name、Parent和Description的處理方法還是不能統一,Parse里面針對 end變量的處理看起來也是很重復,但實際上這是無法在這樣設計的前提下消除的。所以這個代碼也是不好的,充其量只是比第一份代碼強一點點。

實 際上,代碼之所以要寫的好,之所以不能repeat東西,是因為產品狗總是要改需求,不改代碼你就要死,改代碼你就要加班,所以為了減少修改代碼的痛苦, 我們不能repeat任何信息。舉個例子,有一天產品狗說,要把分隔符從分號改成空格!一下子就要改兩個地方了。description后面要加tag! 這樣你處理description的方法又要改了因為他是以空格結尾不是0結尾。

因此針對這個片段,我們需要把它改成這樣:

vector<wstring> SplitString(const wchar_t* config, wchar_t delimiter)
{
    vector
<wstring> fragments;
    
while(auto next = wcschr(config, delimiter))
    {
        fragments.push_back(wstring(config, next));
        config 
= next + 1;
    }
    fragments.push_back(wstring(config));
    
return fragments; // C++11就是好!
}

void ReadConfig(const wchar_t* config)
{
    auto fragments 
= SplitString(config, L';');
    
if(fragments.size() != 3)
    {
        
throw ArgumentException(L"Illegal config string");
    }
    DoName(fragments[
0]);
    DoParent(fragments[
1]);
    DoDescription(fragments[
2]);
}

我們可以發現,分號(L';')在這里只出現了一次,異常內容也只出現了一次,而且處理name、parent和 description的代碼也沒有什么區別了,檢查錯誤也更簡單了。你在這里還給你的Library增加了一個SplitString函數,說不定在以 后什么地方就用上了,比Parse這種專門的函數要強很多倍。

大家可以發現,在這里重復的東西并不僅僅是復制了代碼,而是由于你把 同一個信息散播在了代碼的各個部分導致了有很多相近的代碼也散播在各個地方,而且還不是那么好通過抽成函數的方法來解決。因為在這種情況下,就算你把重復 的代碼抽成了Parse函數,你把函數調用了幾次實際上也等于重復了信息。因此正確的方法就是把做事情的方法變一下,寫成SplitString。這個 SplitString函數并不是通過把重復的代碼簡單的抽取成函數而做出來的。去掉重復的信息會讓你的代碼的結構發生本質的變化

這個問題其實也有很多變體:

  • 不能有Magic Number。L';'出現了很多遍,其實就是個Magic Number。所以我們要給他個名字,譬如說delimiter。
  • 不要復制代碼。這個應該不用我講了。
  • 解耦要做成正交的。SplitString雖然不是直接沖著讀config來寫的,但是它反映了一個在其它地方也會遇到的常見的問題。如果用Parse的那個版本,顯然只是看起來解決了問題而已,并沒有給你帶來任何額外的效益。

信息一旦被你repeat了,你的代碼就會不同程度的出現各種腐爛或者破窗,上面那三條其實只是我能想到的比較常見的表現形式。這件事情也告訴我們,當高手告訴你什么什么不能做的時候,得想一想背后的原因,不然跟封建迷信有什么區別。

posted on 2014-07-15 06:44 陳梓瀚(vczh) 閱讀(15879) 評論(9)  編輯 收藏 引用 所屬分類: 啟示

評論:
# re: 靠譜的代碼和DRY[未登錄] 2014-07-16 04:23 | 煙圈
vally vally god   回復  更多評論
  
# re: 靠譜的代碼和DRY[未登錄] 2014-07-18 02:02 | korall
提供機制優先于提供功能  回復  更多評論
  
# re: 靠譜的代碼和DRY 2014-07-20 10:33 | cymheart
當需求不同時,實現同樣功能的代碼算法也可能是不同的。
如果需求上一開始就強調config文件中哪些字符會變動,算法設計上肯定要做到對應。如果需求不明,也只能簡單的用博主所說的第一段代碼了。

  回復  更多評論
  
# re: 靠譜的代碼和DRY 2014-08-10 19:33 | 飛奔吧少年
各種都真不錯!  回復  更多評論
  
# re: 靠譜的代碼和DRY 2014-08-10 19:34 | 飛奔吧少年
主頁真不錯!  回復  更多評論
  
# re: 靠譜的代碼和DRY 2014-08-31 22:51 | 清醒瘋子利炳根
反復看了幾遍,看明白了:)集中在一個地方調用處理,優于分散在不同的地方來調用。

不單單是把代碼抽取出來建立子程,子程里的實現也要不重復:)

第三個例程的代碼,確實要比第二個那樣的把子程作為參數調用的,好很多  回復  更多評論
  
# re: 靠譜的代碼和DRY 2015-01-13 06:08 | 男人沒錢
DRY的說法很靠譜,我的看法是“做任何事情之前,先減少心智負載”,就是把那些可以簡化掉的行為變成一個接口,以后來了類似的需求,只需要往這個接口里一扔就好了。希望vczh多給我們普及一些簡單易懂的東西。。  回復  更多評論
  
# re: 靠譜的代碼和DRY 2015-03-01 02:51 | ivy
感覺是借用了vb的概念呀  回復  更多評論
  
# re: 靠譜的代碼和DRY 2015-03-05 22:38 | sptt
感同身受~  回復  更多評論
  
青青草原综合久久大伊人导航_色综合久久天天综合_日日噜噜夜夜狠狠久久丁香五月_热久久这里只有精品
  • <ins id="pjuwb"></ins>
    <blockquote id="pjuwb"><pre id="pjuwb"></pre></blockquote>
    <noscript id="pjuwb"></noscript>
          <sup id="pjuwb"><pre id="pjuwb"></pre></sup>
            <dd id="pjuwb"></dd>
            <abbr id="pjuwb"></abbr>
            99热免费精品在线观看| 欧美顶级大胆免费视频| 欧美日韩福利视频| 亚洲精品乱码久久久久久黑人 | 欧美激情一区二区三区在线| 国产一级一区二区| 久久精品国产亚洲一区二区三区| 亚洲一区二区三区四区在线观看 | 美女视频黄 久久| 欧美一区二区私人影院日本| 国产日韩欧美成人| 久久久精品动漫| 久久久成人精品| 亚洲第一综合天堂另类专| 欧美 日韩 国产精品免费观看| 久久精品国产亚洲一区二区三区| 在线视频国内自拍亚洲视频| 欧美激情第9页| 欧美日韩国产首页| 午夜在线精品| 亚洲天堂免费在线观看视频| 亚洲国产专区| 欧美精品三级日韩久久| 国产区在线观看成人精品| 欧美一区二区成人| 久久久久久一区二区三区| 一区二区三区产品免费精品久久75 | 欧美成人福利视频| 99亚洲视频| 久久综合狠狠| 久久精品99| 国产精品久久久久一区二区三区| 久久婷婷国产综合国色天香| 欧美日本簧片| 亚洲精品国产品国语在线app | 亚洲日本电影| 最新日韩在线视频| 久久精品视频在线| 亚洲欧美日韩电影| 国产精品久久久久一区二区| 亚洲欧美日韩精品久久奇米色影视| 欧美一级在线播放| 久久国产精品第一页| 国产欧美日韩视频| 久久9热精品视频| 欧美激情视频在线播放 | 久久久夜色精品亚洲| 久久一区中文字幕| 亚洲自拍偷拍一区| 欧美一区国产二区| 国产午夜精品美女视频明星a级| 亚洲网站在线| 欧美福利在线| 性欧美大战久久久久久久久| 激情一区二区| 欧美日韩一区二区国产| 久久婷婷麻豆| 一区二区在线观看视频| 亚洲欧美制服中文字幕| 欧美成人精品高清在线播放| 一区二区三区高清不卡| 国产午夜精品久久久久久免费视| 久久久久国内| 欧美亚洲免费| 亚洲一二三四久久| 亚洲精品国产精品国自产观看| 久久国产精品久久精品国产 | 亚洲综合色在线| 亚洲国产精品第一区二区三区| 一区二区欧美日韩视频| 欧美午夜宅男影院在线观看| 看欧美日韩国产| 欧美一区二区在线免费播放| 亚洲精品九九| 亚洲免费观看视频| 亚洲激情综合| 欧美激情在线免费观看| 一区二区三区久久网| 91久久久久久| 日韩午夜在线播放| 宅男噜噜噜66一区二区| 中文国产一区| 久久国产精品免费一区| 一区二区久久久久久| 亚洲日本中文| 99国产精品一区| 午夜欧美理论片| 伊人久久亚洲热| 99视频在线精品国自产拍免费观看| 亚洲欧美日韩国产成人精品影院| 久久久久久亚洲精品杨幂换脸 | 欧美va亚洲va国产综合| 欧美日韩午夜视频在线观看| 在线成人亚洲| 性久久久久久久久久久久| 亚洲福利在线看| 欧美一区二区三区免费观看| 欧美日韩另类综合| 亚洲国产精品一区二区第四页av| 亚洲小说欧美另类婷婷| 亚洲激情欧美激情| 免费欧美在线| 亚洲第一在线综合在线| 欧美**字幕| 久久婷婷国产综合尤物精品| 国产精品一区二区在线观看网站| 亚洲精品乱码久久久久久蜜桃麻豆 | 久久一区二区三区超碰国产精品| 国产精品二区在线| 亚洲欧美日韩专区| 亚洲视频在线观看| 国产欧美一区二区三区另类精品 | 午夜视频在线观看一区二区三区 | 一本久道久久综合中文字幕| 亚洲国产一区二区三区在线播| 欧美不卡激情三级在线观看| 亚洲国产欧美不卡在线观看| 亚洲第一二三四五区| 欧美日韩国产影片| 欧美伊人久久| 欧美激情第3页| 国产精品久久精品日日| 欧美一区二区精美| 免费欧美网站| 久久久久免费观看| 欧美日韩国产综合网| 久久精品免费看| 欧美日韩视频第一区| 久久久女女女女999久久| 欧美精品在线观看| 久久一区二区视频| 国产精品久久久久毛片大屁完整版| 欧美一区二区三区在| 欧美日韩亚洲三区| 欧美.www| 亚洲电影免费| 欧美一级视频精品观看| 一区二区三区国产| 欧美激情一区二区久久久| 久久中文欧美| 国产中文一区| 久久久中精品2020中文| 欧美在线观看你懂的| 国产精品日韩电影| 亚洲欧美激情四射在线日| 香蕉久久夜色| 国产精品hd| 亚洲永久免费视频| 午夜精品久久久久影视| 国产精品国产自产拍高清av| 在线一区二区三区做爰视频网站| 亚洲网友自拍| 国产精品亚洲网站| 性欧美大战久久久久久久免费观看| 欧美在线视频播放| 亚洲黑丝一区二区| 欧美午夜a级限制福利片| 欧美一级专区免费大片| 美女日韩欧美| 午夜精品www| 亚洲日韩欧美视频| 国产精品日韩高清| 欧美成人精品在线| 亚洲专区免费| 亚洲高清资源| 久久久水蜜桃| 亚洲欧美综合另类中字| 亚洲激情亚洲| 国产午夜精品久久久久久久| 欧美顶级大胆免费视频| 午夜久久久久| 99国产精品99久久久久久粉嫩| 国产伦精品一区二区| 欧美极品一区| 免费成人av在线| 久久精品国产综合精品| 亚洲欧美精品| 亚洲视频 欧洲视频| 欧美午夜精品久久久久免费视| 美女精品在线观看| 狠狠久久亚洲欧美专区| 欧美中在线观看| 麻豆九一精品爱看视频在线观看免费| 亚洲人成网站精品片在线观看| 好吊色欧美一区二区三区四区| 国产精品三级视频| 国产精品美女久久久浪潮软件| 欧美日韩精品免费观看视一区二区 | 欧美日本免费| 国产精品久久久久久久久免费樱桃 | 欧美精品一区在线观看| 欧美视频二区| 国产精品视频网| 在线精品视频一区二区三四| 在线不卡中文字幕播放| 日韩午夜视频在线观看| 亚洲淫片在线视频| 久久久亚洲精品一区二区三区| 欧美不卡三区| 亚洲一二三区精品|