配列に文字列を追加するには？

Question

この方法で何が問題になっていますか？単に曲の配列に曲を追加することになっています。ソングは重複することはできません。配列にすでに3曲がある場合は、4番目のソングを配列の3番目のインデックスに追加する必要があります。

import java.util.*; 

public class Player { 
    private String[] songs; 
    private int[] order; 
    private int numSongs; 
    private int numSongsInShuffle; 

    public Player() { 
     songs = null; 
     order = null; 
     numSongs = 0; 
     numSongsInShuffle = 0; 
    } 

    public void addSong(String name) { 
     if ((songs==null) || (numSongs==songs.length)) { 
      //expand 
      String [] temp; 
      temp = new String[2*numSongs]; 
      for (int i=0;i<songs.length;i++) 
       temp[i] = songs[i]; 
      songs = temp; 
     } 
     songs[numSongs] = name; 
     numSongs++; 
    } 

Answer 1

最初の項目に、songsがnullであるので、あなたのコードはここにNullPointerException

​​

を投げています。ヌルチェックを行ったとしても、numSongsは0になるためArrayIndexOutOfBoundsExceptionになるので、tempは空の配列になります。あなたがする必要があるのは、次のような初期化ロジックを分離することです：

public void addSong(String name) { 
    if (songs == null) { 
     // initialize array 
     songs = new String[1]; 
    } else if (numSongs == songs.length) { 
     //expand 
     songs = Arrays.copyOf(songs, 2*numSongs); 
    } 
    songs[numSongs] = name; 
    numSongs++; 
}