为什么这个二进制输出代码会导致内存泄漏

Question

我是 C++ 的新手，我正在尝试输出一个波形文件。我在 C# 和 Java 中使用二进制文件都取得了成功，但我还不熟悉 C++。 我知道通常应该在堆上创建数组和对象。

字符串和第一个吸气剂没问题

每当它到达基类的第二个 getter 时，它就会耗尽内存。 这个类叫做 waveWriter，它扩展了一个叫做 WaveFormat 的类，它包含了 getter

WaveWriter 标题：

class WaveWriter : WaveFormat {
private:
    std::string fileName;

public:
    WaveWriter(uint16_t channels, uint32_t sampleRate,
            uint16_t bitsPerSample, double* lSampleData,
            uint32_t lSampleLength, double* rSampleData,
            uint32_t rSampleLength, bool isFloat, std::string outputName);

    ~WaveWriter();

    void writeWave() {

        std::ofstream myFile;
        myFile = std::ofstream(fileName, std::ios::out | std::ios::binary);

        // write the header
        // sGroupID
        myFile << S_GROUP_ID_HEADER;
        // dwfilelength
        myFile.write(reinterpret_cast<const char *> (GetDwFileLength()),
                sizeof (GetDwFileLength()));
        // sRiffType
        myFile << S_RIFF_TYPE;
        // write the format
        // sGroupID
        myFile << S_GROUP_ID_FORMAT;
        // dwChunkSize
        myFile.write(reinterpret_cast<const char *> (GetDwFormatChunkSize()),
                sizeof (GetDwFormatChunkSize()));
        // wFormatTag
        ........ blah blah blah

        // close file
        myFile.close();

        return;
    }
};

这个类的cpp：

WaveWriter::WaveWriter(uint16_t channels, uint32_t sampleRate,
        uint16_t bitsPerSample, double* lSampleData,
        uint32_t lSampleLength, double* rSampleData,
        uint32_t rSampleLength, bool isFloat, std::string outputName) :
WaveFormat(channels, sampleRate, bitsPerSample,
lSampleData, lSampleLength, rSampleData,
rSampleLength, isFloat) {
    outputName.append(".wav");
    this->fileName = outputName;
}

WaveWriter::~WaveWriter() {
    this->~WaveFormat();
}

WaveFormat 的标头包含私有变量、构造函数和像这样访问私有变量的 getter：

public:
    uint16_t GetCbSize() {
        return cbSize;
    }

    uint32_t GetDwAvgBytesPerSec() {
        return dwAvgBytesPerSec;
    }

    uint32_t GetDwChannelMask() {
        return dwChannelMask;
    }......

Answer 1

这是基于您的函数名称的推测，但我认为这段代码：

myFile.write(reinterpret_cast<const char *> (GetDwFileLength()),sizeof (GetDwFileLength()));

是不正确的。 假设GetDwFileLength()返回大小作为值，将其转换为const char *是不正确的。 您需要将其保存在另一个参数中并发布其地址以进行转换。 像这样的东西：

auto val = GetDwFileLength();
myFile.write(reinterpret_cast<const char *> (&val), sizeof (val));

我在您的代码中多次看到类似的错误。 这个错误会导致无效的内存访问。

此外，您应该使用virtual基析构函数，而不是从派生类调用基析构函数。 永远不要在派生类中调用基析构函数。