Question

看网上很多人谭浩强c语言教材的这个程序有错误，我看不出错误在哪里，请高手告诉我

void copy_string(char *from, char * to)
{
for(；*from!='\0'；from++,to++)
*to=*from；
*to='\0'；
}
void main()
{
char *a="I am a teacher"；
char *b="You are a student."；
printf("\nstring a=%s\nstring b=%s\n",a,b)；
copy_string(a,b)；
printf("\nstring a=%s\nstring b=%s\n",a,b)；
}

Answer 1

错的一塌糊涂，烂书一本

void copy_string(char *from, char * to)
//第一句就错，不是一个良好的习惯，应该定义 void copy_string(const char *from, char * to, int nSize)，方式from字符串被误修改，防止 to字符串越界访问
{
for(；*from!='\0'；from++,to++)
*to=*from；
*to='\0'； //这句应该在循环外，程序逻辑错误，导致最终to是个空串
}
//main函数定义不符合C语言规范，应该定义 int main(void)
void main()
{
char *a="I am a teacher"；// 这句应该是 const char *a="I am a teacher"；
char *b="You are a student."； //b作为目标缓存，应该开成数组。这种写法b指向的内存应该分配在只读区里。在vc下用32位编译一下吧，对*b的赋值操作必然会导致死机
printf("\nstring a=%s\nstring b=%s\n",a,b)；
copy_string(a,b)；
printf("\nstring a=%s\nstring b=%s\n",a,b)；
}