Duplicate Input Boxes

[habester]

Hi everyone,

I'm using JOptionPane.showInputDialog for a program that gets a number, than draws out that many random ellipses. Everything works fine except that for some reason, there are two input boxes that pop up every time. I can't figure out why. Any ideas?

import javax.swing.*;
import java.awt.geom.*;
import java.util.Scanner;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.Color;

public class P6_16JH extends JApplet {

	public void paint(Graphics g) {
	
		Graphics2D g2 = (Graphics2D)g;
		
		String response = JOptionPane.showInputDialog(null, "Enter number of circles to draw:");
		int n = Integer.parseInt(response);
		
		for(int i = 0; i<n; i++) {
			double w = Math.random()*getWidth();
			double h = Math.random()*getHeight();
			double x = Math.random()*(getWidth()-w);
			double y = Math.random()*(getHeight()-h);
			Ellipse2D.Double ellipse = new Ellipse2D.Double(x, y, w, h);
			g2.setColor(new Color((int)(Math.random()*255), (int)(Math.random()*255), (int)(Math.random()*255)));
			g2.draw(ellipse);
			g2.fill(ellipse);
		}
	}
}

[masijade]

Do not do any kind of real "work" in the paint method of a Component. Don't override paint, at all. Override paintComponent. And, even there, do not do any real "work" there. Ask for the input outside of the method and set an instance variable with the result (look into AtomicInteger just to make sure that the value is consistent) then call repaint() on the component.

[habester]

Thank you! That got rid of that problem.

[camickr]

Custom painting is done by extending paintComponent() of a JPanel or JComponent. The component is then added to the content pane of the applet.

Never override paint() of a top level container (JApplet, JFrame, JDialog).