更改传递给 void 函数的 String 值

Question

我正在编写一种方法，从作为指向 void 函数的指针传递的字符串中删除空格，如下 -

char *  mystrcpy(char * dest, char * str) {
    dest = malloc(sizeof(str));
    char * tep = dest;
    while(*dest++ = *str++) {

    }
    return tep;
}

void trimWhiteSpace(char * str){

    int counter = 0;
    int i = 0;

    char * res = malloc(sizeof(str));
    for(i = 0; str[i]; i++) {
        if(str[i] != ' ')
            res[counter++] = str[i];
    }
    res[counter] = '\0';


    mystrcpy(str, res);
    printf("I got %s\n", str);
}

但是，当我尝试使用 main 测试我的方法时 -

int main() {
    char * myStr = (char*)malloc(sizeof(char) * 50);
    myStr = "Hello    World   !  ";
    trimWhiteSpace(myStr);
    printf("%s", myStr);
    return 0;
}

程序打印

"HelloWorld!"

在 void trimWhiteSpace(char * str) 方法内部，但打印未修改字符串的 main 方法并非如此。

如何确保 main 方法打印的字符串与 trimWhiteSpace 方法打印的字符串相同？

Answer 1

您的程序因内存泄漏而启动。:)

int main() {
    char * myStr = (char*)malloc(sizeof(char) * 50);
    myStr = "Hello    World   !  ";
    //...

首先分配内存并将其地址分配给指针myStr

    char * myStr = (char*)malloc(sizeof(char) * 50);

在下一条语句中，您使用字符串文字的地址重新分配了指针

    myStr = "Hello    World   !  ";

考虑到字符串文字在 C 和 C++ 中是不可变的。任何修改字符串文字的尝试都会导致未定义的行为。

您应该使用标准 C 函数 strcpy 将字符串文字复制到分配的内存中。

strcpy( myStr, "Hello    World   !  " );

或者另一个错误。您正在使用应用于指针的运算符sizeof

char *  mystrcpy(char * dest, char * str) {
    dest = malloc(sizeof(str));
                  ^^^^^^^^^^^

通常sizeof( char * )等于4或8，具体取决于程序编译的环境。

您应该使用另一个标准 C 函数 strlen。例如

char *  mystrcpy(char * dest, char * str) {
    dest = malloc( strlen( str ) + 1);

程序中还有其他错误。

尽管如此，删除空格的函数可以写得更简单。

这是一个演示程序

#include <stdio.h>

char * trimWhiteSpace( char * str )
{ 
    char *q = str; 
    char c = ' ';

    while ( *q && *q != c ) ++q; 

    char *p = q; 

    while ( *q ) 
    { 
        if ( *++q != c ) *p++ = *q; 
    } 

    return str; 
} 

int main( void ) 
{
    char myStr[] = "Hello    World   !  ";

    puts( myStr );
    puts( trimWhiteSpace( myStr ) );
}    

它的输出是

Hello    World   !  
HelloWorld!