回答編集履歴

見直しキャンペーン中

2023/07/28 15:00

投稿

TN8001

スコア10166

answer CHANGED Viewed

@@ -1,244 +1,244 @@
-[Java - シューティングゲームで、当たり判定を実装したい｜teratail](https://teratail.com/questions/351596)
-から1週間だいぶ試行錯誤しておられるようですね。
-しかし残念ながら方向が違うような気がします。
-Processingは仕組み上クラス内からグローバル変数にアクセスし放題なのですが、そのせいでスパゲッティコードにもなりやすいです（ありがたい時もあるんですが^^;
-例えば`ENEMY`の`x`・`y`が外に出てしまって（`ex`・`ey`）、前回より退化してしまっています。
-> 敵や弾の速度が不安定になってしまっています。
-ちゃんとは追っていませんが、おそらく`ex`・`ey`・`sx`・`sy`のせいです。
-> boolean でhitflagを立てて、衝突範囲内に入った時にtrueを返すような
-敵が複数いるのですから、フラグも複数あるべきでは？
-ひとつでもできないことはないでしょうが、ややこしくなるしすでにクラスがあるんですからそっちに定義すればいいと思います。
-個人的に気になった点を変更しました。
-* 命名規則を標準的なものに
-  Javaではクラス名は通常パスカルケース（例:PascalCase）です。
-  大文字だけ（例:UPPER_SNAKE_CASE）は定数に使用します。
-  （まあ「個人で作っているものは好きにつけりゃいいじゃん」と思っていますが、標準のもの（`ArrayList`等）と混じると読みにくいので^^;
-* 同じような意味の`move`・`display`と、`update`・`draw`があるのは気持ち悪いので統一
-  ２つのプログラムを合体させた感じ？（いろいろ参考にされていると思いますが
-* つど`loadImage`してしまっているのは大変無駄
-  せっかくクラスを作ったのだから、メンバ変数に入れときましょう。
-* `N`や`n`はわかりにくいので、`length`や拡張for文に置き換える
-  特にグローバル変数は多いほど読みにくくなるので、使わずに済む場合は積極的に減らします。
-```Processing
-Player player;
-Enemy[] enemys = new Enemy[5];
-Star[] stars = new Star[100];
-ArrayList<Shot> shotList = new ArrayList<Shot>();
-void setup() {
-  size(300, 600);
-  frameRate(30);
-  rectMode(CENTER);
-  imageMode(CENTER);
-  noCursor();
-  player = new Player(loadImage("hikou.png"));
-  PImage png = loadImage("enemy.png");
-  for (int i = 0; i < enemys.length; i++) {
-    enemys[i] = new Enemy(png);
-  }
-  for (int i = 0; i < stars.length; i++) {
-    stars[i] = new Star();
-  }
-}
-void draw() {
-  background(0);
-  player.update();
-  for (Star star : stars) {
-    star.update();
-    star.draw();
-  }
-  boolean gameOver = false;
-  for (Enemy enemy : enemys) {
-    enemy.update();
-    // 弾に当たっているか確認のため全弾ループ
-    for (int i = shotList.size () - 1; i >= 0; i--) {
-      Shot shot = shotList.get(i);
-      if (enemy.isHit(shot.x, shot.y)) { // shotの座標と接触していれば...
-        enemy.isDead = true; // フラグで管理するならこう
-        //enemy.init(); // でも敵がいなくなってしまうので単に初期化でいい気が^^;
-        shotList.remove(i);
-        println("hit");
-      }
-    }
-    // ついでに自機との接触も
-    if (enemy.isHit(player.x, player.y)) { // playerの座標と接触していれば...
-      gameOver = true;
-    }
-    enemy.draw();
-  }
-  for (int i = shotList.size () - 1; i >= 0; i--) {
-    Shot shot = shotList.get(i);
-    shot.update();
-    if (shot.isAlive()) {
-      shot.draw();
-    } else {
-      shotList.remove(i);
-    }
-  }
-  player.draw();
-  if (gameOver) {
-    fill(255);
-    textSize(40);
-    textAlign(CENTER, CENTER);
-    text("GAME OVER", width / 2, height / 2);
-    noLoop();
-  }
-}
-void mousePressed() {
-  shotList.add(new Shot(player.x, player.y));
-}
-class Player {
-  PImage image;
-  float x;
-  float y;
-  Player(PImage _image) {
-    image = _image;
-  }
-  void update() {
-    x = mouseX;
-    y = mouseY;
-  }
-  void draw() {
-    image(image, x, y);
-  }
-}
-class Enemy {
-  PImage image;
-  float x;
-  float y;
-  float vy = 5;
-  float size = 40;
-  boolean isDead; // フラグを作るならココ
-  Enemy(PImage _image) {
-    image = _image;
-    init();
-    y = random(-2 * height, -height);
-  }
-  void init() {
-    isDead = false;
-    y = random(-20);
-    x = random(20, width - 20);
-  }
-  void draw() {
-    if (isDead) return;
-    image(image, x, y);
-  }
-  void update() {
-    if (isDead) return;
-    y += vy;
-    if (height + size < y) {
-      init();
-    }
-  }
-  // Shot#isAliveと同じような感じで指定座標（_x・_y）との接触判定
-  boolean isHit(float _x, float _y) {
-    if (isDead) return false;
-    float r = size / 2;
-    return x - r < _x && _x < x + r &&  y - r < _y && _y < y + r;
-  }
-}
-class Star {
-  float x;
-  float y;
-  float vy;
-  float len;
-  Star() {
-    init();
-    y = random(-height, height);
-  }
-  void init() {
-    y = random(-2 * height, -1);
-    x = random(0, width);
-    vy = random(10, 18);
-    len = random(15, 25);
-  }
-  void update() {
-    y += vy;
-    if (height < y + len) {
-      init();
-    }
-  }
-  void draw() {
-    stroke(200);
-    line(x, y, x, y + len);
-  }
-}
-class Shot {
-  float x;
-  float y;
-  float vy = -15;
-  float size = 6;
-  Shot(float _x, float _y) {
-    x = _x;
-    y = _y;
-  }
-  void update() {
-    y += vy;
-  }
-  void draw() {
-    stroke(0);
-    fill(255, 0, 0);
-    rect(x, y, size, size);
-  }
-  boolean isAlive() {
-    float r = size / 2;
-    return 0 <= x + r && x - r <= width && 0 <= y + r && y - r <= height;
-  }
-}
-```
----
-別にこれが正解というわけではありません。
+[Java - シューティングゲームで、当たり判定を実装したい｜teratail](https://teratail.com/questions/351596)
+から1週間だいぶ試行錯誤しておられるようですね。
+しかし残念ながら方向が違うような気がします。
+Processingは仕組み上クラス内からグローバル変数にアクセスし放題なのですが、そのせいでスパゲッティコードにもなりやすいです（ありがたい時もあるんですが^^;
+例えば`ENEMY`の`x`・`y`が外に出てしまって（`ex`・`ey`）、前回より退化してしまっています。
+> 敵や弾の速度が不安定になってしまっています。
+ちゃんとは追っていませんが、おそらく`ex`・`ey`・`sx`・`sy`のせいです。
+> boolean でhitflagを立てて、衝突範囲内に入った時にtrueを返すような
+敵が複数いるのですから、フラグも複数あるべきでは？
+ひとつでもできないことはないでしょうが、ややこしくなるしすでにクラスがあるんですからそっちに定義すればいいと思います。
+個人的に気になった点を変更しました。
+* 命名規則を標準的なものに
+  Javaではクラス名は通常パスカルケース（例:PascalCase）です。
+  大文字だけ（例:UPPER_SNAKE_CASE）は定数に使用します。
+  （まあ「個人で作っているものは好きにつけりゃいいじゃん」と思っていますが、標準のもの（`ArrayList`等）と混じると読みにくいので^^;
+* 同じような意味の`move`・`display`と、`update`・`draw`があるのは気持ち悪いので統一
+  ２つのプログラムを合体させた感じ？（いろいろ参考にされていると思いますが
+* つど`loadImage`してしまっているのは大変無駄
+  せっかくクラスを作ったのだから、メンバ変数に入れときましょう。
+* `N`や`n`はわかりにくいので、`length`や拡張for文に置き換える
+  特にグローバル変数は多いほど読みにくくなるので、使わずに済む場合は積極的に減らします。
+```Processing
+Player player;
+Enemy[] enemies = new Enemy[5];
+Star[] stars = new Star[100];
+ArrayList<Shot> shotList = new ArrayList<Shot>();
+void setup() {
+  size(300, 600);
+  frameRate(30);
+  rectMode(CENTER);
+  imageMode(CENTER);
+  noCursor();
+  player = new Player(loadImage("hikou.png"));
+  PImage png = loadImage("enemy.png");
+  for (int i = 0; i < enemies.length; i++) {
+    enemies[i] = new Enemy(png);
+  }
+  for (int i = 0; i < stars.length; i++) {
+    stars[i] = new Star();
+  }
+}
+void draw() {
+  background(0);
+  player.update();
+  for (Star star : stars) {
+    star.update();
+    star.draw();
+  }
+  boolean gameOver = false;
+  for (Enemy enemy : enemies) {
+    enemy.update();
+    // 弾に当たっているか確認のため全弾ループ
+    for (int i = shotList.size () - 1; i >= 0; i--) {
+      Shot shot = shotList.get(i);
+      if (enemy.isHit(shot.x, shot.y)) { // shotの座標と接触していれば...
+        enemy.isDead = true; // フラグで管理するならこう
+        //enemy.init(); // でも敵がいなくなってしまうので単に初期化でいい気が^^;
+        shotList.remove(i);
+        println("hit");
+      }
+    }
+    // ついでに自機との接触も
+    if (enemy.isHit(player.x, player.y)) { // playerの座標と接触していれば...
+      gameOver = true;
+    }
+    enemy.draw();
+  }
+  for (int i = shotList.size () - 1; i >= 0; i--) {
+    Shot shot = shotList.get(i);
+    shot.update();
+    if (shot.isAlive()) {
+      shot.draw();
+    } else {
+      shotList.remove(i);
+    }
+  }
+  player.draw();
+  if (gameOver) {
+    fill(255);
+    textSize(40);
+    textAlign(CENTER, CENTER);
+    text("GAME OVER", width / 2, height / 2);
+    noLoop();
+  }
+}
+void mousePressed() {
+  shotList.add(new Shot(player.x, player.y));
+}
+class Player {
+  PImage image;
+  float x;
+  float y;
+  Player(PImage _image) {
+    image = _image;
+  }
+  void update() {
+    x = mouseX;
+    y = mouseY;
+  }
+  void draw() {
+    image(image, x, y);
+  }
+}
+class Enemy {
+  PImage image;
+  float x;
+  float y;
+  float vy = 5;
+  float size = 40;
+  boolean isDead; // フラグを作るならココ
+  Enemy(PImage _image) {
+    image = _image;
+    init();
+    y = random(-2 * height, -height);
+  }
+  void init() {
+    isDead = false;
+    y = random(-20);
+    x = random(20, width - 20);
+  }
+  void draw() {
+    if (isDead) return;
+    image(image, x, y);
+  }
+  void update() {
+    if (isDead) return;
+    y += vy;
+    if (height + size < y) {
+      init();
+    }
+  }
+  // Shot#isAliveと同じような感じで指定座標（_x・_y）との接触判定
+  boolean isHit(float _x, float _y) {
+    if (isDead) return false;
+    float r = size / 2;
+    return x - r < _x && _x < x + r &&  y - r < _y && _y < y + r;
+  }
+}
+class Star {
+  float x;
+  float y;
+  float vy;
+  float len;
+  Star() {
+    init();
+    y = random(-height, height);
+  }
+  void init() {
+    y = random(-2 * height, -1);
+    x = random(0, width);
+    vy = random(10, 18);
+    len = random(15, 25);
+  }
+  void update() {
+    y += vy;
+    if (height < y + len) {
+      init();
+    }
+  }
+  void draw() {
+    stroke(200);
+    line(x, y, x, y + len);
+  }
+}
+class Shot {
+  float x;
+  float y;
+  float vy = -15;
+  float size = 6;
+  Shot(float _x, float _y) {
+    x = _x;
+    y = _y;
+  }
+  void update() {
+    y += vy;
+  }
+  void draw() {
+    stroke(0);
+    fill(255, 0, 0);
+    rect(x, y, size, size);
+  }
+  boolean isAlive() {
+    float r = size / 2;
+    return 0 <= x + r && x - r <= width && 0 <= y + r && y - r <= height;
+  }
+}
+```
+---
+別にこれが正解というわけではありません。
 10人いたら10通りのコードになるでしょうし、あまりいじらないようにしたので「ちょっとどうかな？」という部分もあります。