[已解决]自制strcpy异常

风眠

效果

1. gcc my_strcpy.c -o test&&./test
   
2. 请输入一个字符串到str1中：I love FishC.com!
   
3. 开始拷贝str1的内容到str2中...
   
4. 拷贝完毕！
   
5. 现在,str2中的内容是: är_ÿ

复制代码

代码

1. #include <stdio.h>
   
2. 
   
3. #define MAX 1024
   
4. 
   
5. int main() {
   
6.   char str1[MAX];
   
7.   char str2[MAX];
   
8. 
   
9.   char *p1 = str1;
   
10.   char *p2 = str2;
    
11. 
    
12.   printf("请输入一个字符串到str1中：");
    
13.   fgets(str1, MAX, stdin);
    
14. 
    
15.   printf("开始拷贝str1的内容到str2中...\n");
    
16.   while ((*p1++ = *p2++) != '\0')
    
17.     ;
    
18. 
    
19.   printf("拷贝完毕！\n现在,str2中的内容是:%s\n", str2);
    
20. 
    
21.   return 0;
    
22. }
    

复制代码

最佳答案

月排行榜 / 总排行榜

陶远航

2023-10-4 09:33:53

您的代码中存在一个错误，导致 strcpy 函数无法正常工作。在拷贝字符串时，您将 p1 和 p2 的赋值顺序搞反了。正确的实现应该是 *p2++ = *p1++。以下是修复后的代码：

1. #include <stdio.h>
   
2. #define MAX 1024
   
3. 
   
4. int main() {
   
5.     char str1[MAX];
   
6.     char str2[MAX];
   
7. 
   
8.     char *p1 = str1;
   
9.     char *p2 = str2;
   
10. 
    
11.     printf("请输入一个字符串到str1中：");
    
12.     fgets(str1, MAX, stdin);
    
13. 
    
14.     printf("开始拷贝str1的内容到str2中...\
    
15. ");
    
16.     while ((*p2++ = *p1++) != '\\0')
    
17.         ;
    
18. 
    
19.     printf("拷贝完毕！\
    
20. 现在,str2中的内容是:%s\
    
21. ", str2);
    
22. 
    
23.     return 0;
    
24. }
    

复制代码

现在，当您使用 gcc my_strcpy.c -o test && ./test 编译并运行此代码时，您应该会看到预期的输出，即将 str1 中的字符串成功拷贝到 str2 中。
如果问题已经解决，请设置最佳答案

跳转到最佳答案楼层

isdkz

这段代码存在两个问题：

1.  p2  指针未初始化，指向的是  str2  数组的首地址，而  str2  数组中没有任何有效数据，因此在执行  while  循环时会访问到未初始化的内存，导致出现异常。

2.  while  循环的终止条件是  *p1++ = *p2++ != '\0' ，这里使用了赋值运算符和比较运算符，但是它们的优先级是不同的，应该使用括号来明确优先级，正确的写法应该是  (*p1++ = *p2++) != '\0' 。

修改后的代码如下：

1. #include <stdio.h>
   
2. 
   
3. #define MAX 1024
   
4. 
   
5. int main() {
   
6.     char str1[MAX];
   
7.     char str2[MAX] = {0}; // 初始化str2数组
   
8.     char *p1 = str1;
   
9.     char *p2 = str2;
   
10. 
    
11.     printf("请输入一个字符串到str1中：");
    
12.     fgets(str1, MAX, stdin);
    
13. 
    
14.     printf("开始拷贝str1的内容到str2中...\n");
    
15.     while ((*p2++ = *p1++) != '\0')
    
16.         ;
    
17. 
    
18.     printf("拷贝完毕！\n现在,str2中的内容是:%s\n", str2);
    
19. 
    
20.     return 0;
    
21. }
    

复制代码

陶远航

您的代码中存在一个错误，导致 strcpy 函数无法正常工作。在拷贝字符串时，您将 p1 和 p2 的赋值顺序搞反了。正确的实现应该是 *p2++ = *p1++。以下是修复后的代码：

1. #include <stdio.h>
   
2. #define MAX 1024
   
3. 
   
4. int main() {
   
5.     char str1[MAX];
   
6.     char str2[MAX];
   
7. 
   
8.     char *p1 = str1;
   
9.     char *p2 = str2;
   
10. 
    
11.     printf("请输入一个字符串到str1中：");
    
12.     fgets(str1, MAX, stdin);
    
13. 
    
14.     printf("开始拷贝str1的内容到str2中...\
    
15. ");
    
16.     while ((*p2++ = *p1++) != '\\0')
    
17.         ;
    
18. 
    
19.     printf("拷贝完毕！\
    
20. 现在,str2中的内容是:%s\
    
21. ", str2);
    
22. 
    
23.     return 0;
    
24. }
    

复制代码

现在，当您使用 gcc my_strcpy.c -o test && ./test 编译并运行此代码时，您应该会看到预期的输出，即将 str1 中的字符串成功拷贝到 str2 中。
如果问题已经解决，请设置最佳答案

Mike_python小

问题描述：

用户提供了一个C语言程序，该程序的目的是将一个字符串从str1复制到str2。但是，程序的输出似乎是错误的。用户希望我们找出问题所在，并提供修复后的代码。

问题分析：

在提供的代码中，我们发现了一个关键错误。在while循环中，str1和str2的内容被错误地复制。根据代码的目的，我们应该将str1的内容复制到str2，而不是相反。

修复后的代码：

2. #include <stdio.h>
3. #define MAX 1024
5. int main() {
6.     char str1[MAX];
7.     char str2[MAX];
9.     char *p1 = str1;
10.     char *p2 = str2;
12.     printf("请输入一个字符串到str1中：");
13.     fgets(str1, MAX, stdin);
15.     printf("开始拷贝str1的内容到str2中...\
16. ");
17.     while ((*p2++ = *p1++) != '\\0')
18.         ;
20.     printf("拷贝完毕！\
21. 现在,str2中的内容是:%s\
22. ", str2);
24.     return 0;
25. }

复制代码

修复说明：

我们将while循环中的赋值操作更正为*p2++ = *p1++，这样就可以正确地将str1的内容复制到str2了。现在，程序应该能够按预期工作，将输入的字符串从str1复制到str2。

球一个最佳答案谢谢啦！这对我非常重要！
本段回答来自Chat GPT-4，如有错误，多多包涵！

风眠

isdkz 发表于 2023-10-4 09:33
这段代码存在两个问题：

1.  p2  指针未初始化，指向的是  str2  数组的首地址，而  str2  数组中没有任 ...

我用了括号啊

风眠

陶远航 发表于 2023-10-4 09:33
您的代码中存在一个错误，导致 strcpy 函数无法正常工作。在拷贝字符串时，您将 p1 和 p2 的赋值顺序搞反了 ...

我怎么这么粗心

风眠

陶远航 发表于 2023-10-4 09:33
您的代码中存在一个错误，导致 strcpy 函数无法正常工作。在拷贝字符串时，您将 p1 和 p2 的赋值顺序搞反了 ...

对了，2楼是不是ChatGpt3?有点哪啥......

陶远航

风眠 发表于 2023-10-4 09:36
对了，2楼是不是ChatGpt3?有点哪啥......

gpt4通常聪明一点

风眠

陶远航 发表于 2023-10-4 09:37
gpt4通常聪明一点

好的，现在我信你是ChatGPT4了......

六翻了

陶远航 发表于 2023-10-4 09:33
您的代码中存在一个错误，导致 strcpy 函数无法正常工作。在拷贝字符串时，您将 p1 和 p2 的赋值顺序搞反了 ...

膜拜大佬