2012-05-03 91 views
0

这是一个简单的opengl/sdl程序。一般来说,一个巨大的deque,其中程序保存显示数据,例如一个三角形将被插入/从这样的出列中退出(GL_Begin .... Vertice 1 .... Vertice 2 ... Vertice 3 .. 。GL_End)C++奇怪的函数指针行为

的产品结构

enum Shapes{ 
    LINE, 
    POLYGON, 
    TRIANGLE 
}; 

struct Item{ 
    int id; 
    int type; 
    Shapes shape; 
    double x; 
    double y; 
    double z; 
}; 

...

void GEngine::CC2D(int id,double x,double y){ // Change Coordinates ... Move the Item 

    for(int i=0;i<Items.size();i++){ 
     Item* item = &Items[i];     // Pointer to the item INSIDE the deque because we need to edit it, we can not create a new instance of Item 

     if(item->id == id && item->type == 2){ 
      item->x += x; 
      item->y += y; 
     } 
    } 

    DrawScene(); 
} 

void GEngine::PollEvents(void (*Event)()){ 
    int mbpressed = 0; // Mouse button pressed flag 
    int mouse_xpre = 0; 
    int mouse_ypre = 0; 
    int move_id = 0; // TESTING 
    while(1){ 
     while(SDL_PollEvent(&event)){ 

      switch(event.type){ 
       case SDL_MOUSEMOTION:{ 
        if(mbpressed == 1){ 
         mouse_x = event.motion.x; // X2 
         mouse_y = event.motion.y; // Y2 
        // (*Event)(); 

        // CC2D( ( X2 - X1 ),( Y2 - Y1 ) 
         CC2D(move_id,(mouse_x-mouse_xpre),(mouse_y-mouse_ypre));  // The difference between the current and the previous mouse position is equal to the DX and DY which will be used to move the vertices. 
         mouse_xpre = mouse_x;  // X1 
         mouse_ypre = mouse_y;  // Y1 

         SDL_Delay(20); 

        } 
        break; 

现在的问题。如果我使用CC2D(move_id,(mouse_x-mouse_xpre),(mouse_y-mouse_ypre));它工作正常,但如果我将CC2D功能添加到事件中,则算法会失败,而不是使用鼠标移动的项目会远离屏幕。

换句话说。

case SDL_MOUSEMOTION:{ 
         if(mbpressed == 1){ 
          mouse_x = event.motion.x; // X2 
          mouse_y = event.motion.y; // Y2 
         // (*Event)(); 

         // CC2D( ( X2 - X1 ),( Y2 - Y1 ) 
         CC2D(move_id,(mouse_x-mouse_xpre),(mouse_y-mouse_ypre));  // The difference between the current and the previous mouse position is equal to the DX and DY which will be used to move the vertices. 
         mouse_xpre = mouse_x;  // X1 
         mouse_ypre = mouse_y;  // Y1 

         SDL_Delay(20); 

        } 
        break; 
} 

^工作得很好。

GEngine gEngine; 

void Event(){ 
      gEngine.CC2D(0,(gEngine.mouse_x-gEngine.mouse_xpre),(gEngine.mouse_y-gEngine.mouse_ypre)); 
    } 

int main(int argc, char *argv[]){ 

... code ... 

gEngine.PollEvents(&Event); 

return 0; // NEVER REACHED 
} 

case SDL_MOUSEMOTION:{ 
        if(mbpressed == 1){ 
         mouse_x = event.motion.x; // X2 
         mouse_y = event.motion.y; // Y2 
         (*Event)(); 

         mouse_xpre = mouse_x;  // X1 
         mouse_ypre = mouse_y;  // Y1 

         SDL_Delay(20); 

        } 
        break; 
} 

不...

为了进一步简化事情:

Case SDL_MOUSEMOTION: ...... code ..... CC2D(params) ........ 

工作正常,但

Event(){ CC2D(params) } // Main.cpp 
Case SDL_MOUSEMOTION: ...... code ..... (*Event)() ........ // GEngine.cpp 

未能如预期

+1

...... *墙上的文字* ......而不是用鼠标移动的项目移离屏幕。 :) –

+0

什么是“(* Event)();”,它的只是伪代码,如“...”,还是语法错误?记住一些cccompilers允许做一些werid的东西,而不会产生异常;-) – umlcat

+0

是的抱歉,错过了。 @umlcat否,这是通过函数指针调用函数的有效语法。 –

回答

3

您在这里声明局部变量里面PollEvents工作:

int mouse_xpre = 0; 
int mouse_ypre = 0; 

注意,它们被命名为mouse_xpremouse_ypre。然后在回调中你正在做

gEngine.CC2D(0,(gEngine.mouse_x-gEngine.mouse_xpre),(gEngine.mouse_y-gEngine.mouse_ypre)); 
//          ^^^^^^^^^^       ^^^^^^^^^^ 

访问成员变量。您需要删除局部变量的声明,以便它们不隐藏成员变量,并且您将在这两个地方使用成员变量。

+0

谢谢!这就是不使用构造函数的代价。出于某种奇怪的原因,我决定在已经在类中定义的整数之前添加* int *。 –