Discussion :

[Bissmark]

Bonsoir,
Je debute en Java et voila mon code;
Je genere donc un nombre aléatoirement, que je souhaite comparer à un tableau de valeur. Mais mon boolean me retourne tout le temps la meme valeur(false)
Je sais que je pourrais utiliser "%2" mais je prefere cette methode puisqu'elle est utilisés plusieurs fois pour qualifier le nombre(paire, rouge)...Je me comprend...
Pour ce qui du boolean je l'ai declaré dans les attributs et initialisé à faux.
Quelqu'un peut m'aider?

[thelvin]

Remarque préliminaire #1 : pour montrer ton code, fais un simple copier/coller et entoure-le de la balise [CODE]. N'attache pas d'image, ça sert à autre chose.

Remarque préliminaire #2 : cet extrait de code n'est pas suffisant. On ne va pas deviner ce que sont toutes ces variables. Il pourrait y avoir un problème de ce côté-là.

Concernant ton problème, tu devrais peut-être arrêter la boucle une fois que tu as trouvé un nombre qui te convient. En l'état, ta boucle parcourt absolument tout le tableau, sans s'arrêter avant la fin, et ne garde que le tout dernier résultat.

[Bissmark]

Voici le reste;
Pour ce qui est d'arrêter la boucle j'ai essayé avec un break mais n'y suis pas parvenu!

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
protected boolean nombreEstPair=false;
protected int[]pair=new int[]{2,4,6,8,10,12,14,16,18,20,22,24,26,28,30,32,34,36};
public int tirage(){
		Random r=new Random();
		int nombre=r.nextInt(37);
	//permet d'obtenir un nombre pseudo-aléatoire compris entre 0(inclus) et 37(exclus)
		return nombre;
	}
 
public boolean estPair(){
		for (int i=0;i<this.pair.length;i++){
			if (this.nombre==pair[i]){
				this.nombreEstPair=true;
			}else{this.nombreEstPair=false;
			}	
		}return nombreEstPair;	
	}

[thelvin]

Pour ce qui est d'arrêter la boucle j'ai essayé avec un break mais n'y suis pas parvenu!

Mais encore ? Tu aurais peut-être pu nous montrer ?

Fin bon, exemple d'utilisation :

Code :

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

1
2
3
4
5
6
7
for(int i = 0; i < nombreDIterations(); i++) {
  faireDesTrucs(i);
  if(ilEstTempsDeSArreter(i)) {
    break;
  }
  faireDAutresTrucsSiOnVeut(i);
}

Sinon, vu que c'est dans une méthode qui contient juste cette boucle, on peut en sortir avec un return :

Code :

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

1
2
3
4
5
6
7
8
9
boolean verifierUnTruc() {
  for(int i = 0; i < nombreDIterations(); i++) {
    int calcul = calculerPourCetteIteration(i);
    if(leCalculVerifieBien(calcul)) {
      return true;
    }
  }
  return false;
}

[Bissmark]

Mais encore ? Tu aurais peut-être pu nous montrer ?

Fin bon, exemple d'utilisation :

Code :

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

1
2
3
4
5
6
7
for(int i = 0; i < nombreDIterations(); i++) {
  faireDesTrucs(i);
  if(ilEstTempsDeSArreter(i)) {
    break;
  }
  faireDAutresTrucsSiOnVeut(i);
}

Sinon, vu que c'est dans une méthode qui contient juste cette boucle, on peut en sortir avec un return :

Code :

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

1
2
3
4
5
6
7
8
9
boolean verifierUnTruc() {
  for(int i = 0; i < nombreDIterations(); i++) {
    int calcul = calculerPourCetteIteration(i);
    if(leCalculVerifieBien(calcul)) {
      return true;
    }
  }
  return false;
}

Mais excuses il est bien vraie que mettre ma tentative concernant le break aurai été bien venu. Mais en toute franchise j'y vais à tâton est par consequent avais essayé le break avant de demander votre aide;
Alors voila ce que j'ai avec un break (ca ne marche pas) en essayant d'appliquer la methode ci-dessus.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
	public boolean estPair(){
		boolean est_pair=false;
		for(int i = 0; i < pair.length; i++) {
			  est_pair=true;
			  if(this.nombre==pair[i]) {
			    break;
			  }
 
			}
		return est_pair;
	}

[v.charlet]

Tu as essayé de coder comme cela (en espérant que this.nombre contient la bonne valeur que tu veux tester)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public boolean estPair(){
 
                this.nombreEstPair=false;
		for (int i=0;i<this.pair.length;i++){
			if (this.nombre==pair[i]){
				this.nombreEstPair=true;
                                break;
			}
 
		}
                return this.nombreEstPair;	
	}

[tchize_]

Il faut penser à arrêter la boucle quand tu as trouvé ton nombre dans le tableau.

PS: pourquoi se casser la tête à faire un printscreen, à le découper et à l'attacher plutot que de copier / coller ton code directement sur le forum. En plus, ce n'est pas pratique à lire /tester / corriger.