私は新しいプロジェクトで働いていましたが、なぜ失敗するのかわからない問題に遭遇しました。
この行を削除すると、textYが_Block_Type_Is_Valid(pHead-> nBlockUse)エラーを返します。だから私は何が間違っているのですか?
これはソースコードです:
Text.h
#ifndef TEXT_H
#define TEXT_H
typedef boost::shared_ptr<Font> FontPtr;
class Text
{
public:
Text(FontPtr font, char *text)
{
str = new char[35];
this->font = font; str = text;
}
Text(const Text& cSource);
Text& operator=(const Text& cSource);
~Text();
.
.
.
.
private:
FontPtr font;
char *str;
GLuint texture;
GLfloat pos_x, pos_y, width, height;
};
#endif
Text.cpp
Text::Text(const Text& cSource)
{
font = cSource.font;
texture = cSource.texture;
pos_x = cSource.pos_x;
pos_y = cSource.pos_y;
width = cSource.width;
height = cSource.height;
int sizeString = 35;
if (cSource.str)
{
str = new char[sizeString];
strncpy(str, cSource.str, sizeString);
}
else
{
str = 0;
}
}
Text& Text::operator=(const Text& cSource)
{
delete[] str;
font = cSource.font;
texture = cSource.texture;
pos_x = cSource.pos_x;
pos_y = cSource.pos_y;
width = cSource.width;
height = cSource.height;
int sizeString = 35;
if (cSource.str)
{
str = new char[sizeString];
strncpy(str, cSource.str, sizeString);
}
else
{
str = 0;
}
return *this;
}
Text::~Text()
{
delete[] str;
}
Font.h
#ifndef FONT_H
#define FONT_H
class Font
{
public:
Font(TTF_Font *font, SDL_Color color)
{
this->font = font; this->color = color;
}
~Font();
.
.
.
private:
TTF_Font *font;
SDL_Color color;
};
#endif
Font.cpp
Font::~Font()
{
TTF_CloseFont(font);
}
CGameApplication.cpp
.
.
.
.
void CGameApplication::initializeApplicationFonts()
{
TTF_Font* font;
SDL_Color color;
font = TTF_OpenFont("test.ttf", 15);
color.r = color.g = color.b = 255;
GApp->addFont(font, color);
Text *text = new Text(GApp->getFonts().at(0), " ");
text->setTexture( CTextM->textToGLTexture(GApp->getFonts().at(0), text) );
text->setPosX(20); text->setPosY(20);
GApp->addText(new Text(*text));
Text *textY = new Text(GApp->getFonts().at(0), " ");
textY->setTexture( CTextM->textToGLTexture(GApp->getFonts().at(0), textY) );
textY->setPosX(80); textY->setPosY(20);
GApp->addText(new Text(*textY));
delete textY; //-----> This line crashes the program with that error
}
.
.
.
GameApp.h
#ifndef GAMEAPP_H
#define GAMEAPP_H
class GameApp
{
public:
GameApp(){
}
//~GameApp();
void addFont(TTF_Font *font, SDL_Color color) {
vFonts.Push_back(FontPtr( new Font(font, color) ) ); }
vector<FontPtr> getFonts() { return vFonts; }
void addText(Text *text) {
vTexts.Push_back(new Text(*text));}
private:
SDL_Surface *gameMainSurface;
vector<Image*> vImages;
std::vector<FontPtr> vFonts;
vector<Text*> vTexts;
vector<Tile*> vTiles;
Map *currentMap;
};
#endif
したがって、問題はオブジェクトtextYを破棄すると、TTF_Fontへのポインタが破棄されることだと思います。しかし、オブジェクトテキストをベクターに追加するときは、コピーコンストラクターを使用するので、さまざまなポインターが問題なくコピーされるため、わかりません。
std::string
を使用してください。そのエラーは、何か、またはそのようなものを二重に削除したことを意味します。自分のメモリを管理していなかった場合には発生しない問題です。コードには、std::string
にはないメモリリークやその他のバグが散らばっています。
私が見ることができることから、エラーはText
のデフォルトのctorに関係しています。 char*
ポインターを受け取り、文字列用のスペースを割り当てますが、実際にtext
をstr
にコピーせず、ポインターを割り当てるだけです!あなたはそれをコピー・トラクターで修正します。今、この例を考えてみましょう:
class Foo{
public:
Foo(char* text){
str = text;
}
~Foo(){
delete str;
}
private:
char* str;
};
int main(){
Foo f("hi");
}
C++ 03(後方互換性のため...)では、このコードに示すように、リテラル文字列("hi"
)を非const char*
ポインターにバインドできます。 C++ 11はありがたいことにそれを修正し、これは実際にはもはやコンパイルされないはずです。文字列は.exeの読み取り専用セクションに配置されているため、delete
ableではないため、リテラル文字列の削除は明らかに機能しません。リテラル文字列からText
オブジェクトをインスタンス化した場合、これがエラーの原因だと思います。
これは、スタック上に作成されたchar[]
から作成した場合にも発生することに注意してください。
char text[] = "hi";
Foo f(text);
Foo
がスタックオブジェクトをdelete
しようとするからです。
これが発生する可能性のあるもう1つのケースは、オブジェクトを二重削除した場合です。
char* text = new char[3];
Foo f(text);
delete text;