Probleme d'execution : le programme doit se fermer (mode console)

Version imprimable

30/01/2009, 15h02
LePassager

Probleme d'execution : le programme doit se fermer (mode console)

bonjour tout le monde,

je me demande les raisons dans le but de comprendre pourquoi parfois mes programmes " crashent " meme s'ils ne contiennent aucune erreur, dés que la fentere en mode console se lance un message du systeme s'affiche alors indiquant que le programme doit se fermer:? !!!!

c'est bizzare non ?

j'utilise Dev-C++ dans sa derniere version et sous OS Vista

merci de me repondre
30/01/2009, 15h40
Goten

sans code... mais en général ça vient d'un comportement indéterminé du code..
30/01/2009, 15h42
3DArchi

Citation:

Envoyé par LePassager

pourquoi parfois mes programmes " crashent " meme s'ils ne contiennent aucune erreur,

On a tous des programmes sans erreur .... jusqu'au premier bug ;)
Montre-nous quelques sources pour avoir des infos plus précises.

merci pour la reponse

voila un bout de code sachant que j'ai tester toutes les fonctions de toutes les classes créées avant de l'utiliser dans cette methode

Code:

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
void Lancer(vector<Carte>& Jeu, Joueur& joueur1, Joueur& joueur2)
{
        Packet Pg(Jeu);
        vector<Carte>  pj1, pj2;
 
        Pg.SplitPacket(pj1, pj2);
 
        Packet p1(pj1);
        Packet p2(pj2);
 
        Main m1 = joueur1.GetMainJoueur();
        Main m2 = joueur2.GetMainJoueur();
 
        vector<Carte> tmp1(p1.GetlePacket());
        vector<Carte> tmp2(p2.GetlePacket());
 
        Carte c1(m1.RetirerCarte(tmp1));
        Carte c2(m2.RetirerCarte(tmp2));
 
 
 
        while(PackJoueur2.size()!= 0 && PackJoueur1.size()!= 0)
        {
            p1.MelangerPacket();
            p2.MelangerPacket();
 
            if( c1.Comparer(c2) == 1 )
            {
                joueur1.SetScore(1);
                joueur2.SetScore(1);
                joueur1.InformationsJoueur();
 
                joueur2.InformationsJoueur();
                cout<<endl;
            }
            else if( c1.Comparer(c2) == 2 )
            {      
                Joueur2.SetScore(3);
                Joueur2.InformationsJoueur();
                cout<<endl;
            }
            else if( c1.Comparer(c2) == 3 )
            {      
                Joueur1.SetScore(3);
                Joueur1.InformationsJoueur();
                cout<<endl;
            }
 
            cout << "Pack1 : " << PackJoueur1.size() 
                 <<" Pack2 : " << PackJoueur2.size() <<endl;
            cout << endl << endl;
            Sleep(3); 
        }
 
        if( joueur1.GetScore() == joueur2.GetScore() )
        {
            cout<<"la partie s'est terminée avec égalité !"<<endl;
            joueur1.InformationsJoueur();cout<<endl;
            joueur2.InformationsJoueur();cout<<endl;
        }
        else if( Joueur1.GetScore() > Joueur2.GetScore() )
        {
 
            Joueur1.InformationsJoueur();cout<<" a gangné !"<<endl;
            Joueur2.InformationsJoueur();cout<<" a perdu !"<<endl;
        }
        else if( Joueur1.GetScore() < Joueur2.GetScore() )
        {
 
            Joueur2.InformationsJoueur();cout<<" a gangné !"<<endl;
            Joueur1.InformationsJoueur();cout<<" a perdu !"<<endl;
        }
 
 
 
}

Est ce que je vous explique le code ?
generalement c'est jeu de cartes

Citation:

Envoyé par LePassager

merci pour la reponse

voila un bout de code sachant que j'ai tester toutes les fonctions de toutes les classes créées avant de l'utiliser dans cette methode

Bonne démarche. Ca évite effectivement beaucoup de soucis.

Citation:

Envoyé par LePassager

Code:

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
void Lancer(vector<Carte>& Jeu, Joueur& joueur1, Joueur& joueur2)
                  {
                        Packet Pg(Jeu);
                        vector<Carte>  pj1, pj2;
 
                        Pg.SplitPacket(pj1, pj2);
 
                        Packet p1(pj1);
                        Packet p2(pj2);
 
                        Main m1 = joueur1.GetMainJoueur();
                        Main m2 = joueur2.GetMainJoueur();
 
                        vector<Carte> tmp1(p1.GetlePacket());
                        vector<Carte> tmp2(p2.GetlePacket());
 
                        Carte c1(m1.RetirerCarte(tmp1));
                        Carte c2(m2.RetirerCarte(tmp2));
 
 
 
                        while(PackJoueur2.size()!= 0 && PackJoueur1.size()!= 0)
                           {
                                 p1.MelangerPacket();
                                 p2.MelangerPacket();
 
                                 if( c1.Comparer(c2) == 1 )
                                 {
                                       joueur1.SetScore(1);
                                       joueur2.SetScore(1);
                                       joueur1.InformationsJoueur();
 
                                       joueur2.InformationsJoueur();
                                       cout<<endl;
                                 }
                                 else if( c1.Comparer(c2) == 2 )
                                 {      
                                       Joueur2.SetScore(3);
                                       Joueur2.InformationsJoueur();
                                       cout<<endl;
                                 }
                                 else if( c1.Comparer(c2) == 3 )
                                 {      
                                       Joueur1.SetScore(3);
                                       Joueur1.InformationsJoueur();
                                       cout<<endl;
                                 }
 
                                 cout<<"Pack1 : "<<PackJoueur1.size()<<" Pack2 : "<<PackJoueur2.size()<<endl;
                                 cout<<endl;cout<<endl;
                                 Sleep(3); 
                           }
 
                       if( joueur1.GetScore() == joueur2.GetScore() )
                       {
                             cout<<"la partie s'est terminée avec égalité !"<<endl;
                             joueur1.InformationsJoueur();cout<<endl;
                             joueur2.InformationsJoueur();cout<<endl;
                       }
                       else if( Joueur1.GetScore() > Joueur2.GetScore() )
                       {
 
                             Joueur1.InformationsJoueur();cout<<" a gangné !"<<endl;
                             Joueur2.InformationsJoueur();cout<<" a perdu !"<<endl;
                       }
                       else if( Joueur1.GetScore() < Joueur2.GetScore() )
                       {
 
                             Joueur2.InformationsJoueur();cout<<" a gangné !"<<endl;
                             Joueur1.InformationsJoueur();cout<<" a perdu !"<<endl;
                       }
 
 
 
                  }

Est ce que je vous explique le code ?
generalement c'est jeu de cartes

Ca manque encore d'information. Ce qui ne me saute pas aux yeux c'est comment tu sors de ta boucle while. Outre que PackJoueur1 et PackJoueur2 ne sont pas définis dans le bout de code que tu postes, les méthodes indiquées dans la boucle while ne semblent pas retirer des nouvelles cartes des paquets. D'où l'impression que tu ne sors jamais de la boucle.

30/01/2009, 17h27
LePassager

mecri pour le temps que tu m'accordes ;)

en fait,c'est la methode RetirerCarte() qui permet de determiner la fin de la boucle et par lasuite d'en sortir, puisqu'elle supprime la premiere carte du conteneur puis retourne la carte supprimée
et par la suite la valeur contenue dans size() decremente

Citation:

Envoyé par LePassager

mecri pour le temps que tu m'accordes ;)

en fait,c'est la methode RetirerCarte() qui permet de determiner la fin de la boucle et par lasuite d'en sortir, puisqu'elle supprime la premiere carte du conteneur puis retourne la carte supprimée
et par la suite la valeur contenue dans size() decremente

Sauf que je ne vois pas où elle est appelée dans le while :

Code:

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
 
                        while(PackJoueur2.size()!= 0 && PackJoueur1.size()!= 0)
                           {
                                 p1.MelangerPacket();
                                 p2.MelangerPacket();
 
                                 if( c1.Comparer(c2) == 1 )
                                 {
                                       joueur1.SetScore(1);
                                       joueur2.SetScore(1);
                                       joueur1.InformationsJoueur();
 
                                       joueur2.InformationsJoueur();
                                       cout<<endl;
                                 }
                                 else if( c1.Comparer(c2) == 2 )
                                 {      
                                       Joueur2.SetScore(3);
                                       Joueur2.InformationsJoueur();
                                       cout<<endl;
                                 }
                                 else if( c1.Comparer(c2) == 3 )
                                 {      
                                       Joueur1.SetScore(3);
                                       Joueur1.InformationsJoueur();
                                       cout<<endl;
                                 }
 
                                 cout<<"Pack1 : "<<PackJoueur1.size()<<" Pack2 : "<<PackJoueur2.size()<<endl;
                                 cout<<endl;cout<<endl;
                                 Sleep(3); 
                           }

30/01/2009, 17h31
LePassager

Citation:

Envoyé par 3DArchi

Bonne démarche. Ca évite effectivement beaucoup de soucis.

merci de m'encourager :D

le probleme ce que lors de l'appel de la methode Lancer() dans le main principal

le programme se ferme sans que j'obiens une erreur ou une indication

c'est pire comme resultat !:?
30/01/2009, 17h33
3DArchi

Rajoutes des traces pour voir comment ton programme se comporte (dump vers cout ou vers un fichier log) : d'abord dans chaque étape de ton main, puis de plus en plus finement en fonction de là où ça s'arrête.
Des bons testU c'est une super démarche que beaucoup devrait suivre.
Ensuite, un bon debug (avec un debugger ou avec des traces), c'est ++ !
30/01/2009, 17h36
Sylvain Togni

Un petit "Rebuild All" puis lancement en mode debug pas-à-pas pour voire où ça plante exactement.

Citation:

Envoyé par 3DArchi

Sauf que je ne vois pas où elle est appelée dans le while :

Code:

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
 
                        while(PackJoueur2.size()!= 0 && PackJoueur1.size()!= 0)
                           {
                                 p1.MelangerPacket();
                                 p2.MelangerPacket();
 
                                 if( c1.Comparer(c2) == 1 )
                                 {
                                       joueur1.SetScore(1);
                                       joueur2.SetScore(1);
                                       joueur1.InformationsJoueur();
 
                                       joueur2.InformationsJoueur();
                                       cout<<endl;
                                 }
                                 else if( c1.Comparer(c2) == 2 )
                                 {      
                                       Joueur2.SetScore(3);
                                       Joueur2.InformationsJoueur();
                                       cout<<endl;
                                 }
                                 else if( c1.Comparer(c2) == 3 )
                                 {      
                                       Joueur1.SetScore(3);
                                       Joueur1.InformationsJoueur();
                                       cout<<endl;
                                 }
 
                                 cout<<"Pack1 : "<<PackJoueur1.size()<<" Pack2 : "<<PackJoueur2.size()<<endl;
                                 cout<<endl;cout<<endl;
                                 Sleep(3); 
                           }

si je vous comprends, l'appel s'effectue dans la boucle de façon automatique
j'explique :
lorsque la carte sera supprimée du conteneur grace a RetirerCarte()
le conteneur s'il contenait 26 cartes moins une qui vient d'etre supprimée
le reste est 25

le premiere test sur la condition de la boucle est size() = 26
le deuxieme test size() = 25
.
.
.
le dernier test de la boucle size vaut Zéro pour les deux conteneurs PackJoueur1 et 2

30/01/2009, 17h54
LePassager

Citation:

Envoyé par 3DArchi

Rajoutes des traces pour voir comment ton programme se comporte (dump vers cout ou vers un fichier log) : d'abord dans chaque étape de ton main, puis de plus en plus finement en fonction de là où ça s'arrête.
Des bons testU c'est une super démarche que beaucoup devrait suivre.
Ensuite, un bon debug (avec un debugger ou avec des traces), c'est ++ !

ouiiii, vous avez raison

je l'ai fait deja

j'affiche le tout depuis la creation des conteneurs jusqu'a l'affichage des valeurs des cartes supprimées et le resultat final

mais ca reste en mettant le code dans la methode Lancer() ca crash :?

sans cette methode le programme fonctionne super correctement
30/01/2009, 17h58
LePassager

Citation:

Envoyé par Sylvain Togni

Un petit "Rebuild All" puis lancement en mode debug pas-à-pas pour voire où ça plante exactement.

je vais le faire :P et je posterai le resultat

mais avant je dois terminer un bout de code :D

merci amigos;)
30/01/2009, 19h13
3DArchi

Citation:

Envoyé par LePassager

si je vous comprends, l'appel s'effectue dans la boucle de façon automatique
j'explique :
lorsque la carte sera supprimée du conteneur grace a RetirerCarte()
le conteneur s'il contenait 26 cartes moins une qui vient d'etre supprimée
le reste est 25

le premiere test sur la condition de la boucle est size() = 26
le deuxieme test size() = 25
.
.
.
le dernier test de la boucle size vaut Zéro pour les deux conteneurs PackJoueur1 et 2

Je ne vois pas l'appel à RetirerCarte DANS la boucle while; à moins que tu n'aies oublié de le recopier en postant ton bout de code.
Met des traces dans ta fonction Lancer et dans ta boucle while.
30/01/2009, 19h44
LePassager

Citation:

Envoyé par 3DArchi

Je ne vois pas l'appel à RetirerCarte DANS la boucle while; à moins que tu n'aies oublié de le recopier en postant ton bout de code.
Met des traces dans ta fonction Lancer et dans ta boucle while.

le code est complètement correcte:P et j'ai rien oublié :d

si vous revoyez le code que j'ai mis :

RetirerCarte est appelée par un objet de une classe main (hand) d'un joueur (classe aussi) qui possede a son tour cete main
cette methode permet de tirer une carte du conteneur c-a-d supprimer une carte puis retounrnée cette derniere pour faire la comparaison et augmenter le score du gagnat
30/01/2009, 19h48
LePassager

j'ai oublié de dire que tirer une carte implique enlever celle la du conteneur

ce qui est a la fin : la condition de la boucle teste si les deux conteneurs contiennent encore des elements de type carte ou non :D
si oui elle retire et compare les cartes et bien sur le score est augmenté sinon

on quitte et on affiche le gagnant
30/01/2009, 19h48
3DArchi

Dans quelle méthode?
MelangerPacket()?
Comparer(c2)?
SetScore(1)?
InformationsJoueur()?
size()?
ce sont les seuls méthodes à l'intérieur du while que je vois.
30/01/2009, 20h01
LePassager

Citation:

Envoyé par 3DArchi

Dans quelle méthode?
MelangerPacket()?
Comparer(c2)?
SetScore(1)?
InformationsJoueur()?
size()?
ce sont les seuls méthodes à l'intérieur du while que je vois.

aucune d'elles

vous allez la trouver dans le constructeur de recopie des cartes

je vous comprends maintenant
moi je parle avec mon code devant les yeux sous Dev-C++ :aie:

je sais pas comment les objets cartes ne sont pas mis dans la boucle :mouarf:

---

je m'excuse:oops:
30/01/2009, 20h06
LePassager

une derniere question :

pour la methode Lancer

vaut il mieux la laisser sans parametres ou bien je rajoute des parametres comme le joueur1, joueur2 et l'ensemble de cartes (paquet) ?

puisque cette methode ne sert qu'à lancer la partie du jeu ?
02/02/2009, 09h26
3DArchi

Quel est l'objet qui a la responsabilité de la méthode Lancer (LeJeu)?
Quel est la durée de vie des objets Jeu/Joueur1/Joueur2 : est-elle synchrone de Lancer -> Variable Locale, elle est synchrone de LeJeu-> compositon?, elle est indépendante: variable main + paramètre? Je ne connais pas suffisamment ton programme pour voir quelle est la conception la + approprié.
Dernière remarque, faire un tirage dans le constructeur par copie est dangereux d'un point de vue maintenance/relecture du code. Spontanément, personne ne le verra.
05/02/2009, 00h10
LePassager

Citation:

Envoyé par 3DArchi

Quel est l'objet qui a la responsabilité de la méthode Lancer (LeJeu)?
Quel est la durée de vie des objets Jeu/Joueur1/Joueur2 : est-elle synchrone de Lancer -> Variable Locale, elle est synchrone de LeJeu-> compositon?, elle est indépendante: variable main + paramètre? Je ne connais pas suffisamment ton programme pour voir quelle est la conception la + approprié.
Dernière remarque, faire un tirage dans le constructeur par copie est dangereux d'un point de vue maintenance/relecture du code. Spontanément, personne ne le verra.

j'ai pas bien compris ce que vous voulez dire par synchrone ?

la fonction lancer ne sert qu'à simuler une partie du jeu carte
05/02/2009, 06h57
3DArchi
Citation:

Envoyé par LePassager

j'ai pas bien compris ce que vous voulez dire par synchrone ?

qui ont la même durée de vie.
Si tu fais :
Code:

1 2 3 4 5 6 7 8 9 10 int main() { // ... initialisations .... Joueur j1; Joueur j2; Paquet paquet; Lancer(j1,j2,Paquet); // ... des-initialisation sans plus utiliser j1, j2 et paquet }
Alors autant mettre j1, j2 et paquet dans la fonction lancer.
Si par contre, tu t'en sert avant ou après, alors passe-les en paramètre.
06/02/2009, 00h17
LePassager
Citation:
Envoyé par 3DArchi

qui ont la même durée de vie.
Si tu fais :

Code:

1 2 3 4 5 6 7 8 9 10 int main() { // ... initialisations .... Joueur j1; Joueur j2; Paquet paquet; Lancer(j1,j2,Paquet); // ... des-initialisation sans plus utiliser j1, j2 et paquet }

Alors autant mettre j1, j2 et paquet dans la fonction lancer.
Si par contre, tu t'en sert avant ou après, alors passe-les en paramètre.
Ah oui ! merci j'ai compris merci bien
donc il vaut mieux que ca soit synchrone

encore un grand merci ^^