dados variáveis ??C ++ ser substituído
Pergunta
Já faz alguns anos desde que eu escrevi C / C ++, e agora eu estou enfrentando um problema que eu simplesmente não consigo resolver por conta própria.
Dado o seguinte struct:
struct InputData
{
float diameter;
float length;
int vertIndex;
struct InputData *parent;
vector<InputData*> children;
bool deadEnd;
InputData(float dia, float lngth)
{
diameter = dia;
length = lngth;
vertIndex = NULL;
parent = NULL;
deadEnd = false;
}
};
I começar por definir um número de nós, e sua relação pai / filho:
InputData i0 = InputData(3.0f, 3.0f);
InputData i1 = InputData(2.0f, 2.0f);
InputData i2 = InputData(1.0f, 1.0f);
InputData i3 = InputData(1.0f, 1.0f);
InputData i4 = InputData(1.0f, 1.0f);
InputData i5 = InputData(1.01f, 0.5f);
i0.children.push_back(&i1);
i1.children.push_back(&i2);
i2.children.push_back(&i3);
i3.children.push_back(&i4);
i4.children.push_back(&i5);
i1.parent = &i0;
i2.parent = &i1;
i3.parent = &i2;
i4.parent = &i3;
i5.parent = &i4;
Note que i5 como o único nó não tem filhos.
Eu, então, passar a fazer algum trabalho usando esses dados (chamando BuildMeshVertices (& i0, e vértices) do main ()), e acabam adicionando uma criança para i5:
void BuildMeshVertices(InputData* current, vector<SimpleVertex> *vertices)
{
//Do work
if(current->children.size() == 1)
{
BuildMeshVertices(current->children[0], vertices);
}
else if(current->children.size() == 0 && current->deadEnd == false)
{
InputData iDeadEnd = InputData(1.01f, 0.5f);
iDeadEnd.deadEnd = true;
iDeadEnd.parent = current;
current->children.push_back(&iDeadEnd);
BuildMeshVertices(&iDeadEnd, vertices);
}
}
Depois que tudo está bem. i0 tem um filho (i1), i1 tem um uma criança (i2), assim por diante e assim por diante e i5 agora tem um filho também.
Eu chamo outra função (BuildMeshIndices ()), e de repente algumas linhas para esta função (linha 63) os dados para a criança recém-adicionado para i5 está sendo substituído. i5 ainda aponta para o filho direito, mas os dados para esta criança é subitamente ilegível.
Aqui está uma imagem antes e depois (desculpe sobre o link, mas não foram permissão para usar tags IMG)
Eu não consigo descobrir por que isso acontece, mas eu tenho a sensação de que tem algo a ver com a minha gestão memória fraca?
Atualizar Também ele não tem que ser feito desta forma. Se, por exemplo mudando o vetor crianças a um vetor de valores é a forma preferida C ++, eu prefiro muito mais que isso. Eu tentei a comentar sobre as respostas, mas eu não tenho certeza que vocês estão vendo os comentários (segundo o FAQ você precisa de 50 reputação de deixar comentários)?
Abaixo está o código fonte completo (com tudo desnecessário retirados, mas o suficiente para reproduzir o erro):
#include "stdafx.h"
#include <vector>
using std::vector;
struct InputData
{
float diameter;
float length;
int vertIndex;
struct InputData *parent;
vector<InputData*> children;
bool deadEnd;
InputData(float dia, float lngth)
{
diameter = dia;
length = lngth;
vertIndex = NULL;
parent = NULL;
deadEnd = false;
}
};
//--------------------------------------------------------------------------------------
// Vertex types
//--------------------------------------------------------------------------------------
struct SimpleVertex
{
float Pos;
SimpleVertex(float Position)
{
Pos = Position;
}
};
void BuildMeshVertices(InputData* current, vector<SimpleVertex> *vertices)
{
current->vertIndex = vertices->size();
//Add vertices..
if(current->children.size() == 1)
{
BuildMeshVertices(current->children[0], vertices);
}
else if(current->children.size() == 0 && current->deadEnd == false)
{
InputData iDeadEnd = InputData(1.01f, 0.5f);
iDeadEnd.deadEnd = true;
iDeadEnd.parent = current;
current->children.push_back(&iDeadEnd);
BuildMeshVertices(&iDeadEnd, vertices);
}
}
void BuildMeshIndices(InputData* current, vector<unsigned long> *indices)
{
indices->push_back(current->vertIndex+2);
indices->push_back(current->vertIndex+0);
indices->push_back(current->vertIndex+1);
indices->push_back(current->vertIndex+3);
indices->push_back(current->vertIndex+0);
indices->push_back(current->vertIndex+2);
InputData *parent = current->parent;
unsigned long vOffset;
if(parent != NULL && parent->children.size() == 1)
{
vOffset = (unsigned long)current->vertIndex;
indices->push_back(vOffset+7);
indices->push_back(vOffset+5);
indices->push_back(vOffset+4);
indices->push_back(vOffset+6);
indices->push_back(vOffset+5);
indices->push_back(vOffset+7);
indices->push_back(vOffset+10);
indices->push_back(vOffset+8);
indices->push_back(vOffset+9);
indices->push_back(vOffset+11);
indices->push_back(vOffset+8);
indices->push_back(vOffset+10);
indices->push_back(vOffset+15);
indices->push_back(vOffset+13);
indices->push_back(vOffset+12);
indices->push_back(vOffset+14);
indices->push_back(vOffset+13);
indices->push_back(vOffset+15);
indices->push_back(vOffset+18);
indices->push_back(vOffset+16);
indices->push_back(vOffset+17);
indices->push_back(vOffset+19);
indices->push_back(vOffset+16);
indices->push_back(vOffset+18);
}
if(current->children.size() == 1 && current->deadEnd == false)
{
BuildMeshIndices(current->children[0], indices);
}
}
int _tmain(int argc, _TCHAR* argv[])
{
InputData i0 = InputData(3.0f, 3.0f);
InputData i1 = InputData(2.0f, 2.0f);
InputData i2 = InputData(1.0f, 1.0f);
InputData i3 = InputData(1.0f, 1.0f);
InputData i4 = InputData(1.0f, 1.0f);
InputData i5 = InputData(1.01f, 0.5f);
i0.children.push_back(&i1);
i1.children.push_back(&i2);
i2.children.push_back(&i3);
i3.children.push_back(&i4);
i4.children.push_back(&i5);
i1.parent = &i0;
i2.parent = &i1;
i3.parent = &i2;
i4.parent = &i3;
i5.parent = &i4;
// Create vertex buffer
vector<SimpleVertex> vertices;
BuildMeshVertices(&i0, &vertices);
// Create index buffer
vector<unsigned long> indices;
BuildMeshIndices(&i0, &indices);
return 0;
}
Solução
Alterar ponteiros crus para ponteiros inteligentes você vai começar o passeio de problemas de gerenciamento de memória.
Você não precisa copiar todo impulso para o seu projeto, cabeçalhos apenas necessários.
#include <vector>
#include <boost/shared_ptr.hpp>
#include <boost/weak_ptr.hpp>
struct InputData
{
float diameter;
float length;
unsigned long vertIndex;
boost::weak_ptr<InputData> parent;
std::vector< boost::shared_ptr<InputData> > children;
bool deadEnd;
InputData(float dia, float lngth, boost::weak_ptr<InputData> p = boost::weak_ptr<InputData>(), bool de = false)
: diameter(dia), length(lngth), vertIndex(0), parent(p), deadEnd(de) {}
};
struct SimpleVertex
{
float Pos;
SimpleVertex(float position) : Pos(position) {}
};
void BuildMeshVertices(boost::shared_ptr<InputData> current, std::vector<SimpleVertex>& vertices)
{
current->vertIndex = vertices.size();
//Add vertices..
if(current->children.size() == 1)
{
BuildMeshVertices(current->children[0], vertices);
}
else if(current->children.size() == 0 && current->deadEnd == false)
{
// this was a stack variable, so the pointer became invalid when going out of ambit.
boost::shared_ptr<InputData> iDeadEnd( new InputData(1.01f, 0.5f, current, true) );
current->children.push_back(iDeadEnd);
BuildMeshVertices(iDeadEnd, vertices);
}
}
void BuildMeshIndices(boost::shared_ptr<InputData> current, std::vector<unsigned long>& indices)
{
unsigned long vi = current->vertIndex;
unsigned long ioffset[] = { vi+2, vi, vi+1, vi+3, vi, vi+2};
indices.insert(indices.end(), ioffset, ioffset+6);
boost::shared_ptr<InputData> parent = current->parent.lock();
if (parent && parent->children.size() == 1)
{
unsigned long offs = current->vertIndex;
unsigned long voffset[] =
{ offs+7, offs+5, offs+4, offs+6, offs+5, offs+7,
offs+10, offs+8, offs+9, offs+11, offs+8, offs+10,
offs+15, offs+13, offs+12, offs+14, offs+13, offs+15,
offs+18, offs+16, offs+17, offs+19, offs+16, offs+18 };
indices.insert(indices.end(), voffset, voffset+24);
}
if(current->children.size() == 1 && current->deadEnd == false)
{
BuildMeshIndices(current->children[0], indices);
}
}
int main()
{
boost::shared_ptr<InputData> i0( new InputData(3.0f, 3.0f) );
boost::shared_ptr<InputData> i1( new InputData(2.0f, 2.0f) );
boost::shared_ptr<InputData> i2( new InputData(1.0f, 1.0f) );
boost::shared_ptr<InputData> i3( new InputData(1.0f, 1.0f) );
boost::shared_ptr<InputData> i4( new InputData(1.0f, 1.0f) );
boost::shared_ptr<InputData> i5( new InputData(1.01f, 0.5f) );
i0->children.push_back(i1);
i1->children.push_back(i2);
i2->children.push_back(i3);
i3->children.push_back(i4);
i4->children.push_back(i5);
i1->parent = i0;
i2->parent = i1;
i3->parent = i2;
i4->parent = i3;
i5->parent = i4;
// Create vertex buffer
std::vector<SimpleVertex> vertices;
BuildMeshVertices(i0, vertices);
// Create index buffer
std::vector<unsigned long> indices;
BuildMeshIndices(i0, indices);
return 0;
}
Pense que você ainda tem uma metade C, metade C ++ código sujo ... você deve escolher um idioma.
Outras dicas
Você está empurrando o ponteiro para um objeto pilha em seu vetor. Uma vez escopo folhas execução desse objeto pilha será destruído ea memória serão reutilizados, resultando em um valor falso. Tentar
InputData *iDeadEnd = new InputData(1.01f, 0.5f);
iDeadEnd->deadEnd = true;
iDeadEnd->parent = current;
current->children.push_back(iDeadEnd);
Em seguida, você vai ter que liberar essa memória no momento apropriado.
Você deve usar memória dinâmica de trabalhar com ponteiros. InputData ??em> será destruído quando você sai da função BuildMeshVertices , de modo que os dados serão garbaged ou você receberá uma exceção de memória.
Você deve fazer algo como
InputData * iDeadEnd = new InputData(1.01f, 0.5f);
em vez de
InputData iDeadEnd = InputData(1.01f, 0.5f);
Você está instanciação iDeadEnd na pilha, e tendo um ponteiro para o endereço de pilha! Quando as funções de terminar e as reversões de pilha, os dados para iDeadEnd vai garble.
InputData *iDeadEnd = new InputData(1.01f, 0.5f);
iDeadEnd->deadEnd = true;
iDeadEnd->parent = current;
current->children.push_back(iDeadEnd);
BuildMeshVertices(iDeadEnd, vertices);
O problema que você tem agora é que a memória para iDeadEnd precisa ser explicitamente desalocados quando você está feito com ele.
No momento em seus BuildMeshVertices função sai então iDeadEnd (filho de i5) é desconstruída porque você declarou na pilha e por sair da função de todo o quadro de pilha é invalidado e todo objeto desconstruída. Você quer quer alocar dinamicamente iDeadEnd ou radicalmente repensar como você define uma árvore. Você seria melhor ter cada porão struct um vetor de InputData (não InputData *) e, em seguida, configurá-los da seguinte forma:
InputData i0 = InputData(3.0f, 3.0f);
i0.children.push_back( InputData( 2.0f, 2.0f ) );
i0.children[0].children.push_back( InputData( 1.0f, 1.0f ) );
etc
Mesmo que está longe de ser ideal, por razões óbvias. Definindo uma árvore de elementos não é o mais divertido de coisas para fazer no entanto.