面试题解析之代码的健壮性

这道题目跟我上次在百度面试的一道题目基本相似，除了考查面试者数据结构、编码能力、编程风格等，特别考查面试者编写的代码的健壮性，也就是程序中的各种异常情况都必须考虑到。今天刚好看到这个例子，感觉很有代表性，也很能给人启发，整理了下思路果断拿出来分享。

1.Subject:写一个实现字符串拷贝的函数。给定字符串拷贝函数strcpy的原型：

char *strcpy(char *dest,const char *src);

要求：

(1)不调用任何库函数。

(2)说明函数为什么返回char *。

2.Solution:

char *strcpy(char *dest,char *src)

{

if ( (dest==NULL) || (src==NULL) )

{

printf("arg wrong");

return NULL;

}

char *ret_string = dest;

while((*dest++ = *src++) != '\0');

return ret_string;

}

3.常见错误解析：

(1)不检查指针的有效性，函数应该首先判断指针dest和src是否为NULL。如果缺少检查输入参数的代码，说明答题者不注重代码的健壮性。

(2)检查指针是否有效时，使用if((！dest)||(！src))if((dest==0)||(src==0))。说明答题者书写代码不规范。dest和src是指针变量，而！针对的是逻辑变量。使用0而没有使NULL，会降低程序的可维护性,说明答题者不知道使用NULL常量的好处。判断指针是否为空指针时，推荐使用NULL而不是0。

(3)没有返回dest指针，说明答题者不知道为何该函数的返回值类型是char *。返回dest指针的原因是为了实现链式表达式，如：

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

strlen求得是拷贝后的dest所指向的字符串的长度。如果不返回指针dest，那么就要写两个语句，而现在这样只在一个语句里就可以完成。

(4)字符串拷贝是，忘记拷贝最后的'\0'。上面的while语句是最精简的写法。

当然为了提高程序的可读性，最好还是尽量容易让别人看懂。如：

while(*src != '\0')

{

*dest = *src;

dest++;

src++;

}

*dest = '\0';

4.完整实现代码：

#include <stdio.h>

#include <stdlib.h>

char *strcpy(char *dest,char *src)

{

if ( (dest==NULL) || (src==NULL) )

{

printf("arg wrong");

return NULL;

}

char *ret_string = dest;

while(*src != '\0')

{

*dest = *src;

dest++;

src++;

}

*dest = '\0'; //注释掉这行也不会报错，但是注意程序的健壮性。

return ret_string;

}

int main()

{

char str[] = "hello";

int n = sizeof(str); //包括了最后一个字符'\0'

char *p = (char *)malloc(n);

strcpy(p,str);

printf("p:%s\n",p);

return 0;

}

Output:

p：hello