Valeur du getter impossible à tester

Question

Bonsoir à tous,

J'ai un petit problème dans le code suivant : 
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.Timer;

public class Ex {
  int attribut ;
  Timer chrono;
  private ActionListener refreshChrono ;
  
  public Ex(){
   attribut = 0;
   chrono = new Timer(1000,refreshChrono);
   refreshChrono = new ActionListener() {
   public void actionPerformed(ActionEvent e1)
    { 
      attribut = 1;
    }
    };
     
     chrono = new Timer(1000,refreshChrono);
     chrono.start();
     
    }
  
  public static void main(String[] args){
   
   Ex exmple = new Ex();
   System.out.println(exmple.get());
   
   while(true){
     System.out.println(exmple.get());
     
     if(exmple.get() == 1) {
      System.out.println("CoucouMain");
     }
    }
 }

int get() {
   return attribut;
  }
}


En effet quand dans le while du main, je print exmple.get() je rentre dans le if alors que si je commente le print je n'y rentre plus.. 
Si quelqu'un sait m"expliquer pourquoi je suis preneur !
Merci d'avance :)

KX · Accepted Answer

Bonjour,

C'est une question très intéressante, il m'a fallu quelques minutes avant de trouver la solution, c'est le genre de piège où je pourrais encore me faire avoir.

Déjà on peut simplifier ton code et j'en profite pour te montrer une alternative afin d'éviter d'utiliser du code Swing alors que tu n'as pas d'interface graphique (mais le bug reste le même).

import java.util.concurrent.*;
// import javax.swing.Timer;

public class Ex {
    private int attribut;

public Ex() {
        // new Timer(1000, e -> attribut = 1).start();
        Executors.newSingleThreadScheduledExecutor().scheduleAtFixedRate(() -> attribut = 1, 1L, 1L, TimeUnit.SECONDS);
    }

public static void main(String[] args) {
        Ex ex = new Ex();
        while (true) {
            // System.out.println(ex.attribut);
            if (ex.attribut == 1) {
                System.out.println("CoucouMain");
            }
        }
    }
}
Pour corriger le bug, il faut rajouter le mot clé volatile à ton attribut :
private volatile int attribut;
Tu trouveras beaucoup d'explication sur ce mot clé, par exemple ici, ce qu'il se cache derrière c'est une optimisation de la JVM qui met en cache la valeur de l'attribut, mais il faut la forcer à se mettre à jour.

Valeur du getter impossible à tester

1 réponse

Votre réponse