すべてのオブジェクトが含まれているプラ​​イベート静的リスト

Question

私は、多くのボールが走り回っていて、バウンスしているアニメーションプログラムを書いています。

ボールの動作を表すBallクラスを作成しました。

別のボールでボールのコリシンを実装すると、
他のすべてのボールをチェックする必要があります。
私はこのクラスを作った。

public class Ball{ 

    private static final List<Ball> allBalls; 
    static{ 
     allBalls = new ArrayList<>(); 
    } 

    private Ball(){} 

    public static Ball getNewBall(){ 
     Ball ball = new Ball(); 
     allBalls.add(ball); 
     return ball; 
    } 

    public void collision(){ 
     for(Ball b : allBalls){ 
      //check whether b is colliding with me 
      //and if colliding, change speed of me and b. 
     } 
    } 

} 

このようなデザイン（プライベート静的リストのすべてのオブジェクトを保持する）は良いか悪いですか？

Answer 1

あなたが持っているものは、そのものを扱うBallManagerクラスです。

public class BallManager { 
    private static BallManager instance = new BallManager(); 
    private BallManager(){} 
    public static BallManager getInstance() { 
     return instance; 
    } 
    public List<Ball> ballsInPlay = new ArrayList<>(); 
    public void createBall(int x, int y) {} 
    public void checkCollisions() { 
     // loop ball list and check collisions 
     // perform cleanup based on collisions 
    } 
    private void ballCleanup(){} 
} 

public class Ball{ 
    public Ball(){} 
    public void collision(Ball other){} 
} 

注：静的クラスからシングルトンに変更されました。また、checkCollisionsメソッドの後に（）を追加してコンパイルエラーを修正しました。

Answer 2

enumを使用してSingleton BallManagerを作成します。

enum singletonを使用するのが最適です。

前の回答を改善してください。

public enum BallManager { 
     INSTANCE; 
     public List<Ball> ballsInPlay = new ArrayList<>(); 
     public void createBall(int x, int y) {} 
     public void checkCollisions() { 
      // loop ball list and check collisions 
      // perform cleanup based on collisions 
     } 
     private void ballCleanup(){} 
} 

public class Ball{ 
    public Ball(){} 
    public void collision(Ball other){} 
}