Using Icon and myIcon

Question

I am practicing using Icon and myIcon and am getting an error saying that myIcon must be defined in its own file. I am pretty sure I defined it within the code and am confused what I did wrong.

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

public class TestIcon {
    public static void main(String[] args) {
        myIcon icn = new myIcon(40,50);
        JOptionPane.showMessageDialog(null, "Hello World!", "Message", JOptionPane.INFORMATION_MESSAGE, icn);
    }
}

public class myIcon implements Icon{
    private int width;
    private int height;

    public myIcon(int width, int height) {
        this.width=width;
        this.height=height;
    }
    public int getIconWidth(){
        return width;
    }
    public int getIconHeight(){
        return height;
    }
    public void paintIcon(Component c, Graphics g, int x, int y){
        Graphics2D g2 = (Graphics2D) g;
        Ellipse2D.Double ellipse = new Ellipse2D.Double(x,y,width, height);

        g2.setColor(Color.RED);
        g2.fill(ellipse);
    }
 }

Answer 1

Either move myIcon to it's own file or make myIcon an inner class of TestIcon, for example...

import java.awt.Color;
import java.awt.Component;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.geom.Ellipse2D;
import javax.swing.Icon;
import javax.swing.JOptionPane;

public class TestIcon {

    public static void main(String[] args) {
        myIcon icn = new myIcon(40, 50);
        JOptionPane.showMessageDialog(null, "Hello World!", "Message", JOptionPane.INFORMATION_MESSAGE, icn);
    }

    public static class myIcon implements Icon {

        private int width;
        private int height;

        public myIcon(int width, int height) {
            this.width = width;
            this.height = height;
        }

        public int getIconWidth() {
            return width;   
        }

        public int getIconHeight() {
            return height;
        }

        public void paintIcon(Component c, Graphics g, int x, int y) {
            Graphics2D g2 = (Graphics2D) g;
            Ellipse2D.Double ellipse = new Ellipse2D.Double(x, y, width, height);

            g2.setColor(Color.RED);
            g2.fill(ellipse);
        }
    }

}

A inner class imposes some limitations, so depending on what you want to do, will depend on what you do.