質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.51%

  • Java

    15785questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

インスタンスの消し方

解決済

回答 5

投稿

  • 評価
  • クリップ 0
  • VIEW 919

nakamura-

score 40

いまシューティングゲームみたいなものを作っています。
上から降ってくる桜の花びらをプレイヤーが弾で攻撃する感じです。
下のものがプログラムの一部になります。

コード
public class Part01 extends JApplet{

    /**
     * 初期メソッド
     */
    public void init(){
        setSize(1000, 700);                //画面サイズ設定

        MyPanel mp = new MyPanel();        //MyPanelインスタンス化
        getContentPane().add(mp);        //MyPanel追加
    }

}
コード
public class MyPanel extends JPanel implements Runnable{

    Player player;

    int gunCount = 0;
    Gun gun;
    ArrayList<Gun> gunList = new ArrayList<Gun>();

    Sakura sakura;
    ArrayList<Sakura> sakuraList = new ArrayList<Sakura>();

    Random rnd = new Random();

    /**
     * MyPanelのコンストラクタ
     */
    public MyPanel(){
        setBackground(Color.BLACK);        //背景色設定

        /*再描画ループスレッド*/
        Thread loopPaint = new Thread(this);
        loopPaint.start();


        /*画像読み込み、切り取り*/
        try{
            player = new Player("jiki.gif", 0, 0, 32, 32);
        }
        catch(IOException ioe){
            System.out.println("画像がありません");
        }

        CreateThread ct = new CreateThread();
        /*弾ループスレッド*/
        ct.loopGunThread(this);

        /*桜作成ループスレッド*/
        ct.loopCreateSakuraThread(this);

        /*桜ループスレッド*/
        ct.loopSakuraThread(this);
    }

    /**
     * キー入力処理
     */
    @Override
    protected void processKeyEvent(KeyEvent ke){
        /*何かキーが押された場合*/
        if(ke.getID() == KeyEvent.KEY_PRESSED){
            System.out.println("キー" + ke.getKeyCode() + "が押された");

            /*押されたキーが左右だった場合*/
            player.updata(ke.getKeyCode());

            /*押されたキーがENTERだった場合*/
            if(ke.getKeyCode() == KeyEvent.VK_ENTER){
                gun = new Gun(10000, 10000);        //Enter押されたらインスタンス生成
                gunList.add(gun);                    //配列に追加

                gunList.get(gunCount).startLine(player.m_x + 16, player.m_y);
                gunCount++;
            }
        }
    }

    /**
     * 描画メソッド
     */
    @Override
    protected void paintComponent(Graphics g){
        super.paintComponent(g);
        requestFocusInWindow();        //キー入力有効化

        player.draw(g);

        if(gun != null){                                //gunがインスタンス化されている場合(Enter押された場合)
            for(int i = 0; i < gunList.size(); i++){    //弾の数だけ繰り返し
                gunList.get(i).draw(g);
            }
        }

        if(sakura != null){
            for(int i = 0; i < sakuraList.size(); i++){
                sakuraList.get(i).draw(g);
            }
        }

        if(gun != null && sakura != null){

            for(int i = 0; i < gunList.size(); i++){
                for(int k = 0; k < sakuraList.size(); k++){

                    double x1 = (sakuraList.get(k).Screen_vData[0][0] + sakuraList.get(k).Screen_vData[1][0]) / 2;
                    double y1 = (sakuraList.get(k).Screen_vData[0][1] + sakuraList.get(k).Screen_vData[1][1]) / 2;

                    double x2 = gunList.get(i).m_x + 1;
                    double y2 = gunList.get(i).m_y + 1;

                    double length = Math.sqrt(Math.pow(x1 - x2, 2) + Math.pow(y1 - y2, 2));

                    if(length < sakuraList.get(k).scale){

                        System.out.println("-----衝突-----");
                    }
                }
            }
        }
    }

    /**
     * runメソッド(loopPaintスレッド)
     */
    public void run(){
        while(true){
            repaint();
        }
    }

}
コード
public class CreateThread{

    Random rnd = new Random();

    /**
     * 弾ループスレッド作成
     */
    public void loopGunThread(MyPanel mp){
        Thread loopGun = new Thread(new Runnable(){
            public void run(){
                while(true){
                    try{
                        Thread.sleep(50);
                    }
                    catch(InterruptedException e){
                    }

                    if(mp.gun != null){                                //gunがインスタンス化されている場合(Enter押された場合)
                        for(int i = 0; i < mp.gunList.size(); i++){    //弾の数だけ繰り返し
                            if(mp.gunList.get(i).m_y > -10){            //画面内の場合(画面外になると終了)
                                mp.gunList.get(i).updata();
                            }
                        }
                    }
                }
            }
        });
        loopGun.start();
    }

    /**
     * 桜作成ループスレッド作成
     */
    public void loopCreateSakuraThread(MyPanel mp){
        Thread createSakura = new Thread(new Runnable(){
            public void run(){

                while(true){
                    try{
                        Thread.sleep((long)(Math.random() * 1000));
                    }
                    catch(InterruptedException e){
                    }

                    createSakura(mp);

                    mp.sakuraList.add(mp.sakura);
                }
            }
        });
        createSakura.start();
    }

    /**
     * 桜ループスレッド作成
     */
    public void loopSakuraThread(MyPanel mp){
        Thread loopSakura = new Thread(new Runnable(){
            public void run(){

                while(true){
                    try{
                        Thread.sleep(20);
                    }
                    catch(InterruptedException e){
                    }

                    if(mp.sakura != null){
                        for(int i = 0; i < mp.sakuraList.size(); i++){
                            mp.sakuraList.get(i).updata();
                        }
                    }
                }
            }
        });
        loopSakura.start();
    }

    /**
     * 桜インスタンス作成
     */
    public void createSakura(MyPanel mp){
        mp.sakura = new Sakura(     1,  1, 0,
                                -1, -1, 0,
                                 1,  1, 0,
                                 1, -1, 0,
                                 (Math.random() * 1000) - 200, -100,
                                 rnd.nextInt(6) + 5);
    }

}


Sakura,Gun,Playerクラスは省略しています。
弾はEnterキーが押されるたびにインスタンスが生成され、gunListに格納されていきます
この弾を描画するときgunListに入っているだけ(弾の数だけ)繰り返して弾を全て描画しています。
if(ke.getKeyCode() == KeyEvent.VK_ENTER){
gun = new Gun(10000, 10000);//Enter押されたらインスタンス生成
gunList.add(gun);//配列に追加
}
このままだとEnterを押すたびにgunListが増えていき、描画するときにgunListの数だけ繰り返すので処理が重たくなると思います。
弾が画面外になったらgunListから取り除くような処理をしたいのですが、どのように記述すればいいのでしょうか?
よろしくお願いします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 5

checkベストアンサー

+1

「次のように書けば良いよ」と言いたいところですが、

for(int i = 0; i < mp.gunList.size(); i++){ 
    Gun gun = mp.gunList.get(i);
    if(gunが画面外に出た){
        mp.gunList.remove(i);
    }else{
        mp.gunList.get(i).updata();
    }
}

キー入力のイベントが発生するスレッド(イベントディスパッチスレッド)と、
loopGunThreadのスレッドが同時に動作しているため、
gunListへGunを入れたり削除したりするには排他制御が必要です。

排他制御をどうするかはちょっと簡単じゃないので、今回は割愛で失礼します。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/07/11 09:29

    回答ありがとうございます。
    排他制御というのを調べてみたいと思います。

    キャンセル

0

画面外に出たら消すという考え方ではなく、描写の前にその弾が画面内か否かを判定してから描写してみてはどうでしょう?
画面外のものは描写処理されなくなるので、少なくとも処理が軽くなることは期待できると思います。
ただし、Gunインスタンスが残る問題は解決しないので、定期的に画面外の弾を除去する(もちろん排他制御に気を遣って)必要があるかと思いますが。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/07/11 09:37

    回答ありがとうございます。
    そういう考え方もありますね!
    参考にさせていただきます。

    キャンセル

-1

gunListからremoveすれば良いのでは?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/07/11 09:20

    回答ありがとうございました!

    キャンセル

-1

こんなふうに、List#removeを使えばListから該当のインスタンスを除去できます。

if(mp.gunList.get(i).m_y > -10){            //画面内の場合(画面外になると終了)
    mp.gunList.get(i).updata();
}
else {
    // 画面外に出た時
    mp.gunList.remove(i);// gunListからi番目を削除
    i--; // iがずれるので補正
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/07/08 17:58

    排他制御の考慮が必要。

    キャンセル

  • 2016/07/11 09:21

    回答ありがとうございます!
    場合分けでremoveを使えばいいんですね!
    ありがとうございました。

    キャンセル

-1

同期化処理(排他制御)にはいくつか方法があると思います

  • java.util.Collections.synchronizedList(List)を使用する
//FQDN(完全修飾名)
java.util.List<Gun> syncArray = java.util.Collections.synchronizedList(new java.util.ArrayList<Gun>());
  • スレッドを一つにまとめて処理する
long c;
int pup;
boolean cgun;
boolean loop=true;
@Override
public void run(){
  while(loop){
    frame();
    Thread.sleep(16);
  }
}

private void frame(){
  c++;
  player.updata(ke.getKeyCode());
  cgun();
  //1フレームごとの処理
  //cにはloop回数が入ってます
}
private void cgun(){
  gun = new Gun(10000, 10000);        //Enter押されたらインスタンス生成
  gunList.add(gun);                    //配列に追加
  gunList.get(gunCount).startLine(player.m_x + 16, player.m_y);
  gunCount++;
}
 protected void processKeyEvent(KeyEvent ke){
        /*何かキーが押された場合*/
        if(ke.getID() == KeyEvent.KEY_PRESSED){
            System.out.println("キー" + ke.getKeyCode() + "が押された");

            /*押されたキーが左右だった場合*/
            pup=ke.getKeyCode();

            /*押されたキーがENTERだった場合*/
            if(ke.getKeyCode() == KeyEvent.VK_ENTER){
               cgun=true;
            }
        }
    }

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/07/10 18:19

    プログラミング方法によっては、synchronizedListで簡単に解決できるかもしれません。

    でも、イベントディスパッチのスレッドに処理をまとめるのは良くないと考えています。

    キャンセル

  • 2016/07/11 09:30

    回答ありがとうございます。
    synchronizedListというのがあるんですね。
    調べてみたいと思います!

    キャンセル

  • 2016/07/11 16:06 編集

    表現が悪すぎた
    イベントスレッドからはフラグを変更する処理だけをやらせてフラグの処理も含めて全体の処理は一つのスレッドでやらせるってことです
    (これをする場合最初のものは不要です)

    キャンセル

同じタグがついた質問を見る

  • Java

    15785questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。