14

我已经尝试过互联网上将字符附加到 char* 的方法,但它们似乎都不起作用。这是我不完整的解决方案之一:

char* appendCharToCharArray(char * array, char a)
{
    char* ret = "";
    if (array!="") 
    {
        char * ret = new char[strlen(array) + 1 + 1]; // + 1 char + 1 for null;
        strcpy(ret,array);
    }
    else
    {
        ret = new char[2];
        strcpy(ret,array);
    }

    ret[strlen(array)] = a;  // (1)
    ret[strlen(array)+1] = '\0';
    return ret;
}

这仅在传递的数组为“”(内部为空白)时才有效。否则它没有帮助(并且在(1)处出错)。你们能帮我解决这个问题吗?非常感谢先进!

4

4 回答 4

15

删除隐藏在块中的那些char * ret声明external 。因此,您有内存泄漏,另一方面未分配内存。ifretret

要比较 c 风格的字符串,您应该使用strcmp(array,"")not array!=""。您的最终代码应如下所示:

char* appendCharToCharArray(char* array, char a)
{
    size_t len = strlen(array);

    char* ret = new char[len+2];

    strcpy(ret, array);    
    ret[len] = a;
    ret[len+1] = '\0';

    return ret;
}

请注意,您必须处理它在ret某处返回的分配内存delete[]

 

为什么你不使用std::string?它具有.append在字符串末尾附加字符的方法:

std::string str;

str.append('x');
// or
str += x;
于 2013-11-10T16:16:27.480 回答
3

函数名不反映函数的语义。事实上,你不附加一个字符。您创建一个包含原始数组和给定字符的新字符数组。因此,如果您确实需要一个将字符附加到字符数组的函数,我会按以下方式编写它

bool AppendCharToCharArray( char *array, size_t n, char c )
{
    size_t sz = std::strlen( array );

    if ( sz + 1 < n ) 
    {
        array[sz] = c;
        array[sz + 1] = '\0';
    }       

    return ( sz + 1 < n );
} 

如果您需要一个包含原始数组副本和给定字符的函数,那么它可能如下所示

char * CharArrayPlusChar( const char *array, char c )
{
    size_t sz = std::strlen( array );
    char *s = new char[sz + 2];

    std::strcpy( s, array );
    s[sz] = c;
    s[sz + 1] = '\0';

    return ( s );
} 
于 2013-11-10T16:37:30.423 回答
1

具体问题是您要声明一个新变量而不是分配给现有变量:

char * ret = new char[strlen(array) + 1 + 1];
^^^^^^ Remove this

并尝试通过比较指针来比较字符串值:

if (array!="")     // Wrong - compares pointer with address of string literal
if (array[0] == 0) // Better - checks for empty string

尽管根本没有必要进行这种比较;无论字符串是否为空,第一个分支都会做正确的事情。

更普遍的问题是您在 C++ 中处理讨厌的、容易出错的 C 样式字符串操作。使用std::string它将为您管理所有内存分配:

std::string appendCharToString(std::string const & s, char a) {
    return s + a;
}
于 2013-11-10T16:19:45.963 回答
0
char ch = 't';
char chArray[2];
sprintf(chArray, "%c", ch);
char chOutput[10]="tes";
strcat(chOutput, chArray);
cout<<chOutput;

输出:

test
于 2018-01-12T06:52:25.257 回答