Добавление в JList с помощью ActionListener и обновление кадра GUI после добавления

Question

Я работаю над проектом (да, это для школы, нет, я не хочу, чтобы кто-нибудь написал это для меня. У меня слишком много гордости!), Используя некоторые компоненты графического интерфейса Java.Это все еще в довольно грубой стадии, и одна вещь мешает мне закончить это.Я стараюсь не просить о помощи, если я действительно не нуждаюсь в ней, потому что обычно, когда я спрашиваю, это оказывается простой ошибкой, так что, если это так, не волнуйтесь обо мнеВо всяком случае, так далее к коду.Это групповой проект, поэтому некоторые мои комментарии адресованы моему партнеру.Я бы спросил их, но это 4 утра ... Во всяком случае, вот оно.Не уверен, почему это во всех этих отдельных коробках.Слушатель, с которым я связываюсь, - ActionPerformed, в нижней части.Заранее благодарю за любую помощь.

import javax.swing.*;
import javax.swing.event.ListSelectionEvent;
import javax.swing.event.ListSelectionListener;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;


public class GUI extends JFrame
{
  private JPanel panel;
  private JFrame frame;
  private JTextArea text;
  private MP3List list = new MP3List();
  private JList songList;
  private JScrollPane scrollList;
  private JMenuBar menuBar;
  private JMenu menu;
  private JMenuItem menuAdd;
  private String[] songs;
  private static String mp3msg = "Project 2: MP3 Tracker"; // Header for JOptionPane


  public GUI()
  {
    super("mp3");
    panel = new JPanel();
    createGUI();
    add(panel);
  }


  public void createGUI()
  {


//This creates the frame(createGUI)
    frame = new JFrame();

//Here, I made an array of the song titles and gave them to a JList
//for display. Do you think we should sort the songs?
    songs = new String[list.getSize()];
    for (int i = 0; i < list.getSize(); i++)
    {
      songs[i] = list.get(i).getSongTitle();
    }
    songList = new JList(songs);

//Set the selection mode to single as I want to fill in fields with info on a clicked song. More on
//that to come.
    songList.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
//Made a scroll bar(vertical and horizontal just in case)
    scrollList = new JScrollPane(songList, JScrollPane.VERTICAL_SCROLLBAR_ALWAYS,
        JScrollPane.HORIZONTAL_SCROLLBAR_ALWAYS);


    text = new JTextArea(30, 30);
    text.setText("This section will hold information about songs and somesuch.");


    menuBar = new JMenuBar();
    menu = new JMenu("File");
    menuAdd = new JMenuItem("Add Song");
    menuAdd.addActionListener(new menuListener());
    menu.add(menuAdd);
    menuBar.add(menu);


    frame.setLayout(new Border());

    songList.addListSelectionListener(new ListSelectionListener()
    {
      public void valueChanged(ListSelectionEvent e)
      {

        int selectedVar;
        selectedVar = songList.getSelectedIndex();
        text.setText(("Song Title: " + list.get(selectedVar).getSongTitle())
            + ("\nArtist: " + list.get(selectedVar).getArtistName())
            + ("\nPlayback Time: " + list.get(selectedVar).getPlayBackTime())
            + ("  ||  Cost: " + list.get(selectedVar).getDownloadCost())
            + ("  ||  Size: " + list.get(selectedVar).getFileSize()));
      }
    });

  }


  public class Border extends JFrame implements LayoutManager
  {
    private static final long serialVersionUID = 1L;
    private final int WINDOW_WIDTH = 400;
    private final int WINDOW_HEIGHT = 300;

    public Border()
    {
      super("MP3 Editor");
      setSize(WINDOW_WIDTH, WINDOW_HEIGHT);
      setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      add(scrollList, BorderLayout.CENTER);
      add(text, BorderLayout.SOUTH);
      setJMenuBar(menuBar);
      setVisible(true);
    }


    @Override
    public void addLayoutComponent(String name, Component comp)
    {
      // TODO Auto-generated method stub


    }

    @Override
    public void layoutContainer(Container parent)
    {
      // TODO Auto-generated method stub

    }

    @Override
    public Dimension minimumLayoutSize(Container parent)
    {
      // TODO Auto-generated method stub
      return null;
    }

    @Override
    public Dimension preferredLayoutSize(Container parent)
    {
      // TODO Auto-generated method stub
      return null;
    }

    @Override
    public void removeLayoutComponent(Component comp)
    {
      // TODO Auto-generated method stub

    }
  }

  public class menuListener extends JMenuItem implements ActionListener
  {

    /**
     *
     */
    private static final long serialVersionUID = 1L;

/*public menuListener()
{
  menuItem.addActionListener(this);
}*/

    public void actionPerformed(ActionEvent e)
    {
      MP3 aSong = getInfo();
      list.add(aSong);
      songs = new String[list.getSize()];
      for (int i = 0; i < list.getSize(); i++)
      {
        songs[i] = list.get(i).getSongTitle();
      }
      songList = new JList(songs);
      scrollList = new JScrollPane(songList, JScrollPane.VERTICAL_SCROLLBAR_ALWAYS,
          JScrollPane.HORIZONTAL_SCROLLBAR_ALWAYS);
      int index = 0;
      songList.setSelectedIndex(index);
      songList.ensureIndexIsVisible(index);
    }

  }

  /**
   * Input one set of MP3 file information and produce a report <br>
   * <p/>
   * <hr>
   * Date created: Sep 22, 2010 <br>
   * Date last modified: Sep 22, 2010 <br>
   * <p/>
   * <hr>
   */

  public static MP3 getInfo()
  {

    // Gather all information using JOptionPane
    String title = JOptionPane.showInputDialog(null,
        "Enter Title: ",
        mp3msg,
        JOptionPane.QUESTION_MESSAGE);
    String artist = JOptionPane.showInputDialog(null,
        "Enter Artist: ",
        mp3msg,
        JOptionPane.QUESTION_MESSAGE);
    int seconds = Integer.parseInt(JOptionPane.showInputDialog(null,
        "Enter playback time in seconds: ",
        mp3msg,
        JOptionPane.QUESTION_MESSAGE));
    double cost = Double.parseDouble(JOptionPane.showInputDialog(null,
        "Enter download cost: ",
        mp3msg,
        JOptionPane.QUESTION_MESSAGE));
    double size = Double.parseDouble(JOptionPane.showInputDialog(null,
        "Enter file size in megabytes (MB): ",
        mp3msg,
        JOptionPane.QUESTION_MESSAGE));
    MP3 asong = new MP3(title, artist, seconds, cost, size);
    return asong;
  }


}

Answer 1

Есть несколько небольших проблем с вашим кодом, но я думаю, что наиболее важной для вас является часть, где вы добавляете новый MP3 в слушатель действия (часть, с которой, как вы сказали, вы хотите помочь).Здесь вы создаете новый JList и перезаписываете предыдущий JList, и вы создаете для него новую панель прокрутки, но вы не добавляете новый JList в свой графический интерфейс, так что старыйодин остается (и поэтому вы никогда не увидите никаких изменений).

В идеале, вы должны держать один и тот же список повсюду и просто обновлять содержимое.Есть несколько способов сделать это, но один из самых простых - просто позвонить songList.setListData(songs).Или вместо создания массива строк создайте Vector, поскольку списки JList работают и с векторами.

Другие проблемы, на которые вы можете обратить внимание:

• private JFrame frame; - этоникогда не используется.
• Ваш класс Border является JFrame, но также используется как LayoutManager, что крайне запутанно и неправильно.Вы должны избавиться от этого.

Лично я бы порекомендовал вам переписать графический интерфейс с нуля.Вот простой пример с рабочим JList для изучения:

package examples;

import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.util.Vector;

import javax.swing.AbstractAction;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JList;
import javax.swing.JOptionPane;
import javax.swing.JScrollPane;

public class GUI extends JFrame {

    private final Vector<String&gt myVector = new Vector<String>();
    private final JList myList = new JList();

    public static void main(String... args) {
        new GUI().setVisible(true);
    }

    public GUI() {
        super("List");
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setLayout(new BorderLayout());
        add(new JButton(new AddItemAction()), BorderLayout.NORTH);
        add(new JScrollPane(myList), BorderLayout.CENTER);
        pack();
    }

    private class AddItemAction extends AbstractAction {
        public AddItemAction() {
            super("Add Item");
        }

        @Override
        public void actionPerformed(ActionEvent e) {
            String newItem = JOptionPane.showInputDialog("Add a new item:");

            if (newItem != null) {
                myVector.add(newItem);
                myList.setListData(myVector);
            }
        }
    }

}

Answer 2

Прочитайте JList API и перейдите по ссылке на раздел в учебном руководстве по Swing «Как использовать списки», в котором приведены рабочие примеры добавления значения в JList при нажатии кнопки.Сравните ваш код с рабочим кодом, чтобы увидеть, в чем разница.