Я хочу, чтобы значения в ArrayList показывались после нажатия кнопки

Question

Как только нажата кнопка, я получаю NullPointerException, где я ожидал, что значение в ArrayList будет напечатано на экране. Например, если я нажму кнопку, указывающую на «A», на экране появится «A», если я нажму кнопку «B», на экране будет напечатан «B».

public class Media extends JPanel { 

    //Declares our media player component 
    private JPanel video_pnl, control_pnl; 
    private JButton play_btn; 
    private JLabel loc_lbl; 
    private int increment; 
    ArrayList<String> file_location; 

    public Media(ArrayList<String> file_location) { 
     this.file_location = file_location; 
     increment = 0; 
     while (increment < file_location.size()) { 
      video_pnl = new JPanel(); 
      video_pnl.setLayout(new BoxLayout(video_pnl, BoxLayout.Y_AXIS)); 
      loc_lbl = new JLabel(); 
      loc_lbl.setText(file_location.get(increment)); 
      play_btn = new JButton("Play"); 
      control_pnl = new JPanel(); 
      control_pnl.setLayout(new FlowLayout(FlowLayout.CENTER)); 
      play_btn.setActionCommand("play"); 
      video_pnl.add(loc_lbl); 
      control_pnl.add(play_btn); 
      video_pnl.add(control_pnl, BorderLayout.SOUTH); 

      Handler handler = new Handler(); 
      play_btn.addActionListener(handler); 

      video_pnl.revalidate(); 
      add(video_pnl); 
      increment++; 
     } 
    } 

    private class Handler implements ActionListener { 

     @Override 
     public void actionPerformed(ActionEvent e) { 
      if (e.getActionCommand().equals("play")) { 
       play(); 
      } 
    //   if (e.getSource() == play_btn){ 
    //    play(); 
    //   } 
     } 
    } 

    public void play() { 
     for (int i = 0; i < file_location.size(); i++) { 
      System.out.println(file_location.get(i)); 
     } 
    } 

    public static void main(String[] args) { 
     //Declare and initialize local variables 
     ArrayList<String> file_location = new ArrayList<>(); 
     file_location.add("A"); 
     file_location.add("B"); 
     file_location.add("C"); 
     file_location.add("D"); 
     file_location.add("E"); 

     //creates instances of the VlcPlayer object, pass the mediaPath and invokes the method "run" 
     Media mediaplayer = new Media(file_location); 
     JFrame ourframe = new JFrame(); 
     ourframe.setContentPane(mediaplayer); 
     ourframe.setLayout(new GridLayout(5, 1)); 
     ourframe.setSize(300, 560); 
     ourframe.setVisible(true); 
     ourframe.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
    } 
} 

Answer 1

Добавить:

this.file_location = file_location; 

в конструкторе (в этой строке: государственные средства массовой информации (ArrayList file_location) {)

Answer 2

В конструкторе, вы обратитесь к аргументу file_location построить ваши панели , но вы никогда не храните массив в своей переменной класса. Так как он инициализирован null по умолчанию, вы получаете NPE, когда ссылаетесь на него в методе play().

Инициализируйте своего члена класса в своем конструкторе значением аргумента (this.file_location = file_location), это исправит вашу проблему.

На стороне примечания я предлагаю вам использовать имена аргументов, которые еще не используются для членов класса. Это становится легче при чтении кода, чтобы заметить, если вы ссылаетесь на аргумент или член, и будет предотвращать проблему, которую вы только что наблюдали.