Adding components dynamically to a nested JPanel

[Harihara Sudan]

Hi,

I am a new Java Programmer.could you help me in resolving this.

I am trying to created a JPanel withing a main JPanel. For the nested one, i have included Grid Layout and i am trying to add components to it dynamically.
i.e i have added a combo box to the nested JPanel and based on the selection it makes, i want to add few more components to the same JPanel.

For this i created some code, but the problem is irrespective of the size, i mention for the combo box, it takes up the whole size of the JPanel and hence, the other components which needs to get added to the panel, based upon the selection of the combo box, are not getting added to the JPanel.

I searched through the net and understood that this has something to do with the layout manager only. But i am unable to debug this out..

Please help me with this

Note - while running the code, please enter the number of rows to be added as 1, since ive not yet handled multiple rows

import java.io.*;
import javax.swing.*;
import java.awt.*;

import java.awt.event.*;
import java.awt.BorderLayout;
import java.awt.Component;
import java.awt.Container;
import java.awt.Graphics;
import java.awt.GridLayout;
import java.awt.event.WindowAdapter;
import javax.swing.JLabel;

import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.event.MouseMotionListener;
import java.io.IOException;
import java.net.*;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTextField;


public class tmp extends JFrame implements MouseListener, MouseMotionListener {
	

	public static JFrame frame;
	public static JTextField nor;
	public static JPanel panel;
	public static JPanel[] panels = new JPanel[500];
	public static int ctr = 0,inc=0;
	public static JScrollBar jb1,jb2;
	public static JScrollPane jp;
	public static JFileChooser jf1,jf2;
	public static JCheckBox[] jr = new JCheckBox[600];
	public static JTextField[] at = new JTextField[600];
	public static String svc_name,svc_vers;
	public static JComboBox[] combo = new JComboBox[500];
	public void Front_end()
	{
		
		frame = new JFrame("abc");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setBounds(0,0,1000,5000);
		panel = new JPanel();
		GUI();
		
		jp = new JScrollPane(panel);		
		jp.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_ALWAYS );
		frame.add(jp);
		
		
		frame.setVisible(true);
		
	}
	public void fillPanel(int tmp)
	 {
		
		panels[tmp].setLayout(new GridLayout(2,7,8,8));
		panels[tmp].setBounds(5,150,100,200);
		
		panel.add(panels[tmp]);
		
		String choices[] = {" ","Offer Name change","Offer Price change","New Offer","Offer Code Change"};
		inc = tmp;
		
		combo[inc] = new JComboBox(choices);
		combo[inc].setSelectedIndex(0);
		combo[inc].setSize(50, 50);
		combo[inc].setBounds(20,160,70,100);
		panels[tmp].add(combo[inc]);
		combo[inc].addItemListener(new ItemListener(){
			  public void itemStateChanged(ItemEvent ie){
			  String str = (String)combo[inc].getSelectedItem();
		
				if(str.equals("Offer Name change"))
				{
					System.out.println("We have a match");
					offerNameChange(inc);
					
				}
				else if(str.equals("Offer Price change"))
				{
					System.out.println("Second match");
				}
			}
		});
		
	}
	public void offerNameChange(int countr)
	{
		JLabel name = new JLabel("Name of the offer");
		name.setBounds(120, 300, 100, 100);
		panels[countr].add(name);
		panel.add(panels[countr]);
		System.out.println("Success");
	}
	
	public void mouseEntered(MouseEvent e) { }
	public void mouseExited   ( MouseEvent e ){}
	public void mousePressed  ( MouseEvent e ) {}
	public void mouseDragged(MouseEvent e) {}
	public void mouseMoved(MouseEvent e) { }
	public void mouseReleased ( MouseEvent e ) {}
	public void mouseClicked(java.awt.event.MouseEvent e) { }
	public void GUI()
	{
		panel.setLayout(null);
		panel.setSize(3000,5000);
		Dimension dim = new Dimension(3000,5000);
		panel.setPreferredSize(dim);
		
		JLabel head = new JLabel("ghg");
		Dimension dd = new Dimension();
		dd.height = 200;
		dd.width = 100;
		head.setSize(dd);
		head.setBounds(300, 10, 200, 50);
		panel.add(head);
		
		JLabel norw = new JLabel("No. of rows more to be added");
		norw.setBounds(600,45,200,50);
		panel.add(norw);
		
		nor = new JTextField();
		nor.setBounds(770,55,25,25);
		panel.add(nor);
		
		JButton nrw = new JButton("Add");
		nrw.setBounds(800,55,60,25);
		panel.add(nrw);
	
	 nrw.addActionListener(new ActionListener()
    {
        public void actionPerformed(ActionEvent ae)
        {
        	int i=Integer.parseInt(nor.getText());
        	int temp = ctr;
        	ctr = ctr + i;
        	for (int ii=0;ii<i;ii++)
        	{
        		panels[temp] = new JPanel();
        		fillPanel(temp);
         
        		temp++;
        	}
        	System.out.println("Out of the loop");
    		ctr++;
        	panel.revalidate();
        	panel.repaint();
        	
}
});
	 }
	public static void main(String[] args){
		System.out.println("Hi");
		tmp har = new tmp();
		har.Front_end();
	}
}

Also, i would like to include multiple such panels arranged vertically growing downwards. Could someone suggest some idea for that.
The constraint in this case is, at first ill go and add 10 rows and then later depending on my requirement, ill go and delete a particular row in the middle (say 5th) using the delete button, which would be provided at last of every row, so the rows must get rearranged in order to fill the gap made by the removal on the middle row. Please help me with this too..

Thanks in advance

[Michael Dunn]

you have too much of this stuff:

combo[inc].setSize(50, 50);
combo[inc].setBounds(20,160,70,100);

do you know what the lines do?

I'd suggest you start over, adding one component at a time, getting it in it's correct postion,
testing a listener if it needs one (simple System.out.println("OK"); to test), compiling/running etc.

if you get stuck you'll have a much simplified code to post.

[Darryl Burke]

Hello Harihara, and welcome to the Ranch! We don't have too many rules here, but we do ask that you BeForthrightWhenCrossPostingToOtherSites.
http://www.java-forums.org/awt-swing/51976-problem-adding-multiple-components-dynamically-nested-jpanel.html

edit: And http://www.codeguru.com/forum/showthread.php?p=2044723

[Harihara Sudan]

Thanks for your reply,

Now i again tried starting it over and still facing the same issue.
Am, sorry if i am sounding crazy, since i am a newbie to this.i know i am messing up with the code.

As you had said, ive posted only the nested JPanel which is actually posing the problem

The main JPanle is named as Panel, which is added to a frame.
Since i need multiple panels within this Panel, i had used an array of JPanels within this Panel.
This array is the panels[].

And i thought the combo[inc].setSize and setBounds, does the same thing, where the latter in addition to setting the size fixes the x and y coordinate within the panel. I tried adding both since i thought either of that might work for me.

Now regardless of whatever i attempt to modify, the nested panel adds only the combo box and makes the combo box to fit the whole size of the frame and hence after selecting the choice, i am unable to add the textbox, through the function offerNameChange.

as you had suggested, i added the System.out.println to check the flow through the item listener, and the flow was completed till the end since all the messages were printed.

public void fillPanel(int tmp)
	 {
		panels[tmp]=new JPanel();
		panels[tmp].setLayout(new GridLayout(1,7,8,8));
		panels[tmp].setBounds(30,150,150,50);
		panel.add(panels[tmp]);
		String choices[] = {" ","Offer Name change","Offer Price change","New Offer","Offer Code Change"};
		inc = tmp;
		System.out.println("The value of tmp is "+tmp);
		combo[inc] = new JComboBox(choices);
		combo[inc].setSelectedIndex(0);
		combo[inc].setBounds(50, 100, 100, 20);
		panels[tmp].add(combo[inc]);
		
		combo[inc].addItemListener(new ItemListener(){
			  public void itemStateChanged(ItemEvent ie){
				  System.out.println("This is invoked");
			  String str = (String)combo[inc].getSelectedItem();
			  System.out.println("The string is"+str+"a");
				if(str.equals("Offer Name change"))
				{
					System.out.println("We have a match");
					JLabel name = new JLabel("Name of the offer");
					name.setBounds(100, 300, 100, 100);
					panels[inc].add(name);
					offerNameChange(inc);
					
				}
				else if(str.equals("Offer Price change"))
				{
					System.out.println("Second match");
				}
			}
		});
		
	
		System.out.println("After adding cmbo box to the panel");
		}
	public void offerNameChange(int countr)
	{
		System.out.println("Inside this");
		JLabel name = new JLabel("Name of the offer");
		System.out.println("The value of counter is"+countr);
		name.setBounds(100, 300, 100, 100);
		panels[countr].add(name);
		System.out.println("Success");	
	}

And i am sorry, actually i was having some timeline to show some progress and so i posted it in the other forum, but i didnt get any answer from the other one till now.

[Ranganathan Kaliyur Mannar]

There are several issues with your code. First of all, you are not setting any layouts to the main panel - this means, the main panel uses FlowLayout. Do you know how FlowLayout handles the components? You just need to call setPreferredSize...

And, you are calling setBounds on the panel - I don't know how FlowLayout handles location of components, but logically, you are adding new panels to the same 'location' and with same size - so, you will have all panels one over the other...

and you should use actionListener for the combo box too unless you need to deal with old and new values (I think ItemListener fires events twice - once for old and new values).

I would suggest you to go through the layout tutorial first.