編輯本段出現頻率最高的筆試題strcpy寫法
阿新 • • 發佈:2019-01-31
1.strcpy的實現程式碼 char * strcpy(char * strDest,const char * strSrc) { char * strDestCopy=strDest; //[3] if ((strDest==NULL)||(strSrc==NULL)) //[1] throw "Invalid argument(s)"; //[2] while ((*strDest++=*strSrc++)!='\0'); //[4] return strDestCopy; }
錯誤的做法: [1] (A)不檢查指標的有效性,說明答題者不注重程式碼的健壯性。 (B)檢查指標的有效性時使用((!strDest)||(!strSrc))或(!(strDest&&strSrc)),說明答題者對C語言中型別的隱式轉換沒有深刻認識。在本例中char *轉換為bool即是型別隱式轉換,這種功能雖然靈活,但更多的是導致出錯概率增大和維護成本升高。所以C++專門增加了bool、true、false三個關鍵字以提供更安全的條件表示式。 (C)檢查指標的有效性時使用((strDest==0)||(strSrc==0)),說明答題者不知道使用常量的好處。直接使用字面常量(如本例中的0)會減少程式的可維護性。0雖然簡單,但程式中可能出現很多處對指標的檢查,萬一出現筆誤,編譯器不能發現,生成的程式內含邏輯錯誤,很難排除。而使用NULL代替0,如果出現拼寫錯誤,編譯器就會檢查出來。
[2] (A)return new string("Invalid argument(s)");,說明答題者根本不知道返回值的用途,並且他對記憶體洩漏也沒有警惕心。從函式中返回函式體內分配的記憶體是十分危險的做法,他把釋放記憶體的義務拋給不知情的呼叫者,絕大多數情況下,呼叫者不會釋放記憶體,這導致記憶體洩漏。 (B)return 0;,說明答題者沒有掌握異常機制。呼叫者有可能忘記檢查返回值,呼叫者還可能無法檢查返回值(見後面的鏈式表示式)。妄想讓返回值肩負返回正確值和異常值的雙重功能,其結果往往是兩種功能都失效。應該以丟擲異常來代替返回值,這樣可以減輕呼叫者的負擔、使錯誤不會被忽略、增強程式的可維護性。
[3] (A)忘記儲存原始的strDest值,說明答題者邏輯思維不嚴密。 [4] (A)迴圈寫成while (*strDest++=*strSrc++);,同[1](B)。 (B)迴圈寫成while (*strSrc!='\0') *strDest++=*strSrc++;,說明答題者對邊界條件的檢查不力。迴圈體結束後,strDest字串的末尾沒有正確地加上'\0'。 2.返回strDest的原始值使函式能夠支援鏈式表示式,增加了函式的“附加值”。同樣功能的函式,如果能合理地提高的可用性,自然就更加理想。
鏈式表示式的形式如: int iLength=strlen(strcpy(strA,strB)); 又如: char * strA=strcpy(new char[10],strB); 返回strSrc的原始值是錯誤的。其一,源字串肯定是已知的,返回它沒有意義。其二,不能支援形如第二例的表示式。其三,為了保護源字串,形參用const限定strSrc所指的內容,把const char *作為char *返回,型別不符,編譯報錯。 在上面的語句中,迴圈語句while ((*strDest++=*strSrc++)!='\0');較難理解,可以把這句理解為以下操作。 第一種: while(1) { char
temp; temp=*strDest=*strSrc; strDest++; strSrc++; if('\0'==temp) break; } 第二種: while(*strSrc!='\0') { *strDest=*strSrc; strDest++; strSrc++; } *strDest=*strSrc; 個人意見:throw用法明顯的執行不通,如果要判斷加#include<assert.h> 如果表示式的值為假,整個程式將退出,並輸出一條錯誤資訊。如果表示式的值為真則繼續執行後面的語句。
使用這個巨集前需要包含標頭檔案assert.h #include <assert.h> #include<iostream.h> char * strcpy(char * strDest ,const char *strSrc) { assert((strDest!=NULL) && (strSrc !=NULL)); char * strDestCopy=strDest; while ((*strDest++=*strSrc++)!='\0'); return strDestCopy; } void
main() { char a[20], c[]="i am teacher!"; strcpy(a,c); cout<<a<<endl; }