練習(xí)一
以下函數(shù)完全沒有檢查可能的數(shù)據(jù)錯誤以及可能的執(zhí)行失敗。請指出此函數(shù)中所有可能發(fā)生錯誤的地方。本題并不考慮出現(xiàn)異常。
int *alloc_and_init(string file_name)
{
ifstream infile(file_name);
int elem_cnt;
infile >> elem_cnt;
int *pi = allocate_array(elem_cnt);
int elem;
int index = 0;
while(infile >> elem)
{
pi[index++] = elem;
}
sort_array(pi, elem_cnt);
register_data(pi);
return pi;
}
這是書中第203頁的練習(xí)7.1。
我自己的答案:打開文件后,未對infile進(jìn)行判斷,是否打開文件成功;pi是否分配成功,未進(jìn)行判斷,它是否為null。
侯捷老師給的答案如下:
第一個錯誤便是“型別不符”。ifstream constructor 接受的參數(shù)型別是const char*而非string。這個沒有注意到。解決方法是利用string的c_str member function取得其c-style字符串表現(xiàn)形式:
ifstream infile(file_name.c_str());
第二個錯誤是檢查infile是否成功開啟。
if ( !infile ) // 開啟失敗
第三個錯誤就是infile >> elem_cnt 可能執(zhí)行失敗。
如,文件內(nèi)含的是文字,那么企圖“讀入某個數(shù)值并置于elem_cnt內(nèi)”的操作便告失敗。此外,文件也有可能是空的。必須檢查讀取是否成功。
infile >> elem_cnt;
if (! infile) // 讀取失敗
第四個錯誤int *pi = allocate_array(elem_cnt);
無論何時,當(dāng)我們處理指針時,必須隨時注意指針是否的確指向?qū)嶋H存在的對象。如果allocate_array()無法配置足夠內(nèi)存,pi便會被設(shè)為0,我們必須檢驗如下:
if ( ! pi ) // allocate_array() 沒有配置到內(nèi)存
需要說明的是:程序的假設(shè)是(1)elem_cnt代表文件中的元素個數(shù);(2)數(shù)組索引值index絕不會發(fā)生溢出。但是除非我們檢查,否則實在無法保證index永遠(yuǎn)不大于elem_cnt。
第一個錯誤和第三個錯誤,沒有考慮到。分析看,對于“型別不符”這個問題,一直沒有注意到。此外,對于讀入文字,沒有思考那么多。
努力學(xué)習(xí)ing……