1. 程式人生 > >編輯本段出現頻率最高的筆試題strcpy寫法

編輯本段出現頻率最高的筆試題strcpy寫法

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;  }