plz solve this error

[silversurfer2in]

i am working on a project in which there is an actionListener for a button...when the user will click it, the code inside will create a JInternalFrame on which i have added a button....now the problem is how to add an actionListener for the button i.e on JInternalFrame
my code goes like this

public void actionPerformed(ActionEvent e)
    {
        ...............
        ...............
        ...............
        JButton cancel = new JButton("Cancel");
	        cancel.setFont(new Font("ARIAL",Font.BOLD,12));
	        cancel.setBounds(410, 570, 75, 25);
	        cancel.addActionListener(this);

on running the code i am having the following error

Exception in thread "AWT-EventQueue-0" java.lang.IllegalArgumentException: illegal component position
	at java.awt.Container.addImpl(Container.java:1048)
	at javax.swing.JLayeredPane.addImpl(JLayeredPane.java:212)
	at javax.swing.JDesktopPane.addImpl(JDesktopPane.java:470)
	at java.awt.Container.add(Container.java:365)
	at financialaccounting.FinancialAccounting.actionPerformed(FinancialAccounting.java:287)
	at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
	at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
	at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
	at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
	at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
	at java.awt.Component.processMouseEvent(Component.java:6263)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
	at java.awt.Component.processEvent(Component.java:6028)
	at java.awt.Container.processEvent(Container.java:2041)
	at java.awt.Component.dispatchEventImpl(Component.java:4630)
	at java.awt.Container.dispatchEventImpl(Container.java:2099)
	at java.awt.Component.dispatchEvent(Component.java:4460)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4574)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4238)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4168)
	at java.awt.Container.dispatchEventImpl(Container.java:2085)
	at java.awt.Window.dispatchEventImpl(Window.java:2475)
	at java.awt.Component.dispatchEvent(Component.java:4460)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:174)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)

please help me :(

[Fubarable]

It's hard to help you, as you never show what line is causing this exception. By the way, you really shouldn't be using null layout and absolute positioning. You're far better off using one or more layout managers.

[silversurfer2in]

i tried to use the grid layout here but it was not working so i was forced to implement this one...
the 'cancel' button is on the internal frame....when i click it this error comes

[silversurfer2in]

in simple words i want to know how to implement the actionlistener for the 'cancel' button

[Cluster Storm]

public void actionPerformed(ActionEvent e)
    {
        ...............
        ...............
        ...............
        JButton cancel = new JButton("Cancel");
	        cancel.setFont(new Font("ARIAL",Font.BOLD,12));
	        cancel.setBounds(410, 570, 75, 25);
	        cancel.addActionListener(this);
}

in my point of view your are doing wrong use of adding action listener because you are adding action listener with button inside the actionPerformed method and this method is called when you register your listener with button outside the method. This wont work any more . . .
In method you write your actions which you want to perform.
you should write your code like this

JButton cancel = new JButton( "Cancel" );
cancel.setFont( new Font( "ARIAL", Font.BOLD, 12) );
cancel.setBounds( 410, 570, 75, 25);

cancel.addActionListener( 
               new ActionListener()
               {
                     public void actionPerformed( ActionEvent e )
                     {
                           System.out.println( "You click Cancel" );
                     }

               } // end actionListener
);

i explained what i understand ?

Thanks . . .:)

[silversurfer2in]

@ cluster storm

thanx it worked...but i would still like to know why can't the actionlistener for the 'cancel' button be implemented outside this method :)

[Cluster Storm]

@ cluster storm

thanx it worked...but i would still like to know why can't the actionlistener for the 'cancel' button be implemented outside this method :)

If you look at your posted code this does not make any sense that you are registering actionListener with your cancel button inside the interface implementation method . . . Let suppose you register it inside your actionPerformed method and when the screen appear button also not displaying . . . because actionPerformed method is called when you register it before or after the methods body . . . Execution start line by line . . . when execution line reach the actionPerformed method, it skipped . . . and your button creation, its location etc are not performed because it depend upon the addActionListener statement . . . when cancel.addActionListener is executed it call the actionPerformed method and then the execution control shifted to this method . . . and so on . . .

Again i say that i explained what i understand ??

Thanks . . . ;)

[silversurfer2in]

If you look at your posted code this does not make any sense that you are registering actionListener with your cancel button inside the interface implementation method . . . Let suppose you register it inside your actionPerformed method and when the screen appear button also not displaying . . . because actionPerformed method is called when you register it before or after the methods body . . . Execution start line by line . . . when execution line reach the actionPerformed method, it skipped . . . and your button creation, its location etc are not performed because it depend upon the addActionListener statement . . . when cancel.addActionListener is executed it call the actionPerformed method and then the execution control shifted to this method . . . and so on . . .

Again i say that i explained what i understand ??

Thanks . . . ;)

well what you say is perfectly ok...but the thing what is bugging me is that i have created an actionListener method for the cancel button outside this interface i.e. outside this actionPerformed method but it was not working in that case....

[Cluster Storm]

well what you say is perfectly ok...but the thing what is bugging me is that i have created an actionListener method for the cancel button outside this interface i.e. outside this actionPerformed method but it was not working in that case....

could you please show that code ?

[silversurfer2in]

public void actionPerformed(ActionEvent e)
    {
        ...............
        ...............
        ...............
        JButton cancel = new JButton("Cancel");
	        cancel.setFont(new Font("ARIAL",Font.BOLD,12));
	        cancel.setBounds(410, 570, 75, 25);
	        cancel.addActionListener(this);
}
public void actionPerformed(ActionEvent e){
System.out.println("cancel is clicked");
}

[Cluster Storm]

public void actionPerformed(ActionEvent e)
    {
        ...............
        ...............
        ...............
        JButton cancel = new JButton("Cancel");
	        cancel.setFont(new Font("ARIAL",Font.BOLD,12));
	        cancel.setBounds(410, 570, 75, 25);
	        cancel.addActionListener(this);
}
public void actionPerformed(ActionEvent e){
System.out.println("cancel is clicked");
}

Dear, you are making mistake which is that you are creating actionPerformed method two times this is not possible . . . Every ActionListener must have one actionPerformed method not two ( ActionListener documentation ).
You cannot change the signature of the interface . . . For this you must have two ActionListener one for one method and one for other like

okButton.addActionListener( 
		new ActionListener()
		{
			public void actionPerformed( ActionEvent e )
			{
				System.out.print( "Ok Button" );
			}
		}
);

cancelButton.addActionListener( 
		new ActionListener()
		{
			public void actionPerformed( ActionEvent e )
			{
				System.out.print( "Cancel Button" );
			}
		}
);

but it is also possible that tow button have same actionListener and we recognize each button by getSource() method like

private class ok_cancel implements ActionListener
{
	public void actionPerformed( ActionEvent e )
	{
		if( e.getSource() == okButton )
			System.out.print( "okButton" );
		else if( e.getSource() == cancelButton )
				System.out.print( "Cancel Button" );
	}
}

okButton.addActionListener( new ok_cancel() );
cancelButton.addActionListener( new ok_cancel() );

I think you get my point !

[Fubarable]

You may wish to create and post an SSCCE that demonstrates what you're trying to do, and gives us compilable code that we can run and adapt. Please see the link in my signature for more details.

[Cluster Storm]

Thanks for point me on my mistake . . .

[silversurfer2in]

@ cluster storm

thanx a lot man.........:)

[Cluster Storm]

You are welcome !!!