Tabbed Pane help

Question

Thanks everyone for the help in the past. I am on to a new section of the program that i am writing and after searching the web and these forums for several hours i am still at a loss. I have a jtabbedpane that i want to add and remove tabs when the user clicks a button. I have the buttons working and can add tabs and delete them at will. My issue is that when i add a new tab, the previous tabs lose thier content. When i create the new tabs i add them to an ArrayList so that i can get at the information later but can only get the newest tab to display the information. If i make a simple test that adds a new tab with a textbox, or a label things work fine. All the tabs have new content in them but when i try to add a panel that is more than a single element the early tabs lose everything. Here is my sample code:
import java.awt.*;
import java.awt.event.ActionEvent;
import java.util.ArrayList;
import javax.swing.*;
import javax.swing.event.ChangeEvent;
import javax.swing.event.ChangeListener;
import com.jgoodies.forms.factories.*;
import com.jgoodies.forms.layout.*;
import info.clearthought.layout.*;
public class TryMain {
     public static void main(String[] args) {
          RunMe();
     private static void RunMe() {
          TryMain Test = new TryMain();
     private TryMain() {
          MainWindow = new JFrame();
          mainWindowToolBar = new JToolBar();
          newWeekButton = new JButton();
          deleteWeekButton = new JButton();
          mainTabs = new JTabbedPane();
          weekTabs = new JPanel();
          sortByLabel = new JLabel();
          sortByChooser = new JComboBox();
          playerNameLabel = new JLabel();
          averageScoreLabel = new JLabel();
          averageRankLabel = new JLabel();
          playerName1 = new JLabel();
          playerAvgRank1 = new JLabel();
          playerAvgScrore1 = new JLabel();
          CellConstraints cc = new CellConstraints();
          TabsList = new ArrayList();
          WeekNum = 0;
          //======== MainWindow ========
               MainWindow.setDefaultCloseOperation(JFrame.HIDE_ON_CLOSE);
               MainWindow.setTitle("test");
               MainWindow.setSize(700, 575);
               MainWindow.setLocation(100, 100);
               MainWindow.setVisible(true);
               mainTabs.setTabLayoutPolicy(JTabbedPane.WRAP_TAB_LAYOUT);
               Container MainWindowContentPane = MainWindow.getContentPane();
               MainWindowContentPane.setLayout(new FormLayout(
                    ColumnSpec.decodeSpecs("default:grow"),
                    new RowSpec[] {
                         FormFactory.DEFAULT_ROWSPEC,
                         FormFactory.LINE_GAP_ROWSPEC,
                         new RowSpec(RowSpec.FILL, Sizes.DEFAULT, FormSpec.DEFAULT_GROW)
               //======== mainWindowToolBar ========
                    mainWindowToolBar.setFloatable(false);
                    mainWindowToolBar.setOpaque(false);
                    //---- newWeekButton ----
                    Action newTabAction = new AbstractAction("New Week") {
                    public void actionPerformed(ActionEvent evt) {
                         mainTabs.addTab("Week-"+(WeekNum+1), new JScrollPane(addTabbedPaneTab(WeekNum)));
                           mainTabs.setSelectedIndex(WeekNum);
                           WeekNum = WeekNum + 1;
                    newWeekButton.setAction(newTabAction);
                    mainWindowToolBar.add(newWeekButton);
                    //---- deleteWeekButton ----
                    Action RemoveTabAction = new AbstractAction("Delete Week") {
                    public void actionPerformed(ActionEvent evt) {
                         if (WeekNum != 0) {
                              WeekNum = (mainTabs.getTabCount() - 1);     
                             mainTabs.removeTabAt(mainTabs.getSelectedIndex());
                             TabsList.remove(mainTabs.getSelectedIndex());
                    deleteWeekButton.setAction(RemoveTabAction);
                    mainWindowToolBar.add(deleteWeekButton);
               MainWindowContentPane.add(mainWindowToolBar, cc.xy(1, 1));
               //======== mainTabs ========
               for (int i = 0; i < 4; i++) {
                    mainTabs.addTab("Week-"+(WeekNum+1), new JScrollPane(addTabbedPaneTab(WeekNum)));
                    mainTabs.setSelectedIndex(WeekNum);
                    WeekNum = WeekNum + 1;
               MainWindowContentPane.add(mainTabs, cc.xy(1, 3));
               mainTabs.addChangeListener(new ChangeListener() {
                public void stateChanged(ChangeEvent evt) {     
                     setFocus();
     private static JFrame MainWindow;
     private static JToolBar mainWindowToolBar;
     private static JButton newWeekButton;
     private static JButton deleteWeekButton;
     private static JTabbedPane mainTabs;
     private static JPanel weekTabs;
     private static JLabel sortByLabel;
     private static JComboBox sortByChooser;
     private static JLabel playerNameLabel;
     private static JLabel averageScoreLabel;
     private static JLabel averageRankLabel;
     private static JLabel playerName1;
     private static JLabel playerAvgRank1;
     private static JLabel playerAvgScrore1;
     private static ArrayList TabsList;
     private static int WeekNum;
     public JPanel addTabbedPaneTab(int wnum) {
          TabsList.add(new JPanel());
          ((JPanel)TabsList.get(wnum)).setLayout(new TableLayout(new double[][] {
                    {TableLayout.PREFERRED, TableLayout.PREFERRED, TableLayout.PREFERRED, TableLayout.PREFERRED, TableLayout.PREFERRED, TableLayout.PREFERRED, TableLayout.PREFERRED, TableLayout.PREFERRED},
                    {TableLayout.PREFERRED, TableLayout.PREFERRED, TableLayout.PREFERRED, TableLayout.PREFERRED, TableLayout.PREFERRED}}));
               ((TableLayout)((JPanel)TabsList.get(wnum)).getLayout()).setHGap(5);
               ((TableLayout)((JPanel)TabsList.get(wnum)).getLayout()).setVGap(5);
               //---- sortByLabel ----
               sortByLabel.setText("Sort By:");
               ((JPanel)TabsList.get(wnum)).add(sortByLabel, new TableLayoutConstraints(0, 0, 0, 0, TableLayoutConstraints.FULL, TableLayoutConstraints.FULL));
               //---- sortByChooser ----
               sortByChooser.setModel(new DefaultComboBoxModel(new String[] {
                    "Name",
                    "Average Rank",
               sortByChooser.setOpaque(false);
               ((JPanel)TabsList.get(wnum)).add(sortByChooser, new TableLayoutConstraints(1, 0, 2, 0, TableLayoutConstraints.FULL, TableLayoutConstraints.FULL));
               //---- playerNameLabel ----
               playerNameLabel.setText("Name");
               ((JPanel)TabsList.get(wnum)).add(playerNameLabel, new TableLayoutConstraints(1, 1, 1, 1, TableLayoutConstraints.FULL, TableLayoutConstraints.FULL));
               //---- averageScoreLabel ----
               averageScoreLabel.setText("Average Score");
               ((JPanel)TabsList.get(wnum)).add(averageScoreLabel, new TableLayoutConstraints(5, 1, 5, 1, TableLayoutConstraints.FULL, TableLayoutConstraints.FULL));
               //---- averageRankLabel ----
               averageRankLabel.setText("Avg. Rank");
               ((JPanel)TabsList.get(wnum)).add(averageRankLabel, new TableLayoutConstraints(6, 1, 6, 1, TableLayoutConstraints.FULL, TableLayoutConstraints.FULL));
               //---- playerName1 ----
               playerName1.setText("Joe Smith");
               ((JPanel)TabsList.get(wnum)).add(playerName1, new TableLayoutConstraints(1, 2, 1, 2, TableLayoutConstraints.FULL, TableLayoutConstraints.FULL));
               //---- playerAvgRank1 ----
               playerAvgRank1.setText("1");
               ((JPanel)TabsList.get(wnum)).add(playerAvgRank1, new TableLayoutConstraints(6, 2, 6, 2, TableLayoutConstraints.FULL, TableLayoutConstraints.FULL));
               //---- playerAvgScrore1 ----
               playerAvgScrore1.setText("32.8");
               ((JPanel)TabsList.get(wnum)).add(playerAvgScrore1, new TableLayoutConstraints(5, 2, 5, 2, TableLayoutConstraints.FULL, TableLayoutConstraints.FULL));
               return ((JPanel)TabsList.get(wnum));
     public static void setFocus() {
        ((JPanel)TabsList.get(mainTabs.getSelectedIndex())).revalidate();
}thanks for the help in advance.

Answer 1

You are trying to re-use the same component to be two different things.You create a bunch of statics, like "static JLabel playerName1", then you do a setText on it and add it to a JPanel. But then the next time around you do a setText on the SAME instance and try to add it to a different JPanel. A component can't be on two different panels at the same time. You must create a new instance for each panel. It would probably be a lot simpler if you just dumped all the statics, and instead wrote more conventional code like:
JLabel name=new JLabel("Name");
panel.add(name,whatever_constraints);And by the way, I think the code in addTabbedPaneTab would be a whole lot more readable if you created a variable to hold a reference to the panel, rather than going back to the get every time, like
public JPanel addTabbedPaneTab(int wnum) {
  JPanel week=new JPanel();
  week.setLayout(... whatever ...);
  JLabel sortByLabel=new JLabel("Sort By");
  week.add(sortByLabel,... constraints ...);
  JComboBox sortByChooser=new JComboBox(new DefaultComboBoxModel(... whatever ...));
  sortByChooser.setOpaque(false);
  week.add(sortByChooser,... constraints...);
  JLabel playerNameLabel=new JLabel("Name");
  week.add(playerNameLabel,... constraints ...);          
  JLabel averageScoreLabel=new JLabel("Average Score");
  week.add(averageScoreLabel,... constraints ...);          
  JLabel averageRankLabel=new JLabel("Avg Rank");
  week.add(averageRankLabel,... constraints ...);          
  JLabel playerName1=new JLabel("Joe Smith");
  week.add(playerName1,... constraints ...);               
  JLabel playerAvgRank1=new JLabel("1");
  week.add(playerAvgRank1,... constraints ...);
  JLabel playerAvgScrore1=new JLabel("32.8");
  week.add(playerAvgScrore1,... constraints ...);
  TabsList.add(week);
  return week;               
}I also don't see why you need TabsList, it's just a duplicate of the list of tabs that you could pull out of the JTabbedPane itself anyway.