接口设计规则一：让你的接口会说话

在面试过程中，我时常会问一道简单的题目：
请找出如下程序不妥之处：
void s_c(const char *s, const char *p)
{
while (*s) {
*p = *s;
}
}
首先，这道简单的题目考察的是找错能力，比如结束符和const的应用。
其次，考察的是接口设计能力，这也是重点。
没有返回值，因此对于一些异常现象不好对外表现，const是否应用正确,
函数名使用是否见其名知其意，函数参数是否见其名知其意，有没有对接
口的简述，指针，引用，就量传递使用是否正确等等。
比如改成如下：
/**
* brief: 拷贝字符串，从源串到目的串
* param[in]: src 源串地址
* param[out]: dest 目的串地址
* return: 成功 0
* 失败 错误码
*/
int strcpy(const char *src, char *dest);
这样只接口就可以很清楚的了解它的功能和用法了。
当然，在代码实现过程中，还有许多其它需要考虑的点，
比如输入是否合法，出错处理，边界处理等等。
现修改如下：
/**
* brief: 拷贝字符串，从源串到目的串
* param[in]: src 源串地址
* param[out]: dest 目的串地址
* return: 成功 0
* 失败 错误码
*/
int strcpy(const char *src, char *dest)
{
if (NULL == src || NULL == dest) {
return -1;
}

while (*dest = *src);

return 0;
}
真正细心的你，觉得这样的程序正确吗？效率又如何？
如果有问题，如何改进？