Discussion :

[Mike888]

Salut !


J'ai un bouton, qui, quand j'appuie dessus me déclenche la génération aléatoire d'un nombre entre 1 et 25. Mais il y a qqch qui doit clocher car quelques fois, mon bouton reste comme "enfoncé" quand je clique dessus et le prog me génère alors une floppée de nombres sans plus s'arrêter.
Le problème ne vient pas du listener mais bien du code avec le random, j'ai vérifié !...

Si vous voyez ce qui cloche...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
public int nbrrandom(){
		int r;
		boolean test=false;
		do {
			r=(int)(Math.random()*25+1);System.out.println(r);
			for (int i=0;i<5;i++) {
			if (billes[i]==r) test=true;
			}
		}
		while (test==true);
		billes[5-nbrbillesrestantes]=r;
		return r;
	}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
 
...
 
gbc.gridx = 2; gbc.gridy = 5;		
gbc.gridwidth = 0; gbc.gridheight = 1;
gbc.weightx = 1; gbc.weighty = 1;	
gbc.gridheight=GridBagConstraints.REMAINDER;
jb9.setText("Lancer la bille");		
MouseListener buttonListener2 = new MouseAdapter(){
	public void mouseClicked(MouseEvent ev) {
		JButton obj = (JButton)ev.getSource();
		if (obj.getActionCommand().startsWith("jb9")) {
			if (SwingUtilities.isLeftMouseButton(ev) || ev.isPopupTrigger()){
				if (bingomodele.getMise()>0) {
					if (bingomodele.getNbrbillesrestantes()>0){								
						int nbr=bingomodele.nbrrandom();
 
				/*  ...   */
 
					}
				}
			}
		}					
	}
};
jb9.setActionCommand("jb9");
jb9.addMouseListener(buttonListener2);
jPanel1.add(jb9,gbc);
 
...

[eric39]

Bonjour,

La méthode 'nbrrandom', utilise une boucle do while, qui continu tant que test==true. Si dans ta boucle la valeur de test passe à true, la boucle ne s'arrêtera jamais (car test ne passe jamais à false.).

Donc, si tu passes dans

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
if (billes[i]==r) test=true;

La boucle ne s'arrêtera jamais.

a+

[Mike888]

Ah mais oui... Je suis trop bête...

Merci !