C 中的手动拆分函数

Question

这段代码快把我逼疯了。目标是根据逗号拆分为 char[]。它在java中工作。但在 C 中打印出奇怪的输出。我怀疑错误出现在循环的第二次迭代中的 28 处，我试图添加 5 ，数组变成了奇怪的字符。

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

main()
{
    char asshat[] = {'4','5','5',',','7','4','7','\0'};

    int firstSize = 0;//
    int secondSize = 0;//

    //new = 4 \0 \0
    char first[] = {'0', '\0'};//
    //new = 
    char second[] = {'0', '\0'};//
    char *first_ptr = first;
    char *second_ptr = second;

    int takingFirst = 1;
    int takingSecond = 0;

    int i;
    for (i = 0; i < strlen(asshat); i++)
    {
        if (asshat[i] != ',')
        {
            if (takingFirst == 1)
            {
                first_ptr[firstSize] = asshat[i];//ERROR here when you add 5 you s**t bricks
                firstSize++;
                if (asshat[i+1] != ',')
                {
                    char new[firstSize+2];
                    int k;
                    for (k = 0; k < strlen(first_ptr); k++)
                    {
                        new[k] = first_ptr[k];
                    }
                    new[firstSize] = '0';
                    new[firstSize+1] = '\0';
                    first_ptr = new;
                }
            }
            if (takingSecond == 1)
            {
                second_ptr[secondSize] = asshat[i];
                secondSize++;
                if (asshat[i+1] != '\0')
                {
                    char new[secondSize+2];
                    int k;
                    for (k = 0; k < strlen(second_ptr); k++)
                    {
                        new[k] = second_ptr[k];
                    }
                    new[secondSize+1] = '\0';
                    second_ptr = new;
                }
            }
        }
        else
        {
            takingFirst = 0;
            takingSecond = 1;
        }
    }
    printf("%d\n",strlen(first_ptr));
    printf("%c%c%c\n",first_ptr[0],first_ptr[1],first_ptr[2]);
    printf("%s\n",first_ptr);

    printf("%d\n",strlen(second_ptr));
    printf("%c%c%c\n",second_ptr[0],second_ptr[1],second_ptr[2]);
    printf("%s\n",second_ptr);
}

Answer 1

将 new 声明为局部变量，并让 first_ptr 指向该局部变量的内存位置：

{
    char new[firstSize+2];
    ...  
    first_ptr = new;
}

然后局部变量超出范围，其内存将被其他变量重用变量/...在未来创建。 first_ptr 仍然指向该内存位置，但那里的数组不再存在。当尝试通过first_ptr访问数组时，任何事情都可能发生，并且您很可能会发现意外的值。

Answer 2

很难阅读您的解决方案正在做什么。我尝试重新实现它以使其更简单，同时保持指针播放：

#include <stdio.h>

int 
main(int argc, char *argv[])
{   
  /* Keeping variable names consistent :) */
  char asshat[] = "456,747";
  char *first = asshat, *second = asshat, *c = asshat;

  while (*c) {
    if (*c == ',') {
      *c = '\0'; second = ++c;
    } else c++;
  }   
  printf("first: %s\nsecond: %s\n", first, second);
  return 0;
}   

这会产生：

first: 456
second: 747

Answer 3

您可以通过大约一行切片来完成:)

假设至少有一个 , ...

char asshat[] = "hello,world";
char* second = asshat;
while (*second++ != ',');
*(second - 1) = 0;

printf("first: %s; second: %s\n",asshat,second);

您可以在此基础上检查是否有逗号（如果有，它将严重崩溃）不是）。

Answer 4

要在 C 中拆分字符串，我建议您将艰苦的工作留给 strtok()。如果没有必要，不要重新发明轮子。

Answer 5

我假设您正在尝试将“455”复制到第一个，并将“747”复制到第二个？如果是这样，则说明您没有为此分配足够的空间。 char first[] = {'0', '\0'} 仅分配大小为 2 的数组；您将需要 4。第二个 与此相同。

您试图通过弄清楚您在原始字符串中的位置以及是否尝试复制到first或来解决同一级别的太多问题>第二 和跟踪每个子字符串的长度；这会加剧你的困惑，并使问题变得比实际需要的更加困难。

首先分离关注点：您需要一个函数来查找输入中的下一个子字符串，另一个函数将该子字符串复制到其目标中。

坦率地说，分割字符串的最佳方法是使用标准库函数 strtok、strchr 或 strcspn 中的任何一个，并使用strcpy 或 strncpy 将子字符串复制到目标数组。有很多很好的理由不使用strtok，但它是实现您正在寻找的最简单方法之一。

以下是使用 strtok 的一般过程：

char *token = strtok(asshat, ",");  // return a pointer to the substring
                                    // before the first comma
while (token)
{
  strcpy(destination[i++], token);   // assumes destination exists and is
                                     // large enough to hold the token contents
  token = strtok(NULL, ",");         // find the next substring in the sequence;
                                     // pass NULL to indicate you're continuing
                                     // in the same string
}

正如我上面提到的，您有不想想要使用 strtok 的原因。它修改输入字符串（用 0 替换分隔符），因此您不能在字符串文字上使用它。您可能还想保留原始字符串。此外，strtok 不是可重入的，也不是线程安全的。

将其应用于您的具体情况，您可以执行以下操作：

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

int main(void)
{
  char asshat[] = "455,747";
  char first[4] = "";
  char second[4] = "";
  char *destination[] = {first, second, NULL};
  size_t i = 0;

  char *token = strtok(asshat, ",");
  while (token && destination[i])  // continue while we get a token AND
  {                                // there's a place to save it
    strcpy(destination[i++], token);
    token = strtok(NULL, ",");
  }

  printf("first = %s\n", first);
  printf("second = %s\n", second);

  return 0;
}