Solveur du jeu le compte est bon

**mty60000** · 01/04/2017, 15h53

Bonjour à tous, j'essaye actuellement de développer un solveur pour le jeu du compte est bon. J'ai commencé par faire un algo et je l'ai testé en Java. Cela ne va pas du tout.
Le but dans mon algo est de construire un arbre avec pour chaque nœud, une solution avec ses informations. A chaque fois que je crée un nœud je le compare avec la "meilleure" solution actuelle (la solution la plus proche avec le moins d'opération).
Pour cela j'utilise une fonction récursive. Le problème c'est que je ne l'appel pas avec les bonnes informations.
Voici mon programme pour le moment:

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
 
import java.io.*;
import java.util.ArrayList;
 
class Information {
	protected long P1;
	protected char operateur;
	protected long P2;
	protected long resultat;
	protected int niveau;
	protected ArrayList<Long> plaquettes;
	protected int nbplaquettes;
 
	public Information(long P1, char operateur, long P2, long resultat, int niveau, ArrayList<Long> plaquettes, int nbplaquettes) {
		this.P1 = P1;
		this.operateur = operateur;
		this.P2 = P2;
		this.resultat = resultat;
		this.niveau = niveau;
		this.plaquettes = plaquettes;
		this.nbplaquettes = nbplaquettes;
	}
	public Information(int niv, ArrayList<Long> pla, int nbpla) {
		this.P1 = -1;
		this.operateur = ' ';
		this.P2 = -1;
		this.resultat = 
		this.niveau = niv;
		this.plaquettes = pla;
		this.nbplaquettes = nbpla;
	}
	public long getP1() {
		return P1;
	}
	public char getOperateur() {
		return operateur;
	}
	public long getP2() {
		return P2;
	}
	public long getResultat() {
		return resultat;
	}
	public int getNiveau() {
		return niveau;
	}
	public ArrayList<Long> getPlaquettes() {
		return plaquettes;
	}
	public int getNbPlaquettes() {
		return nbplaquettes;
	}
 
}
 
class Noeud {
	protected Information info;
	//protected ArrayList<Noeud> fils;
	protected Noeud pere;
 
	/*public Noeud(Information info, ArrayList fils, Noeud pere){
		this.info = info;
		this.fils = fils;
		this.pere = pere;
	}*/
	public Noeud(Information info, Noeud pere){
		this.info = info;
		this.pere = pere;
	}
	public Noeud(Information infor) {
		this.info = infor;
		//this.fils = null;
		this.pere = null;
	}
	public Noeud() {
		this.info = null;
		//this.fils = null;
		this.pere = null;
	}
	public Noeud setInfo(Information i) {
		this.info.P1 = i.P1;
		this.info.operateur = i.operateur;
		this.info.P2 = i.P2;
		this.info.resultat = i.resultat;
		this.info.niveau = i.niveau;
		this.info.plaquettes = i.plaquettes;
		this.info.nbplaquettes = i.nbplaquettes;
		return this;
	}
	public Noeud getPere() {
		return pere;
	}
	public Information getInfo() {
		return info;
	}
	/*public ArrayList getFils() {
		return fils;
	}*/
}
 
public class Solution {
 
	public static long pow(long x, long y) {
		long res = x;
		for(int i=0; i<y; i++) {
			res=res*x;
		}
		return res;
	}
 
	public static long Operation(long N1, long N2, int ope) {
		if(ope == 0) {
			return(N1+N2); 
		}
		else{
			if(ope == 1) {
				if(N1>N2) {
					return(N1-N2); 
				} 
				else {
					return(N2-N1);
				}
			}
			else{
				if(ope == 2) {
					return(N1*N2); 
				}
				else{
					if(ope == 3) {
						if((N1>N2) && (N1%N2==0) && (N2 != 0)) {
							return(N1/N2); 
						}
						else if((N2>N1) && (N2%N1==0) && (N1 != 0)) {
							return(N2/N1);
						}
					}
					else{
						if(ope == 4) {
							return(pow(N1,N2));
						}
						else{
							if(ope == 5) {
								return(pow(N2,N1)); 
							}
							else{
								if(ope == 6) {
									if(N2 != 0) {
										return(N1%N2);
									} 
								}
								else{
									if(ope == 7) {
										if(N1 != 0) {
											return(N2%N1);
										}
									}
								}
							}
						}
					}
				}
			}
		}
		return(-1);
	}
 
	public static void Affiche(Noeud best) {
		int niv = best.info.niveau;
		while(niv>1) {
			System.out.println("op1: "+best.info.getP1()+"op2: "+best.info.getP2()+"resultat: "+best.info.getResultat());
			best.info.P1 = best.pere.info.getP1();
			best.info.operateur = best.pere.info.getOperateur();
			best.info.P2 = best.pere.info.getP2();
			niv--;
		}
	}
 
	static int i=0;
	static int j=0;
	static int o=0;
	static int calc = 0;
	static int niveau = 1;
 
	public static void Find(Noeud N, Noeud best, int cible) {
		//System.out.println("Je suis dans Find");
 
		long res = 0;
		char operateurs [] = {'+','-','*','/','^','%'};
		long test = 0;
		long test2 = 0;
		long op1 = 0;
		long op2 = 0;
		int nbplaques = 0;
		char op;
		long tmp1 = 0;
		long tmp2 = 0;
 
 
		tmp1 = best.info.getResultat();
		//System.out.println("---CIBLE:"+cible);
		//System.out.println("---TMP1:"+tmp1);
		test = tmp1 - cible;		
		if(test<0){
			test=test*(-1);
		}		
		tmp2 = N.info.getResultat();
		//System.out.println("---TMP2:"+tmp2);
		test2 = tmp2 - cible;
 
		if(test2<0){
			//System.out.println("JE SUIS NEGATIF");
			test2=test2*(-1);
		}
		//System.out.println("TEST2:"+test2);
 
		if(test2 < test) {
			//System.out.println("N.P1 INFERIEUR A BEST");
			best.info.P1 = N.info.getP1();
			best.info.operateur = N.info.getOperateur();
			best.info.P2 = N.info.getP2();
			best.info.resultat = N.info.getResultat();
			best.info.niveau = N.info.getNiveau();
			best.info.plaquettes = N.info.getPlaquettes();
		}
		else {
			if(test2 == test) {
				if(N.info.niveau < best.info.niveau) {
					best.info.P1 = N.info.getP1();
					best.info.operateur = N.info.getOperateur();
					best.info.P2 = N.info.getP2();
					best.info.resultat = N.info.getResultat();
					best.info.niveau = N.info.getNiveau();
					best.info.plaquettes = N.info.getPlaquettes();
				} 
			}
		}
 
 
		int max = N.info.getNbPlaquettes();
		//System.out.println("---///"+max+"---");
 
 
 
 
 
 
		if(max>=2) {
			ArrayList<Long> tmp = new ArrayList<Long>();
			tmp.addAll(N.info.getPlaquettes());
			//System.out.println(tmp.get(1));
 
			tmp.remove(0);
 
			Information infom = new Information(0,' ',0,100000,6,N.info.getPlaquettes(),1);
			Noeud N1 = new Noeud(infom);
 
			for(i=0; i<max-1; i++) {
				for(j=i+1; j<max; j++) {					
					for(o=0; o<8; o++) {
						int actuelle = niveau + 1;						
						res = Operation(N.info.getPlaquettes().get(0), N.info.getPlaquettes().get(1), o);
						tmp.add(res);
						tmp.remove(0);						
						op1 = N.info.getPlaquettes().get(0);
						//System.out.println(op1);
						if(o==4 || o==5) {
							op = operateurs[4];
						}
						else {
							if (o==6 || o==7) {
								op = operateurs[5];
							}
							else {
								op = operateurs[o];
							}
						} 
						op2 = N.info.getPlaquettes().get(1);
						nbplaques = N.info.getNbPlaquettes()-1;
						Information infor = new Information(op1,op,op2,res,actuelle,tmp,nbplaques);
						//System.out.println(infor.P1);
						N1.setInfo(infor);
						//System.out.println("-----------------------");
						System.out.println("NIVEAU: "+best.info.getNiveau());
						System.out.println("Operateur1: "+N1.info.getP1()+" Operation: "+N1.info.getOperateur()+" Operateur2: "+N1.info.getP2()+" Resultat: "+N1.info.getResultat());
						//System.out.println(N1.info.getP1());
						N1.pere = N;								
						Find(N1, best, cible);
						calc ++;
						//System.out.println("----Calculs----"+calc);
					}
				}
			}
 
			//Affiche(best);
		}		
		if((i==max-1) && (j==max) && (o==7))  {
			//System.out.println("Je vais dans Affiche");
 
			Affiche(best);
		}
	}
 
	public static void main(String[] args) {
 
		ArrayList<Long> init = new ArrayList<Long>();
		//ArrayList fils = new ArrayList();
		long tmp = 0;
		tmp = 10;
		init.add(tmp);
 
		tmp = 5;
		init.add(tmp);
 
		tmp = 7;
		init.add(tmp);
 
		tmp = 2;
		init.add(tmp);
 
		tmp = 25;
		init.add(tmp);
 
		tmp = 9;
		init.add(tmp);
 
 
		int totalPlaquettes = 6;
		int target = 859;
		int valeurMax = 100000;
 
		Information infor = new Information(0,' ',0,0,1,init,totalPlaquettes);
		Information infom = new Information(0,' ',0,valeurMax,6,init,1);			
		//fils.add(meilleur);
		Noeud racine = new Noeud(infor,null);
		//System.out.println(racine.info.getPlaquettes().get(1));
		Noeud meilleur = new Noeud(infom,racine);
 
		Find(racine, meilleur, target);
	}
}

Donc, que je résume, avec ce programme, je gère mal mes listes de plaquettes restantes pour chaque noeud et je ne fais l'appel de la fonction Find au bon endroit.

Je vous serez donc reconnaissant de bien vouloir me venir en aide.

Merci d'avance

**joel.drigo** · 03/04/2017, 12h43

Salut,

Il faut que tu sois plus précis et factuel. Déjà on ne connait pas ton algorithme. Personnellement, je n'ai jamais eu à réfléchir à la problématique, et je n'ai pas vraiment besoin de le faire, sans parler des règles mêmes du "compte est bon" (j'ignorais par exemple que le modulo est une opération possible et ça me surprend fortement). Même si ça ne devrait pas être difficile d'établir rapidement un algorithme de base, si tu pouvais nous expliquer dans les grands lignes ce que tu cherches à encoder, ça nous permettrait de te répondre plus facilement. Surtout que lire ton code pour le comprendre n'est pas une sinécure. J'ai l'impression qu'il est bien compliqué pour ce qu'il y a faire.

Tu dis que tu n'appelles pas la méthode récursive avec les bonnes valeurs, mais il n'y a que toi qui sait qu'elles seraient les bonnes valeurs. Il faut nous dire ce à quoi tu t'attends. Idem pour l'endroit : c'est quoi le bon endroit d'appel ? Et qu'est-ce qui ne va pas dans les plaquettes restantes : qu'est-ce que tu y trouves par exemple, et qu'est-ce qu'on devrait y trouver ?

**mty60000** · 03/04/2017, 14h54

Bonjour, tout d'abord merci d'avoir pris le temps de me répondre.
Il est vrai que je n'ai pas expliqué les règles. Dans ce projet, je veux coder un solveur pour le jeu le compte est bon avec quelques petites modifications (modulo et mise à la puissance).
Ce que je souhaite faire c'est sortir une meilleures solutions (la valeur la plus proche de la cible avec le moins de calculs possibles). Pour cela, j'essaye de faire un arbre avec pour chaque noeud, une opération (les 2 opérandes, l'opération, le résultat, la liste de plaquettes mise à jour). J'essaye de construire l'arbre avec une fonction récursive (Find). Dans cette fonction, je test le meilleur noeud actuel (Best) avec le noeud crée dans l'appel précédent.
Je pense que le problème c'est que je ne remonte pas dans l'arbre pour faire les calculs avec toutes les combinaisons.
J'espère avoir apporté assez d'informations.

**joel.drigo** · 03/04/2017, 16h16

Bon, j'ai tenté de résoudre le problème vite fait (dans le principe, parce que je ne fais qu'évaluer l'ensemble des solutions possibles) pour voir les différences avec ta solution.

A priori, je ne pense pas que ta récursion soit directement en cause, ta double boucle (ou triple si on compte la boucle sur les opérateurs) étant similaire à la mienne. Je pense que le problème vient plutôt de ta liste de plaquette.

Tu dupliques tout d'abord la liste passée en paramètre, et tu supprimes le premier élément. Ensuite dans la double-boucle, tu ajoutes le résultat, puis enlève le premier élément.

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
/*...*/
 
tmp.remove(0);
 
Information infom = new Information(0,' ',0,100000,6,N.info.getPlaquettes(),1);
Noeud N1 = new Noeud(infom);
 
for(i=0; i<max-1; i++) {
    for(j=i+1; j<max; j++) {					
         for(o=0; o<8; o++) {
	       int actuelle = niveau + 1;						
		res = Operation(N.info.getPlaquettes().get(0), N.info.getPlaquettes().get(1), o);
		tmp.add(res);
		tmp.remove(0);		
/*...*/

Et c'est là qu'il y a quelque chose qui m'échappe dans le raisonnement.

Mon raisonnement : à chaque niveau, on fait la double-boucle, pour chaque paire d'opérandes, on fait une copie de la liste de plaquettes, on retire les 2 plaques correspondantes à la paire d'opérandes, on ajoute le résultat à la liste, et on fait l'appel récursif. Donc en gros, pour chaque paire d'opérandes, je repars de la liste de plaquettes en entrée du niveau courant.

Ton code, si ne je ne me trompe pas : à chaque niveau, on fait une copie de la liste de plaquettes, on retire la première, on fait la double-boucle, pour chaque paire d'opérandes, on retire la première plaquette de la liste, on ajoute le résulat, et on fait l'appel récursif. Première différence essentielle, au fur et à mesure du parcourt des plaquettes, tu les supprimes, fait une opération dessus, mais elles ne sont pas utilsées pour l'opérateur alternatif suivant, alors qu'elles le devraient.
En gros, si on a 1,2,3, 4

on fait la copie de la liste, puis on retire le premier => 2,3,4
on boucle sur les opérateurs +, -, /...
1. on fait 1 + 2 = 3, on retire 2, on ajoute 3 => 3,4,3
2. on fait l'appel récursif
  1. on fait la copie de la liste, on retire le premier => 4,3
  2. on boucle sur les opérateurs +, -, /...
    1. je limite ici la simulation de la récursion...
3. la récursion ayant travaillé sur une copie, on retrouve 3,4,3 intacte, et on traite l'opérateur suivant
  1. on fait 3 - 4 = 1, on retire 3, on ajoute 1 => 4,3,1
  2. on fait l'appel récursif
    1. seulement, il y a un problème ici : chaque opérateur devrait dans un même niveau tester les 2 mêmes opérandes. On essaye de voir ce que donne l'addition, puis la soustraction, puis la multiplication, etc.
      On devrait donc simuler ici 1 - 2, pas 3 - 4. Pire on cumule les résulats des différentes opérations, qui normalement devraient être alternatives.

Voici uniquement ma récursivité pour construire l'ensemble des solutions :

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
31
32
33
34
 
private void buildTree(List<Long> plaques, int compte) {
   for(int i=0; i<plaques.size(); i++) { // je parcours toutes les plaques (je m'en fous du cas plaques.size()-1, parce qu'avec la boucle sur j, ce cas, sera d'office éléminé,
       for(int j=i+1; j<plaques.size(); j++) { // je parcours les plaques suivantes dans ma liste
           List<Long> copie = new ArrayList<>(plaques); //  je fais une copie de la liste
           // je prélève la paire de plaques que je veux traiter
           final long op2 = copie .remove(j); // on doit enlever j avant i, à cause du décalage (ou alors, il faudrait faire copie.remove(j>i?j-1:j));
           final long op1 = copie .remove(i);
           buildTree(copie, compte, op1, op2);  // j'appelle la méthode qui fait le test des opérateurs (j'ai mis ça à part pour plus de clart)
       }
   }
}
 
private void buildTree(List<Long> plaques, int compte, long op1, int op2) {  
   for(Operator ope : Operator.values()) { // pour tous les opérateurs (ici je teste chaque opérateur avec ma paire d'opérandes)
      if ( ope.isEnabled(op1, op2) ) { // si l'opérateur est applicable (j'ai simplifié mon traitement d'opérateurs en éliminant cas impossible et redondant)
          long result = ope.eval(op1,op2); // je calcule le résultat 
          if ( result<0 ) { 
	      throw new IllegalStateException("Overflow: " + ope.toExpression(op1,op2) +" = " +result);
          }
          Node node = new Node(ope, op1, op2, result); // je crées mon noeud
          add(node); // et le stocke (ici je cherche juste à déterminer toutes les séries d'opérations possibles
          if ( result!=compte && !plaques.isEmpty() ) { // si le résulat n'est pas le compte et qu'il existe encore des plaques à traiter
	       plaques.add(result); // j'inclues les résulats dans les calculs suivant
               try {
                     node.buildTree(plaques, compte); // je fais mon appel récursif
               }
               finally { 
                  plaques.remove(plaques.size()-1); // je supprime le résultat de l'opérateur courant de la liste des plaques (il ne doit être utilisé que dans les tests ultérieures, pas de même niveau)
               }
          }
     }
  }
}

**mty60000** · 03/04/2017, 17h06

Merci beaucoup, je vais essayer tout ça et vous donner une réponse en retour quand ça sera fait

Solveur du jeu le compte est bon

Java

Vue hybride

Discussions similaires

Partager

Partager