c++ - SDL2 - Visual Studio 2017 SDL_FreeWAV 访问冲突
问题描述
我正在使用 SDL2,并创建了一个Audio
类来处理游戏音乐和音效。声音完美无缺,但只要Audio
类被破坏,SDL_FreeWAV()
调用就会引发访问冲突:
Program.exe 中的 0x000000006C7A8737 (SDL2.dll) 引发异常:0xC0000005:访问冲突读取位置 0x00007FF4A9080008
音频.h:
#pragma once
namespace Audio {
class Audio {
public:
Audio ();
Audio (char*, char*);
~Audio ();
void pause (int);
private:
Uint32 wav_length;
Uint8 *wav_buffer = NULL;
SDL_AudioSpec wav_spec;
SDL_AudioDeviceID device_id;
int success;
};
class Music {
public:
Music ();
Music (char*);
~Music ();
private:
Audio *audio = NULL;
};
class Effect {
public:
Effect ();
Effect (char*);
~Effect ();
private:
};
};
音频.cpp
#include "stdafx.h"
#include "Audio.h"
#include "SDL_audio.h"
namespace Audio {
Audio::Audio () {
//Default constructor
}
Audio::Audio (char *filename, char *channelName) {
if (SDL_LoadWAV (filename, &this->wav_spec, &this->wav_buffer, &this->wav_length) == NULL) {
std::cout << "[-] SDL: " << SDL_GetError () << "\n";
exit (ERROR_SDL_AUDIO_WAV_LOAD);
}
this->device_id = SDL_OpenAudioDevice (channelName, 0, &this->wav_spec, NULL, 0);
this->success = SDL_QueueAudio (this->device_id, this->wav_buffer, this->wav_length);
}
Audio::~Audio () {
SDL_CloseAudioDevice (this->device_id);
SDL_FreeWAV (this->wav_buffer); // <-- access violation here
}
void Audio::pause (int on) {
SDL_PauseAudioDevice (this->device_id, on);
}
};
解决方案
只要您有默认构造函数并且没有定义自己的构造函数,C++ 编译器就会免费为您提供 amove constructor
和 a copy constructor
。他们浅拷贝对象的所有成员。
当您使用operator=
(将对象分配给另一个对象)时,它使用copy
构造函数。当您返回一个临时对象(右值)时,它使用move
构造函数。
遗憾的是,在这种情况下,默认的复制和移动构造函数不适合您。
考虑一下:
Audio a = Audio("filename", "channel");
在这个看似简单的代码行中,您是:
- 制作一个临时(右值)
Audio
对象 - 打电话
Audio::operator=
- 使用
move constructor
- 删除临时对象
所以在这行有效的 C++ 之后,a
有:
device_id
那是关闭的。wav_buffer
那被释放了。
那么我们如何解决这个问题呢?
Audio::Audio (char *filename, char *channelName) {
if (SDL_LoadWAV (filename, &this->wav_spec, &this->wav_buffer, &this->wav_length) == NULL) {
std::cout << "[-] SDL: " << SDL_GetError () << "\n";
exit (ERROR_SDL_AUDIO_WAV_LOAD);
}
this->device_id = SDL_OpenAudioDevice (channelName, 0, &this->wav_spec, NULL, 0);
this->success = SDL_QueueAudio (this->device_id, this->wav_buffer, this->wav_length);
}
Audio::Audio(Audio&& other) : // move constructor
wav_length(other.wav_length),
wav_buffer(other.wav_buffer),
wav_spec(other.wav_spec),
device_id(other.device_id),
success(other.success)
{
other.wav_buffer = nullptr;
}
Audio::~Audio () {
if(wav_buffer != nullptr) {
SDL_CloseAudioDevice (this->device_id);
SDL_FreeWAV (this->wav_buffer); // Gives access violation
}
}
现在,当一个Audio
对象被移动时,它wav_buffer
会被取消,这样当它被销毁时就不会被清理。
推荐阅读
- javascript - 当 div 窗口被调整大小时,div 不会回到它的位置
- javascript - JS。contentEditable 跟踪更改(更正),如 Word 样式
- crystal-reports - Crystal Reports - 公式中的 Where 子句用于计算第二个表中字段的总和
- gnuplot - 如何用gnuplot制作三元图?
- excel - 更改 Excel CommandBar 的字体和背景颜色
- windows - 使用 Ansible 将文件复制到 Windows NFS
- utf-8 - SQL_HANDLE_ENV 上的驱动程序 SQLAllocHandle UTF-8 失败
- onchange - 如何使用 Onchange 调用 javascript
- c++ - 如果for循环以01-09开头,如何启动我的else?
- snowflake-cloud-data-platform - VARIANT 数据类型大小限制