C++ 无法删除 char*，不断损坏堆

Question

语言：C++，编译器：MSVS (/Za) 和 g++（是的，它必须同时适用），级别：初学者

我正在尝试从 char* 中删除数据，以便重新分配它并继续我的程序。我创建了一个新的 char*，将命令行参数的值分配给它，进行一些验证，然后如果验证失败，它应该解除分配 char* 并让我将新数据分配给 var 但是我得到一个“堆检测到损坏”Visual Studio 中的错误。我很好奇我当前代码的修复以及可以更清晰/简洁地完成的任何其他方式。

在 main() 中：

//...
//make non constant versions to be messed with
char* ba = new char[ strlen(argv[4]) + 1 ];
char* num = new char[ strlen(argv[2]) + 1 ];

//make non constant versions of commandline stuff
nonConstant( argv[4], argv[2], ba, num );

//do the conversion and printing
convert( ba, num );
//...

convert 这样做：

//...
if( error ) {
    getNew(ba, num, &error);
}
//...

这里是getNew：

void getNew( char* ba, char* num, bool error ) {

//if there's an error in their numbers let them input new stuff and check to see if that's valid
while( error ) {
    //tell the user that the input was bad an prompt for more, use getline because strings are weird
    //cin stuff here (this part works, no problems)

    //free up base and num so I can reassign them
    delete[] ba; //<--this line fails
    delete[] num;

    //set lengths = to lengths of input + 1 for \0
    ba = new char[ inputBa.length() + 1 ];
    num = new char[ inputNum.length() + 1 ];

    //do the assignment of new input back to base and num
    inputBa.copy( ba, inputBa.length(), 0 );
    inputNum.copy( num, inputNum.length(), 0 );

    //ensure that the input was good this time
    validateInput( ba, num, error );
}

Answer 1

停止！立即切换到std::string。

这将修复您的代码，并根据要求更清晰/简洁。

Answer 2

通过 char *& 传递参数将解决问题。

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

void reassgnReference( char * &ba, char * &num );
void reassgnPointerToPointer( char ** ba, char ** num );

int main( int argc, char ** argv )
{
    if ( argc > 4 )
    {
        char * ba = new char[strlen( argv[ 4 ] ) + 1];
        char * num = new char[strlen( argv[ 2 ] ) + 1];

        strncpy( ba, argv[ 4 ], strlen( argv[ 4 ] ) );
        strncpy( num, argv[ 2 ], strlen( argv[ 2 ] ) );

        printf( "In the beginning, ba = %s, num = %s\n", ba, num );

        reassgnReference( ba, num );
        printf( "reassgnReference(), ba = %s, num = %s\n", ba, num );

        reassgnPointerToPointer( &ba, &num );
        printf( "reassgnPointerToPointer(), ba = %s, num = %s\n", ba, num );
}
else
{
    printf( "%s Expects at least 4 arguments\n", argv[ 0 ] );
}

return( 0 );
}

void reassgnReference( char *& ba, char *& num )
{
    delete[] ba;
    delete[] num;

    char const * const newBa = "ba from reference";
    char const * const newNum = "num from reference";

    ba = new char[strlen( newBa ) + 1];
    num = new char[strlen( newNum ) + 1];

    strncpy( ba, newBa, strlen( newBa ) );
    strncpy( num, newNum, strlen( newNum ) );
}

void reassgnPointerToPointer( char ** ba, char ** num )
{ 
    delete[] *ba;
    delete[] *num;

    char const * const newBa = "ba from pointer to pointer";
    char const * const newNum = "num from pointer to pointer";

    *ba = new char[strlen( newBa ) + 1];
    *num = new char[strlen( newNum ) + 1];

    strncpy( *ba, newBa, strlen( newBa ) );
    strncpy( *num, newNum, strlen( newNum ) );
}

Answer 3

如果看起来您的 getNew 函数可以重新分配本地 ba 和 num 指向的内存，即 delete[] 它和 new[] 它再次。

但是ba 和num 在调用者中，在convert 中呢？另外，main 中的 ba 和 num 呢？它们按值传递给getNew，这意味着它们保持不变。它们继续指向它们原来的（现在已经死掉的）内存位置。我猜你会继续在convert 和main 中使用那些现在毫无意义的ba 和num 值，直到它们导致崩溃。例如，如果您再次使用这些旧指针值调用 getNew，getNew 可能会在 delete[] 处失败。

此外，目前尚不清楚error 在convert 中是什么。当getNew 实际上需要bool 参数时，为什么要将它作为&error （可能是指针）传递？它会编译，但无论如何它看起来像一个问题。