什么是好的程序員？是不是懂得很多技術細節？還是懂底層編程？還是編程速度比較快？我覺得都不是。對于一些技術細節來說和底層的技術，只要看幫助，查資料就能找到，對于速度快，只要編得多也就熟能生巧了。

我認為好的程序員應該有以下幾方面的素質：

1、有專研精神，勤學善問、舉一反三。

2、積極向上的態度，有創造性思維。

3、與人積極交流溝通的能力，有團隊精神。

4、謙虛謹慎，戒驕戒燥。

5、寫出的代碼質量高。包括：代碼的穩定、易讀、規范、易維護、專業。

這些都是程序員的修養，這里我想談談“編程修養”，也就是上述中的第5點。我覺得，如果我要了解一個作者，我會看他所寫的小說，如果我要了解一個畫家，我會看他所畫的圖畫，如果我要了解一個工人，我會看他所做出來的產品，同樣，如果我要了解一個程序員，我想首先我最想看的就是他的程序代碼，程序代碼可以看出一個程序員的素質和修養，程序就像一個作品，有素質有修養的程序員的作品必然是一圖精美的圖畫，一首美妙的歌曲，一本賞心悅目的小說。

我看過許多程序，沒有注釋，沒有縮進，胡亂命名的變量名，等等，等等，我把這種人統稱為沒有修養的程序，這種程序員，是在做創造性的工作嗎？不，完全就是在搞破壞，他們與其說是在編程，還不如說是在對源程序進行“加密”，這種程序員，見一個就應該開除一個，因為他編的程序所創造的價值，遠遠小于需要在上面進行維護的價值。

程序員應該有程序員的修養，那怕再累，再沒時間，也要對自己的程序負責。我寧可要那種動作慢，技術一般，但有良好的寫程序風格的程序員，也不要那種技術強、動作快的“搞破壞”的程序員。有句話叫“字如其人”，我想從程序上也能看出一個程序員的優劣。因為，程序是程序員的作品，作品的好壞直接關系到程序員的聲譽和素質。而“修養”好的程序員一定能做出好的程序和軟件。

有個成語叫“獨具匠心”，意思是做什么都要做得很專業，很用心，如果你要做一個“匠”，也就是造詣高深的人，那么，從一件很簡單的作品上就能看出你有沒有“匠”的特性，我覺得做一個程序員不難，但要做一個“程序匠”就不簡單了。編程序很簡單，但編出有質量的程序就難了。

我在這里不討論過深的技術，我只想在一些容易讓人忽略的東西上說一說，雖然這些東西可能很細微，但如果你不注意這些細微之處的話，那么他將會極大的影響你的整個軟件質量，以及整個軟件程的實施，所謂“千里之堤，毀于蟻穴”。

“細微之處見真功”，真正能體現一個程序的功底恰恰在這些細微之處。

這就是程序員的——編程修養。我總結了在用C/C++語言（主要是C語言）進行程序寫作上的三十二個“修養”，通過這些，你可以寫出質量高的程序，同時也會讓看你程序的人漬漬稱道，那些看過你程序的人一定會說：“這個人的編程修養不錯”。

1、版權和版本

好的程序員會給自己的每個函數，每個文件，都注上版權和版本。

對于C/C++的文件，文件頭應該有類似這樣的注釋：

/************************************************************************ * * 文件名：network.c * * 文件描述：網絡通訊函數集 * * 創建人：Hao Chen, 2003年2月3日 * * 版本號：1.0 * * 修改記錄： * ************************************************************************/ 而對于函數來說，應該也有類似于這樣的注釋： /*================================================================ * * 函 數 名：XXX * * 參 數： * * type name [IN] : descripts * * 功能描述: * * .............. * * 返 回 值：成功TRUE，失敗FALSE * * 拋出異常： * * 作 者：ChenHao 2003/4/2 * ================================================================*/

這樣的描述可以讓人對一個函數，一個文件有一個總體的認識，對代碼的易讀性和易維護性有很大的好處。這是好的作品產生的開始。

2、縮進、空格、換行、空行、對齊

i) 縮進應該是每個程序都會做的，只要學程序過程序就應該知道這個，但是我仍然看過不縮進的程序，或是亂縮進的程序，如果你的公司還有寫程序不縮進的程序員，請毫不猶豫的開除他吧，并以破壞源碼罪起訴他，還要他賠償讀過他程序的人的精神損失費。縮進，這是不成文規矩，我再重提一下吧，一個縮進一般是一個TAB鍵或是4個空格。（最好用TAB鍵）

ii) 空格。空格能給程序帶來什么損失嗎？沒有，有效的利用空格可以讓你的程序讀進來更加賞心悅目。而不一堆表達式擠在一起。看看下面的代碼：

ha=(ha*128+*key++)%tabPtr->size; ha = ( ha * 128 + *key++ ) % tabPtr->size;

有空格和沒有空格的感覺不一樣吧。一般來說，語句中要在各個操作符間加空格，函數調用時，要以各個參數間加空格。如下面這種加空格的和不加的：

if ((hProc=OpenProcess(PROCESS_ALL_ACCESS,FALSE,pid))==NULL){ } if ( ( hProc = OpenProcess(PROCESS_ALL_ACCESS, FALSE, pid) ) == NULL ){ }

iii) 換行。不要把語句都寫在一行上，這樣很不好。如：

for(i=0;i'9')&&(a[i]<'a'||a[i]>'z')) break;

我靠，這種即無空格，又無換行的程序在寫什么啊？加上空格和換行吧。

for ( i=0; i '9' ) && ( a[i] < 'a' || a[i] > 'z' ) ) { break; } }

好多了吧？有時候，函數參數多的時候，最好也換行，如：

CreateProcess( NULL, cmdbuf, NULL, NULL, bInhH, dwCrtFlags, envbuf, NULL, &siStartInfo, &prInfo );

條件語句也應該在必要時換行：

if ( ch >= '0' || ch <= '9' || ch >= 'a' || ch <= 'z' || ch >= 'A' || ch <= 'Z' )

iv) 空行。不要不加空行，空行可以區分不同的程序塊，程序塊間，最好加上空行。如：

HANDLE hProcess; PROCESS_T procInfo; /* open the process handle */ if((hProcess = OpenProcess(PROCESS_ALL_ACCESS, FALSE, pid)) == NULL) { return LSE_MISC_SYS; } memset(&procInfo, 0, sizeof(procInfo)); procInfo.idProc = pid; procInfo.hdProc = hProcess; procInfo.misc |= MSCAVA_PROC; return(0);

v) 對齊。用TAB鍵對齊你的一些變量的聲明或注釋，一樣會讓你的程序好看一些。如：

typedef struct _pt_man_t_ { int numProc; /* Number of processes */ int maxProc; /* Max Number of processes */ int numEvnt; /* Number of events */ int maxEvnt; /* Max Number of events */ HANDLE* pHndEvnt; /* Array of events */ DWORD timeout; /* Time out interval */ HANDLE hPipe; /* Namedpipe */ TCHAR usr[MAXUSR];/* User name of the process */ int numMsg; /* Number of Message */ int Msg[MAXMSG];/* Space for intro process communicate */ } PT_MAN_T;

怎么樣？感覺不錯吧。

這里主要講述了如果寫出讓人賞心悅目的代碼，好看的代碼會讓人的心情愉快，讀起代碼也就不累，工整、整潔的程序代碼，通常更讓人歡迎，也更讓人稱道。現在的硬盤空間這么大，不要讓你的代碼擠在一起，這樣它們會抱怨你虐待它們的。好了，用“縮進、空格、換行、空行、對齊”裝飾你的代碼吧，讓他們從沒有秩序的土匪中變成一排排整齊有秩序的正規部隊吧。

3、程序注釋

養成寫程序注釋的習慣，這是每個程序員所必須要做的工作。我看過那種幾千行，卻居然沒有一行注釋的程序。這就如同在公路上駕車卻沒有路標一樣。用不了多久，連自己都不知道自己的意圖了，還要花上幾倍的時間才看明白，這種浪費別人和自己的時間的人，是最為可恥的人。

是的，你也許會說，你會寫注釋，真的嗎？注釋的書寫也能看出一個程序員的功底。一般來說你需要至少寫這些地方的注釋：文件的注釋、函數的注釋、變量的注釋、算法的注釋、功能塊的程序注釋。主要就是記錄你這段程序是干什么的？你的意圖是什么？你這個變量是用來做什么的？等等。

不要以為注釋好寫，有一些算法是很難說或寫出來的，只能意會，我承認有這種情況的時候，但你也要寫出來，正好可以訓練一下自己的表達能力。而表達能力正是那種悶頭搞技術的技術人員最缺的，你有再高的技術，如果你表達能力不行，你的技術將不能得到充分的發揮。因為，這是一個團隊的時代。

好了，說幾個注釋的技術細節：

i) 對于行注釋（“//”）比塊注釋（“/* */”）要好的說法，我并不是很同意。因為一些老版本的C編譯器并不支持行注釋，所以為了你的程序的移植性，請你還是盡量使用塊注釋。

ii) 你也許會為塊注釋的不能嵌套而不爽，那么你可以用預編譯來完成這個功能。使用“#if 0”和“#endif”括起來的代碼，將不被編譯，而且還可以嵌套。

4、函數的[in][out]參數

我經常看到這樣的程序：

FuncName(char* str) { int len = strlen(str); ..... } char* GetUserName(struct user* pUser) { return pUser->name; }

不！請不要這樣做。

你應該先判斷一下傳進來的那個指針是不是為空。如果傳進來的指針為空的話，那么，你的一個大的系統就會因為這一個小的函數而崩潰。一種更好的技術是使用斷言（assert），這里我就不多說這些技術細節了。當然，如果是在C++中，引用要比指針好得多，但你也需要對各個參數進行檢查。

寫有參數的函數時，首要工作，就是要對傳進來的所有參數進行合法性檢查。而對于傳出的參數也應該進行檢查，這個動作當然應該在函數的外部，也就是說，調用完一個函數后，應該對其傳出的值進行檢查。

當然，檢查會浪費一點時間，但為了整個系統不至于出現“非法操作”或是“Core Dump”的系統級的錯誤，多花這點時間還是很值得的。

5、對系統調用的返回進行判斷

繼續上一條，對于一些系統調用，比如打開文件，我經常看到，許多程序員對fopen返回的指針不做任何判斷，就直接使用了。然后發現文件的內容怎么也讀出不，或是怎么也寫不進去。還是判斷一下吧：

fp = fopen("log.txt", "a"); if ( fp == NULL ){ printf("Error: open file error "); return FALSE; }

其它還有許多啦，比如：socket返回的socket號，malloc返回的內存。請對這些系統調用返回的東西進行判斷。

6、if 語句對出錯的處理

我看見你說了，這有什么好說的。還是先看一段程序代碼吧。

if ( ch >= '0' && ch <= '9' ){ /* 正常處理代碼 */ }else{ /* 輸出錯誤信息 */ printf("error ...... "); return ( FALSE ); }

這種結構很不好，特別是如果“正常處理代碼”很長時，對于這種情況，最好不要用else。先判斷錯誤，如：

if ( ch < '0' || ch > '9' ){ /* 輸出錯誤信息 */ printf("error ...... "); return ( FALSE ); } /* 正常處理代碼 */ ......

這樣的結構，不是很清楚嗎？突出了錯誤的條件，讓別人在使用你的函數的時候，第一眼就能看到不合法的條件，于是就會更下意識的避免。

7、頭文件中的#ifndef

千萬不要忽略了頭件的中的#ifndef，這是一個很關鍵的東西。比如你有兩個C文件，這兩個C文件都include了同一個頭文件。而編譯時，這兩個C文件要一同編譯成一個可運行文件，于是問題來了，大量的聲明沖突。

還是把頭文件的內容都放在#ifndef和#endif中吧。不管你的頭文件會不會被多個文件引用，你都要加上這個。一般格式是這樣的：

#ifndef <標識> #define <標識> ...... ...... #endif

<標識>在理論上來說可以是自由命名的，但每個頭文件的這個“標識”都應該是唯一的。標識的命名規則一般是頭文件名全大寫，前后加下劃線，并把文件名中的“.”也變成下劃線，如：stdio.h

#ifndef _STDIO_H_ #define _STDIO_H_ ...... #endif

（BTW：預編譯有多很有用的功能。你會用預編譯嗎？）

8、在堆上分配內存

可能許多人對內存分配上的“棧 stack”和“堆 heap”還不是很明白。包括一些科班出身的人也不明白這兩個概念。我不想過多的說這兩個東西。簡單的來講，stack上分配的內存系統自動釋放，heap上分配的內存，系統不釋放，哪怕程序退出，那一塊內存還是在那里。stack一般是靜態分配內存，heap上一般是動態分配內存。

由malloc系統函數分配的內存就是從堆上分配內存。從堆上分配的內存一定要自己釋放。用free釋放，不然就是術語——“內存泄露”（或是“內存漏洞”）—— Memory Leak。于是，系統的可分配內存會隨malloc越來越少，直到系統崩潰。還是來看看“棧內存”和“堆內存”的差別吧。

棧內存分配 —————

char *AllocStrFromStack() { char pstr[100]; return pstr; }

堆內存分配 —————

char *AllocStrFromHeap(int len) { char *pstr; if ( len <= 0 ) return NULL; return ( char* ) malloc( len ); }

對于第一個函數，那塊pstr的內存在函數返回時就被系統釋放了。于是所返回的char*什么也沒有。而對于第二個函數，是從堆上分配內存，所以哪怕是程序退出時，也不釋放，所以第二個函數的返回的內存沒有問題，可以被使用。但一定要調用free釋放，不然就是Memory Leak！

在堆上分配內存很容易造成內存泄漏，這是C/C++的最大的“克星”，如果你的程序要穩定，那么就不要出現Memory Leak。所以，我還是要在這里千叮嚀萬囑付，在使用malloc系統函數（包括calloc，realloc）時千萬要小心。

記得有一個UNIX上的服務應用程序，大約有幾百的C文件編譯而成，運行測試良好，等使用時，每隔三個月系統就是down一次，搞得許多人焦頭爛額，查不出問題所在。只好，每隔兩個月人工手動重啟系統一次。出現這種問題就是Memery Leak在做怪了，在C/C++中這種問題總是會發生，所以你一定要小心。一個Rational的檢測工作——Purify，可以幫你測試你的程序有沒有內存泄漏。

我保證，做過許多C/C++的工程的程序員，都會對malloc或是new有些感冒。當你什么時候在使用malloc和new時，有一種輕度的緊張和惶恐的感覺時，你就具備了這方面的修養了。

對于malloc和free的操作有以下規則：

配對使用，有一個malloc，就應該有一個free。（C++中對應為new和delete）

盡量在同一層上使用，不要像上面那種，malloc在函數中，而free在函數外。最好在同一調用層上使用這兩個函數。

malloc分配的內存一定要初始化。free后的指針一定要設置為NULL。

注：雖然現在的操作系統（如：UNIX和Win2k/NT）都有進程內存跟蹤機制，也就是如果你有沒有釋放的內存，操作系統會幫你釋放。但操作系統依然不會釋放你程序中所有產生了Memory Leak的內存，所以，最好還是你自己來做這個工作。（有的時候不知不覺就出現Memory Leak了，而且在幾百萬行的代碼中找無異于海底撈針，Rational有一個工具叫Purify，可能很好的幫你檢查程序中的Memory Leak）

9、變量的初始化

接上一條，變量一定要被初始化再使用。C/C++編譯器在這個方面不會像JAVA一樣幫你初始化，這一切都需要你自己來，如果你使用了沒有初始化的變量，結果未知。好的程序員從來都會在使用變量前初始化變量的。如：

對malloc分配的內存進行memset清零操作。（可以使用calloc分配一塊全零的內存）

對一些棧上分配的struct或數組進行初始化。（最好也是清零）

不過話又說回來了，初始化也會造成系統運行時間有一定的開銷，所以，也不要對所有的變量做初始化，這個也沒有意義。好的程序員知道哪些變量需要初始化，哪些則不需要。如：以下這種情況，則不需要。

char *pstr; /* 一個字符串 */ pstr = ( char* ) malloc( 50 ); if ( pstr == NULL ) exit(0); strcpy( pstr, "Hello Wrold" );

但如果是下面一種情況，最好進行內存初始化。（指針是一個危險的東西，一定要初始化）

char **pstr; /* 一個字符串數組 */ pstr = ( char** ) malloc( 50 ); if ( pstr == NULL ) exit(0); /* 讓數組中的指針都指向NULL */ memset( pstr, 0, 50*sizeof(char*) );

而對于全局變量，和靜態變量，一定要聲明時就初始化。因為你不知道它第一次會在哪里被使用。所以使用前初始這些變量是比較不現實的，一定要在聲明時就初始化它們。如：

Links *plnk = NULL; /* 對于全局變量plnk初始化為NULL */

10、h和c文件的使用

H文件和C文件怎么用呢？一般來說，H文件中是declare（聲明），C文件中是define（定義）。因為C文件要編譯成庫文件（Windows下是.obj/.lib，UNIX下是.o/.a），如果別人要使用你的函數，那么就要引用你的H文件，所以，H文件中一般是變量、宏定義、枚舉、結構和函數接口的聲明，就像一個接口說明文件一樣。而C文件則是實現細節。

H文件和C文件最大的用處就是聲明和實現分開。這個特性應該是公認的了，但我仍然看到有些人喜歡把函數寫在H文件中，這種習慣很不好。（如果是C++話，對于其模板函數，在VC中只有把實現和聲明都寫在一個文件中，因為VC不支持export關鍵字）。而且，如果在H文件中寫上函數的實現，你還得在makefile中把頭文件的依賴關系也加上去，這個就會讓你的makefile很不規范。

最后，有一個最需要注意的地方就是：帶初始化的全局變量不要放在H文件中！

例如有一個處理錯誤信息的結構：

char* errmsg[] = { /* 0 */ "No error", /* 1 */ "Open file error", /* 2 */ "Failed in sending/receiving a message", /* 3 */ "Bad arguments", /* 4 */ "Memeroy is not enough", /* 5 */ "Service is down; try later", /* 6 */ "Unknow information", /* 7 */ "A socket operation has failed", /* 8 */ "Permission denied", /* 9 */ "Bad configuration file format", /* 10 */ "Communication time out", ...... ...... };

請不要把這個東西放在頭文件中，因為如果你的這個頭文件被5個函數庫（.lib或是.a）所用到，于是他就被鏈接在這5個.lib或.a中，而如果你的一個程序用到了這5個函數庫中的函數，并且這些函數都用到了這個出錯信息數組。那么這份信息將有5個副本存在于你的執行文件中。如果你的這個errmsg很大的話，而且你用到的函數庫更多的話，你的執行文件也會變得很大。

正確的寫法應該把它寫到C文件中，然后在各個需要用到errmsg的C文件頭上加上 extern char* errmsg[]; 的外部聲明，讓編譯器在鏈接時才去管他，這樣一來，就只會有一個errmsg存在于執行文件中，而且，這樣做很利于封裝。

我曾遇到過的最瘋狂的事，就是在我的目標文件中，這個errmsg一共有112個副本，執行文件有8M左右。當我把errmsg放到C文件中，并為一千多個C文件加上了extern的聲明后，所有的函數庫文件尺寸都下降了20%左右，而我的執行文件只有5M了。一下子少了3M啊。

備注

有朋友對我說，這個只是一個特例，因為，如果errmsg在執行文件中存在多個副本時，可以加快程序運行速度，理由是errmsg的多個復本會讓系統的內存換頁降低，達到效率提升。像我們這里所說的errmsg只有一份，當某函數要用errmsg時，如果內存隔得比較遠，會產生換頁，反而效率不高。

這個說法不無道理，但是一般而言，對于一個比較大的系統，errmsg是比較大的，所以產生副本導致執行文件尺寸變大，不僅增加了系統裝載時間，也會讓一個程序在內存中占更多的頁面。而對于errmsg這樣數據，一般來說，在系統運行時不會經常用到，所以還是產生的內存換頁也就不算頻繁。權衡之下，還是只有一份errmsg的效率高。即便是像logmsg這樣頻繁使用的的數據，操作系統的內存調度算法會讓這樣的頻繁使用的頁面常駐于內存，所以也就不會出現內存換頁問題了。

11、出錯信息的處理

你會處理出錯信息嗎？哦，它并不是簡單的輸出。看下面的示例：

if ( p == NULL ){ printf ( "ERR: The pointer is NULL " ); }

告別學生時代的編程吧。這種編程很不利于維護和管理，出錯信息或是提示信息，應該統一處理，而不是像上面這樣，寫成一個“硬編碼”。第10條對這方面的處理做了一部分說明。如果要管理錯誤信息，那就要有以下的處理：

/* 聲明出錯代碼 */ #define ERR_NO_ERROR 0 /* No error */ #define ERR_OPEN_FILE 1 /* Open file error */ #define ERR_SEND_MESG 2 /* sending a message error */ #define ERR_BAD_ARGS 3 /* Bad arguments */ #define ERR_MEM_NONE 4 /* Memeroy is not enough */ #define ERR_SERV_DOWN 5 /* Service down try later */ #define ERR_UNKNOW_INFO 6 /* Unknow information */ #define ERR_SOCKET_ERR 7 /* Socket operation failed */ #define ERR_PERMISSION 8 /* Permission denied */ #define ERR_BAD_FORMAT 9 /* Bad configuration file */ #define ERR_TIME_OUT 10 /* Communication time out */ /* 聲明出錯信息 */ char* errmsg[] = { /* 0 */ "No error", /* 1 */ "Open file error", /* 2 */ "Failed in sending/receiving a message", /* 3 */ "Bad arguments", /* 4 */ "Memeroy is not enough", /* 5 */ "Service is down; try later", /* 6 */ "Unknow information", /* 7 */ "A socket operation has failed", /* 8 */ "Permission denied", /* 9 */ "Bad configuration file format", /* 10 */ "Communication time out", }; /* 聲明錯誤代碼全局變量 */ long errno = 0; /* 打印出錯信息函數 */ void perror( char* info) { if ( info ){ printf("%s: %s ", info, errmsg[errno] ); return; } printf("Error: %s ", errmsg[errno] ); }

這個基本上是ANSI的錯誤處理實現細節了，于是當你程序中有錯誤時你就可以這樣處理：

bool CheckPermission( char* userName ) { if ( strcpy(userName, "root") != 0 ){ errno = ERR_PERMISSION_DENIED; return (FALSE); } ... } main() { ... if (! CheckPermission( username ) ){ perror("main()"); } ... }

一個即有共性，也有個性的錯誤信息處理，這樣做有利同種錯誤出一樣的信息，統一用戶界面，而不會因為文件打開失敗，A程序員出一個信息，B程序員又出一個信息。而且這樣做，非常容易維護。代碼也易讀。

當然，物極必反，也沒有必要把所有的輸出都放到errmsg中，抽取比較重要的出錯信息或是提示信息是其關鍵，但即使這樣，這也包括了大多數的信息。

12、常用函數和循環語句中的被計算量

看一下下面這個例子：

for( i=0; i<1000; i++ ){ GetLocalHostName( hostname ); ... }

GetLocalHostName的意思是取得當前計算機名，在循環體中，它會被調用1000次啊。這是多么的沒有效率的事啊。應該把這個函數拿到循環體外，這樣只調用一次，效率得到了很大的提高。雖然，我們的編譯器會進行優化，會把循環體內的不變的東西拿到循環外面，但是，你相信所有編譯器會知道哪些是不變的嗎？我覺得編譯器不可靠。最好還是自己動手吧。

同樣，對于常用函數中的不變量，如：

GetLocalHostName(char* name) { char funcName[] = "GetLocalHostName"; sys_log( "%s begin......", funcName ); ... sys_log( "%s end......", funcName ); }

如果這是一個經常調用的函數，每次調用時都要對funcName進行分配內存，這個開銷很大啊。把這個變量聲明成static吧，當函數再次被調用時，就會省去了分配內存的開銷，執行效率也很好。

13、函數名和變量名的命名

我看到許多程序對變量名和函數名的取名很草率，特別是變量名，什么a,b,c,aa,bb,cc，還有什么flag1,flag2, cnt1, cnt2，這同樣是一種沒有“修養”的行為。即便加上好的注釋。好的變量名或是函數名，我認為應該有以下的規則：

直觀并且可以拼讀，可望文知意，不必“解碼”。

名字的長度應該即要最短的長度，也要能最大限度的表達其含義。

不要全部大寫，也不要全部小寫，應該大小寫都有，如：GetLocalHostName 或是 UserAccount。

可以簡寫，但簡寫得要讓人明白，如：ErrorCode -> ErrCode, ServerListener -> ServLisner，UserAccount -> UsrAcct 等。

為了避免全局函數和變量名字沖突，可以加上一些前綴，一般以模塊簡稱做為前綴。

全局變量統一加一個前綴或是后綴，讓人一看到這個變量就知道是全局的。

用匈牙利命名法命名函數參數，局部變量。但還是要堅持“望文生意”的原則。

與標準庫（如：STL）或開發庫（如：MFC）的命名風格保持一致。

14、函數的傳值和傳指針

向函數傳參數時，一般而言，傳入非const的指針時，就表示，在函數中要修改這個指針把指內存中的數據。如果是傳值，那么無論在函數內部怎么修改這個值，也影響不到傳過來的值，因為傳值是只內存拷貝。

什么？你說這個特性你明白了，好吧，讓我們看看下面的這個例程：

void GetVersion(char* pStr) { pStr = malloc(10); strcpy ( pStr, "2.0" ); } main() { char* ver = NULL; GetVersion ( ver ); ... ... free ( ver ); }

我保證，類似這樣的問題是一個新手最容易犯的錯誤。程序中妄圖通過函數GetVersion給指針ver分配空間，但這種方法根本沒有什么作用，原因就是——這是傳值，不是傳指針。你或許會和我爭論，我分明傳的時指針啊？再仔細看看，其實，你傳的是指針其實是在傳值。

15、修改別人程序的修養

當你維護別人的程序時，請不要非常主觀臆斷的把已有的程序刪除或是修改。我經常看到有的程序員直接在別人的程序上修改表達式或是語句。修改別人的程序時，請不要刪除別人的程序，如果你覺得別人的程序有所不妥，請注釋掉，然后添加自己的處理程序，必竟，你不可能100%的知道別人的意圖，所以為了可以恢復，請不依賴于CVS或是SourceSafe這種版本控制軟件，還是要在源碼上給別人看到你修改程序的意圖和步驟。這是程序維護時，一個有修養的程序員所應該做的。

如下所示，這就是一種比較好的修改方法：

/* * ----- commented by haoel 2003/04/12 ------ * * char* p = ( char* ) malloc( 10 ); * memset( p, 0, 10 ); */ /* ------ Added by haoel 2003/04/12 ----- */ char* p = ( char* )calloc( 10, sizeof char ); /* ---------------------------------------- */ ...

當然，這種方法是在軟件維護時使用的，這樣的方法，可以讓再維護的人很容易知道以前的代碼更改的動作和意圖，而且這也是對原作者的一種尊敬。

以“注釋 — 添加”方式修改別人的程序，要好于直接刪除別人的程序。

16、把相同或近乎相同的代碼形成函數和宏

有人說，最好的程序員，就是最喜歡“偷懶”的程序，其中不無道理。

如果你有一些程序的代碼片段很相似，或直接就是一樣的，請把他們放在一個函數中。而如果這段代碼不多，而且會被經常使用，你還想避免函數調用的開銷，那么就把他寫成宏吧。

千萬不要讓同一份代碼或是功能相似的代碼在多個地方存在，不然如果功能一變，你就要修改好幾處地方，這種會給維護帶來巨大的麻煩，所以，做到“一改百改”，還是要形成函數或是宏。

17、表達式中的括號

如果一個比較復雜的表達式中，你并不是很清楚各個操作符的憂先級，即使是你很清楚優先級，也請加上括號，不然，別人或是自己下一次讀程序時，一不小心就看走眼理解錯了，為了避免這種“誤解”，還有讓自己的程序更為清淅，還是加上括號吧。

比如，對一個結構的成員取地址：

GetUserAge( &( UserInfo->age ) );

雖然，&UserInfo->age中，->操作符的優先級最高，但加上一個括號，會讓人一眼就看明白你的代碼是什么意思。

再比如，一個很長的條件判斷：

if ( ( ch[0] >= '0' || ch[0] <= '9' ) && ( ch[1] >= 'a' || ch[1] <= 'z' ) && ( ch[2] >= 'A' || ch[2] <= 'Z' ) )

括號，再加上空格和換行，你的代碼是不是很容易讀懂了？

18、函數參數中的const

對于一些函數中的指針參數，如果在函數中只讀，請將其用const修飾，這樣，別人一讀到你的函數接口時，就會知道你的意圖是這個參數是[in]，如果沒有const時，參數表示[in/out]，注意函數接口中的const使用，利于程序的維護和避免犯一些錯誤。

雖然，const修飾的指針，如：const char* p，在C中一點用也沒有，因為不管你的聲明是不是const，指針的內容照樣能改，因為編譯器會強制轉換，但是加上這樣一個說明，有利于程序的閱讀和編譯。因為在C中，修改一個const指針所指向的內存時，會報一個Warning。這會引起程序員的注意。

C++中對const定義的就很嚴格了，所以C++中要多多的使用const，const的成員函數，const的變量，這樣會對讓你的代碼和你的程序更加完整和易讀。（關于C++的const我就不多說了）

19、函數的參數個數（多了請用結構）

函數的參數個數最好不要太多，一般來說6個左右就可以了，眾多的函數參數會讓讀代碼的人一眼看上去就很頭昏，而且也不利于維護。如果參數眾多，還請使用結構來傳遞參數。這樣做有利于數據的封裝和程序的簡潔性。

也利于使用函數的人，因為如果你的函數個數很多，比如12個，調用者很容易搞錯參數的順序和個數，而使用結構struct來傳遞參數，就可以不管參數的順序。

而且，函數很容易被修改，如果需要給函數增加參數，不需要更改函數接口，只需更改結構體和函數內部處理，而對于調用函數的程序來說，這個動作是透明的。

20、函數的返回類型，不要省略

我看到很多程序寫函數時，在函數的返回類型方面不太注意。如果一個函數沒有返回值，也請在函數前面加上void的修飾。而有的程序員偷懶，在返回int的函數則什么不修飾（因為如果不修飾，則默認返回int），這種習慣很不好，還是為了原代碼的易讀性，加上int吧。

所以函數的返回值類型，請不要省略。

另外，對于void的函數，我們往往會忘了return，由于某些C/C++的編譯器比較敏感，會報一些警告，所以即使是void的函數，我們在內部最好也要加上return的語句，這有助于代碼的編譯。

21、goto語句的使用

N年前，軟件開發的一代宗師——迪杰斯特拉(Dijkstra)說過：“goto statment is harmful !!”，并建議取消goto語句。因為goto語句不利于程序代碼的維護性。

這里我也強烈建議不要使用goto語句，除非下面的這種情況：

#define FREE(p) if(p) { free(p); p = NULL; } main() { char *fname=NULL, *lname=NULL, *mname=NULL; fname = ( char* ) calloc ( 20, sizeof(char) ); if ( fname == NULL ){ goto ErrHandle; } lname = ( char* ) calloc ( 20, sizeof(char) ); if ( lname == NULL ){ goto ErrHandle; } mname = ( char* ) calloc ( 20, sizeof(char) ); if ( mname == NULL ){ goto ErrHandle; } ...... ErrHandle: FREE(fname); FREE(lname); FREE(mname); ReportError(ERR_NO_MEMOEY); }

也只有在這種情況下，goto語句會讓你的程序更易讀，更容易維護。（在用嵌C來對數據庫設置游標操作時，或是對數據庫建立鏈接時，也會遇到這種結構）

22、宏的使用

很多程序員不知道C中的“宏”到底是什么意思？特別是當宏有參數的時候，經常把宏和函數混淆。我想在這里我還是先講講“宏”，宏只是一種定義，他定義了一個語句塊，當程序編譯時，編譯器首先要執行一個“替換”源程序的動作，把宏引用的地方替換成宏定義的語句塊，就像文本文件替換一樣。這個動作術語叫“宏的展開”

使用宏是比較“危險”的，因為你不知道宏展開后會是什么一個樣子。例如下面這個宏：

#define MAX(a, b) a>b?a:b

當我們這樣使用宏時，沒有什么問題：MAX( num1, num2 ); 因為宏展開后變成 num1>num2?num1:num2；。但是，如果是這樣調用的，MAX( 17+32, 25+21 ); 呢，編譯時出現錯誤，原因是，宏展開后變成：17+32>25+21?17+32:25+21，哇，這是什么啊？

所以，宏在使用時，參數一定要加上括號，上述的那個例子改成如下所示就能解決問題了。

#define MAX( (a), (b) ) (a)>(b)?(a):(b)

即使是這樣，也不這個宏也還是有Bug，因為如果我這樣調用 MAX(i++, j++); ， 經過這個宏以后，i和j都被累加了兩次，這絕不是我們想要的。

所以，在宏的使用上還是要謹慎考慮，因為宏展開是的結果是很難讓人預料的。而且雖然，宏的執行很快（因為沒有函數調用的開銷），但宏會讓源代碼澎漲，使目標文件尺寸變大，（如：一個50行的宏，程序中有1000個地方用到，宏展開后會很不得了），相反不能讓程序執行得更快（因為執行文件變大，運行時系統換頁頻繁）。

因此，在決定是用函數，還是用宏時得要小心。

23、static的使用

static關鍵字，表示了“靜態”，一般來說，他會被經常用于變量和函數。一個static的變量，其實就是全局變量，只不過他是有作用域的全局變量。比如一個函數中的static變量：

char* getConsumerName() { static int cnt = 0; .... cnt++; .... }

cnt變量的值會跟隨著函數的調用次而遞增，函數退出后，cnt的值還存在，只是cnt只能在函數中才能被訪問。而cnt的內存也只會在函數第一次被調用時才會被分配和初始化，以后每次進入函數，都不為static分配了，而直接使用上一次的值。

對于一些被經常調用的函數內的常量，最好也聲明成static（參見第12條）

但static的最多的用處卻不在這里，其最大的作用的控制訪問，在C中如果一個函數或是一個全局變量被聲明為static，那么，這個函數和這個全局變量，將只能在這個C文件中被訪問，如果別的C文件中調用這個C文件中的函數，或是使用其中的全局（用extern關鍵字），將會發生鏈接時錯誤。這個特性可以用于數據和程序保密。

24、函數中的代碼尺寸

一個函數完成一個具體的功能，一般來說，一個函數中的代碼最好不要超過600行左右，越少越好，最好的函數一般在100行以內，300行左右的孫函數就差不多了。有證據表明，一個函數中的代碼如果超過500行，就會有和別的函數相同或是相近的代碼，也就是說，就可以再寫另一個函數。

另外，函數一般是完成一個特定的功能，千萬忌諱在一個函數中做許多件不同的事。函數的功能越單一越好，一方面有利于函數的易讀性，另一方面更有利于代碼的維護和重用，功能越單一表示這個函數就越可能給更多的程序提供服務，也就是說共性就越多。

雖然函數的調用會有一定的開銷，但比起軟件后期維護來說，增加一些運行時的開銷而換來更好的可維護性和代碼重用性，是很值得的一件事。

25、typedef的使用

typedef是一個給類型起別名的關鍵字。不要小看了它，它對于你代碼的維護會有很好的作用。比如C中沒有bool，于是在一個軟件中，一些程序員使用int，一些程序員使用short，會比較混亂，最好就是用一個typedef來定義，如：

typedef char bool;

一般來說，一個C的工程中一定要做一些這方面的工作，因為你會涉及到跨平臺，不同的平臺會有不同的字長，所以利用預編譯和typedef可以讓你最有效的維護你的代碼，如下所示：

#ifdef SOLARIS2_5 typedef boolean_t BOOL_T; #else typedef int BOOL_T; #endif typedef short INT16_T; typedef unsigned short UINT16_T; typedef int INT32_T; typedef unsigned int UINT32_T; #ifdef WIN32 typedef _int64 INT64_T; #else typedef long long INT64_T; #endif typedef float FLOAT32_T; typedef char* STRING_T; typedef unsigned char BYTE_T; typedef time_t TIME_T; typedef INT32_T PID_T;

使用typedef的其它規范是，在結構和函數指針時，也最好用typedef，這也有利于程序的易讀和可維護性。如：

typedef struct _hostinfo { HOSTID_T host; INT32_T hostId; STRING_T hostType; STRING_T hostModel; FLOAT32_T cpuFactor; INT32_T numCPUs; INT32_T nDisks; INT32_T memory; INT32_T swap; } HostInfo; typedef INT32_T (*RsrcReqHandler)( void *info, JobArray *jobs, AllocInfo *allocInfo, AllocList *allocList);

C++中這樣也是很讓人易讀的：

typedef CArray HostInfoArray;

于是，當我們用其定義變量時，會顯得十分易讀。如：

HostInfo* phinfo; RsrcReqHandler* pRsrcHand;

這種方式的易讀性，在函數的參數中十分明顯。

關鍵是在程序種使用typedef后，幾乎所有的程序中的類型聲明都顯得那么簡潔和清淅，而且易于維護，這才是typedef的關鍵。

26、為常量聲明宏

最好不要在程序中出現數字式的“硬編碼”，如：

int user[120];

為這個120聲明一個宏吧。為所有出現在程序中的這樣的常量都聲明一個宏吧。比如TimeOut的時間，最大的用戶數量，還有其它，只要是常量就應該聲明成宏。如果，突然在程序中出現下面一段代碼，

for ( i=0; i<120; i++){ .... }

120是什么？為什么會是120？這種“硬編碼”不僅讓程序很讀，而且也讓程序很不好維護，如果要改變這個數字，得同時對所有程序中這個120都要做修改，這對修改程序的人來說是一個很大的痛苦。所以還是把常量聲明成宏，這樣，一改百改，而且也很利于程序閱讀。

#define MAX_USR_CNT 120 for ( i=0; i

這樣就很容易了解這段程序的意圖了。

有的程序員喜歡為這種變量聲明全局變量，其實，全局變量應該盡量的少用，全局變量不利于封裝，也不利于維護，而且對程序執行空間有一定的開銷，一不小心就造成系統換頁，造成程序執行速度效率等問題。所以聲明成宏，即可以免去全局變量的開銷，也會有速度上的優勢。

27、不要為宏定義加分號

有許多程序員不知道在宏定義時是否要加分號，有時，他們以為宏是一條語句，應該要加分號，這就錯了。當你知道了宏的原理，你會贊同我為會么不要為宏定義加分號的。看一個例子：

#define MAXNUM 1024;

這是一個有分號的宏，如果我們這樣使用：

half = MAXNUM/2; if ( num < MAXNUM )

等等，都會造成程序的編譯錯誤，因為，當宏展開后，他會是這個樣子的：

half = 1024;/2; if ( num < 1024; )

是的，分號也被展進去了，所以造成了程序的錯誤。請相信我，有時候，一個分號會讓你的程序出現成百個錯誤。所以還是不要為宏加最后一個分號，哪怕是這樣：

#define LINE "=================================" #define PRINT_LINE printf(LINE) #define PRINT_NLINE(n) while ( n-- >0 ) { PRINT_LINE; }

都不要在最后加上分號，當我們在程序中使用時，為之加上分號，

main() { char *p = LINE; PRINT_LINE; }

這一點非常符合習慣，而且，如果忘加了分號，編譯器給出的錯誤提示，也會讓我們很容易看懂的。

28、||和&&的語句執行順序

條件語句中的這兩個“與”和“或”操作符一定要小心，它們的表現可能和你想像的不一樣，這里條件語句中的有些行為需要和說一下：

express1 || express2

先執行表達式express1如果為“真”，express2將不被執行，express2僅在express1為“假”時才被執行。因為第一個表達式為真了，整個表達式都為真，所以沒有必要再去執行第二個表達式了。

express1 && express2

先執行表達式express1如果為“假”，express2將不被執行，express2僅在express1為“真”時才被執行。因為第一個表達式為假了，整個表達式都為假了，所以沒有必要再去執行第二個表達式了。

于是，他并不是你所想像的所有的表達式都會去執行，這點一定要明白，不然你的程序會出現一些莫明的運行時錯誤。

例如，下面的程序：

if ( sum > 100 && ( ( fp=fopen( filename,"a" ) ) != NULL ) { fprintf(fp, "Warring: it beyond one hundred "); ...... } fprintf( fp, " sum is %id ", sum ); fclose( fp );

本來的意圖是，如果sum > 100 ，向文件中寫一條出錯信息，為了方便，把兩個條件判斷寫在一起，于是，如果sum<=100時，打開文件的操作將不會做，最后，fprintf和fclose就會發現未知的結果。

再比如，如果我想判斷一個字符是不是有內容，我得判斷這個字符串指針是不為空（NULL）并且其內容不能為空（Empty），一個是空指針，一個是空內容。我也許會這樣寫：

if ( ( p != NULL ) && ( strlen(p) != 0 ))

于是，如果p為NULL，那么strlen(p)就不會被執行，于是，strlen也就不會因為一個空指針而“非法操作”或是一個“Core Dump”了。

記住一點，條件語句中，并非所有的語句都會執行，當你的條件語句非常多時，這點要尤其注意。

29、盡量用for而不是while做循環

基本上來說，for可以完成while的功能，我是建議盡量使用for語句，而不要使用while語句，特別是當循環體很大時，for的優點一下就體現出來了。

因為在for中，循環的初始、結束條件、循環的推進，都在一起，一眼看上去就知道這是一個什么樣的循環。剛出學校的程序一般對于鏈接喜歡這樣來：

p = pHead; while ( p ){ ... ... p = p->next; }

當while的語句塊變大后，你的程序將很難讀，用for就好得多：

for ( p=pHead; p; p=p->next ){ .. }

一眼就知道這個循環的開始條件，結束條件，和循環的推進。大約就能明白這個循環要做個什么事？而且，程序維護進來很容易，不必像while一樣，在一個編輯器中上上下下的搗騰。

30、請sizeof類型而不是變量

許多程序員在使用sizeof中，喜歡sizeof變量名，例如：

int score[100]; char filename[20]; struct UserInfo usr[100];

在sizeof這三個的變量名時，都會返回正確的結果，于是許多程序員就開始sizeof變量名。這個習慣很雖然沒有什么不好，但我還是建議sizeof類型。

我看到過這個的程序：

pScore = (int*) malloc( SUBJECT_CNT ); memset( pScore, 0, sizeof(pScore) ); ...

此時，sizeof(pScore)返回的就是4（指針的長度），不會是整個數組，于是，memset就不能對這塊內存進行初始化。為了程序的易讀和易維護，我強烈建議使用類型而不是變量，如：

對于score：sizeof(int) * 100 /* 100個int */ 對于filename：sizeof(char) * 20 /* 20個char */ 對于usr：sizeof(struct UserInfo) * 100 /* 100個UserInfo */

這樣的代碼是不是很易讀？一眼看上去就知道什么意思了。

另外一點，sizeof一般用于分配內存，這個特性特別在多維數組時，就能體現出其優點了。如，給一個字符串數組分配內存，

/* * 分配一個有20個字符串， * 每個字符串長100的內存 */ char* *p; /* * 錯誤的分配方法 */ p = (char**)calloc( 20*100, sizeof(char) ); /* * 正確的分配方法 */ p = (char**) calloc ( 20, sizeof(char*) ); for ( i=0; i<20; i++){ /*p = (char*) calloc ( 100, sizeof(char) );*/ p[i] = (char*) calloc ( 100, sizeof(char) ); }

（注：上述語句被注釋掉的是原來的，是錯誤的，由dasherest朋友指正，謝謝）

為了代碼的易讀，省去了一些判斷，請注意這兩種分配的方法，有本質上的差別。

31、不要忽略Warning

對于一些編譯時的警告信息，請不要忽視它們。雖然，這些Warning不會妨礙目標代碼的生成，但這并不意味著你的程序就是好的。必竟，并不是編譯成功的程序才是正確的，編譯成功只是萬里長征的第一步，后面還有大風大浪在等著你。從編譯程序開始，不但要改正每個error，還要修正每個warning。這是一個有修養的程序員該做的事。

一般來說，一面的一些警告信息是常見的：

1）聲明了未使用的變量。（雖然編譯器不會編譯這種變量，但還是把它從源程序中注釋或是刪除吧）

2）使用了隱晦聲明的函數。（也許這個函數在別的C文件中，編譯時會出現這種警告，你應該這使用之前使用extern關鍵字聲明這個函數）

3）沒有轉換一個指針。（例如malloc返回的指針是void的，你沒有把之轉成你實際類型而報警，還是手動的在之前明顯的轉換一下吧）

4）類型向下轉換。（例如：float f = 2.0; 這種語句是會報警告的，編譯會告訴你正試圖把一個double轉成float，你正在閹割一個變量，你真的要這樣做嗎？還是在2.0后面加個f吧，不然，2.0就是一個double，而不是float了）

不管怎么說，編譯器的Warning不要小視，最好不要忽略，一個程序都做得出來，何況幾個小小的Warning呢？

32、書寫Debug版和Release版的程序

程序在開發過程中必然有許多程序員加的調試信息。我見過許多項目組，當程序開發結束時，發動群眾刪除程序中的調試信息，何必呢？為什么不像VC++那樣建立兩個版本的目標代碼？一個是debug版本的，一個是Release版的。那些調試信息是那么的寶貴，在日后的維護過程中也是很寶貴的東西，怎么能說刪除就刪除呢？

利用預編譯技術吧，如下所示聲明調試函數：

#ifdef DEBUG void TRACE(char* fmt, ...) { ...... } #else #define TRACE(char* fmt, ...) #endif

于是，讓所有的程序都用TRACE輸出調試信息，只需要在在編譯時加上一個參數“-DDEBUG”，如：

cc -DDEBUG -o target target.c

于是，預編譯器發現DEBUG變量被定義了，就會使用TRACE函數。而如果要發布給用戶了，那么只需要把取消“-DDEBUG”的參數，于是所有用到TRACE宏，這個宏什么都沒有，所以源程序中的所有TRACE語言全部被替換成了空。一舉兩得，一箭雙雕，何樂而不為呢？

順便提一下，兩個很有用的系統宏，一個是“FILE”，一個是“LINE”，分別表示，所在的源文件和行號，當你調試信息或是輸出錯誤時，可以使用這兩個宏，讓你一眼就能看出你的錯誤，出現在哪個文件的第幾行中。這對于用C/C++做的大工程非常的管用。

綜上所述32條，都是為了三大目的——

1、程序代碼的易讀性。

2、程序代碼的可維護性，

3、程序代碼的穩定可靠性。

有修養的程序員，就應該要學會寫出這樣的代碼！這是任何一個想做編程高手所必需面對的細小的問題，編程高手不僅技術要強，基礎要好，而且最重要的是要有“修養”！

好的軟件產品絕不僅僅是技術，而更多的是整個軟件的易維護和可靠性。

軟件的維護有大量的工作量花在代碼的維護上，軟件的Upgrade，也有大量的工作花在代碼的組織上，所以好的代碼，清淅的，易讀的代碼，將給大大減少軟件的維護和升級成本。

原文標題：長文：嵌入式程序員的編程修養

文章出處：【微信公眾號：玩轉單片機】歡迎添加關注！文章轉載請注明出處。