Question

所以，我一直在玩c ++，并决定编写一个涉及以二进制模式打开和写入文件的程序。我不太熟悉c ++的iostream功能（我主要做基于API的编程），但我读了几个关于这个主题的技术指南并编写了一些代码。代码用于打开一个文件，将其数据读取到缓冲区，然后将该缓冲区转换为另一种格式并将其写入另一个文件。问题是它不断抛出“Debug Assertion”错误，这显然是围绕无效使用空指针。但是，当我查看代码时，我无法理解它。我可能只是误用了iostream库或者发生了一个简单的逻辑错误。我需要有单独的SetMemBlock函数，因为我打算使用相同的基础来格式化各种函数的不同输出。这只是我的原型。无论如何，这是我快速的脏课程设置：

const DebugMode = true;

class A
{
public:
    bool FileFunction( const char *, const char * );

protected:
    bool SetMemBlock( char *, std::fstream &, std::streamoff & );

private:
    std::fstream SrcFileStream;
    std::fstream DestFileStream;
};

bool A::SetMemBlock( char* MemBlock, std::fstream & FileStream, std::streamoff & Size )
{
    std::streamoff TempOff = 0;
    //This is meant to check for a non-empty buffer and to see if the stream is valid.
    if( MemBlock != 0 || !FileStream.is_open() )
        return false;

    TempOff = FileStream.tellg();
    FileStream.seekg(0, std::ios::end);
    Size = FileStream.tellg();

    MemBlock = new( std::nothrow ) char[ (int) Size ];

    if( MemBlock == 0 )
        return false;

    FileStream.seekg(0, std::ios::beg);
    FileStream.read( MemBlock, (int) Size );

    if( !FileStream )
        return false;

    FileStream.seekg(TempOff);
    return true;
}

bool A::FileFunction( const char * SrcFile, const char * DestFile )
{
    char * MemBlock = 0;
    std::streamoff Size = 0;

    SrcFileStream.open( SrcFile, std::ios::binary | std::ios::in );
    DestFileStream.open( DestFile, std::ios::binary | std::ios::out );

    if( !SrcFileStream.is_open() || !DestFileStream.is_open() )
        return false;

    if( DebugMode )
    {
        std::cout<<"Files opened succesfully...\nNow writing memory block..."<<std::endl;
    }

    if( !SetMemBlock( MemBlock, SrcFileStream, Size ) )
    {
        std::cout<<"An error occured when reading to memory block!"<<std::endl;
        return false;
    }

    if( DebugMode )
    {
        std::cout<<"Memory block written..."<<std::endl;
    }

    DestFileStream.seekp( std::ios::beg );
    DestFileStream.write( MemBlock, Size );

    SrcFileStream.close();
    DestFileStream.close();
    delete[] MemBlock;
    return true;

}

Answer 1

您按价值将MemBlock传递给SetMemBlock。因此，该函数只设置本地副本的值，这对调用函数没有影响;因此，调用函数中MemBlock的值仍然是垃圾。使用它作为指针可能会导致断言（如果你很幸运）或彻头彻尾的崩溃（如果你不是。）你想通过引用传递该参数。

如果您不知道这些术语的含义，Google会“按值传递”和“按参考传递”。你真的需要了解其中的差异！

Answer 2

通过引用传递MemBlock：

bool A::SetMemBlock( char*& MemBlock, std::fstream & FileStream, std::streamoff & Size )

C ++调试断言错误

2 个答案: