不呼叫庫函式，實現strcpy
阿新 • • 發佈：2019-02-08
錯誤的做法：
[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 *返回，型別不符，編譯報錯。