风眠 发表于 2023-10-4 09:33:18

自制strcpy异常

效果
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;
char str2;

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;
}

isdkz 发表于 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;
    char str2 = {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;
}

陶远航 发表于 2023-10-4 09:33:53

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

#include <stdio.h>
#define MAX 1024

int main() {
    char str1;
    char str2;

    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 中。
如果问题已经解决,请设置最佳答案

Mike_python小 发表于 2023-10-4 09:34:05

问题描述:

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

问题分析:

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

修复后的代码:


#include <stdio.h>
#define MAX 1024

int main() {
    char str1;
    char str2;

    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。

球一个最佳答案谢谢啦!这对我非常重要!{:10_254:}{:10_254:}{:10_254:}
本段回答来自Chat GPT-4,如有错误,多多包涵!

风眠 发表于 2023-10-4 09:34:13

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

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

我用了括号啊

风眠 发表于 2023-10-4 09:35:54

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

我怎么这么粗心{:10_266:}

风眠 发表于 2023-10-4 09:36:50

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

对了,2楼是不是ChatGpt3?{:5_94:}有点哪啥......{:9_219:}

陶远航 发表于 2023-10-4 09:37:25

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

gpt4通常聪明一点

风眠 发表于 2023-10-4 09:38:05

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

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

六翻了 发表于 2023-10-5 19:18:59

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

膜拜大佬
页: [1]
查看完整版本: 自制strcpy异常