Java Array List/whileループカウントエラー

Question

私のwhileループは終了しましたが、私の間違いは一度しか実行されないという問題がありますか？

public class ear { 

    private ArrayList<Point> Master = new ArrayList<Point>(); 
    private ArrayList<Point> Shot = new ArrayList<Point>(); 

    public ear() { 
     addMaster(); 
     addShot(); 
     if (Shot.get(0).y == Master.get(0).y) { 
      while (Shot.get(0).x > Master.get(0).x) { 
       System.out.println("MasterY: " + Master.get(0).x); 
       System.out.println("ShotY: " + Shot.get(0).x); 
       moveShot(); 
      } 
     } 
    } 

    public void moveShot() { 
     Point p = new Point(); 
     for (int i = 0; i < Shot.size(); i++) { 
      Shot.get(i).x = p.x; 
      p.x -= 10; 
      p.y = 5; 
      Shot.set(i, p); 
     } 
    } 
} 

耳の目的は、iがショット値xを取得し、それの10を引く移動ショットでメインクラス で作成されます。

Answer 1

1）それは常に真であるとして、これは無意味です：あなたはShot.get(i).xに新しいPointを割り当てるよう

if (Shot.get(0).y == Shot.get(0).y) 

2）ここでPoint p = new Point(); Shot.get(i).x = p.x;は意味がありません。あなたが現在Shotを保存し、その値を変更するには、仲介値Pointを持っているしたい場合は、むしろ逆の作業を行う必要があります

p.x = Shot.get(i).x - 10; 
p.y = Shot.get(i).y + 5; 

し、現在のShotの新しい値としてpを設定します。

Shot.set(i, p); 

最後に

3）、xとyは、あなたが直接仲介Pointオブジェクトを使用せずに、それらを変更することができShotの公共の場であれば：

public void moveShot() { 
    for (int i = 0; i < Shot.size(); i++) { 
     Shot.get(i).x -= 10; 
     Shot.get(i).y += 5; 
    } 
} 

インスタンスの公開フィールドはお勧めできませんが、ゲームでは時々使用されます。あなたは（デフォルトでは）次に(0, 0) であることをPointを作成moveShot()方法で

Answer 2

は、あなたが0に彼らのxの値を設定し、あなたのShotsのすべてを反復処理し、xの値は、もはやあなたのより大きいではありませんMaster x値なので、ループは終了します。