Ending a Timer within itself

[Jean Fontaine]

Hi everybody !


I've got a little problem concerning my program :
I want to end a timer when a certain value reaches zero, but it keeps telling me "local variable timer1 is accessed from within inner class and needs to be declared final", but I don't want to make it final !
I tried many differnet ways, and in the end made my code exactly like this one (or so I think) : http://www.java2s.com/Code/JavaAPI/javax.swing/Timerstop.htm but it still doesn't work...
It's lines 69 and 78.
Here is my code :

public class Combatbak_1 extends javax.swing.JFrame {
     int dinozHP;
     int dinozXP;
     int dinozFireAttack;
     int dinozDefenseNormal;
     
     int monsterHP;
     int monsterAttack;
     int monsterArmor;
     
     int lostHp;
     String monsterName;
     boolean Dinozturn;
     
     

    /** Creates new form Combatbak */
    public Combatbak_1() {
        initComponents();
        Goupignon goupignon = new Goupignon();
        Dinoz dinoz = new Dinoz ();
        
        
        dinozHP = dinoz.getHP();
        dinozXP = dinoz.getXP();
        dinozFireAttack = dinoz.getFireElement()*3+dinoz.getFireElement();
        dinozDefenseNormal = dinoz.getDefenseNormal();
        
        monsterHP = goupignon.getHP();
        monsterAttack = goupignon.getAttack();
        monsterArmor = goupignon.getDefense();
        
        monsterName = goupignon.getName();
        
        Dinozturn = false;
         Timer timer1;
        
        
        
        ActionListener a = new ActionListener () { 
            
            @Override
            public void actionPerformed  ( ActionEvent e) {  
                
          if (dinozHP >= 0 && monsterHP >= 0) {
               if (Dinozturn == false) {   
                   dinozHP = dinozHP -(monsterAttack - dinozDefenseNormal);
                   lostHp =  monsterAttack - dinozDefenseNormal;
                   jLabel1.setText("Goupignon attaque ! Votre Dinoz perd " + lostHp + " PV.");
                   jLabel2.setText("Dinoz HP = " + dinozHP);
                    
               }
               else if (Dinozturn ==true) {  
                   monsterHP = monsterHP - (dinozFireAttack - monsterArmor);
                   lostHp = dinozFireAttack - monsterArmor;
                   jLabel1.setText("Votre Dinoz attaque ! Le Goupignon perd " + lostHp + " PV.");
                   jLabel3.setText("Monstre HP : " + monsterHP);
                    
               }
               if (Dinozturn == false) {
                   Dinozturn = true;
               }
               else if (Dinozturn == true) {
                   Dinozturn = false;
               }
            }  
           if (dinozHP <= 0) {
              jLabel1.setText("Votre Dinoz est mort ! Vous avez perdu !");
              timer1.stop();
          }
                   
                         
                else if (monsterHP <= 0) {                
              jLabel1.setText("Votre adversaire est mort ! Vous avez gagné !");
              if (monsterName.equals("Goupignon") ) {
                  dinozXP = dinozXP + 25;
                  jLabel1.setText("Vous obtenez " + dinozXP + " XP !");
                  timer1.stop();
                  
              }
          }
            }
            
           
        
        };
        
        
         
         
                 /* Fin du timer1 */
                 timer1 = new Timer (1000, a);
                timer1.start();
        
    }

Thank you for your help !

Oracion

[Rob Camick]

if (dinozHP >= 0 && monsterHP >= 0) {   

Why don't you have a problem with the "dinozHP" and "monsterHP" variables?

In other words how are those variables defined differently than the "timer1" variable?

[Rob Spoor]

Can't you use e.getSource(), after casting it to Timer?

[Jean Fontaine]

Wow !
Thanks a lot !
I just had to add "Timer timer1;" at the beginning !
I didn't really get the e.getsource thing, but I'll try to find more about it tomorrow !
Thanks again both of you !

Oracion