字符串複製函數

題目:

已知 strcpy 的函數原型是:char * strcpy( * char strDest , const * char strSrc);

要求:

a、不調用函數庫來實現strcpy函數;

b、解釋爲什麼要返回char *

代碼實現:

        char * strcpy(char * strDest,const char * strSrc) 
        { 
             if ((strDest==NULL)||(strSrc==NULL)) 
                      throw "Invalid argument(s)";
             char * strDestCopy=strDest;   
             while ((*strDest++=*strSrc++)!='\0'); 
             return strDestCopy; 
        } 
從錯誤的做法的分析入手:

a、

首先,(A)不檢查指針的有效性,說明答題者不注重代碼的健壯性。 
            (B)檢查指針的有效性時使用((!strDest)||(!strSrc))或(!(strDest&&strSrc)),說明答題者對C語言中類型的隱式轉換沒有深刻認識。在本例中char *轉換爲bool即是類型隱式轉換,這種功能雖然靈活,但更多的是導致出錯概率增大和維護成本升高。所以C++專門增加了bool、true、false三個關鍵字以提供更安全的條件表達式。
          (C)檢查指針的有效性時使用((strDest==0)||(strSrc==0)),說明答題者不知道使用常量的好處。直接使用字面常量(如本例中的0)會減少程序的可維護性。0雖然簡單,但程序中可能出現很多處對指針的檢查,如果出現筆誤,編譯器不能發現,生成的程序內含邏輯錯誤,很難排除。而使用NULL代替0,如果出現拼寫錯誤,編譯器就會檢查出來。

其次,(A)return new string("Invalid argument(s)";,說明答題者根本不知道返回值的用途,並且他對內存泄漏也沒有警惕心。從函數中返回函數體內分配的內存是十分危險的做法,他把釋放內存的義務拋給不知情的調用者,絕大多數情況下,調用者不會釋放內存,這導致內存泄漏。

            (B)return 0;,說明答題者沒有掌握異常機制。調用者有可能忘記檢查返回值,調用者還可能無法檢查返回值。試圖讓返回值肩負返回正確值和異常值的雙重功能,其結果往往是兩種功能都失效。應該以拋出異常來代替返回值,這樣可以減輕調用者的負擔、使錯誤不會被忽略、增強程序的可維護性

最後, (A)忘記保存原始的strDest值,說明答題者邏輯思維不嚴密。 

            (B)循環寫成while (*strSrc!='\0') *strDest++=*strSrc++;,說明答題者對邊界條件的檢查不力。循環體結束後,strDest字符串的末尾沒有正確地加上'\0'。

b、

strcpy函數第二個參數是字符型常數組,是源字符串,爲已知的。所以char *  其前加上const 表示它是不能改變的值。

返回基類型是 char * ,因爲strDest的原始值使函數能夠支持鏈式表達式,增加了函數的“附加值”。同樣功能的函數,如果能合理地提高的可用性,自然就更加理想。例如:

int length=strlen(strcpy(strDest,"hello world"));

如果返回strSrc的原始值將是錯誤的。其一,源字符串肯定是已知的,返回它沒有意義。其二,爲了保護源字符串,形參用const限定strSrc所指的內容,把const char *作爲char *返回,類型不符,編譯報錯。



發表評論
所有評論
還沒有人評論,想成為第一個評論的人麼? 請在上方評論欄輸入並且點擊發布.
相關文章