2013-11-01 50 views
1

我有这两种方法叫做一个又一个;变量不明原因变化

m_Fence.Initialise(7); 
m_Fence.CreateSquareFence();  

下面是他们的每个细节;

void CFence::Initialise(int size) 
{ 
    m_square_size = size; 
} 

void CFence::CreateSquareFence() 
{ 
    int l_Vector_Pos = 0; 

    //Set initial vector at origin and start of first fence 
    CVector3f l_Fence_Position = CVector3f(0.0f,0.0f,0.0f); 

    int l_side = 0; // Makes sure only 4 sides get created 
    std::string bools[4] = {"plus_z", "plus_x", "minus_z", "minus_x"}; 
     //Draw 1st fence on 0 x-axis 
     while(l_side < 4) 
      { 
       for(int i=0 ; i<=m_square_size ; i++) 
        {     
        //Find which direction it's going and set Fence position and add to vector 
        if(bools[l_side] == "plus_z" && i>0) 
         l_Fence_Position += CVector3f(0.0,0.0,beamLength); 
        else if(bools[l_side] == "plus_x" && i>0) 
         l_Fence_Position += CVector3f(beamLength,0.0,0.0); 
        else if(bools[l_side] == "minus_z" && i>0) 
         l_Fence_Position += CVector3f(0.0,0.0,-beamLength); 
        else if(bools[l_side] == "minus_x" && i>0) 
         l_Fence_Position += CVector3f(-beamLength,0.0,0.0); 

        fences[l_Vector_Pos].setPosition(l_Fence_Position); 
        l_Vector_Pos++; 

        //Increase this int to let it know what side we're creating 
        if(i == m_square_size) 
         { 
          l_side++; 
        } 
       } 
      } 
} 

我通过7到和它最初改变m_square_size变量7,但是当我调用CreateSquareFence方法它最终成为0,我挣扎明白。

+1

设置一个内存断点。 –

+0

你可以将'm_square_size'封装在getter方法中。 –

+1

“栅栏”是如何定义初始化的?你有可能在这里超越数组吗? – Angew

回答

1

谈到我的意见为答案

确保fences是足够大,你没有达到出界和王牌邻近mnemory的(更普遍,调用未定义行为)。


在旁注中,该代码看起来太复杂。你可以这样简化:

void CFence::CreateSquareFence() 
{ 
    int l_Vector_Pos = 0; 

    //Set initial vector at origin and start of first fence 
    CVector3f l_Fence_Position = CVector3f(0.0f, 0.0f, 0.0f); 

    CVector3f offsets[4] = { 
    CVector3f(0.0, 0.0, beamLength), 
    CVector3f(beamLength, 0.0, 0.0), 
    CVector3f(0.0, 0.0, -beamLength), 
    CVector3f(-beamLength, 0.0, 0.0) 
    }; 
    for (int l_side = 0; l_side < 4; ++l_side) { 
    for (int i = 0; i <= m_square_size; ++i) { 
     if (i>0) 
     l_Fence_Position += offsets[l_side]; 
     fences[l_Vector_Pos].setPosition(l_Fence_Position); 
     ++l_Vector_Pos; 
    } 
    } 
} 
+0

不错,谢谢你的收拾! – r0bb077