arraylistが返ってきます。

Question

私は、arraylistにいくつかの小惑星オブジェクトを持つゲームを持っています。私のコードをきちんとしたものにするために、私はこのarraylistを別のクラス/メソッドに渡したいと思っています。私はそれが分かったと思った。しかし明らかにそうではなかった。ここに行く： My Level1クラスは、小惑星クラスのpopulateAsteroidメソッドを呼び出す初期化メソッド（init）から開始します。 Level1クラスには、小惑星のアライリストをループして更新する更新機能が含まれています。このすべてが正常に動作します：

public class Level1 { 
    private Asteroid populateAsteroid 
    private Collision collision; 

    public void init() { 

    populateAsteroid = new Asteroid(); 
    // create and populate an arraylist with asteroid objects 
    populateAsteroid.populateAsteroid(1); 
    collision = new Collision(); 
    } 

    public void update() { 
    // get the arraylist 
    ArrayList<Asteroid> asteroidList = populateAsteroid.getAsteroidList(); 

    for(int i = 0; i < asteroidList.size(); i++) {   
     Asteroid a = asteroidList.get(i); 
     a.update(i); 
    } 
    collision.checkCollision(); 
    } 
} 


public class Asteroid {  
    public ArrayList<Asteroid> asteroidList; 

    public void populateAsteroid(int level) { 
    // if the arraylist is null, as when the function is called by init, create the array 
    if (asteroidList == null){ 
     asteroidList = new ArrayList<Asteroid>(); 
    } 

    for (int i = 1 + asteroidList.size(); i <= (asteroidsAtEachLevel[level]); i++){ 
     Asteroid a = new Asteroid(); 
     asteroidList.add(a); 
    } 
    } 

    public ArrayList<Asteroid> getAsteroidList() { 
     return asteroidList; 
    } 
} 

しかし、トラブルがcheckCollision方法

public class Collision { 

    private Asteroid a; 

    public void checkCollision(){ 

    a = new Asteroid(); 
    // get the arraylist     
    ArrayList<Asteroid> asteroidList = a.getAsteroidList(); 

    if (asteroidList == null){ 
     System.out.println("null");} 
    } 
    ..."more code".... 

リストがnullが含まれているレベル1のクラスに更新機能によって呼び出された私の衝突クラス、、入って来、私がLevel1update関数でやっているのと同じようにarraylistを取得していると私は信じています。 レベル1クラスの呼び出しからarraylistがどのように見えないのか？助言がありますか？

Answer 1

Collisionクラスに新しいAsteroidというブランドを作成していますが、初期化されていません。代わりに、以前に初期化した小惑星のインスタンスをクラスに渡す必要があります。例えば

：checkCollision()で

public class Level1 { 

    private Asteroid populateAsteroid 
    private Collision collision; 

    public void init() { 

     populateAsteroid = new Asteroid(); 
     // create and populate an arraylist with asteroid objects 
     populateAsteroid.populateAsteroid(1); 
     collision = new Collision(populateAsteroid); //Pass in the reference to the asteroid. 

} 

と

public class Collision { 

private Asteroid a; 

public void checkCollision(Asteroid rock){ 

    a = rock; 
    // get the arraylist     
    ArrayList<Asteroid> asteroidList = a.getAsteroidList(); 

    if (asteroidList == null){ 
     System.out.println("null");} 
    } 
    ..."more code".... 

Answer 2

あなたはブランドの新しいAsteroidオブジェクトを作成しています。 Asteroidクラスの新しいインスタンスが作成されると、そのコンストラクタでListが作成されることはありません。このため、nullが表示されます。

checkCollision()が正しく機能するためには、先に入力したのと同じAsteroidオブジェクトをチェックする必要があります。

あなたは小惑星のリストに移入されていない、あなたのinit()方法

public void init() { 
    populateAsteroid = new Asteroid(); 
    populateAsteroid.populateAsteroid(1); 
    collision = new Collision(); 
    collision.checkCollision(populateAsteroid) 
} 

Answer 3

を変更すると

public void checkCollision(Asteroid a){    
    ArrayList<Asteroid> asteroidList = a.getAsteroidList(); 

    if (asteroidList == null){ 
     System.out.println("null");} 
    } 
    ..."more code".... 
} 

ようにそれにあなたが以前に作られたAsteroidオブジェクトを渡すことによって、これを達成することができます。あなたは新しいクラスをインスタンス化しますが、リストを作成するために必要なメソッドをしていない：あなたは小惑星クラスオブジェクトを作成すると

a = new Asteroid(); 
a.populateAsteroid(someInteger)    
ArrayList<Asteroid> asteroidList = a.getAsteroidList(); 

、あなただけのリストを宣言しています。あなたはそれに値を代入されていません。

public ArrayList<Asteroid> asteroidList; 

それは、ArrayListの新しいインスタンスが作成されpopulateAsteroid方法までではありません。それを宣言に残すと、nullが返されます。したがって、次のif条件は常にtrueを返します。

if (asteroidList == null){ 
    asteroidList = new ArrayList<Asteroid>(); 
}