垃圾“程序是怎样练成的”——关于《C程序设计伴侣》第A章(四)
前文鏈接:http://www.cnblogs.com/pmer/archive/2012/12/13/2817180.html
【樣本】
?
?
【評析】
邊施工邊設(shè)計還不算,更雷人的是最后才考慮“組裝”。這是典型的“自底向上”而非結(jié)構(gòu)化程序設(shè)計所提倡的“自頂向下”。一方面作者夸大其詞地胡扯什么“在C語言程序設(shè)計當中,“自頂向下,逐步求精”就像一句具有魔力的咒語,只要我們一念這個咒語,任何負責困難的問題都會迎刃而解”(P40),另一方面又在編碼時踐踏“自頂向下”這條原則。這是典型的打著紅旗反紅旗。
【樣本】
?
?
【評析】
自從dijkstra指出goto有害之后,這種耗子窩流程圖就難得一見了。各位,趕緊出來瞧鼠窩。
怎么看怎么像挖了個大坑,然后跳了下去,把自己給埋了起來而出不來,然后不得已又挖了一個洞才好不容易鉆了出來。
【樣本】
?
?
【評析】
有這樣的流程圖,代碼混亂、復雜也就不足為奇了。
首先
?
這種結(jié)構(gòu)很糟糕,還不如寫成
?
while(true){/*……*/if(strlen(wd)>0){/*……*/continue;}break;}?
寫出如此糟糕結(jié)構(gòu)最重要的原因在于作者不懂得如何定義變量,把wd這個char [30]定義在了while語句的循環(huán)體內(nèi)部,然而矛盾的是這個while語句是用來處理wd的。這讓人想起了一個笑話,一個笨婆娘縫被子,結(jié)果把自己給縫進去被子里面去了。這里的wd就是如此。本應該定義在while語句之外,卻被“縫”在了循環(huán)體內(nèi)。最后想出來也只好使用和笨婆娘一樣的最后招數(shù)——“break”了。
如果把wd定義在while語句之外,while語句就簡潔多了。
?
樣本中的strlen(wd)>0也是一種拙劣的寫法,因為它等價于 *wd !='\0'。每次循環(huán)進行一次函數(shù)調(diào)用和每次循環(huán)只進行一次“!=”運算,效率顯然是天壤之別。
樣本中的NULL == pre在邏輯上是錯誤的,應該寫成NULL == file->list。
除此之外
?
是一種很笨拙的寫法。這種寫法把鏈表的head和結(jié)點的next成員區(qū)別開來,在這里必然要寫個if-else語句。更簡潔的寫法的基礎(chǔ)是把兩者視為同一種東西,這樣就可以統(tǒng)一對待了。造成這種笨拙的另一個原因是作者僵化固執(zhí)地把新結(jié)點加在鏈表結(jié)尾,實際上這樣做毫無必要。
綜上所述,這個函數(shù)可以更好地寫為
? 當然,代碼中還有其他毛病,但這些毛病與程序的總體思路的錯誤或其他函數(shù)相關(guān),在這里沒辦法進一步糾正。
總結(jié)
以上是生活随笔為你收集整理的垃圾“程序是怎样练成的”——关于《C程序设计伴侣》第A章(四)的全部內(nèi)容,希望文章能夠幫你解決所遇到的問題。
- 上一篇: 肱二头肌长头腱的作用
- 下一篇: 办公室网络打印机的连接