weijun_zhang 发表于 2021-4-22 13:38:05

S1E22动动手第1题:用指针1拷贝字符串 —— strncpy 函数

本帖最后由 weijun_zhang 于 2021-4-22 13:41 编辑

S1E22动动手第1题进阶题:用指针1拷贝字符串 —— strncpy 函数
请问下方函数为何错误呢?

#include <stdio.h>

int main()
{
       char str1,str2;
       char *target1=str1;
       char *target2=str2;
       int n;
      
       printf("请输入一个字符串到str1中:");
         fgets(str1,1028,stdin);
       printf("请输入需要拷贝的字符个数:");
             scanf("%d",&n);
       printf("开始拷贝。。。\n");      
          // printf("str2的内容是:");   
         while(n>0)
         {

                           if(*target1=='\0')//此处是==,不是=
                    {
                                  break;
                           }
            if((int)(*target1)<0)    //汉字占2个字符的处理
               {
                         *target2=*target1;
                             target1++;
                   target2++;
                  *target2=*target1;
                                   target1++;
                                   target2++;
                           }
            else
                 {
                          *target2=*target1;
                          target1++;
            target2++;
                        }
                          n--;
                   }

      printf("str2的内容是:%s",str2);

      return 0;
}


下方为老师的代码
#include <stdio.h>

#define MAX 1024

int main()
{
      char str1;
      char str2;

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

      char ch;
      int n;

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

      printf("请输入需要拷贝的字符个数:");
      scanf("%d", &n);

      printf("开始拷贝 str1 的内容到 str2 中...\n");
      while (n--)
      {
               ch = *target2++ = *target1++;
               if (ch == '\0')
               {
                     break;
               }
               if ((int)ch < 0)
               {
                     *target2++ = *target1++;
                     *target2++ = *target1++;
               }
      }

      *target2 = '\0';

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

      return 0;
}

weijun_zhang 发表于 2021-4-22 13:41:07

我已经自己搞定啦。

元豪 发表于 2023-4-30 10:20:25

这段代码中的错误在于,当源字符串 str1 中需要拷贝的字符个数 n 大于实际字符个数时,目标字符串 str2 仍然没有以 \0 结尾。这将导致在输出目标字符串 str2 的内容时,会出现意料之外的结果,并且可能会访问到未分配的内存空间而产生不可预测的错误。

正确的方法是,在拷贝完成后,手动向目标字符串 str2 的结尾位置添加 \0,即用 target2 指针指向目标字符串 str2 的最后一个字符,然后将其赋值为 '\0'。这样才能保证目标字符串 str2 的有效性,避免不必要的错误。

以下是修改后的示例代码:

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

int main()
{
    char str1, str2;
    char *target1 = str1;
    char *target2 = str2;
    int n;

    printf("请输入一个字符串到 str1 中:");
    fgets(str1, 1028, stdin);
    printf("请输入需要拷贝的字符个数:");
    scanf("%d", &n);
    printf("开始拷贝。。。\n");

    while (n > 0 && *target1 != '\0') {
      if ((int)(*target1) < 0) {
            // 汉字占两个字符
            *target2++ = *target1++;
            *target2++ = *target1++;
      } else {
            *target2++ = *target1++;
      }
      n--;
    }

    // 手动添加结尾标志
    *target2 = '\0';

    printf("str2 的内容是:%s\n", str2);

    return 0;
}
修改后的代码使用了 C 标准库中的 string.h 头文件中提供的字符串函数,避免了手动处理 \0 结尾标志的繁琐过程。同时,为了确保源字符串和目标字符串的安全性,还需要在拷贝时判断源字符串是否已经结束(即 *target1 == '\0'),如果结束了就退出循环,否则继续拷贝。
页: [1]
查看完整版本: S1E22动动手第1题:用指针1拷贝字符串 —— strncpy 函数