-----------
我經(jīng)?吹竭@樣的程序:
FuncName(char* str)
{
int len = strlen(str);
.....
}
char*
GetUserName(struct user* pUser)
{
return pUser->name;
}
不!請不要這樣做。
你應該先判斷一下傳進來的那個指針是不是為空。如果傳進來的指針為空的話,那么,你的一個大的系統(tǒng)就會因為這一個小的函數(shù)而崩潰。一種更好的技術(shù)是使用斷言(assert),這里我就不多說這些技術(shù)細節(jié)了。當然,如果是在C++中,引用要比指針好得多,但你也需要對各個參數(shù)進行檢查。
寫有參數(shù)的函數(shù)時,首要工作,就是要對傳進來的所有參數(shù)進行合法性檢查。而對于傳出的參數(shù)也應該進行檢查,這個動作當然應該在函數(shù)的外部,也就是說,調(diào)用完一個函數(shù)后,應該對其傳出的值進行檢查。
當然,檢查會浪費一點時間,但為了整個系統(tǒng)不至于出現(xiàn)"非法操作"或是"Core Dump"的系統(tǒng)級的錯誤,多花這點時間還是很值得的。
5、對系統(tǒng)調(diào)用的返回進行判斷
--------------
繼續(xù)上一條,對于一些系統(tǒng)調(diào)用,比如打開文件,我經(jīng)?吹,許多程序員對fopen返回的指針不做任何判斷,就直接使用了。然后發(fā)現(xiàn)文件的內(nèi)容怎么也讀出不,或是怎么也寫不進去。還是判斷一下吧:
fp = fopen("log.txt", "a");
if ( fp == NULL ){
printf("Error: open file error\n");
return FALSE;
}
其它還有許多啦,比如:socket返回的socket號,malloc返回的內(nèi)存。請對這些系統(tǒng)調(diào)用返回的東西進行判斷。
6、if 語句對出錯的處理
-----------
我看見你說了,這有什么好說的。還是先看一段程序代碼吧。
if ( ch >= '0' && ch <= '9' ){
/* 正常處理代碼 */
}else{
/* 輸出錯誤信息 */
printf("error ......\n");
return ( FALSE );
}
這種結(jié)構(gòu)很不好,特別是如果"正常處理代碼"很長時,對于這種情況,最好不要用else。先判斷錯誤,如:
if ( ch < '0' ch > '9' ){
/* 輸出錯誤信息 */
printf("error ......\n");
return ( FALSE );
}
/* 正常處理代碼 */
......
這樣的結(jié)構(gòu),不是很清楚嗎?突出了錯誤的條件,讓別人在使用你的函數(shù)的時候,第一眼就能看到不合法的條件,于是就會更下意識的避免。
7、頭文件中的#ifndef
----------
千萬不要忽略了頭件的中的#ifndef,這是一個很關(guān)鍵的東西。比如你有兩個C文件,這兩個C文件都include了同一個頭文件。而編譯時,這兩個C文件要一同編譯成一個可運行文件,于是問題來了,大量的聲明沖突。
還是把頭文件的內(nèi)容都放在#ifndef和#endif中吧。不管你的頭文件會不會被多個文件引用,你都要加上這個。一般格式是這樣的:
#ifndef <標識>
#define <標識>
......
......
#endif
<標識>在理論上來說可以是自由命名的,但每個頭文件的這個"標識"都應該是唯一的。標識的命名規(guī)則一般是頭文件名全大寫,前后加下劃線,并把文件名中的"."也變成下劃線,如:stdio.h
#ifndef _STDIO_H_
#define _STDIO_H_
......
#endif
(BTW:預編譯有多很有用的功能。你會用預編譯嗎?)
8、在堆上分配內(nèi)存
---------
可能許多人對內(nèi)存分配上的"棧 stack"和"堆 heap"還不是很明白。包括一些科班出身的人也不明白這兩個概念。我不想過多的說這兩個東西。簡單的來講,stack上分配的內(nèi)存系統(tǒng)自動釋放, heap上分配的內(nèi)存,系統(tǒng)不釋放,哪怕程序退出,那一塊內(nèi)存還是在那里。stack一般是靜態(tài)分配內(nèi)存,heap上一般是動態(tài)分配內(nèi)存。
由malloc系統(tǒng)函數(shù)分配的內(nèi)存就是從堆上分配內(nèi)存。從堆上分配的內(nèi)存一定要自己釋放。用free釋放,不然就是術(shù)語--"內(nèi)存泄露"(或是"內(nèi)存漏洞 ")-- Memory Leak。于是,系統(tǒng)的可分配內(nèi)存會隨malloc越來越少,直到系統(tǒng)崩潰。還是來看看"棧內(nèi)存"和"堆內(nèi)存"的差別吧。
棧內(nèi)存分配
-----
char*
AllocStrFromStack()
{
char pstr[100];
return pstr;
}
堆內(nèi)存分配
-----
char*
AllocStrFromHeap(int len)
{
char *pstr;
if ( len <= 0 ) return NULL;
return ( char* ) malloc( len );
}
對于第一個函數(shù),那塊pstr的內(nèi)存在函數(shù)返回時就被系統(tǒng)釋放了。于是所返回的char*什么也沒有。而對于第二個函數(shù),是從堆上分配內(nèi)存,所以哪怕是程序退出時,也不釋放,所以第二個函數(shù)的返回的內(nèi)存沒有問題,可以被使用。但一定要調(diào)用free釋放,不然就是Memory Leak!
在堆上分配內(nèi)存很容易造成內(nèi)存泄漏,這是C/C++的最大的"克星",如果你的程序要穩(wěn)定,那么就不要出現(xiàn)Memory Leak。所以,我還是要在這里千叮嚀萬囑付,在使用malloc系統(tǒng)函數(shù)(包括calloc,realloc)時千萬要小心。
記得有一個UNIX上的服務(wù)應用程序,大約有幾百的C文件編譯而成,運行測試良好,等使用時,每隔三個月系統(tǒng)就是down一次,搞得許多人焦頭爛額,查不出問題所在。只好,每隔兩個月人工手動重啟系統(tǒng)一次。出現(xiàn)這種問題就是Memery Leak在做怪了,在C/C++中這種問題總是會發(fā)生,所以你一定要小心。一個Rational的檢測工作--Purify,可以幫你測試你的程序有沒有內(nèi)存泄漏。





