processing ある状態になったら、その処理をはじめからやりなおすコードを知りたい

Question

前提

int i;

int x=0;

int y=0;

int count=20;
int z=4;
int u=8;

int timer;
float speed =2;

final int sec=20;

void playing() {

background(255);
int i=int(random(z, u));
text("score :"+score, 10, 100);
text("Time :"+ (20-(millis()-timer)/1000), 10, 40);

line(300, 450, 700, 450);

if(millis()-timer>=20*1000){
scene=4;
} else {
if (x<350)
{
x+=3;
} else {
x=350;
y+=i;
if (y<330) {
if (keyCode==LEFT) {
return ;
}
} else if (330<=y&&y<=370) {
if (keyCode==LEFT) {
y=350;
score++;
}else{return;}
}else{return;}
image(img, 450, y, 50, 50);
image(img, 500, y1, 50, 50);
image(img, 550, y2, 50, 50);

}
  fill(244); 
rect(x, 350, 300, 40);
rect(x, 410, 300, 40);
fill(255, 0, 0);
rect(x, 390, 300, 20);

}
}

returnと書かれているところに来たら、
void playingの時間とスコア以外の処理をまた一から始める処理をしたいのですが
どうしたらいいでしょうか？

あと、score++;があるところの処理に来ると、スコアがぶわあああっと増えてしまいます。
score++;の処理に来たら、スコアが1増えるだけでいいんですけど、、そこもお願いします。

Comments

[thkana]

イチから始める...とはどうなることか、なにがどうなればイチから始まるのか、それをちゃんと考えましたか?
まずそれを考えてください。

[thkana]

投稿の形式の問題として、コード部分をすべて
```
コード部分
```
となるよう独立した行の```で囲ってください。(始まりの方は``` Processing とするとなお可)
ヘルプに書いてあることだと思うんですけどねぇ。

Answer 1

returnと書かれているところに来たら、
void playingの時間とスコア以外の処理をまた一から始める処理をしたい

このゲームの理解があっているかわかりませんが、上から降ってくる何かをキャッチするような感じですかね？
早すぎても遅すぎてもダメでやり直しになる、と。
しかし真ん中のreturnはやり直しではないですね（目押ししにくくするため画像を消す？）

上と下のreturnの前に、x = y = 0;とでもすればいいんじゃないですかね。

あと、score++;があるところの処理に来ると、スコアがぶわあああっと増えてしまいます。
score++;の処理に来たら、スコアが1増えるだけでいいんですけど

そのifに毎フレーム入ってしまいますから、そりゃそうなりますよね。
yを370以上にするか、yを変えたくなければkeyCodeを変えるかですかね。

int x;
int y;
int z = 4;
int u = 8;
int timer;
int scene;
int score;
PImage img;

void setup() {
  size(700, 500);
  img = loadImage("https://www.gravatar.com/avatar/e633c3b40490dc5f87e3fd52b295fac9?d=identicon", "png");
}

void draw() {
  playing();
}

void playing() {
  background(255);

  text("score :" + score, 10, 100);
  text("Time :" + (20 - (millis() - timer) / 1000), 10, 40);

  line(300, 330, 700, 330); // デバッグ用
  line(300, 370, 700, 370); // デバッグ用
  line(300, 450, 700, 450);

  //if (20 * 1000 <= millis() - timer) {
  //  scene = 4;
  //  return;
  //}

  if (x < 350) {
    x += 3;
  } else {
    x = 350;
    y += int(random(z, u));

    if (y < 330 && keyPressed && keyCode == LEFT) { // 早すぎ
      x = y = 0;
      return;
    }
    if (370 < y) { // 遅すぎ
      x = y = 0;
      return;
    }

    if (330 <= y) {
      if (keyPressed && keyCode == LEFT) { // キャッチ
        keyCode = 0;
        y = 350;
        score++;
      } else {
        return; // 目押しよけ？ ちらつくしコレいりますかね？
      }
    }

    line(300, y, 700, y); // デバッグ用
    image(img, 450, y, 50, 50);
    //image(img, 500, y1, 50, 50);
    //image(img, 550, y2, 50, 50);
  }

  fill(244);
  rect(x, 350, 300, 100);
  fill(255, 0, 0);
  rect(x, 390, 300, 20);
}

ifのネストがややこしすぎます。早期リターン等で平易にすべきです（意味を理解するのに時間がかかりました^^;

Answer 2

ある状態になったら、その処理をはじめからやりなおすコードを知りたい

やりなおす、と考えるとややこしくなりますんで、
ここは考え方を変え、

処理をループさせておいて、やり直しさせたくないときにそのループから抜けるようにする、
と考えるとコードにしやすいかと思います

Answer 3

まずは後半だけ。

score++;の処理に来たら、スコアが1増えるだけ

その通りに動いていますよ。

score++;があるところの処理に来ると、スコアがぶわあああっと増えてしまいます

score++では1しか増加しなくても、score++;があるところに何度も来ているから、です。
現象を正しく捉えないと、正しい対策を立てられません。

  if (330<=y&&y<=370) {
    if (keyCode==LEFT) {
      y=350;
      score++;

と書いて、yがこの範囲で、「今押されたのが左矢印キーだったら」となって欲しいと考えたのかと思いますが、プログラムに書かれている条件は「最後に押されたのが左矢印キーだったら」です。
つまり、一度左矢印キーを押すと、(y=350;とも書かれているし)その条件はず～っと成立したままになります。
プログラムはあなたが考えたように動くのではなく、ソースコードに書いてあるとおりに動きます。

押されたというのは、直前まで押されていなくて、いまは押されているということ。
押されているというのは、履歴は関係なく、とにかく現時点において押されているということ。
「押された」ことを検出したいのなら、「直前までキーが押されていなかった」あるいは「直前まで押されていたのが左矢印キーでなかった」ことと「いま押されているのが左矢印であること」ことをあわせて判別しなければいけません。