Thread: Getting Pictures to Spin

Hi,

i have two programs where i want the pictures to spin. On one program the pictures spin and on the other they dont. The code is the same so im not sure whats going on.

Program where pictures do spin

 if (e.getSource() == btnStopSpin )
        stopTimer();
 
      if (e.getSource() == btnSpin ){
        counter = 0;
        timer = new Timer(100, this);
        timer.start() ;
      }
 
      if (e.getSource() == timer ){
        setIcon() ;
      }
 
  }// end of Action Performed
 
    public void stopTimer(){
    	timer.stop();
    }
 
 
 
 
    private void setIcon(){
 
    	switch (k) {
    		case 0:
    			btnSpinPictures.setIcon(iconPic1);
    			k=1;
    			counter++;
    			break;
    		case 1:
    			btnSpinPictures.setIcon(iconPic2);
    			k=2;
    			counter++;
    			break;
    		case 2:
    			btnSpinPictures.setIcon(iconPic3);
    			k=0;
    			counter++;
    			break;
 
    	}
 
       	if (counter >20)
    		stopTimer();
 
 
    }
 
 
}

Program where the pictures don spin

 if(event.getSource()==btnStart){
			counter = 0;
			timer = new Timer(10,this);
			timer.start();
 
		}
 
		if(event.getSource()==timer) {
			setIcon ();
		}
		if(event.getSource()==timer){
			timer.stop();
		}
}
 
}
 
		public void stopTimer(){
			timer.stop();
 
	}
 
 
 
 
 
public void setIcon(){
 
			one = random.nextInt(8);
 
 
		switch (one){
 
		case 1:
			lblDisplayPic1.setIcon(iconPic1);
			one = 1;
			counter ++;
			break;
 
		case 2:
			lblDisplayPic1.setIcon(iconPic2);
			one = 2;
			counter ++;
			break;
 
		case 3:
			lblDisplayPic1.setIcon(iconPic3);
			one = 3;
			counter ++;
			break;
 
		case 4:
			lblDisplayPic1.setIcon(iconPic4);
			one = 4;
			counter ++;
			break;
 
		case 5:
			lblDisplayPic1.setIcon(iconPic5);
			one = 5;
			counter ++;
			break;
 
		case 6:
			lblDisplayPic1.setIcon(iconPic6);
			one = 6;
			counter ++;
			break;
 
		case 7:
			lblDisplayPic1.setIcon(iconPic7);
			one = 7;
			counter ++;
			break;
 
		case 8:
			lblDisplayPic1.setIcon(iconPic8);
			one = 8;
			counter ++;
			break;
		}
 
		two = random.nextInt(8);
 
switch (two){
 
		case 1:
			lblDisplayPic2.setIcon(iconPic1);
			two = 1;
			counter ++;
			break;
 
		case 2:
			lblDisplayPic2.setIcon(iconPic2);
			two = 2;
			counter ++;
			break;
 
		case 3:
			lblDisplayPic2.setIcon(iconPic3);
			two = 3;
			counter ++;
			break;
 
		case 4:
			lblDisplayPic2.setIcon(iconPic4);
			two = 4;
			counter ++;
			break;
 
		case 5:
			lblDisplayPic2.setIcon(iconPic5);
			two = 5;
			counter ++;
			break;
 
		case 6:
			lblDisplayPic2.setIcon(iconPic6);
			two = 6;
			counter ++;
			break;
 
		case 7:
			lblDisplayPic2.setIcon(iconPic7);
			two = 7;
			counter ++;
			break;
 
		case 8:
			lblDisplayPic2.setIcon(iconPic8);
			two = 8;
			counter ++;
			break;
 
}
three = random.nextInt(8);
 
switch (three){
 
		case 1:
			lblDisplayPic3.setIcon(iconPic1);
			three = 1;
			counter ++;
			break;
 
		case 2:
			lblDisplayPic3.setIcon(iconPic2);
			three = 2;
			counter ++;
			break;
 
		case 3:
			lblDisplayPic3.setIcon(iconPic3);
			three = 3;
			counter ++;
			break;
 
		case 4:
			lblDisplayPic3.setIcon(iconPic4);
			three = 4;
			counter ++;
			break;
 
		case 5:
			lblDisplayPic3.setIcon(iconPic5);
			three = 5;
			counter ++;
			break;
 
		case 6:
			lblDisplayPic3.setIcon(iconPic6);
			three = 6;
			counter ++;
			break;
 
		case 7:
			lblDisplayPic3.setIcon(iconPic7);
			three = 7;
			counter ++;
			break;
 
		case 8:
			lblDisplayPic3.setIcon(iconPic8);
			three = 8;
			counter ++;
			break;
 
 
			}
checkWin();  
 
if(counter >20)
	stopTimer();
 
}

Id appreciate any help

Thank you

Like you see the reels spin on a fruit machine.

 import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
class SpinSimulator extends JFrame implements ActionListener{
  private int k = 0;
  private int counter;
  private JButton btnSpinPictures,btnSpin, btnStopSpin, btnExit;
  private Timer timer;
  private Icon iconPic1, iconPic2, iconPic3;
 
 
  public static void main(String[] args) {
 
	  SpinSimulator spinApp = new SpinSimulator();
      spinApp.setVisible(true) ;
  }
  public SpinSimulator(){
 
	 super(".... Spinning Simulator....");
 
      btnSpinPictures = new JButton("");
      btnSpinPictures.setPreferredSize(new Dimension(100, 100)) ;
      btnSpinPictures.setBackground(Color.yellow );
      btnExit = new JButton("Quit?");
      btnExit.setBackground(Color.cyan ) ;
      btnSpin = new JButton("Spin");
      btnSpin.setBackground(Color.green ) ;
      iconPic1 = new ImageIcon("bin/Pic1.jpg");
      iconPic2 = new ImageIcon("bin/Pic4.jpg");
      iconPic3 = new ImageIcon("bin/Pic5.jpg");
 
      setBackground(Color.pink) ;
      setLayout(new FlowLayout()) ;
      add(btnSpinPictures) ;
      add(btnSpin) ;
      add(btnExit) ;
 
      btnSpin.addActionListener(this) ;
      btnExit.addActionListener(this) ;
 
      setSize(200, 250);
 
  }
 
 
    public void actionPerformed( ActionEvent e){
 
      if (e.getSource() == btnExit){
        JOptionPane.showMessageDialog(null, "Quiting Program :-)") ;
        System.exit(0) ;
      }
 
 
 
 
      if (e.getSource() == btnSpin ){
        counter = 0;
        timer = new Timer(100, this);
        timer.start() ;
      }
 
      if (e.getSource() == timer ){
        setIcon() ;
      }
 
  }
 
    public void stopTimer(){
    	timer.stop();
    }
 
    private void setIcon(){
 
 
    	switch (k) {
    		case 0:
    			btnSpinPictures.setIcon(iconPic1);
    			k=1;
    			counter++;
    			break;
    		case 1:
    			btnSpinPictures.setIcon(iconPic2);
    			k=2;
    			counter++;
    			break;
    		case 2:
    			btnSpinPictures.setIcon(iconPic3);
    			k=0;
    			counter++;
    			break;
 
    	}
 
       	if (counter >20)
    		stopTimer();
 
 
    }
 
 
}

thats the basic program. To see how it runs you would have to add the pics.

is it because its a Jbutton where the pictures are shown?

i suppose so yeah. So it would flicker through each of the pictures eventually stopping on a random one

Sorry, the code i just posted was for the program that works. What the program does is what i would like my other program to do but for some reason that i dont know of it wont. Instead of showing some images and stopping, it just shows one random image.

The code for my other program is:

 import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import java.util.*;
import javax.swing.JButton;
import javax.swing.JLabel;
import javax.swing.Icon;
import javax.swing.JTextField;
 
import javax.swing.Timer;
 
 
 
public class ElectronicSlot8 extends JFrame implements ActionListener
 
{
 
	private Icon iconPic1, iconPic2, iconPic3, iconPic4, iconPic5, iconPic6, iconPic7, iconPic8;
	private JLabel lblBalance, lblWinnings;
	private JTextField txtBalance, txtWinnings; 
	private JButton btnBye, btnHoldLeft, btnHoldRight, btnHoldMiddle;
	private Timer timer; 
	private JButton btnStart, btnMoney, btnCashout, btnTransfer;
	private int one, two, three, counter;
	private int balance, winnings;
 
 
 
 
	private Random random;
	private JButton btnSpinPicture1, btnSpinPicture2, btnSpinPicture3;
 
 
 
 
	public static void main(String[] args) 
 
	{
		ElectronicSlot8 electonicslot8 = new ElectronicSlot8();
		electonicslot8.setVisible(true);
	}
		public ElectronicSlot8()
 
{
 
			balance = 0;
			random = new Random();
 
		setLayout(new FlowLayout());
 
		btnSpinPicture1 = new JButton();
		btnSpinPicture1.setPreferredSize(new Dimension (150,150));
		btnSpinPicture1.setBackground (Color.gray);
 
		btnSpinPicture2 = new JButton();
		btnSpinPicture2.setPreferredSize(new Dimension (150,150));
		btnSpinPicture2.setBackground (Color.gray);
 
		btnSpinPicture3 = new JButton();
		btnSpinPicture3.setPreferredSize(new Dimension (150,150));
		btnSpinPicture3.setBackground (Color.gray);
 
 
		txtBalance = new JTextField(3);
		txtBalance.setFont(new Font("Arial", Font.BOLD, 16));
		txtBalance.setBackground(Color.white);
		txtBalance.setForeground(Color.blue);
		txtBalance.setText(balance + "");
 
		txtWinnings = new JTextField(3);
		txtWinnings.setFont(new Font("Arial", Font.BOLD, 16));
		txtWinnings.setBackground(Color.white);
		txtWinnings.setForeground(Color.blue);
		txtWinnings.setText(balance + "");
 
		btnTransfer = new JButton("Add to Balance");
		btnTransfer.setEnabled(false);
		btnMoney = new JButton("Add £1");
		btnStart = new JButton("Play");
		btnStart.setEnabled(false);
		btnBye = new JButton("Exit");
		btnCashout = new JButton("Withdraw");
 
		btnHoldLeft = new JButton("Hold");
		btnHoldLeft.setPreferredSize(new Dimension(150,25));
		btnHoldLeft.setEnabled(false);
 
		btnHoldRight = new JButton("Hold");
		btnHoldRight.setPreferredSize(new Dimension(150,25));
		btnHoldRight.setEnabled(false);
 
		btnHoldMiddle = new JButton("Hold");
		btnHoldMiddle.setPreferredSize(new Dimension(150,25));
		btnHoldMiddle.setEnabled(false);
 
 
		lblBalance = new JLabel("Balance");
		lblBalance.setPreferredSize(new Dimension(100,50));
		lblBalance.setBackground(Color.black);
 
		lblWinnings = new JLabel("Winnings");
		lblWinnings.setPreferredSize(new Dimension(100,50));
		lblWinnings.setBackground(Color.black);
 
 
		add(txtBalance);
		add(lblBalance);
		add(txtWinnings);
		add(lblWinnings);
		add(btnStart);
		add(btnBye);
		add(btnSpinPicture1);
		add(btnSpinPicture2);
		add(btnSpinPicture3);
		add(btnHoldLeft);
		add(btnHoldMiddle);
		add(btnHoldRight);
		add(btnMoney);
		add(btnCashout);
		add(btnTransfer);
 
 
		btnMoney.addActionListener(this);
		btnStart.addActionListener(this);
		btnTransfer.addActionListener(this);
		btnBye.addActionListener(this);
		btnCashout.addActionListener(this);
		btnHoldLeft.addActionListener(this);
		btnHoldMiddle.addActionListener(this);
		btnHoldRight.addActionListener(this);
 
		iconPic1 = new ImageIcon("bin/Pic1.jpg");
		iconPic2 = new ImageIcon("bin/Pic2.jpg");
		iconPic3 = new ImageIcon("bin/Pic3.jpg");
		iconPic4 = new ImageIcon("bin/Pic4.jpg");
		iconPic5 = new ImageIcon("bin/Pic5.jpg");
		iconPic6 = new ImageIcon("bin/Pic6.jpg");
		iconPic7 = new ImageIcon("bin/Pic7.jpg");
		iconPic8 = new ImageIcon("bin/Pic8.jpg");
 
		random = new Random();
 
		setTitle("Electronic Bandit");
		setSize(500,320);
 
 
}	
 
 
	public void actionPerformed (ActionEvent event)
 
{
 
		if(event.getSource()==btnBye)
		{
			JOptionPane.showMessageDialog(null,"Thank You For Playing....Bye!!!");
 
			System.exit(0);
		}
 
		if(event.getSource()==btnCashout)
		{
		JOptionPane.showMessageDialog(null, "You have collected  " +winnings+"p, Bye!!!");
 
			System.exit(0);
 
		}
 
		if (event.getSource()==btnMoney){
			txtBalance.setForeground(Color.black);
			balance = balance + 100;
			txtBalance.setText(balance + "");
 
		}
 
		if (event.getSource() ==btnMoney)
		{
		btnStart.setEnabled(true);
		}
 
		if (event.getSource()==btnTransfer){
			balance = winnings + balance;
			txtBalance.setText(balance +"");
			JOptionPane.showMessageDialog(null,"You have transferred  " +winnings+"p");
		}
 
		if (event.getSource()==btnTransfer){
			txtWinnings.setText("0");
		}
 
		if (event.getSource()==btnTransfer){
			winnings =0;
			btnTransfer.setEnabled(false);
			JOptionPane.showMessageDialog(null,"You now have no winnings to transfer!!!");
 
		}
		if(winnings >=20){
			btnTransfer.setEnabled(true);
			}
 
 
		if (event.getSource() ==btnStart)
		{
			btnHoldLeft.setEnabled(true);
			btnHoldMiddle.setEnabled(true);
			btnHoldRight.setEnabled(true);
		}
 
 
 
			if (event.getSource() ==btnStart){
				balance = balance - 20;
				txtBalance.setText(balance +"");
			}
			if (balance>=20){
				btnStart.setEnabled(true); 
 
		}
 
			if(balance <20){
				balance = 0;
 
				btnStart.setEnabled(false);
 
				JOptionPane.showMessageDialog(null,"After this spin you will not have enough Credit. Please insert more Money!!!");
			}
 
 
 
			if(event.getSource()==btnStart){
			counter = 0;
			timer = new Timer(100,this);
			timer.start();
 
		}
 
		if(event.getSource()==timer) {
			setIcon ();
		}
		if(event.getSource()==timer){
			timer.stop();
		}
}
 
 
 
		public void stopTimer(){
			timer.stop();
 
	}
 
 
 
 
 
public void setIcon(){
 
			one = random.nextInt(8);
 
 
		switch (one){
 
		case 0:
			btnSpinPicture1.setIcon(iconPic1);
			one = 1;
			counter ++;
			break;
 
		case 1:
			btnSpinPicture1.setIcon(iconPic2);
			one = 2;
			counter ++;
			break;
 
		case 2:
			btnSpinPicture1.setIcon(iconPic3);
			one = 3;
			counter ++;
			break;
 
		case 3:
			btnSpinPicture1.setIcon(iconPic4);
			one = 4;
			counter ++;
			break;
 
		case 4:
			btnSpinPicture1.setIcon(iconPic5);
			one = 5;
			counter ++;
			break;
 
		case 5:
			btnSpinPicture1.setIcon(iconPic6);
			one = 6;
			counter ++;
			break;
 
		case 6:
			btnSpinPicture1.setIcon(iconPic7);
			one = 7;
			counter ++;
			break;
 
		case 7:
			btnSpinPicture1.setIcon(iconPic8);
			one = 0;
			counter ++;
			break;
		}
 
		two = random.nextInt(8);
 
switch (two){
 
		case 0:
			btnSpinPicture2.setIcon(iconPic1);
			two = 1;
			counter ++;
			break;
 
		case 1:
			btnSpinPicture2.setIcon(iconPic2);
			two = 2;
			counter ++;
			break;
 
		case 2:
			btnSpinPicture2.setIcon(iconPic3);
			two = 3;
			counter ++;
			break;
 
		case 3:
			btnSpinPicture2.setIcon(iconPic4);
			two = 4;
			counter ++;
			break;
 
		case 4:
			btnSpinPicture2.setIcon(iconPic5);
			two = 5;
			counter ++;
			break;
 
		case 5:
			btnSpinPicture2.setIcon(iconPic6);
			two = 6;
			counter ++;
			break;
 
		case 6:
			btnSpinPicture2.setIcon(iconPic7);
			two = 7;
			counter ++;
			break;
 
		case 7:
			btnSpinPicture2.setIcon(iconPic8);
			two = 0;
			counter ++;
			break;
 
}
three = random.nextInt(8);
 
switch (three){
 
		case 0:
			btnSpinPicture3.setIcon(iconPic1);
			three = 1;
			counter ++;
			break;
 
		case 1:
			btnSpinPicture3.setIcon(iconPic2);
			three = 2;
			counter ++;
			break;
 
		case 2:
			btnSpinPicture3.setIcon(iconPic3);
			three = 3;
			counter ++;
			break;
 
		case 3:
			btnSpinPicture3.setIcon(iconPic4);
			three = 4;
			counter ++;
			break;
 
		case 4:
			btnSpinPicture3.setIcon(iconPic5);
			three = 5;
			counter ++;
			break;
 
		case 5:
			btnSpinPicture3.setIcon(iconPic6);
			three = 6;
			counter ++;
			break;
 
		case 6:
			btnSpinPicture3.setIcon(iconPic7);
			three = 7;
			counter ++;
			break;
 
		case 7:
			btnSpinPicture3.setIcon(iconPic8);
			three = 0;
			counter ++;
			break;
 
 
			}
checkWin();  
 
if(counter >20)
	stopTimer();
 
}
 
public void checkWin(){
 
	if (one == two && two == three){
	winnings = winnings + 100;
	txtWinnings.setText(winnings + "");
	JOptionPane.showMessageDialog(null, "Winner £1.00");
	}
	else if ( one == two || one == three || two == three){
	winnings = winnings + 50;
	txtWinnings.setText(winnings + "");
	JOptionPane.showMessageDialog(null, "50p Win");
	}
	else if (three == 1){
	winnings = winnings + 20;
	txtWinnings.setText(winnings + "");
	JOptionPane.showMessageDialog(null, "20p Win");
	}
	}
 
	}

No i havn't. Stupid question here but how do i do that?

i have tried what you said on timer and i got the following response:

javax.swing.Timer@a3bcc1

is that correct or not?

I have given up on the hold buttons for now cause they've been making me go mad for days, so i thought id get this spin part to work since im nearly there

Everything works as how it should. I don't know whether the response i got from the timer was correct or not

sorry i meant everything but the timer. I don't know whether im blind or what but im confused as to why the pictures dont act the same as the pictures do in the SpinSimulator program.

Yeah i have been trying to make it a bit tidier and will try to do the array. For the moment it works so im trying to concentrate on the things that arn't. then im going to sort that out

on the SpinSimulator program, when you click the button "Spin", it flicks through three pictures giving the effect that its spinning. When you click the button "Play" on the ElectronicSlot8 program the pictures dont give that effect. They only display one random image instead of flicking through all of them.

Does that make sense? im useless at explaining things.

All i want to do is to get the ElectronicSlot8 program to have the same effect as the SpinSimulator program when the "Play" button is clicked. The code is the same but i'm not getting the same outcome

yeah i would like the three image displays to do that. Just like the SpinSimulator.

oh i thought the coding for them was both the same and thought they would both have done that.

Could you please tell me what am i actually missing?

I have managed to merge the if statements so it looks more tidy.

I cant really make an array because i havnt learnt about these yet and its only testing us on what we have already learnt.

I have tried to do some debugging and it only shows that one picture has been shown in each box. Not sure what else im meant to get out of this

public void actionPerformed (ActionEvent event)
 
{
 
		if(event.getSource()==btnBye){
			JOptionPane.showMessageDialog(null,"Thank You For Playing....Bye!!!");
				System.exit(0);
		}
 
		if(event.getSource()==btnCashout){
			JOptionPane.showMessageDialog(null, "You have collected  " +winnings+"p, Bye!!!");
				System.exit(0);
		}
 
		if (event.getSource()==btnMoney){
			txtBalance.setForeground(Color.black);
			balance = balance + 100;
			txtBalance.setText(balance + "");
			btnStart.setEnabled(true);
		}
 
 
		if (event.getSource()==btnTransfer){
			balance = winnings + balance;
			txtBalance.setText(balance +"");
			txtWinnings.setText("0");
			JOptionPane.showMessageDialog(null,"You have transferred  " +winnings+"p");
			winnings =0;
			btnTransfer.setEnabled(false);
			JOptionPane.showMessageDialog(null,"You now have no winnings to transfer!!!");
		}
 
		if(winnings >=20){
			btnTransfer.setEnabled(true);
			}
 
 
		if (event.getSource() ==btnStart){
			btnHoldLeft.setEnabled(true);
			btnHoldMiddle.setEnabled(true);
			btnHoldRight.setEnabled(true);
			balance = balance - 20;
			txtBalance.setText(balance +"");
		}
 
			if (balance>=20){
					btnStart.setEnabled(true); 
			}else{
					btnStart.setEnabled(false);
						JOptionPane.showMessageDialog(null,"After this spin you will not have enough Credit. Please insert more Money!!!");
			}
 
			if(event.getSource()==btnStart){
			counter = 0;
			timer = new Timer(100,this);
			timer.start();
 
		}
 
		if(event.getSource()==timer) {
			setIcon ();
			timer.stop();
 
		}
}
 
		public void stopTimer(){
			timer.stop();
 
	}

thats the code that i sorted out. I keep trying different things cause im kinda of lost at the moment.