前文鏈接：http://www.cnblogs.com/pmer/archive/2012/12/13/2817180.html

【樣本】

?

?

【評(píng)析】

　　邊施工邊設(shè)計(jì)還不算，更雷人的是最后才考慮“組裝”。這是典型的“自底向上”而非結(jié)構(gòu)化程序設(shè)計(jì)所提倡的“自頂向下”。一方面作者夸大其詞地胡扯什么“在C語言程序設(shè)計(jì)當(dāng)中，“自頂向下，逐步求精”就像一句具有魔力的咒語，只要我們一念這個(gè)咒語，任何負(fù)責(zé)困難的問題都會(huì)迎刃而解”(P40)，另一方面又在編碼時(shí)踐踏“自頂向下”這條原則。這是典型的打著紅旗反紅旗。

【樣本】

?

?

【評(píng)析】

　　自從dijkstra指出goto有害之后，這種耗子窩流程圖就難得一見了。各位，趕緊出來瞧鼠窩。
　　怎么看怎么像挖了個(gè)大坑，然后跳了下去，把自己給埋了起來而出不來，然后不得已又挖了一個(gè)洞才好不容易鉆了出來。

【樣本】

?

?

【評(píng)析】

　　有這樣的流程圖，代碼混亂、復(fù)雜也就不足為奇了。
　　首先

while(true){/*……*/if(strlen(wd)>0){/*……*/}else{break;}}

?

　　這種結(jié)構(gòu)很糟糕，還不如寫成

?

while(true){/*……*/if(strlen(wd)>0){/*……*/continue;}break;}

?

　　寫出如此糟糕結(jié)構(gòu)最重要的原因在于作者不懂得如何定義變量，把wd這個(gè)char [30]定義在了while語句的循環(huán)體內(nèi)部，然而矛盾的是這個(gè)while語句是用來處理wd的。這讓人想起了一個(gè)笑話，一個(gè)笨婆娘縫被子，結(jié)果把自己給縫進(jìn)去被子里面去了。這里的wd就是如此。本應(yīng)該定義在while語句之外，卻被“縫”在了循環(huán)體內(nèi)。最后想出來也只好使用和笨婆娘一樣的最后招數(shù)——“break”了。
　　如果把wd定義在while語句之外，while語句就簡潔多了。

char wd[30] ;while( text = cutword(text,wd) , strlen(wd)>0 ){file->total +=1;word* exist = findnode(file->list,wd);if( NULL == exist){word* node = createnode(wd);if(NULL == pre){file->list = node;}else{pre->next = node;}pre = node;}else{exist->count +=1;}}

?

　　樣本中的strlen(wd)>0也是一種拙劣的寫法，因?yàn)樗葍r(jià)于 *wd !='\0'。每次循環(huán)進(jìn)行一次函數(shù)調(diào)用和每次循環(huán)只進(jìn)行一次“!=”運(yùn)算，效率顯然是天壤之別。
　　樣本中的NULL == pre在邏輯上是錯(cuò)誤的，應(yīng)該寫成NULL == file->list。
　　除此之外

if(NULL == pre){file->list = node;}else{pre->next = node;}pre = node;

?

是一種很笨拙的寫法。這種寫法把鏈表的head和結(jié)點(diǎn)的next成員區(qū)別開來，在這里必然要寫個(gè)if-else語句。更簡潔的寫法的基礎(chǔ)是把兩者視為同一種東西，這樣就可以統(tǒng)一對(duì)待了。造成這種笨拙的另一個(gè)原因是作者僵化固執(zhí)地把新結(jié)點(diǎn)加在鏈表結(jié)尾，實(shí)際上這樣做毫無必要。
　　綜上所述，這個(gè)函數(shù)可以更好地寫為

void parseword(txtfile* file) {char* text = file->text;file->list = NULL;file->total = 0;char wd[30] ;cleantext(text);while( text = cutword(text,wd) , *wd != '\0' ){file->total +=1;word* exist = findnode(file->list,wd);if( NULL == exist){word* node = createnode(wd);node->next = file->list;file->list = node ;}else{exist->count +=1;}} }

?　　當(dāng)然，代碼中還有其他毛病，但這些毛病與程序的總體思路的錯(cuò)誤或其他函數(shù)相關(guān)，在這里沒辦法進(jìn)一步糾正。

總結(jié)

以上是生活随笔為你收集整理的垃圾“程序是怎样练成的”——关于《C程序设计伴侣》第A章（四）的全部內(nèi)容，希望文章能夠幫你解決所遇到的問題。

如果覺得生活随笔網(wǎng)站內(nèi)容還不錯(cuò)，歡迎將生活随笔推薦給好友。