Back to General and Gameplay Programming

the new operator

jLeslie · 2000-05-16T22:47:52

Ok, I''ve created a few classes to hold data about my 3D objects, yadda yadda yadda......But I''m having trouble allocating memory within one of my classes. Here''s the declaration for the main object class which contains pointers to various meshes and skeletons and whatnot that make up the object for my game. class OBJECTCLASS { public: char Name[80]; int Type; int NumAnimations; int NumMeshs; MESHCLASS* cMesh[10]; ANIMATIONCLASS* cAnimations[15]; SKELETONCLASS* cSkeleton; OBJECTCLASS(); ~OBJECTCLASS(); HRESULT LoadObject(char* strFileName); HRESULT AddMesh(MESHCLASS* cMeshObject); HRESULT AddAnimation(ANIMATIONCLASS* cAnimationObject); HRESULT AddSkeleton(SKELETONCLASS* cSkeletonObject); HRESULT UpDateMe(LPDIRECT3DDEVICE7 D3DDevice); private: }; The problem is with the LoadObject function, which is listed here.....The first time through the loop it seems to work fine, but when it finds another mesh to load in the file and enters...if (!strcmp(Chunk_Name, "PolH"))...for the second time, I get an access violation when I try to allocate memory with the new statement for cMesh[...]. HRESULT OBJECTCLASS::LoadObject(char* strFileName) { FILE *myfile = fopen(strFileName, "r"); char Chunk_Name[80]; while((fscanf(myfile, "%s" , Chunk_Name)) != EOF) { if(!strcmp(Chunk_Name, "Bone")) { BONECLASS* cBoneObject; cBoneObject = new BONECLASS; cSkeleton->AddBone(cBoneObject); } if (!strcmp(Chunk_Name, "Join")) { JOINTCLASS* cJoint; cJoint = new JOINTCLASS; BONECLASS* cBone; cBone = new BONECLASS; cJoint = LoadJoint(myfile); cBone = cSkeleton->FindBone(cJoint->Bone1); cBone->AddJoint(cJoint); cBone = cSkeleton->FindBone(cJoint->Bone2); cBone->AddJoint(cJoint); } if (!strcmp(Chunk_Name, "PolH")) { cMesh[NumMeshs] = new MESHCLASS; cMesh[NumMeshs]->LoadMesh(myfile); NumMeshs++; } } return S_OK; }

General and Gameplay Programming Programming

Started by jLeslie May 15, 2000 04:48 PM

11 comments, last by jLeslie 23 years, 11 months ago

Anonymous

May 16, 2000 04:24 PM

I haven''t read every line of your code, so maybe there''s some other problem, but these lines in your destructor look wrong:

for (int i = 0;i < NumChildren;i++)
{
if (Children[NumChildren])
delete Children[NumChildren];
}

I think it should be:

for (int i = 0;i < NumChildren;i++)
{
if (Children)
delete Children; } Peter