为什么strsep()不能使用指向堆栈的指针？

Question

使用函数strsep查找字符串的第一个单词似乎存在指针兼容性问题。到目前为止，我一直认为char *s和char s[]是完全可以互换的。但看起来并非如此。我在堆栈上使用数组的程序失败，并显示以下消息：

foo.c: In function ‘main’:
foo.c:9:21: warning: passing argument 1 of ‘strsep’ from incompatible pointer type [-Wincompatible-pointer-types]
  char *sub = strsep(&s2, " ");
                     ^
In file included from foo.c:2:0:
/usr/include/string.h:552:14: note: expected ‘char ** restrict’ but argument is of type ‘char (*)[200]’
 extern char *strsep (char **__restrict __stringp,

我不明白这个问题。使用malloc的程序可以工作。

这是可行的：

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

int main(void)
{
    char s1[] = "Hello world\0";
    char *s2 = malloc(strlen(s1)+1);
    strcpy(s2, s1);
    char *sub = strsep(&s2, " ");

    printf("%s\n", sub);

    return 0;
}

这不是：

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

int main(void)
{
    char s1[] = "Hello world\0";
    char s2[200];
    strcpy(s2, s1);
    char *sub = strsep(&s2, " ");

    printf("%s\n", sub);

    return 0;
}

问题出在哪里？(为strcpy感到抱歉)。为什么指针指向堆栈还是堆对函数很重要？我理解为什么你不能访问二进制/文本段中的字符串，但是堆栈有什么问题呢？

Answer 1

 note: expected ‘char ** restrict’ but argument is of type ‘char (*)[200]’

您的警告会准确地告诉您问题所在。你有两种不同的类型。

char *s2;        /* declares a character pointer */

而

char s2[200];   /* declares an array of char[200] */

当您将的地址作为指针时，结果是指针到指针。当您获取 an数组的地址时，结果是指向数组的指针。当您取消引用指针到指针时，结果是一个指针。当您取消引用指向数组的指针时，结果是一个数组。

strsep没有被设计为接受指向数组的指针作为参数(这将阻止它根据需要重新分配)

Answer 2

@DavidRankin关于为什么它不起作用是正确的。但您仍然可以编写代码，以便它可以在堆栈上使用变量。

要使用数组代替malloc()，可以创建另一个指向数组的指针，并将其用作strsep()的参数，如version1()函数所示。

我知道这可能只是一个示例，但是您提供的malloc()和strsep()示例可能会导致内存错误，因为strsep()会更新指针(它会修改它所指向的地址)。因此，您必须保存malloc()返回的原始地址，以便正确地释放该内存。请参见version2()示例。

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


void version1(void)
{
    char s1[] = "Hello world";
    char s2[200];
    char* s3 = s2;
    char *sub;

    strcpy(s3, s1); // or s2 in place of s3

    while ((sub = strsep(&s3, " ")))
    {
        printf("%s\n", sub);
    }
}

void version2(void)
{
    char s1[] = "Hello world";
    char *mymem = malloc(strlen(s1)+1);
    char *s2 = mymem;
    char *sub;

    strcpy(s2, s1);

    while ((sub = strsep(&s2, " ")))
    {
        printf("%s\n", sub);
    }

    free(mymem);
}

int main(int argc, char* argv[])
{
    printf("Version1\n");
    version1();

    printf("\nVersion2\n");
    version2();

    return 0;
}