Discussion :

[nunurs83]

Bonjour voila j'ai un petit problème... J'ai deux structures :

Code :

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

1
2
3
4
5
6
7
typedef struct TMaillon Maillon;
 
struct TMaillon {
   void *motchaine; /* mot lu et stocke dans la liste chaînée */
   LC *ptrLigneColonne; /* ponteur sur la structure LC */
   Maillon *suivant; /* adresse du prochain maillon de la liste chaînée */
};

Code :

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

1
2
3
4
5
6
7
/* la structure TLC */
typedef struct TLC LC;
 
struct TLC {
    int Ligne, Colonne; /* ligne et colonne d'un mot */
    LC *suivant; /* adresse du prochain maillon de la liste chaînée LC */
};

La première structure possède un pointeur permettant d'acceder aux données de la deuxième.
Cependant dès que je tente de faire une lecture sur la structure LC par l'intermediaire du pointeur ptrLigneColonne, mon programme s'arrete sur le message d'erreur : "erreur de segmentation".

sachant que je tente d'accéder par ce code :

Code :

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

MaChaine_move->ptrLigneColonne->Ligne

et que la structure LC se remplie bien...

Merci pour vos conseils.

[Swoög]

Salut !

Question bête : tu initialises MaChaine_move->ptrLigneColonne ?

Et pourquoi un void* ? si c'est une chaîne de caractères, pourquoi ne pas utiliser un char* ?

[nunurs83]

C'est pour un projet scolaire...ils veulent appliquer le principe de généricité donc je suis obligé de mettre un void*
Arf en effet j'ai testé la validité de MaChaine_move->ptrLigneColonne et il n'est pas valide...mais je vois pas à quoi l'initialiser

[Swoög]

comment initialises-tu MaChaine_move ?

[nunurs83]

MaChaine_move me permet juste de parcourir ma liste chaînée

Code :

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

1
2
3
4
5
for(MaChaine_move=MaChaine; MaChaine_move!=NULL; MaChaine_move=MaChaine_move->suivant)
{
   printf("%s ",MaChaine_move->motchaine);
   if(MaChaine_move->ptrLigneColonne) printf("huhu"); //test validité pointeur 
}

[Swoög]

dans ce cas, c'est la création de ta liste chaînée qui est foireuse

[nunurs83]

dans ce cas, c'est la création de ta liste chaînée qui est foireuse

Je ne pense pas puisque je peux la parcourir sans problème et afficher son contenu.

[Skyounet]

Ta liste est peut-être bonne mais pas les pointeurs sur LC.

Montre ton code en entier.

[nunurs83]

je crée un nouveau maillon de ma liste chaînée, je remplie ce maillon avec les données (cest-à-dire le mot), puis j'appelle ma fonction de création de la deuxième liste (type LC) :

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
27
 Maillon *inserer_debut(Maillon *NewMaillon, int Taillemot, char *MotLu, int Numligne)
{
      Maillon *vMaillon = NULL;
      /* on alloue la memoire pour le nouvel element */
    if((vMaillon = (Maillon *)malloc(sizeof(Maillon))) == NULL)
    {    perror("malloc failed ");
        exit(1);
    }
    
    /* Creation du tableau pour le mot */
    vMaillon->motchaine = malloc (Taillemot*sizeof(char));
    if( vMaillon->motchaine == NULL )
    {
        fprintf(stderr,"Allocation impossible pour motchaine");
        exit(EXIT_FAILURE);
    }
   /* on copie le mot pour le stocker */
    strcpy(vMaillon->motchaine, MotLu);

    /* on appelle maintenant la fonction pour crée un maillon de type LC */
    LC* ptrLigneColonne = inserer_debutLC(ptrLigneColonne, Numligne);
 
    /* comme il s'agit du premier élément, il n'y a rien après */
      vMaillon->suivant = NULL;
      /* on renvoie notre element */
      return(vMaillon);
}

Ma fonction inserer_debutLC :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
LC *inserer_debutLC(LC *NewLC, int Numligne)
{
    LC *vMesLC = NULL;
    /* on alloue la memoire pour le nouvel element */
    if((vMesLC = (LC *)malloc(sizeof(LC))) == NULL)
    {    perror("malloc failed ");
          exit(1);
    }
    vMesLC->Ligne = Numligne;
    printf("%d", Numligne); /* test pour vérifier le remplissage */
    vMesLC->suivant = NULL;
    return(vMesLC);
}

[Swoög]

à quoi te servent Maillon *NewMaillon et LC *NewLC dans tes fonctions ? tu ne les utilises jamais !

de plus, tu n'assignes ptrLigneColonne à rien... à mon avis il y a de très gros problèmes là... c'est tout à fait normal que ça plante !!

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
Maillon *inserer_debut(int Taillemot, char *MotLu, int Numligne)
{
      Maillon *vMaillon;
      /* on alloue la memoire pour le nouvel element */
    if((vMaillon = (Maillon *)malloc(sizeof(Maillon))) == NULL)
    {    perror("malloc failed ");
        exit(EXIT_FAILURE);
    }
 
    /* Creation du tableau pour le mot */
    vMaillon->motchaine = malloc (Taillemot*sizeof(char));
    if( vMaillon->motchaine == NULL )
    {
        fprintf(stderr,"Allocation impossible pour motchaine");
        exit(EXIT_FAILURE);
    }
   /* on copie le mot pour le stocker */
    strcpy(vMaillon->motchaine, MotLu);
 
    /* on appelle maintenant la fonction pour crée un maillon de type LC */
    vMaillon->ptrLigneColonne = inserer_debutLC(Numligne);
 
    /* comme il s'agit du premier élément, il n'y a rien après */
      vMaillon->suivant = NULL;
      /* on renvoie notre element */
      return(vMaillon);
}
 
 
 
 
 
LC *inserer_debutLC(int Numligne)
{
    LC *vMesLC;
    /* on alloue la memoire pour le nouvel element */
    if((vMesLC = (LC *)malloc(sizeof(LC))) == NULL)
    {    perror("malloc failed ");
          exit(EXIT_FAILURE);
    }
    vMesLC->Ligne = Numligne;
    printf("%d", Numligne); /* test pour vérifier le remplissage */
    vMesLC->suivant = NULL;
    return(vMesLC);
}

et ça c'est sans corrigé la copie pour la chaîne de caractère (en théorie, tu n'as pas besoin de fournir sa taille, strlen te permet facilement de la récupérer... et si tu souhaites copier une taille fixée d'avance, tu devrais plutôt utiliser strncpy pour éviter les débordements...

[nunurs83]

Arf heu oui se sont de vieux restes *passe encore pour un boulet*
Merci beaucoup