Discussion :

[NicoMasterChief]

Bonjour, j'ai une erreur de segmentation avec mon module EnsembleChaine que je n'arrive pas à résoudre depuis plusieurs heures. Mon module stocke des chaînes de caractère dans un tableau.

Voici ma structure et mon constructeur qui pose problème :

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
struct EnsembleChaine
{
    char** s;
    int nb;
};
 
EnsembleChaine creerEnsembleChaine(void)
{
    EnsembleChaine this;
    this->s=malloc(sizeof(char*)*SIZE_DEFAULT);
    for(int i=0;i<SIZE_DEFAULT;i++)
    {
        this->s[i]=malloc(sizeof(char)*SIZE_DEFAULT);
        for(int j=0;j<SIZE_DEFAULT;j++)
        {
            this->s[i][j]='\0';
        }
    }
    this->nb=0;
    return this;
}

J'ai une erreur de segmentation sur cette ligne :

Code :

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

this->s[i][j]='\0';

Pourtant, pour moi tout est correct. J'ai enlevé la boucle for comme ceci :

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
EnsembleChaine creerEnsembleChaine(void)
{
    EnsembleChaine this;
    this->s=malloc(sizeof(char*)*SIZE_DEFAULT);
    for(int i=0;i<SIZE_DEFAULT;i++)
    {
        this->s[i]=malloc(sizeof(char)*SIZE_DEFAULT);
        //for(int j=0;j<SIZE_DEFAULT;j++)
        //{
            //this->s[i][j]='\0';
        //}
    }
    this->nb=0;
    return this;
}

Et ça fonctionne ! Pourtant quand je laisse la boucle for vide comme ceci :

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
EnsembleChaine creerEnsembleChaine(void)
{
    EnsembleChaine this;
    this->s=malloc(sizeof(char*)*SIZE_DEFAULT);
    for(int i=0;i<SIZE_DEFAULT;i++)
    {
        this->s[i]=malloc(sizeof(char)*SIZE_DEFAULT);
        for(int j=0;j<SIZE_DEFAULT;j++)
        {
            //this->s[i][j]='\0';
        }
    }
    this->nb=0;
    return this;
}

Ça ne fonctionne plus et ça me déplace l'erreur de segmentation ici :

Code :

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

this->s[i]=malloc(sizeof(char)*SIZE_DEFAULT);

Du coup, je ne comprends pas ce qui ne va pas. En espérant une aide de votre part, car là j'en peux plus... Merci.

[Neckara]

Bonjour,

As-tu essayé d'utiliser Valgrind ? Il est assez utile pour voir les erreurs liés à la mémoire (fuites de mémoires, segfaults, ...).

[JulieCarree]

bonjour,
au pif,
tu n'as pas mallocqué this
mais je ne suis pas un spécialiste.

[Neckara]

tu n'as pas mallocqué this
mais je ne suis pas un spécialiste.

Bien vu

Je n'avais même pas vu que c'était du C .

Je pense qu'il vaut mieux éviter "new", "delete" et "this" comme nom de variables en C, on ne sait jamais si on veut par la suite réutiliser le tout en C++

[JulieCarree]

Ouf, j'ai eu peur Neckara,
j'ai pensé qu'en lui proposant l'utilisation de Valgrind, tu ne voulais pas lui montrer son oubli directement,

j'ai cru que j'allai prendre un savon.

[Neckara]

j'ai cru que j'allai prendre un savon.

C'est quoi cette réputation que vous êtes en train de me faire ? Je ne suis pas si méchant que ça

Non, en fait je ne voyais vraiment pas d'où pouvais bien venir l'erreur .

Mais en effet, tu as raison, ce serait bien que NicoMasterChief essaye d'utiliser Valgrind au moins une fois sur ce problème. Valgrind est un outil tellement utile

[Médinoc]

Euh, ça ne devrait même pas compiler, en fait, à moins que quelqu'un ait tapé quelque chose de vraiment terminalement stupide, du genre typedef struct EnsembleChaine* EnsembleChaine;...

[jabbounet]

bonjour

j'ai une petite question: SIZE_DEFAUT*est-il constant (genre un #define SIZE_DEFAUT 20)?

Si oui, il est possible de modifier la structure comme suit et supprimer au passage creerEnsembleChaine...

Code :

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

1
2
3
4
5
6
 
struct EnsembleChaine
{
    char s[SIZE_DEFAUT][SIZE_DEFAUT];
    int nb;
};

Dans ce cas l'initialisation des structures peut se faire de cette façon

Code :

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

1
2
3
 
/* attention je suis un peu rouillé en C je ne sais pas si cette syntaxe est vallable */
struct EnsembleChaine ensembleCh = {  { { 0,} , },  0 };

Si l'utilisation de mémoire dynamique est un impératif ou imposé, tu peux regarder du coté de calloc à la place de malloc pour initialiser tes champs à 0 ( ou '\0').

un memset peut aussi faire l'affaire pour initialiser les éléments de ta structure une fois l'allocation terminée.

[NicoMasterChief]

En effet, j'avais oublié d'allouer this avec malloc, erreur très bête ^^.
Merci à tous pour vos réponses.

Sujet clos.

[Obsidian]

Au passage, on ne dit pas « irrésolvable » mais « insoluble » ! :-)

[Médinoc]

Juste par curiosité, peut-on quand même voir la définition du type EnsembleChaine? Pas la structure struct EnsembleChaine que tu as déjà postée, mais le type (probablement défini avec typedef) qui marche sans le mot-clé struct.

[NicoMasterChief]

Au passage, on ne dit pas « irrésolvable » mais « insoluble » ! :-)

http://fr.wiktionary.org/wiki/irr%C3%A9solvable

Juste par curiosité, peut-on quand même voir la définition du type EnsembleChaine? Pas la structure struct EnsembleChaine que tu as déjà postée, mais le type (probablement défini avec typedef) qui marche sans le mot-clé struct.

Code :

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

typedef struct EnsembleChaine* EnsembleChaine;

Je suis en deuxième année de licence Informatique, et c'est comme ça qu'on fait, on appelle ça l'abstraction de type par pointeur.
J'ai vu ton message précédent. Pourrais-tu m'expliquer pourquoi tu es "allergique" à cette méthode ?

[Médinoc]

Parce que tu as utilisé exactement le même nom que la structure pointée. C'est complètement contre-intuitif, surtout pour des programmeurs qui touchent au C++ de temps en temps car en C++, struct T et T représentent forcément le même type.

Je ne vois aucun problème pour un truc clair et intuitif, du genre:

Code C :

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

typedef struct EnsembleChaine* PEnsembleChaine;

Voire les deux:

Code C :

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

typedef struct EnsembleChaine EnsembleChaine, *PEnsembleChaine;

Mais utiliser le même nom pour la structure et son pointeur, c'est tellement stupide que j'avais peine à concevoir que quelqu'un puisse l'avoir vraiment fait. Qu'est-ce qui t'a pris de te tirer une balle dans le pied ainsi?