Discussion :

[moule]

Bonjour à tous,
Je rencontre un petit problème, la compilation est ok, mais à l'éxecution la console se fige quelques instants et se ferme... Avec un petit debugger ( sur devc++), j'obtient un jolie "Une violation d 'accés (erreur de segmentation)".

En gros :

Le programme plante a chaque fois que je créer un pointeur vers une variable de structure Sommet ET que j'essai d'accéder à un de ses champs
Exemple :

Code :

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

1
2
3
//se situe dans le main
	Sommet * S;
	S->nbArcs = 0; //<== ca plante !

Par contre, si je créer une variable locale S, ca marche.
Exemple :

Code :

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

1
2
Sommet S;
	S.nbArcs = 0; //<== ca marche.

Je donne le main ainsi que la structure ( il y a d'autre fichier mai je ne pense qu'ils agissent sur le problème) :

Le main :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <stdio.h>
#include <stdlib.h>
#include "dicoParser.h"
#include "structures.h"
 
 
int main()
{
  Sommet  *S;
  S->nbArcs = 3;
  S = fileParser(".\\dico.txt");
  system("PAUSE");	
  return 0;
}

Le fichier strcutres.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
#ifndef STRUCTURES_H
#define STRUCTURES_H
 
#include <stdio.h>
#include <stdlib.h>
 
typedef struct sommet
{
    char lettre;
    struct sommet ** arcs; //tableau de pointeur de sommets
    int nbArcs;
    char * motType; //"Nom", "Adj", "Ver"
    char * motBase; //forme de base du mot
}Sommet;
 
Sommet * nouvSommet();
void addArc(Sommet * s);
 
#endif // end ifndef STRUCTURES_H

Le fichier strucutres.c

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
#include "structures.h"
 
Sommet * nouvSommet()
{
    Sommet * S;
    S->lettre = (char)NULL;
    S->arcs = NULL; //tableau de pointeur de sommets
    S->nbArcs = 0;
    S->motType = NULL;
    S->motBase = NULL;
    return S;
}
 
void addArc(Sommet * s)
{
    struct sommet ** arcs;
}

Si vous pouvez em mettre sur une piste ^^

Cordialement.

[Franck.H]

Et elle est passée où l'allocation dynamique avec malloc pour l'adresse de ton pointeur ? J'en vois pas !
Il faut faire une allocation dynamique de ton pointeur avant de vouloir l'utiliser !

[hegros]

Normal tu as juste un pointeur vers un Sommet et non un Sommet.

Quand tu déclares Sommet S tu as bien un Sommet d'alloué sur la pile c'est une variable automatique.

Quand tu déclares Sommet *S tu déclares juste une variable capable de pointer sur un Sommet mais ca n'alloue pas un Sommet pour autant.


Il faut donc que tu alloues dynamiquement ton objet

Code :

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

1
2
3
Sommet * S = malloc( sizeof Sommet ) ;
//ou
Sommet * S = malloc ( sizeof *S ) ;

puis tester que l'allocation à réussi :

Code :

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

1
2
3
4
5
6
7
8
if(S!=NULL) {
//et enfin accéder aux membre
S->nbArcs
 
//ne pas oublier de libérer la mémoire a la fin
free(S),S=NULL;
 
}

[atnnn]

Il n'est souvent pas necessaire de faire de l'allocation explicite. Lorsque tu as deja une structure, tu peut trouver son adresse avec &:

Code :

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

1
2
3
4
 
Sommet s;
Sommet *p = &s;
s->nbArcs = 7;

[hegros]

Il n'est souvent pas necessaire de faire de l'allocation explicite. Lorsque tu as deja une structure, tu peut trouver son adresse avec &:

Code :

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

1
2
3
4
 
Sommet s;
Sommet *p = &s;
s->nbArcs = 7;

Complétement inutile.Autant directement faire s.nbArcs=7.

On utilise l'allocation dynamique quand on ne sait pas à l'avance le nombre de Sommet que l'on veut car en ne sachant pas combien de Sommet j'ai besoin j'aurais du mal à ecrire


Sommet s[100]; Et si j'en ai besoin que de 2 ? Et si j'en ai besoin plus de 100 ?

Réponse :allocation dynamique.

[Médinoc]

Complétement inutile.Autant directement faire s.nbArcs=7.

En effet, mais cela cesse d'être inutile si le pointeur en question est un paramètre de fonction.

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
void ModifieSommet(Sommet *pSom)
{
    pSom->nbArcs = 42;
}
 
int main(void)
{
    Sommet som1;
    Sommet som2;
    Sommet *pSom3 = NULL;
    ModifieSommet(&som1);
    ModifieSommet(&som2);
 
    pSom3 = malloc( sizeof(*pSom3) );
    if(pSom3 != NULL)
    {
        ModifieSommet(pSom3);
        free(pSom3), pSom3=NULL;
    }
    return 0;
}

Enfin, évidemment, avec une fonction aussi simple, on peut mettre simplement son contenu dans le main(). Mais avec fonction bien complexe montre l'utilité d'y passer un pointeur...

[moule]

Merci pour beaucoup . Je peux avancer ^^

Cordialement.

[Emmanuel Delahaye]

Code :

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

1
2
3
//se situe dans le main
	Sommet * S;
	S->nbArcs = 0; //<== ca plante !

Ta maman, elle t'a jamais dit qu'il ne fallait pas utiliser une variable non-initialisée ? Et ton compilateur, il ne dit rien ? Il est peut être mal réglé :

http://emmanuel-delahaye.developpez....tm#cfg_compilo