Java シューティングゲームで複数個の弾を発射する方法

Question

引用テキスト### 前提・実現したいこと

Javaでシューティングゲームを作成しています。
マウスをクリックすると弾が発射されます。
敵のうち、一体だけは弾が３発当たらないと倒せなくなっています。

弾を５発ずつ発射できるようにしたいのですが、１発しか表示されません。

発生している問題・エラーメッセージ

エラーメッセージ

該当のソースコード

ソースコード

(中略)
   // 自機の弾の初期設定
		m=5;
		k=0;
		jiki_tama_x=new int[m];
		jiki_tama_y=new int[m];
		jiki_tama_flg=new int[m];
		for(int j=0;j<m;j++){
		    jiki_tama_x[j] = jiki_tama_y[j] = -100;
		    jiki_tama_flg[j] = 0;
		}
	
（中略）
	/* MouseListener に関連するメソッド */
	public void mouseClicked(MouseEvent e) {//マウスがクリックされたら自機の弾をチェック
	    
	   if (jiki_tama_flg[k]==0) {
		jiki_tama_x[k] = jiki_x + jiki_w/2;
		jiki_tama_y[k] = 400;
		jiki_tama_flg[k] = 1;
	
	    }
	   
	｝	 
	   （中略）
	

	/* ActionListener に関連するメソッド */
	public void actionPerformed(ActionEvent e) {
	    Dimension d = getSize();
	    if (e.getSource()==timer) {
		// 敵機を水平方向に動かす
		for (int i=0; i<n; i++) {
		    teki_x[i] += teki_u[i];
		    if (teki_x[i]<0 || teki_x[i]>(d.width-teki_w)) teki_u[i] = -teki_u[i];
		}
	

		// 自機の弾を動かす
		
		if (jiki_tama_flg[k]==1) {
		    jiki_tama_y[k] -= 10;
		  		   
		    for(int i=0;i<(n-1);i++){//全ての敵機(1回で倒せる)について判定していく
			if(teki_alive[i]==1&&
			   teki_x[i]<jiki_tama_x[k]&&teki_x[i]+teki_w>jiki_tama_x[k]&&
			   teki_y[i]<jiki_tama_y[k]&&teki_y[i]+teki_h>jiki_tama_y[k]){

			    teki_alive[i]=0;//敵の生存状態を破壊（０）にする
			    jiki_tama_flg[k] = 0;//次の弾が打てるように待機中（０）にする
			}
		    }
		    //３回で倒す敵の判定
		    if(teki_alive[12]==3||teki_alive[12]==2||teki_alive[12]==1&&
			   teki_x[12]<jiki_tama_x[k]&&teki_x[12]+teki_w>jiki_tama_x[k]&&
			   teki_y[12]<jiki_tama_y[k]&&teki_y[12]+teki_h>jiki_tama_y[k]){

			    teki_alive[12]-=1;//敵の生存状態を一つ減らす
			    jiki_tama_flg[k] = 0;//次の弾が打てるように待機中（０）にする
			 
		    }

		    k+=1;
		    
		    
		    if(k>=5){
			k=0;
		    }
		    if (jiki_tama_y[k]<0) jiki_tama_flg[k] = 0;
		}

		
		    （中略）
		
		
	}

	/*　画面描画に関連するメソッド */
	public void paintComponent(Graphics g) {
	    super.paintComponent(g);

	   （中略）	    
	  	      
	    // 自機の弾の描画
	    
	    if (jiki_tama_flg[k]==1) {
		g.setColor(Color.yellow);
		g.fillOval(jiki_tama_x[k], jiki_tama_y[k], 5, 10);
	    	    }
	    
	    （中略）
      
    }
}
}

試したこと

弾の描画の部分を

for(int l=0;l<m;l++){
g.setColor(Color.yellow);
g.fillOval(jiki_tama_x[l], jiki_tama_y[l], 5, 10);
}

としてみましたがうまく出来ませんでした。
複数の弾を描画する方法について教えていただきたいです。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[jimbe]

まさか、全てを含む 1 クラスで作っているのでしょうか。
だとしたら、まずクラス分けを覚えられた方が良いかと思います。
Jiki クラス、Teki クラス、Tama クラス等に分け、それぞれに x,y や w,h、alive, flag 等の属性を入れ、その上でそれぞれの動きや衝突判定のメソッドを加えていくと、もう少し見通し良くなると思います。

[URO]

ありがとうございます。クラス分けを実践してみます。

Answer 1

とりあえず自機の移動(A,Dキー)と弾発射だけしてみました。
参考に・・・なるかどうか分かりません。
敵機は入れるとまた大きくなりすぎるので、やめました。

package teratail_java.q370706;

import java.awt.*;
import java.awt.event.*;
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.Timer;

public class Q370706 extends JFrame {
  public static void main(String[] args) {
    new Q370706().setVisible(true);
  }
  Q370706() {
    super("Q370706");
    setDefaultCloseOperation(EXIT_ON_CLOSE);

    GamePanel panel = new GamePanel();
    add(panel);
    pack();

    panel.start();
  }
}

class GamePanel extends JPanel {
  public static final int WIDTH = 300;
  public static final int HEIGHT = 300;

  private Jiki jiki;
  private Controller controller;

  private List<Tama> tamaList = new ArrayList<Tama>();
  GamePanel() {
    super(null);
    setPreferredSize(new Dimension(WIDTH,HEIGHT));
    setMinimumSize(getPreferredSize());
    setFocusable(true);
    setBackground(Color.BLACK);

    jiki = new Jiki(this, (WIDTH-Jiki.WIDTH)/2, HEIGHT-50);

    controller = new Controller();
    addMouseListener(controller);
    addKeyListener(controller);
  }

  void start() {
    new Timer(100, new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        int dx = controller.getDeltaX();
        jiki.setDeltaX(dx);
        jiki.move();
        if(intersect(jiki)) { //外に出そうなら戻す
          jiki.setDeltaX(-dx);
          jiki.move();
        }
        if(controller.fire) {
          jiki.fire();
          controller.fire = false;
        }
        synchronized (tamaList) {
          for(Iterator<Tama> ite=tamaList.iterator(); ite.hasNext(); ) {
            Tama tama = ite.next();
            tama.move();
            if(!contains(tama)) ite.remove();
          }
        }
        repaint();
      }
    }).start();
  }
  //一部でもパネルの中
  boolean contains(GameObject obj) {
    return 0 <= obj.x+obj.w && obj.x < WIDTH &&
            0 <= obj.y+obj.h && obj.y < HEIGHT;
  }
  //一部(以上)パネルの外
  boolean intersect(GameObject obj) {
    return obj.x < 0 || WIDTH < obj.x+obj.w ||
            obj.y < 0 || HEIGHT < obj.y+obj.h;
  }

  int countTama(IFF iff) {
    int count = 0;
    for(Tama tama : tamaList) if(tama.iff == iff) count++;
    return count;
  }
  void add(Tama tama) {
    tamaList.add(tama);
  }
  void remove(Tama tama) {
    tamaList.remove(tama);
  }

  @Override
  public void paintComponent(Graphics g) {
    super.paintComponent(g);

    Graphics2D g2 = (Graphics2D)g;
    jiki.draw(g2);
    synchronized (tamaList) {
      for(Tama tama : tamaList) tama.draw(g2);
    }
  }

  class Controller extends MouseAdapter implements KeyListener {
    private static final int LEFT_KEY = KeyEvent.VK_A;
    private static final int RIGHT_KEY = KeyEvent.VK_D;
    private static final int DELTA = 10;

    private boolean left, right;
    private boolean fire;

    void setLeft(boolean left) { this.left = left; }
    void setRight(boolean right) { this.right = right; }
    void setFire(boolean fire) { this.fire = fire; }

    int getDeltaX() {
      return left == right ? 0 : right ? DELTA : -DELTA;
    }

    @Override
    public void mouseClicked(MouseEvent e) {
      setFire(true);
    }

    @Override
    public void keyTyped(KeyEvent e) {} //ignore
    @Override
    public void keyReleased(KeyEvent e) {
      if(e.getKeyCode() == LEFT_KEY) setLeft(false);
      if(e.getKeyCode() == RIGHT_KEY) setRight(false);
    }
    @Override
    public void keyPressed(KeyEvent e) {
      if(e.getKeyCode() == LEFT_KEY) setLeft(true);
      if(e.getKeyCode() == RIGHT_KEY) setRight(true);
    }
  }

  enum IFF {
    A, B;
  }

  abstract class GameObject {
    protected GamePanel world;
    protected int x, y; //左上座標
    protected int w, h; //幅, 高さ
    protected int dx, dy; //移動距離
    protected IFF iff; //敵味方識別

    GameObject(GamePanel world, IFF iff, int x, int y, int w, int h) {
      this.world = world;
      this.iff = iff;
      this.x = x;
      this.y = y;
      this.w = w;
      this.h = h;
    }

    void move() {
      x += dx;
      y += dy;
    }

    abstract void draw(Graphics2D g);
  }

  abstract class GameTarget extends GameObject {
    GameTarget(GamePanel world, IFF iff, int x, int y, int w, int h) {
      super(world, iff, x, y, w, h);
    }

    void fire() {
      Tama tama = createTama(world.countTama(iff));
      if(tama != null) world.add(tama);
    }

    abstract Tama createTama(int count);
  }

  class Jiki extends GameTarget {
    public static final int WIDTH = 20;
    public static final int HEIGHT = 20;
    private static final int TAMA_MAX = 5;

    private int nPoints = 3;
    private int[] xPoints = new int[nPoints];
    private int[] yPoints = new int[nPoints];

    Jiki(GamePanel world, int x, int y) {
      super(world, IFF.A, x, y, WIDTH, HEIGHT);
      setPolygon();
    }

    void setDeltaX(int dx) {
      this.dx = dx;
    }

    void setPolygon() {
      xPoints[0] = x+10;
      yPoints[0] = y;
      xPoints[1] = x;
      yPoints[1] = y+20;
      xPoints[2] = x+20;
      yPoints[2] = y+20;
    }

    @Override
    Tama createTama(int count) {
      if(count >= TAMA_MAX) return null;
      return new Tama(world, iff, x+(WIDTH-Tama.WIDTH)/2, y-Tama.HEIGHT, -10);
    }
    @Override
    void draw(Graphics2D g) {
      g.setColor(Color.GREEN);
      setPolygon();
      g.fillPolygon(xPoints, yPoints, nPoints);
    }
  }

  //class Teki extends GameTarget {
  //}

  class Tama extends GameObject {
    public static final int WIDTH = 5;
    public static final int HEIGHT = 10;

    Tama(GamePanel world, IFF iff, int x, int y, int dy) {
      super(world, iff, x, y, WIDTH, HEIGHT);
      this.dy = dy;
    }

    @Override
    void draw(Graphics2D g) {
      g.setColor(Color.YELLOW);
      g.fillOval(x, y, w, h);
    }
  }
}