problème de segmentation fault avec les tableaux

Quel débuggeur utiliser sous mac ?

Sujet résolu

Fandekasp

18 mars 2008 à 15:07:22

Bonjour chers zéros !

J'ai un léger problème

Regardez par vous-même :-°

Que veux-tu faire ? (choix : attaquer - boirePotionDeVie - changerArme)

changerArme

Quelle arme voulez-vous prendre ?

0. Epée rouillée

1. Epée aiguisée

2. Epée batarde à 2 mains

3. Glaive de Vercingétorix

4. Lance de chevalier

5. Trident de Poséidon

6. Le canif de Dieu

7. A

8. ???

Segmentation fault

Donc voilà, j'utilise vim sous mac, et je compile de la sorte :

fandekasp$ c++ main.cpp Personnage.cpp Arme.cpp

En faisant des recherches, j'ai cru comprendre qu'on utilisait gdb comme débuggeur (mais ces mêmes personnes compilaient avec gcc, alors je ne sais pas si je peux aussi utiliser gdb.. excusez le zéro qui domine en moi, j'ai un peu de mal avec tout ça

) . C'est pourquoi je viens humblement demander votre avis, que pourrais-je utiliser pour m'en sortir tout seul pour corriger mes erreurs ? merci beaucoup

Sinon, voilà le code spécifique à l'origine de l'erreur... Ne connaissant pas grand chose des tableaux en C++, j'imagine que l'erreur va vous sauter aux yeux looool.

main.cpp

// ...
do
{
// initialisation de mon tableau de strings
string tableauArmes[] = {"Epée rouillée", "Epée aiguisée",
"Epée batarde à 2 mains", "Glaive de Vercingétorix",
"Lance de chevalier", "Trident de Poséidon",
"Le canif de Dieu"};
cout << "Que veux-tu faire ? (choix : attaquer - boirePotionDeVie -
changerArme)" << endl;
string choixTour;
getline(cin, choixTour);
// ...
else if (choixTour == "changerArme")
{
cout << "Quelle arme voulez-vous prendre ? " << endl;
// On fait une boucle de tous les éléments du tableau et on les affiche
for (int i = 0; i<sizeof(tableauArmes); i++)
{
cout << " " << i << ". " << tableauArmes[i] << endl;
}
string nomArme;
getline (cin, nomArme);
// ...

Merci beaucoup pour le temps que vous aurez passé pour moi

chlablak

18 mars 2008 à 15:12:23

fais:

std::cout << sizeof(TableauArmes) << std::endl;

et tu verras tout de suite que tu ne peux pas faire comme ca

utilise std::vector

C++ Références | Idiomes | FAQ

ledemonboiteux

18 mars 2008 à 15:14:01

sizeof(tableauArmes) c'est le nombres de bytes de ton tableau et pas le nombre d'elements qu'il contient.

Tu ne peux pas connaitre a l'execution le nombre d'éléments d'un truc[].
Je te conseille grandement d'utiliser std::vector<std::string> a la place.

sinon tu définit une CONSTANTE préprocesseur qui vaut le nombre d'armes que tu as défini.

#define CONSTANTE_N 7

string tableauArmes[CONSTANTE_N] = {"Epée rouillée", "Epée aiguisée",
"Epée batarde à 2 mains", "Glaive de Vercingétorix",
"Lance de chevalier", "Trident de Poséidon",
"Le canif de Dieu"};

puis dans ta boucle tu fais:

for(i=0;i<CONSTANTE_N;++i){/*bla bla ...*/}

* Un wrapper C++ pour sqlite * Une alternative a boost units

MatteX

18 mars 2008 à 15:33:55

Citation : ledemonboiteux

#define CONSTANTE_N 7

Il est temps de passer au "++"

const int CONSTANTE_N = 7;

1) Typé
2) Appartient à un espace de nom
3) Le compilateur peut s'apercevoir d'une ambiguïté entre 2 entités du même nom.

lmghs

18 mars 2008 à 16:04:49

Citation : ledemonboiteux

a- sizeof(tableauArmes) c'est le nombres de bytes de ton tableau et pas le nombre d'elements qu'il contient.

b- Tu ne peux pas connaitre a l'execution le nombre d'éléments d'un truc[].
c- Je te conseille grandement d'utiliser std::vector<std::string> a la place.

d- sinon tu définit une CONSTANTE préprocesseur qui vaut le nombre d'armes que tu as défini.

#define CONSTANTE_N 7

string tab[CONSTANTE_N] = { ...}

a- C'est vrai

b- c'est faux, avec les tableaux statiques, on peut le savoir
cf p.ex. http://hermitte.free.fr/Info/C++/libs/luc_lib/array_help.hpp (oubliez le pdf dans cette zone du site, il est mauvais)

c- au détail de l'initialisation si on veut coder le biniou en dur ... vector sera effectivement mieux pour des vraies solutions (où tout sera lu depuis un fichier p.ex.)

d- Cette solution n'en est pas une, si la constante déclarée diffère du nombre d'éléments effectivement déclarés, on va avoir des problèmes.
Perso, j'ai plutôt tendance à post-initialiser la constante avec:

std::string tab[] = { ....} ;
const size_t N = array_size(tab);

ledemonboiteux

19 mars 2008 à 9:41:48

Imghs, merci pour tes remarques, je ne conaissais pas la méthode en particulier array_size(...)

Mattex, oui les const c'est mieux que les directives prepross.

* Un wrapper C++ pour sqlite * Une alternative a boost units