鱼C论坛

 找回密码
 立即注册
查看: 1260|回复: 9

[已解决]自制strcpy异常

[复制链接]
发表于 2023-10-4 09:33:18 | 显示全部楼层 |阅读模式

马上注册,结交更多好友,享用更多功能^_^

您需要 登录 才可以下载或查看,没有账号?立即注册

x
效果
gcc my_strcpy.c -o test&&./test
请输入一个字符串到str1中:I love FishC.com!
开始拷贝str1的内容到str2中...
拷贝完毕!
现在,str2中的内容是: är_ÿ
代码
#include <stdio.h>

#define MAX 1024

int main() {
  char str1[MAX];
  char str2[MAX];

  char *p1 = str1;
  char *p2 = str2;

  printf("请输入一个字符串到str1中:");
  fgets(str1, MAX, stdin);

  printf("开始拷贝str1的内容到str2中...\n");
  while ((*p1++ = *p2++) != '\0')
    ;

  printf("拷贝完毕!\n现在,str2中的内容是:%s\n", str2);

  return 0;
}
最佳答案
2023-10-4 09:33:53
您的代码中存在一个错误,导致 strcpy 函数无法正常工作。在拷贝字符串时,您将 p1p2 的赋值顺序搞反了。正确的实现应该是 *p2++ = *p1++。以下是修复后的代码:
#include <stdio.h>
#define MAX 1024

int main() {
    char str1[MAX];
    char str2[MAX];

    char *p1 = str1;
    char *p2 = str2;

    printf("请输入一个字符串到str1中:");
    fgets(str1, MAX, stdin);

    printf("开始拷贝str1的内容到str2中...\
");
    while ((*p2++ = *p1++) != '\\0')
        ;

    printf("拷贝完毕!\
现在,str2中的内容是:%s\
", str2);

    return 0;
}

现在,当您使用 gcc my_strcpy.c -o test && ./test 编译并运行此代码时,您应该会看到预期的输出,即将 str1 中的字符串成功拷贝到 str2 中。
如果问题已经解决,请设置最佳答案
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复

使用道具 举报

发表于 2023-10-4 09:33:31 | 显示全部楼层
这段代码存在两个问题:

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

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

修改后的代码如下:
#include <stdio.h>

#define MAX 1024

int main() {
    char str1[MAX];
    char str2[MAX] = {0}; // 初始化str2数组
    char *p1 = str1;
    char *p2 = str2;

    printf("请输入一个字符串到str1中:");
    fgets(str1, MAX, stdin);

    printf("开始拷贝str1的内容到str2中...\n");
    while ((*p2++ = *p1++) != '\0')
        ;

    printf("拷贝完毕!\n现在,str2中的内容是:%s\n", str2);

    return 0;
}
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2023-10-4 09:33:53 | 显示全部楼层    本楼为最佳答案   
您的代码中存在一个错误,导致 strcpy 函数无法正常工作。在拷贝字符串时,您将 p1p2 的赋值顺序搞反了。正确的实现应该是 *p2++ = *p1++。以下是修复后的代码:
#include <stdio.h>
#define MAX 1024

int main() {
    char str1[MAX];
    char str2[MAX];

    char *p1 = str1;
    char *p2 = str2;

    printf("请输入一个字符串到str1中:");
    fgets(str1, MAX, stdin);

    printf("开始拷贝str1的内容到str2中...\
");
    while ((*p2++ = *p1++) != '\\0')
        ;

    printf("拷贝完毕!\
现在,str2中的内容是:%s\
", str2);

    return 0;
}

现在,当您使用 gcc my_strcpy.c -o test && ./test 编译并运行此代码时,您应该会看到预期的输出,即将 str1 中的字符串成功拷贝到 str2 中。
如果问题已经解决,请设置最佳答案
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2023-10-4 09:34:05 | 显示全部楼层
问题描述:

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

问题分析:

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

修复后的代码:
#include <stdio.h>#define MAX 1024int main() {    char str1[MAX];    char str2[MAX];    char *p1 = str1;    char *p2 = str2;    printf("请输入一个字符串到str1中:");    fgets(str1, MAX, stdin);    printf("开始拷贝str1的内容到str2中...\");    while ((*p2++ = *p1++) != '\\0')        ;    printf("拷贝完毕!\现在,str2中的内容是:%s\", str2);    return 0;}
修复说明:

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

球一个最佳答案谢谢啦!这对我非常重要!
本段回答来自Chat GPT-4,如有错误,多多包涵!
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

 楼主| 发表于 2023-10-4 09:34:13 | 显示全部楼层
isdkz 发表于 2023-10-4 09:33
这段代码存在两个问题:

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

我用了括号啊
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

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

我怎么这么粗心
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

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

对了,2楼是不是ChatGpt3?有点哪啥......
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2023-10-4 09:37:25 | 显示全部楼层
风眠 发表于 2023-10-4 09:36
对了,2楼是不是ChatGpt3?有点哪啥......

gpt4通常聪明一点
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

 楼主| 发表于 2023-10-4 09:38:05 | 显示全部楼层

好的,现在我信你是ChatGPT4了......
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

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

膜拜大佬
想知道小甲鱼最近在做啥?请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

小黑屋|手机版|Archiver|鱼C工作室 ( 粤ICP备18085999号-1 | 粤公网安备 44051102000585号)

GMT+8, 2024-10-6 01:34

Powered by Discuz! X3.4

© 2001-2023 Discuz! Team.

快速回复 返回顶部 返回列表