Reputation: 1
I'm trying to add a JComponent onto a Jpanel whenever I switch my selection for daysOfTheWeek
comboBox. However, it doesn't seem to work, but only works when I put it ouside of theactionPerformed(ActionEvent e)
method. What am I doing wrong? Here is my simplified code:
public class App extends JFrame {
public static final int WIDTH = 1900;
public static final int HEIGHT = 1000;
JPanel scheduleArea;
private String[] days = {"Monday", "Tuesday"};
public App() {
super("Schedule");
scheduleArea = new JPanel();
initializeGraphics();
}
public void initializeGraphics() {
setMinimumSize(new Dimension(WIDTH, HEIGHT));
getContentPane().setBackground(Color.LIGHT_GRAY);
getContentPane().setLayout(null);
createScheduleArea();
setVisible(true);
setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
}
public void createScheduleArea() {
JPanel schedulePanel = new JPanel();
schedulePanel.setBounds(850,40,990,870);
schedulePanel.setLayout(null);
scheduleArea.setBounds(25,105,940,740);
scheduleArea.setBackground(new java.awt.Color(197,218,221));
JComboBox daysOfTheWeek = new JComboBox(days);
daysOfTheWeek.setBounds(750,30,200,45);
schedulePanel.add(daysOfTheWeek);
daysOfTheWeek.addActionListener(new ActionListener() {
@Override
public void actionPerformed(ActionEvent e) {
String selectedDay = (String) daysOfTheWeek.getSelectedItem();
switch (selectedDay) {
case "Monday":
scheduleArea.add(new JLabel("Monday")); // JLabel not added
break;
case "Tuesday":
scheduleArea.add(new JLabel("Tuesday"));
break;
default:
System.out.println("Please select");
}
}
});
schedulePanel.add(scheduleArea);
add(schedulePanel);
}
}
Upvotes: 0
Views: 89
Reputation: 51553
Oracle has a helpful tutorial, Creating a GUI With Swing. Skip the Learning Swing with the NetBeans IDE section. Pay close attention to the Laying Out Components Within a Container section.
Generally, you design a Swing GUI so that you create all the Swing components once. Then, you update the values of the Swing Components.
Generally, you create a Swing GUI from the inside out. You define your Swing components and JPanels
and let the JFrame
size itself.
Here's a GUI I came up with, based on your GUI.
I use a JFrame
. The only time you should extend a JFrame
, or any Java class, is when you want to override one or more of the class methods.
I created two separate JPanels
, one for the JComboBox
and one for the JTextArea
. I added a JButton
to the combo box panel so you could select the same day more than once. I used a JTextArea
so I could define one Swing component and append the text.
Here's the complete runnable code.
import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JComboBox;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
import javax.swing.SwingUtilities;
public class SchedulingApplication implements Runnable {
public static void main(String[] args) {
SwingUtilities.invokeLater(new SchedulingApplication());
}
private JTextArea textArea;
@Override
public void run() {
JFrame frame = new JFrame("Schedule");
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.add(createComboBoxPanel(), BorderLayout.NORTH);
frame.add(createSchedulePanel(), BorderLayout.CENTER);
frame.pack();
frame.setLocationByPlatform(true);
frame.setVisible(true);
}
private JPanel createComboBoxPanel() {
JPanel panel = new JPanel(new FlowLayout(FlowLayout.TRAILING, 5, 5));
panel.setBorder(BorderFactory.createEmptyBorder(0, 5, 5, 5));
String[] days = { "Monday", "Tuesday" };
JComboBox<String> daysOfTheWeek = new JComboBox<>(days);
panel.add(daysOfTheWeek);
JButton button = new JButton("Select");
button.addActionListener(new ActionListener() {
@Override
public void actionPerformed(ActionEvent e) {
String selectedDay = (String) daysOfTheWeek.getSelectedItem();
switch (selectedDay) {
case "Monday":
case "Tuesday":
textArea.append(selectedDay);
textArea.append(System.lineSeparator());
break;
default:
System.out.println("Please select");
}
}
});
panel.add(button);
return panel;
}
private JPanel createSchedulePanel() {
JPanel panel = new JPanel(new BorderLayout());
panel.setBorder(BorderFactory.createEmptyBorder(0, 5, 5, 5));
textArea = new JTextArea(10, 40);
textArea.setBackground(new Color(197, 218, 221));
JScrollPane scrollPane = new JScrollPane(textArea);
panel.add(scrollPane);
return panel;
}
}
Upvotes: 1