Tom's Guide > Forum > Programmation > [C++] Problème de segmentation - std::stack

[C++] Problème de segmentation - std::stack

Forum Programmation : [C++] Problème de segmentation - std::stack

TomsGuide.com : 800 000 inscrits répondent à toutes vos questions high-tech et informatique. Pour obtenir de l'aide, inscrivez-vous gratuitement !
Créer un nouveau sujet Répondre
Mot :    Pseudo :           
 
Sethpolma   19-02-2006 à 14:35:43

Bonjour à tous !

J'expérimente une nouvelle façon de gérer mes événements en SDL. Je voulais utiliser la pile (ou <i>stack</i> ), mais j'ai des petits problèmes de violation d'accès (ou de segmentation).

Voici mon code :

Code :
  1. #include <stack>
  3. struct StateStruct
  4. {
  5.         void (*StatePointer)();
  6. };
  8. stack<StateStruct> StateStack;
  10. ...
  12.     StateStruct state;
  13.     state.StatePointer = Exit;
  14.     StateStack.push(state);
  15.    
  16.     state.StatePointer = Menu;
  17.     StateStack.push(state);
  19. ...



Voilà. Le problème vient de là : si je met cette partie en commentaire, ça marche (mise à part quelques petits problèmes après dans le programme).

Merci beaucoup à tous mes sauveurs ! :)

Répondre
Liens sponsorisés
Inscrivez-vous ou connectez-vous pour masquer ceci.
CRicky   19-02-2006 à 15:21:38
- 0 +

Et si tu commentes juste les push() ?

Répondre à CRicky
Sethpolma   19-02-2006 à 20:14:50
- 0 +

Eh bien, comme il fallait s'y attendre, tout fonctionne bien... D'où est-ce que ça peut venir ? Pourtant, le StateStack est bien déclaré, non ?

Répondre à Sethpolma
CRicky   19-02-2006 à 22:44:17
- 0 +

Je crois me souvenir qu'en C++ les "struct" sont en fait des "class".
quand tu fais la pile, le constructeur par recopie est appelé pour copier l'objet, peut-être qu'il y a un problème de ce côté.

Tu devrais essayer de gérer des pointeurs pour éviter de faire des copies:

Code :
  1. #include <stack>
  3. struct StateStruct
  4. {
  5.         void (*StatePointer)();
  6. };
  8. stack<StateStruct *> StateStack;
  10. ...
  12.     StateStruct state1;
  13.     state1.StatePointer = Exit;
  14.     StateStack.push(&state1);
  15.    
  16.     StateStruct state2;
  17.     state2.StatePointer = Menu;
  18.     StateStack.push(&state2);
  20. ...


mais attention à zone de définition des variables

Répondre à CRicky
Sethpolma   20-02-2006 à 10:08:16
- 0 +

Ca a l'air de marcher là. Merci de ton aide. Enfin... Ca marche... Je n'ai plus de problème de mémoire. Mais à présent, c'est comme si mes fonctions n'étaient pas utilisées... Je vais regarder ! :)

Répondre à Sethpolma
CRicky   20-02-2006 à 13:21:54
- 0 +

Il faut voir que c'est un pointeur que tu passes, il faut donc gérer toi-même la mémoire de tous les objets (création par des new et destruction par des delete) si tu comptes faire sortir un objet de ta fonction.

Répondre à CRicky
Sethpolma   20-02-2006 à 17:16:49
- 0 +

Comment cela ?

Quelques détails supplémentaires : en fait je me sers des quelques lignes précédentes pour indiquer quel est l'état. J'ai pour ce faire trois fonctions : Exit(), Game() et Menu() qui gèrent toutes trois des parties différentes du jeu.

Je me sers du pointeur StatePointer pour simplement renvoyer la fonction à utiliser par l'intermédiaire de :

Code :
  1. int main(int argc, char** argv)
  2. {
  3.     Init();
  4.    
  5.     while(!StateStack.empty())
  6.     {
  7.           StateStack.top()->StatePointer();
  8.     }
  9.    
  10.     Shutdown();
  11.     return 0;
  12. }



Voilà l'ossature de mon programme ! Mais je ne pense pas qu'il faille créer quoi que ce soit, vu que la variable StateStack est créee en globale...

Répondre à Sethpolma
CRicky   20-02-2006 à 20:15:09
- 0 +

Statestack est effectivement créé en globla, mais il faut aussi que les objets mis dedans le soient, car si tu crée state1 en local sans utiliser new dans ta fonction Init(), il sera détruit à la sortie de la fonction, et au final, tu auras dans ta pile un pointeur qui pointera dans une zone mémoire non aloué et où il n'y a rien => résultat crash de l'application quand tu va appeler un élément de la pile.

Donc il te faut faire des new pour tout objets mis dans la pile. Et du coup faire des delete avant la suppression de la pile (car sinon tu supprime la pile sans supprimer les objets contenus).

Répondre à CRicky
Sethpolma   21-02-2006 à 20:30:26
- 0 +

Effectivement ! Ca marche beaucoup mieux maintenant ! Au moins, ça a des points positifs : j'ai jamais autant vu de cours sur les pointeurs et sur les std::stack en si peu de temps !

Eh bien, merci beaucoup ! Je peux enfin tenter de terminer mon programme, avant le prochain problème ! ;)

Répondre à Sethpolma
Créer un nouveau sujet Répondre
Tom's Guide > Forum > Programmation > [C++] Problème de segmentation - std::stack
Aller à :

Il y a 1930 utilisateurs connus et inconnus. Pour voir la liste des connectés connus, cliquez ici.

Plan du forum
Forum Bestofmedia ©
2000-2009 Bestofmedia Group
Page générée en 0,311 secondes
Attention

Vous allez répondre sur un sujet resté inactif pendant plus de 6 mois.
Assurez-vous d'apporter des éléments nouveaux à la discussion avant de poursuivre.

Répondre Annuler
Liens