Discussion :

[SofEvans]

Bonjour

Je voudrais creer pointeur sur un tableau de type Arme, Arme etant une structure definie.
Je dois mal m'y prendre, car je compile sans warning ni error mais lorsque je lance le programme, cela plante.
J'ai un SIGSEV , segmentation fault, mais je dois avouer que je ne vois pas ou.

Voici Arme.h

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
 
#ifndef ARME_H
#define ARME_H
 
 
typedef enum NomArme
{
    LANCE_ROQUETTE,
    LANCE_MISSILE,
    UF_45,
    RAPTOR,
    NombreArme = 4
};
 
typedef struct DefinitionArme
{
 
    /* Caracteristique fixe de l'arme */
    std::string nom;
    float cadence;
 
}Arme, *pArme;
 
pArme CreationListeArmes (void);
 
#endif

et voici Arme.cpp

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
 
#include "arme.h"
 
using namespace std;
 
pArme CreationListeArmes (void)
{
    pArme ListeArme = (pArme) malloc (sizeof(Arme) * NombreArme);
 
    /* On creer toutes les armes */
    ListeArme[LANCE_ROQUETTE].nom = "Lance Roquette";
    ListeArme[LANCE_ROQUETTE].cadence = 80;
 
    ListeArme[LANCE_MISSILE].nom = "Lance Missile";
    ListeArme[LANCE_MISSILE].cadence = 20;
 
    ListeArme[UF_45].nom = "UF 45";
    ListeArme[UF_45].cadence = 990;
 
    ListeArme[RAPTOR].nom = "Raptor";
    ListeArme[RAPTOR].cadence = 678;
 
    return ListeArme;
}

L'appel dans le main :

Code :

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

1
2
 
pArme ListeArme = CreationListeArmes();

Je voudrais savoir ce qui ne va pas, car j'y bloque depuis pas mal de temps et j'avoue mon incompréhension devant ce probleme.

J'ai donc quelques questions :

1 - Le sigsev ne viendrait-il pas du fait que j'utilise des enum pour mon tableau ?
2 - Est ce que j'ai réellement créer un pointeur vers un tableau de type Arme ?
3 - Dans ma fonction, ne faudrait-il pas plutot faire
ListeArme[LANCE_ROQUETTE]->nom = "Lance Roquette"; et non ListeArme[LANCE_ROQUETTE].nom = "Lance Roquette"; ?

Voila, merci de m'avoir lu, bonne journée a vous.

[ram-0000]

2 choses

Ton bug, comme tu fais un malloc, le constructeur de std::string nom de ta classe arme n'est pas appelé. Je pense que le string est dans un état "pâteux".

Il ne faut pas mélanger le C et le C++ (et d'ailleurs, je vais déplacer ton post dans C++)

Secondo, ton énumération me parait dangeureuse

Code :

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

1
2
3
4
5
6
7
8
typedef enum NomArme
{
    LANCE_ROQUETTE,
    LANCE_MISSILE,
    UF_45,
    RAPTOR,
    NombreArme = 4
};

Lorsque tu vas vouloir rajouter une arme, il faudra pas oublier de modifier le "= 4" en "= 5" (et je parie que tu vas oublier).

[SofEvans]

Salut,

Merci de ta reponse

Secondo, ton énumération me parait dangeureuse

Code :

typedef enum NomArme
{
LANCE_ROQUETTE,
LANCE_MISSILE,
UF_45,
RAPTOR,
NombreArme = 4
};

Lorsque tu vas vouloir rajouter une arme, il faudra pas oublier de modifier le "= 4" en "= 5" (et je parie que tu vas oublier).

Non, justement.
J'ai mis cette variable expres afin que plus tard, lorsque je rajouterai une arme, j'aurai juste a modifier cette enum.
Enfin, si tu as plus securisé, je suis preneur.

Il ne faut pas mélanger le C et le C++

Euh, cela veut dire que l'on ne doit pas utiliser des structure avec le C++ ?
Je viens de me rendre compte que j'utilise encore le malloc au lieu du new, mais mis a part cela, quelle instruction "appartienne" au C ?

Ton bug, comme tu fais un malloc, le constructeur de std::string nom de ta classe arme n'est pas appelé. Je pense que le string est dans un état "pâteux".

Ah, ca je ne le savais pas. J'avais lu qu'avec l'objet string, il suffisait juste de faire Arme.nom = "bonjour"; et que cela suffirait.
Apparement non.
Peut-tu m'en dire plus ? (La reference a l'etat "pateux" m'interesse).

PS : Désolé, je me suis effectivement trompé de forum

[Goten]

Pour l'utilisation du C il faisait référence au malloc surtout je pense. En effet le malloc n'appelle pas le constructeur et donc les objets ne sont pas initialisés.
Deux autres choses qui te viennent certainement du C :
Les casts C-like t'oublies à la place utilise des casts du C++ qui sont beaucoup plus safe (les casts en x_cast<> comme static_cast<> dynamic_cast<> etc).
La définition des structures / classe en C++ pas besoin de typedef :

Code :

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

1
2
3
 
typedef struct foo{
} foo;

s'écrit tout simplement en C++ :

Code :

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

1
2
3
4
 
struct foo{};
//plus loin dans le code pour créer une instance :
foo x;

Enfin ici tu n'as pas besoin d'un pointeur, (enfin pas avec le code que tu nous montre) et donc t'embêter avec de l'alloc dynamique. alloue tout ça statiquement sur la pile.

[ram-0000]

1/ Pas besoin de mettre =4 ou =5 derrière NombreArme, le compilateur le fait sans erreur donc laisse le faire.

Code :

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

1
2
3
4
5
6
7
8
typedef enum NomArme
{
LANCE_ROQUETTE,
LANCE_MISSILE,
UF_45,
RAPTOR,
NombreArme     /* doit IMPERATIVEMENT être le dernier item de l'numeration */
};

2/ Si si, on peut utiliser des structures en C++ mais quand ces structure contiennent des objets au sens C++ du terme, il faut les allouer avec new et surtout pas malloc.

3/ malloc() appartient au C (comme printf et scanf) et new appartient au C++ (comme std::cin et std::cout)

4/ Quand tu fais un new d'une structure contenant des objets C++ (std::string dans ton cas), les constructeurs de ces objets sont automatiquement appelés par new.

Dans ton cas, tu faisais un malloc et donc le constructeur de std::string n'était pas appelé (par contre il avait sa place réservée). Comme le constructeur n'est pas appelé, le std::string n'est pas initialisé et contient donc des valeurs aléatoires.

Lorsque tu faisait ensuite std::string nom = "roquette", le std::string devait essayer de libérer son buffer interne ou croire qu'il était initialisé et cela partait en cacahuète

[SofEvans]

Argh

Chuis deg'

Bon, merci de vos précision, je test tout ca et je vous tient au courant.

Difficile de passer au C++ quand on a trop d'automatisme du C

[SofEvans]

Bon, ben tout roule.
Merci a vous

Je vais revoir les base du C++, notamment les structures. Je ne savait pas qu'il y avait une difference entre le C et le C++ pour les cast.

Je met le post en resolu, of course

[3DArchi]

1/ Pas besoin de mettre =4 ou =5 derrière NombreArme, le compilateur le fait sans erreur donc laisse le faire.

Code :

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

1
2
3
4
5
6
7
8
typedef enum NomArme
{
LANCE_ROQUETTE,
LANCE_MISSILE,
UF_45,
RAPTOR,
NombreArme     /* doit IMPERATIVEMENT être le dernier item de l'numeration */
};

Il m'arrive de faire ça :

Code :

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

1
2
3
4
5
6
7
8
enum NomArme
{
LANCE_ROQUETTE =0,
LANCE_MISSILE,
UF_45,
RAPTOR,
NombreArme
};

Et le typedef n'est pas nécessaire en C++.

[SofEvans]

Merci pour vos information.

J'ai effectivement lu que les vector seraient indiqué en C++ et qu'il ne faut plus faire de tableau alloué dynamiquement, meme avec new.

Quelles seraient les consequence d'un new sur un tableau ?

Au fait, j'utilise le typedef car il permet de faire un alias avec un pointeur.

Code :

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

1
2
3
4
5
 
typedef struct DefinitionArme
{
/* Code */
}Arme, *pArme;

Ainsi, il me suffit de faire pArme au lieu de *Arme.
Quel est l'equivalent en C++ ?



Merci aussi pour l'explication du transtypage, je vais approfondir davantage cela.