Discussion :

[antares56]

Bonjour à tous et à toutes !
J'ai donc une question pour vous que je n'arrive pas à résoudre mais je pense être à deux doigts de trouver, je m'explique, je dois faire une fonction qui, pour un tableau 2D ,si la coordonnée i,j n'a aucune cases adjacentes qui sont égales à "*" ( donc si [i][j] == 0)on révèle toutes les cases adjacentes qui sont égales à 0(un démineur en gros quand on affiche toutes les cases autour de celles qu'on a sélectionnée sauf les bombes)
voici mon code (oui y'a beaucoup de if je sais que c'est perfectible )
PS:la fonction "caseCorrecte" vérifie si les coordonnées i,j sont des cases valables du tableau(si elles appartiennent au tableau )

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
 
if (Tadj[i][j]==0) {						
			if (caseCorrecte(i-1,j-1)==true){
				T[i-1][j-1]=1;						
			}
			if (caseCorrecte(i-1,j)==true){
				T[i-1][j]=1;
			}
			if (caseCorrecte(i-1,j+1)==true){
				T[i-1][j+1]=1;
			}
			if (caseCorrecte(i,j-1)==true){
				T[i][j-1]=1;
			}
			if (caseCorrecte(i,j+1)==true){
				T[i][j+1]=1;
			}
			if (caseCorrecte(i+1,j-1)==true){
				T[i+1][j-1]=1;
			}
			if (caseCorrecte(i+1,j)==true){
				T[i+1][j]=1;
			}
			if (caseCorrecte(i+1,j+1)==true){
				T[i+1][j+1]=1;
			}	
 
		}

[PhilippeGibault]

Oui et?
A part des questions de qualité de code, je ne vois pas quoi critiquer?
je ne vois même pas où est ton problème.

Par exemple, à la place de:

Code :

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

1
2
 
if (caseCorrecte(i-1,j-1)==true)

Ecrire:

Code :

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

1
2
 
if (caseCorrecte(i-1,j-1))

je suppose que caseCorrecte envoie un booléen...

Ensuite, on peut un peu optimiser.

Genre:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
if (Tadj[i][j]==0) {
 
    for(int ik = i - 1; ik < i + 2; ik++){
 
        for(jk = j - 1; jk < j + 2; jk++){
 
           if((i != ik) && (j != jk) &&  caseCorrecte(ik,jk){
               T[ik][jk]=1;
           }
       }
    }
}

On peut évidement toujours faire mieux. Ce ne sont que des pistes.

[antares56]

Merci pour votre réponse ! En faites c'est que le code ne fonctionne pas dans certain cas... Et je ne vois pas où est le soucis

[professeur shadoko]

ton problème est un grand classique: "éclabousser" les 8 cases autour d'une cellule (on trouve ça par exemple dans des réalisations de jeu de la vie).

d'abord on peut truquer un peu le tableau: soit en en faisant une matrice torique (mais ça change le jeu) soit en mettant tout autour une "couche de gras" c'est à dire que la première et la dernière ligne, la première et la dernière colonne sont constituées de cellules "mortes" qui ne sont pas affichées et dont le contenu (bidon comme -1) peut facilement être testé sans se poser de question.


Ensuite à chaque coup on peut générer facilement un tableau de 8 paires qui, en étant parcouru, donne les coordonnées des 8 cases adjacentes.

tout ça évite d'écrire des boucles fastidieuses.