プログラミングの課題、手伝ってください！！

質問

この俺が書いたプログラムを見てわかるとおり、今は１つのボールにしか、ファンクションがきいていない状況になってます。

これを３つのボール個々に大きさが代わったり、速さが変わったり、でてくるロケーションが変わったりするファンクションをきかせたいんです。

int width =600, height = 600;
float d = random(5,40);
float d1 = random(5,40);
float d2 = random(5,40);
float d3 = random(5,40);
float x = random(0,width);
float x1 = random(0,width);
float x2 = random(0,width);
float x3 = random(0,width);
float y =0;
float y1 =0;
float y2 =0;
float y3 =0;
float speed1 = random(1,5);
float speed2 = random(2,4);
float speed3 = random(1,9);

void setup()
{
size(width,height);
background(255);
smooth();
noStroke();

}

void draw()
{
background(255);

x1 = x1;
y1 = y1;
speed1 = speed1;
d1 = d1;
fill(255,0,0);
ellipse(x,y,d,d);
if(y<=height)
{
y= y + speed1;
d= d1;
}
else
{
speed1 = random(1,5);
d = random(5,40);
y = 0;
x = random(0,width);
}
x2 = x2;
y2 = y2;
speed2 = speed2;
d2 = d2;
fill(0,255,0);
ellipse(x,y,d,d);
if(y<=height)
{
y= y + speed2;
d = d2;
}
else
{
speed2 = random(2,4);
y = 0;
d = random(5,40);
x = random(0,width);

}
x3 = x3;
y3 = y3;
speed3 = speed3;
d3 = d3;
fill(0,0,255);
ellipse(x,y,d,d);
if(y<=height)
{
y= y + speed3;
d= d3;
}
else
{
speed3 = random(1,9);
y = 0;
d = random(5,40);
x = random(0,width);
}
}

よろしくお願いします。

回答No.1

sekibunx

元のソースコードを極力流用するなら...
普通はx1やらy1やらをそれぞれ配列で用意して，forで処理すると思います．

//x1 = x1;
//y1 = y1;
//speed1 = speed1;
//d1 = d1;
fill(255,0,0);
ellipse(x1,y1,d1,d1);
if(y1 <= height)
{
y1 = y1 + speed1;
//d = d1;
}
else
{
speed1 = random(1,5);
d1 = random(5,40);
y1 = 0;
x1 = random(0,width);
}

//x2 = x2;
//y2 = y2;
//speed2 = speed2;
//d2 = d2;
fill(0,255,0);
ellipse(x2,y2,d2,d2);
if(y2<=height)
{
y2 = y2 + speed2;
//d = d2;
}
else
{
speed2 = random(2,4);
y2 = 0;
d2 = random(5,40);
x2 = random(0,width);

}

//x3 = x3;
//y3 = y3;
//speed3 = speed3;
//d3 = d3;
fill(0,0,255);
ellipse(x3,y3,d3,d3);
if(y3<=height)
{
y3 = y3 + speed3;
//d = d3;
}
else
{
speed3 = random(1,9);
y3 = 0;
d3 = random(5,40);
x3 = random(0,width);
}

元のソースコードでうまく動かなかった原因ですが，
>x1 = x1;
>y1 = y1;
>speed1 = speed1;
>d1 = d1;

ここらへんですね．
x = x1;
y = y1;
speed = speed1;
d = d1;
の誤りです．
わざわざx，y，d，speedに代入する必要もないので，
使用しない様に直しました．

ほかにも
>y= y + speed1;

>d = random(5,40);
>y = 0;
>x = random(0,width);
あたりが間違っていますが...