[newbie] The constructor <Class>(String, ImageIcon) is undefined

[Jon Camilleri]

Well this is what I see:

ActionFrame.java
...
ColorAction yellowAction = new ColorAction("Yellow", new ImageIcon("C:\\pictures\\yellow-ball.gif"), Color.YELLOW);
		ColorAction blueAction = new ColorAction("Blue", new ImageIcon("C:\\pictures\\blue-ball.jpg"), Color.BLUE);
		ColorAction redAction = new ColorAction("Red", new ImageIcon("C:\\pictures\\red-ball.jpg"), Color.RED);
...

AbstractAction.java

public class ColorAction extends AbstractAction {
...
public ColorAction(String name, Icon icon, Color c) 
	{
		putValue(Action.NAME, name);
		putValue(Action.SMALL_ICON, icon);
		putValue(Action.SHORT_DESCRIPTION, icon);
		putValue("color", c);
	}
...
}

Full listing:

ActionTest.java
package homenetwork.bkr.training;

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

public class ActionTest {

	/**
	 * @param args
	 */
	public static void main(String[] args) {
		EventQueue.invokeLater(new Runnable()
		{
			public void run()
			{
				ActionFrame frame = new ActionFrame();
				frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
				frame.setVisible(true);
			}
		}
		);

	}

}

ActionFrame.java
package homenetwork.bkr.training;

import java.awt.Color;
import javax.swing.*;
import javax.swing.InputMap;
import homenetwork.bkr.training.*;

@SuppressWarnings({ "serial", "unused" })
public class ActionFrame extends JFrame {
	
	public ActionFrame()
	{
		setTitle("ActionTest");
		setSize(DEFAULT_WIDTH, DEFAULT_HEIGHT);
		
		JPanel buttonPanel = new JPanel();
		
		//define actions
		ColorAction yellowAction = new ColorAction("Yellow", new ImageIcon("C:\\pictures\\yellow-ball.gif"), Color.YELLOW);
		ColorAction blueAction = new ColorAction("Blue", new ImageIcon("C:\\pictures\\blue-ball.jpg"), Color.BLUE);
		ColorAction redAction = new ColorAction("Red", new ImageIcon("C:\\pictures\\red-ball.jpg"), Color.RED);
		/*TODO read from within the same project
		rename pictures as .PNG.
		*/
		
		//Add buttons for these actions
		buttonPanel.add(new JButton(yellowAction));
		buttonPanel.add(new JButton(blueAction));
		buttonPanel.add(new JButton(redAction));
		
		//Add panel to frame
		add(buttonPanel);
		
		//Associate the Y, B and R keys with names
		InputMap imap = buttonPanel.getInputMap(JComponent.WHEN_ANCESTOR_OF_FOCUSED_COMPONENT);
		imap.put(KeyStroke.getKeyStroke("ctrl Y"), "panel.yellow");
		imap.put(KeyStroke.getKeyStroke("ctrl B"), "panel.blue");
		imap.put(KeyStroke.getKeyStroke("ctrl R"), "panel.red");
		
		//Associate the names with actions
		ActionMap amap = buttonPanel.getActionMap();
		amap.put("panel.yellow", yellowAction);
		amap.put("panel.blue", blueAction);
		amap.put("panel.red", redAction);
	}
	
	public static final int DEFAULT_WIDTH = 300;
	public static final int DEFAULT_HEIGHT = 200;

}

ColorAction.java
package homenetwork.bkr.training;

import java.awt.Color;
import javax.swing.*;
import java.awt.event.*;

public class ColorAction extends AbstractAction {
	/** Constructs a color action
	 * @param name: the name to show on the button.
	 * @param icon: the icon to display on the button.
	 * @param c: the background color
	 */
	public ColorAction(String name, Icon icon, Color c) 
	{
		putValue(Action.NAME, name);
		putValue(Action.SMALL_ICON, icon);
		putValue(Action.SHORT_DESCRIPTION, icon);
		putValue("color", c);
	}
	
	public void actionPerformed(ActionEvent event)
	{
		Color c = (Color) getValue("color");
		buttonPanel.setBackground(c);
	}
	
	private JPanel buttonPanel;
	
	public static final int DEFAULT_WIDTH = 300;
	public static final int DEFAULT_HEIGHT = 200;

}

[Sunil Kumar]

Any stack trace.. line number of error?? anything... Provide some info.
From what i see, you are trying to instantiate ColorAction using (String, ImageIcon) as parameters while your constructor needs 3 (String, Icon, Color)
If ImageIcon is a subclass of Icon, you can still instantiate using (String, ImageIcon, Color). Main point to note is.. the number of arguments should be correct

[Jon Camilleri]

Sunil Kumar wrote:Any stack trace.. line number of error?? anything... Provide some info.
From what i see, you are trying to instantiate ColorAction using (String, ImageIcon) as parameters while your constructor needs 3 (String, Icon, Color)
If ImageIcon is a subclass of Icon, you can still instantiate using (String, ImageIcon, Color). Main point to note is.. the number of arguments should be correct

Compilation error:
Exception in thread "AWT-EventQueue-0" java.lang.Error: Unresolved compilation problems:
The constructor ColorAction(String, ImageIcon) is undefined
The constructor ImageIcon(String, Color) is undefined
Syntax error, insert ")" to complete VariableInitializer
The constructor ColorAction(String, ImageIcon) is undefined
The constructor ImageIcon(String, Color) is undefined
Syntax error, insert ")" to complete VariableInitializer
The constructor ColorAction(String, ImageIcon) is undefined
The constructor ImageIcon(String, Color) is undefined
Syntax error, insert ")" to complete VariableInitializer

at homenetwork.bkr.training.ActionFrame.<init>(ActionFrame.java:19)
at homenetwork.bkr.training.ActionTest$1.run(ActionTest.java:16)
at java.awt.event.InvocationEvent.dispatch(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)

[Henry Wong]

Weird. The compile errors doesn't seem to match the code. Maybe some of your stuff is out of sync -- I would consider deleting your class files, and compile again.

Henry

[Henry Wong]

From reading your other topics, it looks like you are using an IDE to run the program that has compile errors. Don't do that !!!

You need to fix the compile errors. And fix them before you run them. Use the IDE to point them out and fix them -- and fix them all before you run the program. Otherwise you can get weird runtime errors, that doesn't make sense, which is what you have here.

Henry