【至急！】processing 模様を繰り返し配置したい

Question

至急です！！！

****moyou1 と　moyou2 を繰り返し配置したいのですが、
__unexpected token: void __
と表示されてしまいます。他にも悪いところがあると思うので直して頂けると嬉しいです。

---

int i=20;
size(400,400);
colorMode(HSB, 360, 100, 100, 100);
noStroke();
for (int y = 0; y <= height; y+=100) {
for (int x = 0; x <= width; x+=100) { fill(random(360), 100, 100, 50);
if (x % 200 == 0) {
moyou1(x, y); }
else {
moyou2(x,y); }
moyou1(200,150);
moyou2(100,100);
colorMode(HSB,360,100,100,100);
}

void moyou1(float x, float y) {
pushMatrix();
translate(x, y);
for (int i=0;i<36; i++) {
rotate(radians(30));
fill(0, 10, 100, 100);
rect(60, 10, 20, 20);
fill(100, 100, 100, 20);
ellipse(20, 60, 120, 10);
fill(60, 100, 100, 50);
triangle(10, 10, 20, 20, 20, 30);
}
popMatrix();
}
void moyou2(float x, float y) {
pushMatrix();
translate(x, y); for (int i = 0; i < 20; i++) {
rotate(radians(18));
fill(i*18, 100,100,250);
if (i%2 == 0) {
rect(40, 20, 20, 20);
} else {
ellipse(50, 100, 10, 10); }
triangle(100,20,20,30,40,40); }
popMatrix();
}

Answer 1

Processingには、StaticモードとActiveモードというのがあります。
Staticモードは、変数宣言や関数呼び出しなどをベタに書いていくモード、
Activeモードは、関数を定義して、それらをsetup()やdraw()関数の定義の中から呼び出すモード。
この二つは混在できません。

それ以外にもありそうですが、ここはデバッグ代行の場ではないので、少なくともバグのあぶり出しぐらいは自分でやってください。

とりあえず、ProcessingのIDEであればCtrl+Tキーで整形してくれるので、活用することをオススメします。

Comments

[thkana]

以下一般論で、この質問がそうだ、という話ではありませんが。

私は「至急」案件かどうか気にしません。別に回答する義務があるわけでもなく、あなたが急いでいるかどうか全く私には関係のないことだからです。むしろ、自分のペースで回答できないならやめておこう、と考えることも。
また、自分の「至急」にかまけて質問が雑で必要な情報が欠けていたり、結局有用な回答を遠ざける質問になっていることもしばしばです。急いでいればこそ、緻密な質問を心がけたほうが早く解決するのではないでしょうか。

また、「至急」と言われると「お前の至急を押し付けるな」と反感を持つ回答者も少なくないようです。実際、至急案件の多くは「締切まで放置していた課題」の匂いがします。なんで質問者の怠慢のツケで回答者が急がされなきゃいけないの?ということですね。

まぁ、全般的にQ&Aサイトで「至急」として良い結果が得られる場面はあまり見かけません。

その辺考慮の上、それでも「至急」と主張するのがよいと考えるなら止めませんが。