啊这啧啧啧 发表于 2023-4-21 22:30:09

拷贝字符有问题

#include <stdio.h>
#include <string.h>
#define MAX 1024

int main()
{
      char str1;
                char str2;
                char *target1 = str1;
      char *target2 = str2;
               
                printf("请输入一个字符串到 str1 中:");
                fget(str1,MAX,stdin);
                printf("%s\n",str1);
                strcpy(str2,str1);
      printf("%s",str2);
      return 0;
}

isdkz 发表于 2023-4-21 22:37:16

你的程序有一些问题,下面是修复后的代码:

#include <stdio.h>
#include <string.h>
#define MAX 1024

int main()
{
    char str1;
    char str2;

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

    // 使用 strcpy,不需要提供数组下标
    strcpy(str2, str1);
    printf("%s", str2);
    return 0;
}


问题与修改如下:

1. 代码中使用的函数名是 `fget`,正确的函数名应该是 `fgets`。
2. 在调用 `strcpy` 时,应直接使用数组名而不是带有下标的数组名。使用 `strcpy(str2, str1)` 而不是 `strcpy(str2, str1)`。
3. 你的原代码中定义了 `char *target1 = str1;` 和 `char *target2 = str2;`,但在程序中并未使用这两个指针,因此我将它们删除了。如果你打算使用它们,请确保在代码中正确使用。

sfqxx 发表于 2023-4-21 22:40:19

您好,这段代码中有几个问题:

1. `fget(str1,MAX,stdin)` 应该是 `fgets(str1, MAX, stdin)`,即正确的函数名为`fgets`,而且最后一个参数是 `stdin`。
2. 字符串拷贝函数 `strcpy(str2,str1)` 中,应该使用指针变量作为参数,正确写法是 `strcpy(target2, target1)`。
3. 还需要在程序开头添加 `#include <stdlib.h>` 来包含内存分配和释放函数。

经过修改后的代码如下:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define MAX 1024

int main() {
    char str1;
    char str2;

    char *target1 = str1;
    char *target2 = str2;

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

    strcpy(target2, target1);

    printf("%s", str2);

    return 0;
}


请注意,在这个例子中,拷贝操作实际上没有部署新的字符串,所以结果可能并不如您期望。例如,如果我们输入"Hello World",程序将输出一个空白行,因为目标数组还未初始化。要编辑此代码以获得预期效果,请尝试向目标字符串赋值(如 `str2 = '\0'`)然后重复复制步骤。
页: [1]
查看完整版本: 拷贝字符有问题