Double window popup :/

[AndroidAppNewbie]

Hi.
I've made a game.
And when the game is over(the timer reaches 0) the game calls "new win();"
which is my end of game window.. the thing is that when the "win" window get's created. it comes up 2 windows. not one. Below is my win class and my timer class. the win class is inside the class "Move", but that dosen't matter. all the code related to the problem should be located below:

And please note that this code isn't completely finished, just trying to fix this double window prob before i move on.

public class clock implements Runnable, ActionListener{
		private class win extends JFrame implements ActionListener{

			private JButton again;
			private JLabel score1;
			private JLabel score2;
			
			public win(){
				super("Game Over");
				again = new JButton("Play again");
				score1 = new JLabel("Player1(RED) score:" + p1score);
				score2 = new JLabel("Player2(GREEN) score:" + p2score);
				again.addActionListener(this);
				setLayout(new FlowLayout());
				add(again);
				add(score1);
				add(score2);
				setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
				setVisible(false);
				setLocation(900, 250);
				setSize(400, 200);
				setResizable(false);
				
			}
			
			
			public void actionPerformed(ActionEvent e) {
				if(e.getSource() == again){
					new reseter();
					
				}if(e.getSource() == timer){
					left.setText("Time left: " + tic);
					tic--;
				left.setText("Time left: " + tic);
				}
				if(tic == 25){
					delay.stop();
					sc.stop();
					enemytimer.stop();
					tic = 30;
					t1.stop();
					timer.stop();
					setVisible(true);
					new win();
	 			}
				
			}
			
		}
		 Timer timer = new Timer(1000, new win());
		win w = new win();

		
		
		public void run(){
			setVisible(true);	
			
			    timer.start();  
			   
			    
               left.setText("Time left: " + tic);
				
				
				
		}
		
		public void actionPerformed(ActionEvent e) {
			
			
		}
		public void keyPressed(KeyEvent e) {
			
			timer.stop();
             
 					
		}
	}

[AndroidAppNewbie]

Bump, anyone? :/

[AndroidAppNewbie]

Cmon, someone must be able to help :/

[AndroidAppNewbie]

i'll bump one last time to see if anyone cares to respond.

[NRitH]

Kee-rist. Where to begin?

First, you're instantiating 3 windows: one in actionPerformed(), one as an instance variable, and once in the construction of Timer. WHY? You need only one, and can show and hide it when necessary. It looks to me that what's happening is that it's being shown once in actionPerformed(), and once when the Timer expires. That can't be right. To fix it,

1. create an instance variable for the one and only instance of your window
2. replace new win() in actionPerformed() with window.setVisible(true)
3. pass the window instance, not a new one, to the Timer constructor

There's more to it, but I'll leave that up to you.