Re: What is the problem here, LayoutManagers.

From:

"Matt Humphrey" <matth@iviz.com>

Newsgroups:

comp.lang.java.programmer

Date:

Thu, 14 May 2009 12:36:47 -0400

Message-ID:

<lfmdnbbDq_A-1ZHXRVn_vwA@giganews.com>

<iMohed@live.se> wrote in message
news:43093f17-486a-4a71-b38d-013aac85627a@l28g2000vba.googlegroups.com...
Hello guys. I have a Problem in the following code. The result is that
only one MFrame shows up in the window instead of the two constructed.
What is wrong and why ?? Thanks in advance for any aid.

<snipped code>

public View(){

   for (MFrame mF : this.frames) {
     mF = new MFrame();
     mF.setLayout(new BoxLayout(mF, BoxLayout.PAGE_AXIS));
     mF.add(mF.name);
     mF.add(mF.msg);
     mF.buttons.setLayout(new BoxLayout(mF.buttons,
        BoxLayout.LINE_AXIS));
     for (JButton jB : mF.sSP) {
       jB = new JButton();
       mF.buttons.add(jB);
     }
     mF.add(mF.buttons);
     mF.add(mF.status);
     mF.add(mF.score);
     this.add(mF);
   }

   this.setDefaultCloseOperation(EXIT_ON_CLOSE);
   this.setVisible(true);
}

JFrame uses BorderLayout by default which does not accept multiple
components in each region, so this.add(mF) only works for one item.
http://java.sun.com/javase/6/docs/api/javax/swing/JFrame.html You probably
need some other kind of layout manager or panel (flow layout, tabbed pane,
etc.)

Also, your iterator over frames (line 3 above) always returns null for mF.
Assigning mF in the following line does not update the value in this.frames.

Matt Humphrey http://www.iviz.com/