配列を使って複数の線分を描くプログラムについて

質問

配列を使って複数の線分を描くプログラムについて、途中までは完成させたのですが、どうしてもわからない箇所があります。今日中に誰か回答をお願いします。

#include <stdlib.h>
#include <stdlib.h>
#include <stdio.h>
#include <math.h>
#include "inc2D.h"

int *pxs,*pys,*pxe,*pye;
int N;

void draw_line(int xs,int ys,int xe,int ye)
{

// [この中は完成させました]

}

void polygon(N,xs,ys,xe,ye)
int N;
int xs[],ys[],xe[],ye[];
{

/*
　ここがどうしてもわかりません。for文を使った繰り返しではないかと考えて、色々試してみましたが、
　エラーが出たり、表示されなかったりしてどうしても進みません。
*/
void draw_line();

}

void display(void)
{
void polygon();

glClear(GL_COLOR_BUFFER_BIT|GL_DEPTH_BUFFER_BIT);
polygon(N,pxs,pys,pxe,pye);
}

int main(int argc, char *argv[])
{
int i;
glutInit(&argc,argv);
glutInitDisplayMode(GLUT_RGB | GLUT_DEPTH);
glutCreateWindow("01kc999 Tsuyoshi SAITOH");
glutReshapeWindow(wsize,wsize);
glutDisplayFunc(display);
glutReshapeFunc(reshape);
glClear(GL_COLOR_BUFFER_BIT|GL_DEPTH_BUFFER_BIT);
glClearColor(0.0,0.3,0.3,0.0);

scanf("%d",&N);
pxs=(int *)malloc(sizeof(int)*N);
pys=(int *)malloc(sizeof(int)*N);
pxe=(int *)malloc(sizeof(int)*N);
pye=(int *)malloc(sizeof(int)*N);
for(i=0;i<N;i++)
{scanf("%d %d %d %d",&(pxs[i]),&(pys[i]),&(pxe[i]),&(pye[i]));}

glutMainLoop();
return 0;
}

回答No.1

kmee · ベストアンサー

その「いろいろ試した」って何をやって、どんなエラーになったんですか?

draw_lineが「(xs,ys)から(xe,ye)に線を引く」という仕様で、正しく期待通りに動いているなら

void polygon(int N,int xs[],int ys[],int xe[],ye[]) {
int i;
for(i=0;i<N;i++){
draw_line(xs[i],ys[i],xe[i],ye[i]);
}
}

でいいはずですよ。

他の部分が「正しい」ことは、確認できていますか?
例えば
・draw_line(0,0,200,100)等と指定して、(0,0)-(200,100)の線が引けますか?
・scanfの戻り値をチェックしていませんが、正しく期待通りに読み込めてますか?
printf("%d\n",N);
for(i=0;i<N;i++){
printf("%d %d %d %d\n",pxs[i],pys[i],pxe[i],pye[i]);
}
これをglutMainLoop();の前に入れたら、ちゃんと期待する結果になってますか?

あと、関係はないでしょうが、書き方が古いです。
> void polygon(N,xs,ys,xe,ye)
> int N;
> int xs[],ys[],xe[],ye[];
このような宣言、今はまず使いません。

> void draw_line();
> void polygon();
使用する関数を知らせるためのプロトタイプ宣言になりますが、引数が()というのはあまり使いません。
それと、この場合、関数定義がプロトタイプ宣言の前にある(polygonの定義中void draw_line();があるが、draw_lineの定義がその前にある)ので、ここでの記述は不要です。

お礼

回答ありがとうございます。試してみたところ

/usr/bin/ld: cannot find -lglu
collect2: ld はステータス 1 で終了しました

というエラーが出ます。
質問前から多発していたエラーなので、最初はlgluというものが
インストールされていないのではと思い、いろいろなプログラムを試してみたのですが解決せず、
プログラム側に何か問題があるのではと考えていました。
やはり何かインストールされていないのでしょうか。

補足

無事解決しました！　ありがとうございます！

回答No.2

akr

polygonの関数仕様が書かれてませんが、ここは回答者各人が好き勝手に決めて下さいってこと？

名前からして多角形を角関数を描画する関数としますね。引数は第1パラメータから順に、線分の数、始点X軸座標の配列、始点Y軸座標の配列、終点X軸の配列、終点Y軸の配列ってとこにします。

関数の中身は＃１様の回答と同じです。

気になることとしまして、先に書いた関数の仕様ですが、引数が線分の数と座標ですよね。この場合だと各々の線分がつながってるわけではないので、多角形を描画しないですよね。

引数を始点と終点ではなく、単に頂点の座標の配列にしたほうが、関数名と一致しそうなのですがどうでしょうか。

お礼

回答ありがとうございます！　やってみようと思います！