Discussion :

[MrMichou]

Bonjour,
étant débutant en programmation, j'ai travail à faire sur un MasterMind, mais la n'est pas le problème.
Je voulais savoir pourquoi jDeveloper me cite comme erreur que c'est un état impossible a atteindre.

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
        while (pion1 == pion2 || pion1 == pion3 || pion1 == pion4 ){
        pion1 = (int) (Math.random() * 8 ) + 1;
        }
        return pion1;
        while (pion2 == pion1 || pion2 == pion3 || pion2 == pion4 ){
        pion2 = (int) (Math.random() * 8 ) + 1;
        }
        return pion2;
        while (pion3 == pion1 || pion3 == pion2 || pion3 == pion4 ){
        pion3 = (int) (Math.random() * 8 ) + 1;
        }
        return pion3;
        while (pion4 == pion1 || pion4 == pion2 || pion4 == pion3 ){
        pion4 = (int) (Math.random() * 8 ) + 1;
        }
        return pion4;

Comment pourrais-je faire en sorte que cela génère des chiffres différent, et si j'ai commis de grosses fautes ou sont-elles (de syntaxe ou autres )?


Cordialement MrMichou !

[rg77140]

Bonjour,

Je voulais savoir pourquoi jDeveloper me cite comme erreur que c'est un état impossible a atteindre.

Code :

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

1
2
3
4
5
        while (pion1 == pion2 || pion1 == pion3 || pion1 == pion4 ){
        pion1 = (int) (Math.random() * 8 ) + 1;
        }
        return pion1;
       ...

Bonjour,

return quitte immédiatement une fonction en retournant la valeur qui suit le return. Tout ce qui est donc à la suite d'un return ne sera jamais exécuté. Donc tout ce qui est après "return pion1" ne sera jamais exécuté dans ton code, d'où l'erreur qui t'es remontée par ton IDE.

J'ai pas bien compris ce que tu cherches à faire avec cet algorithme. Essaie de l'expliquer en français

Romain.

[MrMichou]

Oui, désolé je l'ai fais un peux à la va vite mon explication :s.

Dans ma méthode je dois générer de façon aléatoire la combinaison de 4 chiffres, mais les 4 chiffres ne doivent pas être semblable.

Merci pour ton explication rg77140.

[rg77140]

Voilà un bout de code qui devrait t'aider :

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
 
/**
* Méthode permettant de generer un nombre different des trois passés en paramètres
*/
public static int genererNombreDifferent(int valeurExclue1, int valeurExclue2, int valeurExclue3)){
        int nombreAleatoire;
        do{
                nombreAleatoire = (int) (Math.random() * 8 ) + 1;
        } while (nombreAleatoire== valeurExclue1 || nombreAleatoire== valeurExclue2 || nombreAleatoire == valeurExclue3);
 
        return nombreAleatoire;
}
 
// génération des 4 nombres
pion1 = genererNombreDifferent(pion2,pion3,pion4);
pion2 = genererNombreDifferent(pion1,pion3,pion4);
pion3 = genererNombreDifferent(pion1,pion2,pion4);
pion4 = genererNombreDifferent(pion1,pion2,pion3);

J'ai pas testé il y a peut-être des petites adaptations...

Romain.

[MrMichou]

Wow , j'aurais jamais penser à utiliser un do while.
Merci beaucoup de ton aide !!!

[professeur shadoko]

tu veux créer une liste de N nombres de 1 à X tirés au hasard sans répétition?
Si X n'est pas trop grand:
- générer une liste des nombres de 1 à X
- faire un Collection.shuffle sur cette liste
- prendre les N premiers éléments
Il y a plus optimum (avec Random nextInt()), mais cette recette est facile à coder (clin d'oeil à ceux qui connaissent APL )

PS: lire la littérature sur les algorithmes de MasterMind : il existe un algo extrèmement performant (pas le plus efficace, mais assez proche de l'optimum théorique) qui s'écrit en quelques lignes de code.

[Sharcoux]

J'ajouterai que cette section de code n'est pas très logique :

pion1 = genererNombreDifferent(pion2,pion3,pion4);
pion2 = genererNombreDifferent(pion1,pion3,pion4);
pion3 = genererNombreDifferent(pion1,pion2,pion4);
pion4 = genererNombreDifferent(pion1,pion2,pion3);

Pourquoi pion1 ne pourrait pas être égal à l'ancienne valeur de pion2, puisque pion2 va changer de valeur.

Une approche correcte serait plutôt :

Code :

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

1
2
3
4
pion1 = genererNombreDifferent();
pion2 = genererNombreDifferent(pion1);
pion3 = genererNombreDifferent(pion1,pion2);
pion4 = genererNombreDifferent(pion1,pion2,pion3);

Bien entendu, on voit que dans ce cas, la réponse la plus pertinente est celle de professeur Shadoko qui évite de faire des tours de boucle pour rien et d'écrire une fonction compliquée pour chaque cas, ou d'utiliser l'Ellipse (...) de Java 5.0.

[MrMichou]

Merci beaucoup pour vos réponse !
@professeur shadoko, je vais regarder pour les algo. même si mon but est d'essayer d'apprendre par moi même, mais cela a l'air intéressant , même si je pense ne pas avoir encore le niveau pour pouvoir dev un mastermind de haut niveau, (Même si cela ne nécessite pas un niveau de pro. ).
Surtout que pour le moment je ne cherche pas a faire un MasterMind de haut niveau avec un environnement Graphique, bien que j'aimerais .
Merci beaucoup.
@Sharcoux, Merci pour ta correction, je n'ai pas encore essayer. Ton raisonnement à l'air logique, j'vais regarder sa de suite.

Merci beaucoup pour vos réponses !!!
Ça fait plaisir de recevoir de l'aide !