定義した関数が動いてくれない

Question

質問の概要

現在MacOSで、純粋なC言語で、OpenGLを使ってルービックレース(スライドパズルに近いもの)を作っています。(不便ですが大学の課題の制約なので仕方ないです。)
結論から言うとコード内のmove関数のなかで呼び出すexchange関数が動いてくれません.
ゲームを作ること自体が初めてで、非常に汚いコードとなってしまったのですがどうかお付き合いお願いします.

トライしようとしていること

スライドパズルのパネルをクリックするとそのパネルを動かすようにしたいです.構造体Panelで作られたp[25]は空白も含めた25個のパネルを現し,メンバに色と位置(ColorとPosition)があります.
moveはクリックしたところのパネルを(動かせる方向は必然的に一つしかないので)動かす関数で,その働きを,二つのパネルを交換するexchange関数で細分化して実装しようとしています.

トライしようとしていること

僕の見立てでは,表示された画面で空白の列にある左側のパネルを押せばそのパネルと空白が交換されるはずなのですが,そうなりません.
押した位置に対し「右に空白があります」などを出力するようにしているので試した限りでは,move関数の定義にある複雑なif文や入子構造に問題はないと思っています.

//描画座標とマウス座標でy軸の向きが逆であることに注意!!
#include<stdio.h>
#include<stdlib.h>
#include<time.h>
#include <GLUT/glut.h>
typedef struct{
    double Color[3];
    int Position[2];
}Panel;
Panel p[25];
double white[3] ={1.0, 1.0, 1.0};
double yellow[3]={1.0, 1.0, 0.0};
double blue[3]  ={0.0, 0.0, 1.0};
double green[3] ={0.0, 1.0, 0.0};
double red[3]   ={1.0, 0.0, 0.0};
double orange[3]={1.0, 0.5, 0.0};
double black[3] ={0.0, 0.0, 0.0};
int position[25][2]=//パネルの左上の座標
{
    {200,200},{240,200},{280,200},{320,200},{360,200},
    {200,160},{240,160},{280,160},{320,160},{360,160},
    {200,120},{240,120},{280,120},{320,120},{360,120},
    {200, 80},{240, 80},{280, 80},{320, 80},{360, 80},
    {200, 40},{240, 40},{280, 40},{320, 40},{360, 40}
};
int WINDOW_WIDTH = 500;   // ウィンドウの横幅
int WINDOW_HEIGHT = 500;  // ウィンドウの高さ
int emp;
void move(int x,int y);
void exchange(Panel p1, Panel p2);
void init(void){
    glClearColor(0.0, 0.0, 0.0, 1.0);
}

void display() {
    int i;
    // ウィンドウ表示内容のクリア
    glClear(GL_COLOR_BUFFER_BIT);
    
    // 枠線
    glColor3d(1.0, 1.0, 1.0);
    glLineWidth(10.0);
    glBegin(GL_LINES);
    glVertex2d(200, 40);
    glVertex2d(200, 240);
    glEnd();
    glBegin(GL_LINES);
    glVertex2d(200, 240);
    glVertex2d(400, 240);
    glEnd();
    glBegin(GL_LINES);
    glVertex2d(400, 240);
    glVertex2d(400, 40);
    glEnd();
    glBegin(GL_LINES);
    glVertex2d(400, 40);
    glVertex2d(200, 40);
    glEnd();
    glLineWidth(1.0);
    
    //中央9×9の枠線
    glColor3d(1.0, 0.0, 1.0);
    glLineWidth(10.0);
    glBegin(GL_LINES);
    glVertex2d(240, 80);
    glVertex2d(240, 200);
    glEnd();
    glBegin(GL_LINES);
    glVertex2d(240, 200);
    glVertex2d(360, 200);
    glEnd();
    glBegin(GL_LINES);
    glVertex2d(360, 200);
    glVertex2d(360, 80);
    glEnd();
    glBegin(GL_LINES);
    glVertex2d(360, 80);
    glVertex2d(240, 80);
    glEnd();
    glLineWidth(1.0);
    
    for(i=0;i<25;i++){
        glColor3d(p[i].Color[0], p[i].Color[1], p[i].Color[2]);
        glRectd(p[i].Position[0], p[i].Position[1], p[i].Position[0]+39, p[i].Position[1]+39);
        //最初二つが左上,あと二つが右下.
    }
    glFlush();
}

void mouse(int button, int state, int x, int y) {
    //クッリクしたところを動かす
    if(button==GLUT_LEFT_BUTTON && state==GLUT_DOWN){
        move(x,y);
    }
}


void reshape(int width, int height) {
    // OpenGLウィンドウの描画範囲を設定
    // 下記は描画範囲が[0, width] x [0, height]となるように設定している
    glViewport(0, 0, width*2, height*2);
    glMatrixMode(GL_PROJECTION);
    glLoadIdentity();
    gluOrtho2D(0.0, (double)width, 0.0, (double)height);
    WINDOW_WIDTH = width;
    WINDOW_HEIGHT = height;
}

int main(int argc, char **argv) {
    time_t t;
    int i,j,tmp;
    char move;
    int panel[25]={1,1,1,1,2,2,2,2,3,3,3,3,4,4,4,4,5,5,5,5,6,6,6,6,0};
    int goal[24] ={1,1,1,1,2,2,2,2,3,3,3,3,4,4,4,4,5,5,5,5,6,6,6,6};
    //初期配置をランダムシャッフルで決める.
    for(i=24;i>0;i--){
        time(&t);
        srand(t);
        j=rand()%(i+1);
        tmp=panel[i];
        panel[i]=panel[j];
        panel[j]=tmp;
    }
    //↑(フィッシャーイエーツシャッフルの改良版であるダステンフェルドのアルゴリズムを使用.)
    
    //コマンドベースコードの初期配列とGUIの初期配置の対応づけ
   for(i=0;i<25;i++){
        if(panel[i]==1){
            for(j=0;j<3;j++){
                p[i].Color[j]=white[j];
            }
            for(j=0;j<2;j++){
                p[i].Position[j]=position[i][j];
            }
        }else
        if(panel[i]==2){
            for(j=0;j<3;j++){
                p[i].Color[j]=yellow[j];
            }
            for(j=0;j<2;j++){
                p[i].Position[j]=position[i][j];
            }
        }else
        if(panel[i]==3){
            for(j=0;j<3;j++){
                p[i].Color[j]=blue[j];
            }
            for(j=0;j<2;j++){
                p[i].Position[j]=position[i][j];
            }
        }else
        if(panel[i]==4){
            for(j=0;j<3;j++){
                p[i].Color[j]=green[j];
            }
            for(j=0;j<2;j++){
                p[i].Position[j]=position[i][j];
            }
        }else
        if(panel[i]==5){
            for(j=0;j<3;j++){
                p[i].Color[j]=red[j];
            }
            for(j=0;j<2;j++){
                p[i].Position[j]=position[i][j];
            }
        }else
        if(panel[i]==6){
            for(j=0;j<3;j++){
                p[i].Color[j]=orange[j];
            }
            for(j=0;j<2;j++){
                p[i].Position[j]=position[i][j];
            }
        }else
        if(panel[i]==0){
            emp=i;
            for(j=0;j<3;j++){
                p[i].Color[j]=black[j];
            }
            for(j=0;j<2;j++){
                p[i].Position[j]=position[i][j];
            }
        }
    }
    
    // GLUTの初期化
    glutInit(&argc, argv);
    // ウィンドウのサイズを設定
    glutInitWindowSize(WINDOW_WIDTH, WINDOW_HEIGHT);
    // ウィンドウの作成 (引数はウィンドウのタイトル)
    glutCreateWindow("RubicRace");
    // 描画に使う関数の登録
    glutDisplayFunc(display);
    // ウィンドウのサイズ変更時に呼ばれる関数の登録
    glutReshapeFunc(reshape);
    // マウス操作時に呼ばれる関数の登録
    glutMouseFunc(mouse);
    //初期化処理
    init();
    // 描画ループの開始
    glutMainLoop();
}
//------------

//関数定義
void move(int x, int y){
    if(x>=200 && x<=400 && y>=260 && y<=460/*200px四方かどうかの判断*/){
        
       if(((x>=p[emp].Position[0] && x<=p[emp].Position[0]+40) || (WINDOW_HEIGHT-y>=p[emp].Position[1] && WINDOW_HEIGHT-y<=p[emp].Position[1]+40)) && !(x>p[emp].Position[0] && x<p[emp].Position[0]+40 && WINDOW_HEIGHT-y>p[emp].Position[1] && WINDOW_HEIGHT-y<p[emp].Position[1]+40)){
           if(x<p[emp].Position[0]){//クッリクの右側に空きがある時(右へスライド)
               exchange(p[(5*(y-260)/40)+(x-200)/40], p[emp]);//実験的に空白と,押した位置パネルのみ交換するようにしている.
               printf("右に空白があります\n");
           }else
           if(x>p[emp].Position[0]+40){//左へ
               exchange(p[(5*(y-260)/40)+(x-200)/40], p[emp]);
               printf("左に空白があります\n");
           }else
           if(WINDOW_HEIGHT-y<p[emp].Position[1]){//上へ
　　　　　　　　　exchange(p[(5*(y-260)/40)+(x-200)/40], p[emp]);
               printf("上に空白があります\n");
           }else
           if(WINDOW_HEIGHT-y>p[emp].Position[1]+40){//下へ
　　　　　　　　　exchange(p[(5*(y-260)/40)+(x-200)/40], p[emp]);
               printf("下に空白があります\n");
           }
           //emp=(5*(y-260)/40)+(x-200)/40;//右辺はクリックをしたパネルのインデント
　　　　　　　//↑予定で行けばクリックしたところが新たな空白にするのですが,
                    //exchangeがうまく行われないのでコメントアウトしています.
       }
        
    }
}
void exchange(Panel p1, Panel p2){
    int i;
    double tmp;
    //色を交換
    for(i=0;i<3;i++){
        tmp=p1.Color[i];
        p1.Color[i]=p2.Color[i];
        p2.Color[i]=tmp;
    }
}

よろしくお願いいたします.m(_ _)m

次なる問題

maisumakunさん,回答ありがとうございます。おかげでexchange関数は動作してくれました。
しかしパネルをクリックした後ウィンドウのサイズを変えるなどしてreshape関数を呼び出さないと変化しません。クッリクと同時にパネルが移動するようにしたいのですがどうしたら良いでしょうか。よろしくお願いします。

Answer 1

C言語では、引数はコピーされます。

void exchange(Panel p1, Panel p2)の中でp1やp2を変更しても、それはコピーに対する操作となり、渡した側には何も影響しません。

void exchange(Panel* p1, Panel* p2)のようにポインタでやり取りする必要があります。

Comments

[takadaibuoyancy]

ありがとうございます！おかげでexchangeが動作しました。もしよろしければ「次なる問題」についても回答いただけると幸いです。

[maisumakun]

すみません、GLUT環境でのプログラミング経験がないのでよくわからないです。

[takadaibuoyancy]

そうですか。でも、ありがとうございました。

Answer 2

次なる問題について:
glutPostRedisplay()と言う再描画を行うコールバック関数をマウスアクションに組み込んだらできました。