予期しないNullPointerExceptionがコンストラクタから配列を割り当てます

Question

私はこれを理解することができないでしょうし、もし皆さんが助けてくれればそれは素晴らしいでしょう！ 私はすでに作成したオブジェクトをコンストラクタに渡して、すべての値を取得できるようにしようとしています。主に

public class Drops { 
    Ship ship; 
    Bullet[] bullet; 
    Aliens[] aliens; 
    Movement movement; 

    public Drops(Ship ship,Bullet[] bull,Aliens[] alienT) { 
    this.ship = ship; 
    for (int a = 0; a < MainGamePanel.maxAliens;a++) { 
     System.out.println(a +" " +alienT[a].x); // THIS WORKS, when nothing 
               // is being assigned, so the values 
               // are being passed correctly. 
     this.aliens[a] = alienT[a]; 
     for (int b = 0; b < MainGamePanel.maxShots;b++){ 
     this.bullet[b] = bull[b]; 
    } 
    } 
    } 
// that is is the class, and also where the error occurs 

私は船が配列雄牛とalienTないこの

drop = new Drops(ship, bull, alienT); 

ようコンストラクタに値を送信しています両方のアレイです。

ありがとうございます！彼らはより短い場合、ループ条件が考慮にalienTとbullの長さを取ることを確認して、また

public Drops(Ship ship,Bullet[] bull,Aliens[] alienT){ 
    this.ship = ship; 
    this.bullet = new Bullet[bull.length]; 
    this.aliens = new Aliens[alianT.length]; 
    // .. 

：

Bullet[] bullet; 
Aliens[] aliens; 

例えば：あなたは配列を初期化する必要が

Answer 1

MainGamePanel.maxAliensとMainGamePanel.maxShotsはArrayIndexOutOfBoundsExceptionになります。

Answer 2

bullパラメータとallienTパラメータをそれぞれCollection<Bullet>とCollection<AllienT>として定義できます。

次に、ArrayList、HashSet、またはあなたの好きなコレクションクラスを渡して、このメソッドを呼び出すことができます。

Answer 3

aliensとbulletメンバー配列のためNPEが得られましたnullです。適切な長さのコンストラクタでインスタンス化していることを確認してください。

public Drops(Ship ship,Bullet[] bull,Aliens[] alienT){ 
    this.ship = ship; 
    this.aliens = new Aliens[alienT.length]; 
    this.bullet = new Bullet[bull.length]; 
    // ... 
}