petit probleme de programmation de memory

**Asmod_D** · 25/09/2007, 00h56

salut,

j'essaie de faire un petit jeu de memory
j'ai juste quelques problemes: parfois il m'indique des touches qui ne sont pas memorisé dans mon tableau chiffre . et lors de l'affichage lorsque j'ai trouve une paire de mots il m'indique toujours les tous indices du tableau cacher ex:

au debut du programme j'ai 1234
5678

apres lorsque je trouve une paire de mots ex kiwi j'ai: 1Kkiwi23kiwi4
5678

et je voudrais bien voir le resultat 1kiwi3kiwi
5678

voici mon programme :

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
#include <iostream.h>
#include <string.h>
#include <time.h>
#include <stdio.h>
 
 
 
int main(int argc,char** argv)
{
 
	srand((unsigned)time(NULL));
	char mots[8][10];
	strcpy(mots[0],"fraise");
	strcpy(mots[1],"fraise");
	strcpy(mots[2],"pomme");
	strcpy(mots[3],"pomme");
	strcpy(mots[4],"kiwi");
	strcpy(mots[5],"kiwi");
	strcpy(mots[6],"orange");
	strcpy(mots[7],"orange");
 
	char mots_melange[8][10];
	int i,aleatoire;
	int tableau_aleatoire[8];
	int cacher[8];
	int nombre1,nombre2;
	bool game=true;
	bool jeu=true;
	int tour=0;
	int chiffre[8];
 
 
	while(game==true&&tour!=4)
		{
 
		if(jeu==true)
		{
			for (i=0;i<8;i++)
				{    
					 cacher[i]=i+1;
					 cout<<cacher[i];
 
					if (i==3)
					{
						cout<<"\n";
					}
					jeu=false;
				}
 
	for (i=0;i<8;i++)
	{
		tableau_aleatoire[i]=0;
		chiffre[i]=0;
	}
 
	for (i=0;i<8;i++)
	{
	do
	   {
		aleatoire=rand()%8;
		strcpy(mots_melange[i],mots[aleatoire]);
 
	   }while(tableau_aleatoire[aleatoire]==1);
		 tableau_aleatoire[aleatoire]=1;
   }
 
}	
	do{	
	cout<<"tapez un nombre : ";
	cin>>nombre1;
	cout<<"\n"<<mots_melange[nombre1];
	cout<<"\n"<<"tapez un deuxieme nombre : ";
	cin>>nombre2;
	cout<<"\n"<<mots_melange[nombre2];
	if (chiffre[nombre1]==1||chiffre[nombre2]==2)
	{
		cout<<"vous avez deja tapez ses nombre recommencez :\n";
	}
	}while(chiffre[nombre1]==1||chiffre[nombre2]==2);
	cout<<mots_melange[nombre2];cout<<"\n";
	if (strcmp(mots_melange[nombre1],mots_melange[nombre2])==0)
	{
 
		chiffre[nombre1]=1;
		chiffre[nombre2]=2;
		tour++;
		for(i=0;i<8;i++)
		{
 
			if (chiffre[i]==1)
			{
				cout<<mots_melange[i];
 
			}
 
 
 
			if (chiffre[i]==2)
			{
				cout<<mots_melange[i];
 
			}
 
 
			if(chiffre[i]!=1!=chiffre[i]!=2)
			{
				cout<<cacher[i];
 
			}
 
 
 
			if (i==3)
			{
				cout<<"\n";
			}
		}
		cout<<"\n";
	}
 
 
	}
 
 
	return 0;
}

merci !

**JolyLoic** · 25/09/2007, 01h09

Je n'ai pas le courage de lire en détail ton code à cette heure ci, mais juste de faire quelques remarques après un bref survol.

Tu utilises des char* et char[], ce qui est source d'erreur. Je te conseille de définir ton tableau plus simplement : std::string mots_melange[8]; (après un #include <string>), ou pour plus de flexibilité sur le nombre d'éléments :
std::vector<std::string> mots_melange;

Ta fonction est trop grosse. Découpe là en fonctions plus élémentaires. Par exemple une fonction qui affiche l'état courant du jeu me semble intéressante.

J'ai l'impression que pour gérer les différentes choses liées à l'état du jeu, tu utilises différents tableaux, l'un pour les noms, l'un pour l'état caché/visible... J'aurais pour ma part défini une structure carte, et j'aurais fait un tableau de cette structure. :

Code :

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

1
2
3
4
5
6
7
struct Carte
{
  bool visible;
  string nom;
};
 
vector<Carte> jeu;

L'idée est d'avoir un truc packagé et cohérent.

**koala01** · 25/09/2007, 01h54

Salut,

La première réaction, à froid, qui me vient à l'esprit serait de t'inciter à utiliser la classe string, accessible via un #include <string> et disponible dans l'espace de nommage std, pour tout ce qui concerne la gestion de chaînes de caractères...

Son utilisatio de base est bien plus facile, intuitive et sécurisée que celle d'un tableau de caractères "C style" identique:

Ainsi, rien ne t'empêche de créer un tableau de chaînes sous la forme de

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
#include <string>
using namespace std;
int main()
{
    string mots[8];
    mots[0]="fraise";
    mots[1]="fraise";
    mots[2]="pomme";
    mots[3]="pomme";
    mots[4]="kiwi";
    mots[5]="kiwi";
    mots[6]="orange";
    mots[7]="orange";
    for(size_t i=0;i<7;i++)
        cout<<mots[i]<<endl;
    return 0;
}

(code minimum compilable pour montrer l'utilisation d'un tableau de chaînes

)

La deuxième remarque que me suggère ton code serait de veiller à utiliser des règles d'indentation strictes, pour permettre une relecture facile (un code est bien plus souvent lu qu'écrit

)

Ton code serait bien plus facile à lire (sans l'avoir modifié) s'il prenait la forme de

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
#include <iostream.h>
#include <string.h>
#include <time.h>
#include <stdio.h>
 
 
 
int main(int argc,char** argv)
{
    srand((unsigned)time(NULL));
    char mots[8][10];
    strcpy(mots[0],"fraise");
    strcpy(mots[1],"fraise");
    strcpy(mots[2],"pomme");
    strcpy(mots[3],"pomme");
    strcpy(mots[4],"kiwi");
    strcpy(mots[5],"kiwi");
    strcpy(mots[6],"orange");
    strcpy(mots[7],"orange");
    char mots_melange[8][10];
    int i,aleatoire;
    int tableau_aleatoire[8];
    int cacher[8];
    int nombre1,nombre2;
    bool game=true;
    bool jeu=true;
    int tour=0;
    int chiffre[8];
    while(game==true&&tour!=4)
    {
        if(jeu==true)
        {
            for (i=0;i<8;i++)
            {    
                cacher[i]=i+1;
                cout<<cacher[i];
                if (i==3)
                {
                    cout<<"\n";
                }
                jeu=false;
            }
            for (i=0;i<8;i++)
            {
                tableau_aleatoire[i]=0;
                chiffre[i]=0;
            }
            for (i=0;i<8;i++)
            {
                do
                {
                    aleatoire=rand()%8;
                    strcpy(mots_melange[i],mots[aleatoire]);
                }while(tableau_aleatoire[aleatoire]==1);
                tableau_aleatoire[aleatoire]=1;
            }
        }	
	do
        {	
            cout<<"tapez un nombre : ";
            cin>>nombre1;
            cout<<"\n"<<mots_melange[nombre1];
            cout<<"\n"<<"tapez un deuxieme nombre : ";
            cin>>nombre2;
            cout<<"\n"<<mots_melange[nombre2];
            if (chiffre[nombre1]==1||chiffre[nombre2]==2)
            {
                cout<<"vous avez deja tapez ses nombre recommencez :\n";
            }
        }while(chiffre[nombre1]==1||chiffre[nombre2]==2);
        cout<<mots_melange[nombre2];cout<<"\n";
        if (strcmp(mots_melange[nombre1],mots_melange[nombre2])==0)
        {
            chiffre[nombre1]=1;
            chiffre[nombre2]=2;
            tour++;
            for(i=0;i<8;i++)
            {
                if (chiffre[i]==1)
                {
                    cout<<mots_melange[i];
                }
                if (chiffre[i]==2)
                {
                    cout<<mots_melange[i];
                }
                if(chiffre[i]!=1!=chiffre[i]!=2)
                {
                    cout<<cacher[i];
                }
                if (i==3)
                {
                    cout<<"\n";
                }
            }
            cout<<"\n";
        }
    }
    return 0;
}

et il met en évidence le fait qu'il te manque un peu de "factorisation" du code...

En effet, l'un des premiers et principaux principes qu'il faut apprendre à mettre en oeuvre est la délégation des tâches (une fonction est responsable d'une seule chose, mais elle le fait bien... cf cette intervention de ma part)

Ainsi, on pourrait envisager une fonction "Melanger" qui s'occupera... du mélange des paires et une autre "jouer"...

La troisième remarque qu'il m'inspire est "sois attentif à utiliser les bons fichiers d'en-tête"...:

<iostream.h> et <string.h> sont avantageusement remplacé par leurs homologues sans extensions (<iostream> et <string>)
stdio.h est une en-tête C, et n'apporte rien ici

Enfin, je dirais que l'utilisation d'un tableau statique n'est pas vraiment l'idéal à utiliser quand on souhaite effectuer un "mélange" des élément en s'assurant l'unicité de l'utilisation... Mais je vais peut être un peu trop vite par rapport à tes connaissances

Pour le reste, tu m'excusera, mais, vu l'heure, j'ai un peu la flegme de vérifier ton code... je *tâcherai* d'y jeter un œil demain dans la journée

petit probleme de programmation de memory

C++

Discussions similaires

Partager

Partager