时间戳

首页 > 解决方案 > 无法将字符从一个字符串分配给另一个

c - 无法将字符从一个字符串分配给另一个

问题描述

所以我有一个函数，它接受一个字符串，去掉特殊格式的字符，并将它分配给另一个字符串以供以后处理。

一个示例调用将是：

act_new("$t does $d");

它应该去掉 $t 和 $d 并将第二个字符串保留为“does”，但它没有分配任何东西。经过几年的闲置后，我重新开始编程，这是别人的代码（A MUD 代码库，Rom），但我觉得我缺少指针分配的基本内容。有小费吗？

（这是截断的代码，其余的直到很久以后才对 str 或 point 进行任何操作）

void act_new(const char *format)
{
  const char *str;
  char *point;

  str = format;

  while ( *str != '\0' ) {
    if ( *str != '$' ) {
      *point++ = *str++;
      continue;
    }
  }
}

标签： cmud

解决方案

正如评论中所指出的，您的代码存在一些问题：

  • point未初始化（垃圾指针值）
  • continue什么都不做
  • $如果遇到a 则无限循环

在编写函数时，还必须记住，如果$遇到 a 当且仅当它不是字符串中的最后一个字符（除了'\0'），则跳过一个额外的字符。

由于您知道需要循环多少次，因此for循环更适合，并且作为奖励，您不必明确检查 a 之后的字符$是否是'\0'在跳过额外字符时format（src下面重命名）。另外，不要忘记终止目标字符串。

此代码将为您处理这些事情：

void act_new(const char *src)
{
    const size_t length = strlen(src);
    char * const dst = (char*)malloc(sizeof(char)*(length+1));
    if(dst == NULL)
        // Error handling left out
        return;

    char *point = dst;

    for(size_t i = 0; i < length; ++i)
    {
        if(src[i] == '$')
        {
            ++i;
            continue;
        }
        *point++ = src[i];
    }
    *point = '\0';  //Terminate string properly

    printf("%s\n", dst);
    free(dst);
}

推荐阅读