程序員:請不要輕視我們的過去
這是當年開發的一個煤炭安全監控軟件里增加新測點的函數。如果在當年看這段代碼,感覺這應該是一段挺不錯的代碼,比如:注釋很多,幾乎每行都有注釋;匈牙利命名法,使用像lp、w這樣的前綴;結構、數組、宏定義,看起來都是有板有眼。
- //-----------------------------------------------------------------------------
- // 函數:NewPoint
- // 功能:分配一個測點定義緩沖區。
- // 入口參數:
- // lpPNum 測點號指針。
- // 出口參數:
- // 如成功,返回在測點定義表中的偏移;如失敗,返回0xffff。
- //-----------------------------------------------------------------------------
- WORD NewPoint(LPCSTR lpPNum)
- {
- WORD i;
- LPTPOINT lpTmpPt;
- // 在測點定義表中找空的緩沖區
- for (i=0;i<PtTab.wCount;i++) if (PtTab.lpPt[i].tpDel) break;
- if (i==PtTab.wCount) {
- // 如未找到,則判斷測點數是否已到***值,如已到,則返回失敗
- if (PtTab.wCount+1>MAXPOINT) return(0xffff);
- // 如未找到***值,則測點總數加1。即在測點定義表的***追加一個緩沖區
- PtTab.wCount++;
- }
- // 初始化緩沖區
- lpTmpPt=&PtTab.lpPt[i];
- // 將刪除標志(即緩沖區空標志)置1,作為后面具體定義時區分增加與修改的標志,
- // 如是修改,則此標志必為0
- lpTmpPt->tpDel=1;
- // 拷貝測點號到緩沖區
- lstrcpy(lpTmpPt->tpNum,lpPNum);
- // 將前一此操作測點的定義復制到緩沖區
- lstrcpy(lpTmpPt->tpName,HistPt.tpName);
- lpTmpPt->tpType=HistPt.tpType;
- lpTmpPt->tpOp=HistPt.tpOp;
- // 賦缺省的狀態與數值
- lpTmpPt->tpState=PS_SUSP;
- lpTmpPt->tpRValue=0;
- lpTmpPt->tpDValue=0;
- // 置缺省的狀態變化時間為當前時間
- _fmemcpy(&lpTmpPt->tpTime,&Time,sizeof(SYSTIME));
- // 將偏移值返回
- return(i);
- }
時間已經過去十幾年了,計算機系統、開發語言、設計思想都有了很大轉變,現在要寫一段同樣功能的代碼,可能完全不同了。
***的變化應該是面向對象技術的應用,測點和測點表都會從struct變為class,測點數組可能會用std::vector或std::list代替,并且用new來動態分配,宏定義也會換成常量或枚舉。而上面這個創建新測點的函數,也會被封裝到測點表這個類中。面向對象技術改變了我們思考和實踐的方式,而且這個改變作用是巨大的。
再有就是一些設計思想的轉變,比如對注釋的看法。以前,注釋多可能意味著可讀性強。但《重構》一書中說:“你看到一段代碼有著長長的注釋,然后發現,這些注釋之所以存在乃是因為代碼很糟糕。”再看看上面這段代碼,前半段,其實是查找一個空的測點位置,完全可以用Extract Method(提煉函數)重構方法將其移入另外一個函數,并給其一個準確的命名,而不需要注釋。后半段,“代碼已經清楚說明了一切,注釋已經變得多余了。”
匈牙利命名法,已成為爭議***的命名法,其類型冗余常常大于它所帶來的收益。它來自微軟的一名匈牙利程序員,但現在微軟的.Net和它的編程語言中,微軟更換了這一法則,在C#中以駱駝命名法和帕斯卡命名法居多。
計算機系統的提升也改變著程序。早年16位OS上編程需要區分遠指針和近指針,但現在32位環境下,C++指針變量前綴一般都是p,很少見到lp。
技術在進步,編程思想在轉變,人的思維也在不斷更新。也許再過十幾年,現在聽都沒有聽說過的新技術會大行其道,你回頭再看看現在的代碼,可能也會有像我今天的感覺。但無論怎樣,請不要輕視我們的過去,因為沒有過去的積累,就沒有今天的進步。
原文鏈接:http://www.cnblogs.com/wanghui9072229/archive/2011/04/25/2028793.html
【編輯推薦】