Segmentation fault

Fermé

safasahnoun - 14 nov. 2012 à 17:08
loupius Messages postés 697 Date d'inscription dimanche 1 novembre 2009 Statut Membre Dernière intervention 31 décembre 2017 - 14 nov. 2012 à 23:06

Bonjour,
J'ai fait un programme et lors de la compilation il me dit segmentation fault
voici la parti ou j' ai le problème:
typedef Pokemon p;
typedef struct maillon *Pokemon_list;
struct maillon {
p donnee;
Pokemon_list suivant;
};

//cette fonction va ajouter un pokemon (type que j'ai construit ) dans une liste :
Pokemon_list add(Pokemon_list liste, Pokemon p)
{Pokemon_list nouvelliste = malloc(sizeof(Pokemon_list));
nouvelliste->donnee = p;
nouvelliste->suivant = liste;
return nouvelliste;
}
//affichage
void get_pokemons_list(Pokemon_list a){
if (a== NULL) {printf ("liste vide");}
printf("[");
while (a->suivant != NULL)
{printf ("%s,",a->donnee.Nom);
a=a->suivant;}
printf("%s]",a->donnee.Nom);}

A voir également:

Segmentation fault
Segmentation fault ✓ - Forum Programmation
Scanf segmentation fault ✓ - Forum Programmation
Apache segmentation fault ✓ - Forum Linux / Unix
Erreur de segmentation - Forum C
Erreur de segmentation C++ ✓ - Forum C++

3 réponses

Réponse 1 / 3

KX Messages postés 16733 Date d'inscription samedi 31 mai 2008 Statut Modérateur Dernière intervention 31 janvier 2024 3 015
14 nov. 2012 à 17:17

Il te manque un else :

void get_pokemons_list(Pokemon_list a)
{
    if (a==NULL)
        printf ("liste vide");
    else
    {
        printf("[");
        while (a->suivant != NULL)
        {
            printf ("%s,",a->donnee.Nom);
            a=a->suivant;
        }
        printf("%s]",a->donnee.Nom);
    }
}

On doit aussi pouvoir faire plus simplement :

void get_pokemons_list(Pokemon_list a)
{
    printf("[");
    for (;a!=NULL;a=a->suivant)
        printf ("%s,",a->donnee.Nom);
    printf("]");
}

safasahnoun
14 nov. 2012 à 17:23

j' ai corrigé mais le probmème de segmentation rest !

KX Messages postés 16733 Date d'inscription samedi 31 mai 2008 Statut Modérateur Dernière intervention 31 janvier 2024 3 015
14 nov. 2012 à 17:29

Il se peut alors que ce soit a->donnee qui vaille NULL et donc donnee.Nom qui plante, mais là c'est que ton code est mal fait au moment tu créés ta liste :

void get_pokemons_list(Pokemon_list a)
{
    printf("[");
    for (;a!=NULL;a=a->suivant)
        printf("%s,",(a->donnee==NULL) ? "NULL" : a->donnee.Nom);
    printf("]");
}

Réponse 2 / 3

fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 835
14 nov. 2012 à 17:50

nouvelliste->donnee = p;
nouvelliste->donnee est de type Pokemon.
Où est réalisée l'allocation ? A moins que tu souhaites faire pointer nouvelliste->donnee vers p, mais dans ce cas, il faut mettre un type pointeur.

Réponse 3 / 3

loupius Messages postés 697 Date d'inscription dimanche 1 novembre 2009 Statut Membre Dernière intervention 31 décembre 2017 148
14 nov. 2012 à 23:06

... lors de la compilation il me dit segmentation fault
Ce ne serait pas plutôt à l'exécution ? Sinon, que dit le compilateur ?

voici la parti ou j' ai le problème
Lorsqu'une erreur de programme entraîne une erreur de segmentation (99/100 un pointeur en galère (mal initialisé, utilisé après delete, ...)), il est loin d'être rare que la manifestation de cette erreur se fasse plus tard, voir beaucoup plus tard, voir pas du tout !
Bonne continuation.

Discussions similaires

Pb avec: Segmentation Fault (Core Dumped)

zsh: segmentation fault

Langage C - C/C++ Erreur de segmentation

Erreur de Segmentation (Core Dumped)

segmentation fault(core dumped ) avec gmp

Discussions similaires

Newsletters