Stubborn Window

[Jayayoh]

Hello. I am new to both this forum, and java programming in general. I have created a swing window, adding a JPanel to it, and adding a JLabel to that. I have some simple code that my computer tech teacher at my school helped me with, that creates an animation. The animation however, is not appearing. This is less of a problem as it has appeared in previous applications. My real problem, is that when resized or minimized, the window turns black. in the case of it being resize, it turns black the portion that was resized. This has happened in several of my applications, and it is really bugging me. Some advice would be appreciated.

*You will notice that the setResizable(false); function has been commented out*

/*  Created By John Omeljaniuk
	Wednesday June 15 2011
	First official Game
*/
//-------------------------Imports-----------------------------------------------------------------
import javax.swing.*;
import java.awt.event.*;
import java.awt.*;
import java.util.*;
//-------------------------Main Program-------------------------------------------------------------
class diceman extends JFrame
{
//--------------------------containers-----------------------------------------------------------
	JPanel lvl1 = new JPanel();
//--------------------------Images------------------------------------------------------------------
//standing dice man
	ImageIcon standing_die[] = {
	new ImageIcon("C:\\TAoDM\\diceman_stand_1.bmp"),
	new ImageIcon("C:\\TAoDM\\diceman_stand_2.bmp"),
	new ImageIcon("C:\\TAoDM\\diceman_stand_3.bmp"),
	new ImageIcon("C:\\TAoDM\\diceman_stand_4.bmp"),
	new ImageIcon("C:\\TAoDM\\diceman_stand_5.bmp"),
	new ImageIcon("C:\\TAoDM\\diceman_stand_4.bmp"),
	new ImageIcon("C:\\TAoDM\\diceman_stand_3.bmp"),
	new ImageIcon("C:\\TAoDM\\diceman_stand_2.bmp")
	};
	JLabel diceman_stand = new JLabel(standing_die[0]);
//--------------------------variables---------------------------------------------------------------
	int i = 0;
	boolean motion = false;
	timer t = new timer();
//--------------------------Main Method-------------------------------------------------------------
	public static void main(String[] args)
	{
		diceman window = new diceman();
	}
//--------------------------Class Constructor--------------------------------------------------------
	public diceman()
	{
		super("The Adventures of Dice Man");
		setSize(500,500);
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		add(lvl1);
		lvl1.setBackground(Color.cyan);
		setVisible(true);
		//setResizable(false);
		lvl1.add(diceman_stand);
		t.start();

	}
//--------------------------painting method-----------------------------------------------------------
	public void paint(Graphics g)
	{
		if(motion == false)
		{
			diceman_stand.setIcon(standing_die[i]);
			i++;
			if(i == standing_die.length)
			{
				i = 0;
			}
		}
	}
//--------------------------Timer Thread--------------------------------------------------------------
class timer extends Thread
{
	public void run()
	{
		while(true)
		{
			try
			{
				sleep(100);
			}
			catch(Exception e)
			{
			}
			repaint();
		}
	}
}
//--------------------------end of timer---------------------------------------------------------------
}
//---------------------------end of main class----------------------------------------------------------

[Norm]

A problem I see is you're setting the window visible before all the components have been added.

[camickr]

My real problem, is that when resized or minimized, the window turns black. in the case of it being resize, it turns black the portion that was resized.

The main problem is probably because you don't invoke super.paint(...) at the start of the method. However, you should never override the paint() method of a top level container (JFrame, JDialog, JWindow or JApplet).

Custom painting is done by overriding the paintComponent() method of a Swing component like JPanel or JComponent and then you add the component to the JFrame (or other top level container). See: Lesson: Performing Custom Painting (The Java™ Tutorials > Creating a GUI With JFC/Swing)

Animation should be done by using a Swing Timer, NOT by uising a Thread and sleep(). See: How to Use Swing Timers (The Java™ Tutorials > Creating a GUI With JFC/Swing > Using Other Swing Features)

[Norm]

Note: JFrame doesn't have a paintComponent() method.

[DarrylBurke]

Moved here from 'New To Java'

db

[Jayayoh]

Thank you so very much. My window and all images are added perfectly now, however, I am still attempting to figure out the swing timer. It is rather confusing. For example: where do I put the action event method, and where do I innitialize the timer? I have tried different locations in the code, but none of them seem to work.

[Norm]

There are many code samples of timer use on the forum. Do a Search for examples.

When you are having problems, post the code and the error messages here for help.

In the code you posted earlier, the paint() method doesn't do any painting. You don't need to put code in it. Put the code in paint() in the timer action method.

[Jayayoh]

WOOOOOOOT! after about two hours of working it out, My animation works...With a swing timer!Thank you so much, everyone who contributed to this post. You have my eternal gratitude.