发出重新分配内存到字符串的问题

Question

C的新手并试图学习。 在这里，我尝试创建一个函数，该函数使用动态内存分配和ByRef复制字符串直至第一个空间。 似乎我在使用Realloc的方式做错了。您能帮我弄清楚我使用动态内存分配的方式有什么问题？

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

void f1(char **c, char *s);

int main() {
    char * s = "this is an example";
    char *c;

    c =(char *) malloc(sizeof(char));
    f1(&c,s);
    free(c);
}

void f1(char **c, char *s)
{
    int i=0;
    while ((s[i])!=' ')
    {
        (*c)[i]=s[i];
        i++;
        (*c)=(char *)realloc ((*c),sizeof(char)*i);
    }

    (*c)[i]='\0';
    printf("\n%s\n",*c);

}

Answer 1

void f1(char** r, char* s) 
{
   // find size of new buffer
   size_t len = 0;
   while(s[len] != '\0' && s[len] != ' ') len++;

   *r = (char*)malloc(len + 1);

    memcpy(*r, s, len);

    (*r)[len] = '\0';
}

Answer 2

已经为一个字符分配了一个字符的内存

c =(char *) malloc(sizeof(char));

的第一次迭代中，

int i=0;
while ((s[i])!=' ')
{
    (*c)[i]=s[i];
    i++;
    (*c)=(char *)realloc ((*c),sizeof(char)*i);
}

在函数调用之前，在填充此内存

    (*c)[i]=s[i];

，然后再次分配了一个字符的内存，

    (*c)=(char *)realloc ((*c),sizeof(char)*i);

因为在循环的第一次迭代中>变得等于1。因此，在循环的第二次迭代中，试图在分配的内存之外写作，从而导致不确定的行为。

您至少需要写作，

    *c = realloc ( *c, i + 1);

也需要更安全地使用中间指针作为例如，

char *tmp = realloc ( *c, i + 1);
if ( tmp != NULL ) *c = tmp;

但是在这种情况下，您还需要更改功能逻辑。

一样更改功能

int f1( char **c, const char *s);

，并且应该更改程序的状况

while ( s[i] != '\0' && s[i] !=' ' )

并且应该像使用您的方法

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

int f1( char **s1, const char *s2 );

int main( void ) 
{
    const char *s = "this is an example";
    char *t = malloc( sizeof( char ) );

    if ( t != NULL )
    {
        t[0] = '\0';
        f1( &t, s);
        puts( t );
    }

    free( t );
}

int f1( char **s1, const char *s2 )
{
    int success = 1;

    for ( size_t i = 0; success && s2[i] != '\0' && !isblank( ( unsigned char )s2[i] ); i++ )
    {
        char *tmp = realloc( *s1, i + 2 );

        success = tmp != NULL;

        if ( success )
        {
            *s1 = tmp;
            ( *s1 )[i] = s2[i];
            ( *s1 )[i+1] = '\0';
        }
    }

    return success;
}

。然而，程序输出是

this

一种具有许多内存重新分配的方法，效率低下。

我会以以下方式编写

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

char * f1( const char *s, const char *delim );

int main( void ) 
{
    const char *s = "this is an example";
    char *t = f1( s, " \t" );

    if ( t != NULL )
    {
        puts( t );
    }

    free( t );
}

char * f1( const char *s, const char *delim )
{
    size_t n = strcspn( s, delim );

    char *result = malloc( n + 1 );

    if ( result != NULL )
    {
        result[n] = '\0';
        memcpy( result, s, n );
    }

    return result;
}

程序输出是

this

Answer 3

正如@unholysheep提到的那样，我用来分配内存的i太小。更改为（*t）=（char*）realloc（（（*t），（i+1）*sizeof（char））;，并且起作用。