[C]erreur de segmentation dans une fonction Fermé

Question

bonjour à tous j'ai une erreur de segmentation dans une focntion voilà l'idée de base ; on creer une fonction du genre : afficheOrigineavecListe(element de ta liste (genre ab), et une liste avec lekel il faut tout afficher) { tant que ta liste nest pas vide si lelement de ta liste que tu parcours et different de Origine, alors tu afifche Origine->element et j'appel cette fonction genre avec : ab, et tab[0] pui ab et tab[1], etc... seq =(Liste **) malloc (NBRSEQ * sizeof(Liste *)); // les tableau des toutes les Nseq for(i=0; imot); for(j=0; jmot, seq[j]); } } void AfficherOrigineListe(char mot [150], Liste *p) { while (p!=NULL) { // printf("%s", p->mot); if(strcmp(p->mot, mot)!=0)_____________________ici l'erreur_____________________ { printf("%s", p->mot); printf("->"); printf("%s", mot); // } p=p->suivant; } } j'attends vos reponse

lami20j · Answer

Salut, l'erreur est peut être ici. while (p!=NULL) { printf("%s", p->mot); for(j=0; jmot, seq[j]); } } La condition sera toujours vraie vu que tu n'incrémentes paswhile (p!=NULL) { printf("%s", p->mot); for(j=0; jmot, seq[j]); } p=p->suivant; }

stroumpf · Answer

merci Lami20, mais meme avec ca ca genere la mem erreur au meme emplacement :(

lami20j · Answer

D'ailleurs il te manque une accolade (pourquoi tu utilises les accolades quand il ne faut pas?) for(i=0; imot); for(j=0; jmot, seq[j]); p=p->suivant; } }

stroumpf · Answer

le probleme reste à ce niveau
          if(strcmp(p->mot, mot)!=0)


quand je fait des printf de cette facon
 printf("%s
", p->mot);
  printf("%s
",mot);
 if(strcmp(p->mot, mot)!=0)
juste avant le declenchement du message d'erreur , il n'affiche pas  ce message printf("%s
", p->mot);
c'est a dire le probleme vient de p
je sais pas si c'est null ou sa manque une allocation ou autre.
je suis perdue vraiement

lami20j · Answer

Ça marche comme ça?void AfficherOrigineListe(char mot [150], Liste *p)
{
  while (p!=NULL){
    printf("%s -> %s
", p->mot,mot);
    p=p->suivant;
  }
}

stroumpf · Answer

non
ya une erruer de segmentation ici : printf("%s -> %s
", p->mot,mot);
je pense que le probleme vient de p->mot

lami20j · Answer

Re,

je vois ça

seq =(Liste **) malloc (NBRSEQ * sizeof(Liste *)); // les tableau des toutes les Nseq
for(i=0; i<NBRSEQ; i++)
{
//GenRA(seq[i], res);
p=seq[i];

Mais je ne vois null part que seq contient quelque chose
tu alloues la mémoire, d'accord mais après????

Pour moi seq ne pointe vers rien du tout

stroumpf · Answer

merci, c'est ce que j'ai remarque:
j'ai fait ca
seq =(Liste **) malloc (NBRSEQ * sizeof(Liste *)); // les tableau des toutes les Nseq
for(i =0;i<NBRSEQ-1; i++)
{
    seq[i]=NULL;


}

    seq[0]=seq2;
mais ca marche pas aussi :(

lami20j · Answer

Ton seq ne contient rien.

Tu dis ça // les tableau des toutes les Nseq


Ben, tu n'as qu'à commencer affecter le tableau seq avec tes Nseq et ensuite le traiter

stroumpf · Answer

je comprends pas Lami, 
tu peux m'eclaircir un peu stp,?
merci

lami20j · Answer

Ben, quand tu fait ça
seq =(Liste **) malloc (NBRSEQ * sizeof(Liste *));

tu ne fait qu'allouer de la mémoire pour un tableau de listes
ensuite tu dois faire pointer chaque élément de ton tableau de listes vers une liste 

après ça tu pourras parcourir ton tableau de listes

chuka · Answer

Salut,
ben comme dis lamij20, t'as rien dans seq.....
peut etre fait un tableau du style:
int i=0;
liste* tab[NBRSEQ];
while(i<=NBRSEQ)
{tab[i]=le_nom_de_ta_liste;
i++;}

@+

stroumpf · Answer

merci chuka
comme ca ca plante : for(i =0;i<NBRSEQ-1; i++)
{
    seq[i]=p;


}
:(

lami20j · Answer

C'est toi qui c'est mieux que nous que seq doit contenir.

C'est à toi de nous dire.
Quand tu crées seq à quoi tu penses qu'il doit contenir?

chuka · Answer

p est bien un pointeur sur liste?
déclare comme list*p ou list p?

stroumpf · Answer

chaque case de tableau va contenir une liste de mots 
typedef struct L
{
    int freq;
    char mot[C_TAILLE_MOT];
    struct L *suivant;
} Liste;
voilà ^^

lami20j · Answer

Re,

comme ca ca plante : for(i =0;i<NBRSEQ-1; i++)
{
seq[i]=p;


} 

En fait quand tu fait ça, tu ne fait que pointer chaque élément de tableau sur la même liste.

Pour chouka

p est un pointeur de type Liste

lami20j · Answer

chaque case de tableau va contenir une liste de mots 

Justement, où sont ces listes de motsseq[0]=ListeDeMot1;
seq[1]=ListeDeMot2;
.
.
.
seq[NBRSEQ]=ListeDeMotNBRSEQ;

stroumpf · Answer

et donc ,
je comprends pas comment faire :(
desolé de vous gener

lami20j · Answer

Dans, ton code, où sont les listes de mots que tu veux affecter à ce tableau de liste?

stroumpf · Answer

voila je les rempli a laide de la fonction GenNseq int main(int argc, char *argv[]) { Liste *seq2=NULL; Liste **seq=NULL; Liste *p; int j; int res=0; FILE *fdesc=fopen(FNAME,"r"); Liste *TableHash[TAILLEHASH]; Liste *p1; int i; for (i = 0; i < TAILLEHASH; ++i) { TableHash[i] = NULL; } if ( fdesc ) { char buff[BUFFSIZE]; char prec[BUFFSIZE]; printf("Parsing file"); while ( get_word(fdesc,prec,BUFFSIZE) ) { printf("."); while ( get_word(fdesc, buff, BUFFSIZE) ) { //printf("b2"); //printf("%s %s ",prec,buff); char s3[BUFFSIZE * 2]; sprintf(s3, "%s %s", prec, buff); puts(s3); strncpy( prec,buff,BUFFSIZE); insere_th(TableHash,s3); } fseek(fdesc, 1, SEEK_CUR); } } printf("la table de hachage avant le filtrage"); dump_table(TableHash); res = numligne(fdesc); printf("%d ", res); //printf("%d ", res); //printf("aaaaaaaaaaaaaaaaaaaaaaaa"); clean_table(TableHash, res); printf("la table de hachage filtrée "); dump_table(TableHash); for (i=0;isuivant) { seq2=InsertionEnTete(seq2, p1->freq, p1->mot); } } } seq =(Liste **) malloc (NBRSEQ * sizeof(Liste *)); // les tableau des toutes les Nseq for(i =0;imot); for(j=0; jmot, seq[j], res); p=p->suivant; } } // free(seq); free(TableHash); return 0; }

lami20j · Answer

Cet appel de fonction  AfficherListe(seq[i]);
affiche le résultat désiré?!

lami20j · Answer

Je ne comprends plus rien

Dans ton message initial tu as la fonction avec 2 paramètres 
void AfficherOrigineListe(char mot [150], Liste *p)
et dans ton dernier code je vois que tu lui passe 3 arguments

AfficherOrigineListe(p->mot, seq[j], res);

Décide toi.

stroumpf · Answer

AfficherOrigineListe(p->mot, seq[j], res); 
comme ca car je vais calculer le support qui va utiliser res : nombre de ligne
mais le mem probleme dans les 2 cas lami

lami20j · Answer

Mais ta fonction est conçue avec 2 paramètres seulement.
Tu ne peux pas passer 3 arguments.

chuka · Answer

Si tu as déclaré :
liste *a;
a=(list*)mallac(1*sizeof(list));
a->freq=ta_valeur;
a->mot="ton_mot";
a->suivant=? //la je sais pas comment tu fais....
normalement si tu as declaré 
liste* tab[NBRSEQ];
int i=0;
tab[i]=a; //ca devrait marcher....
Comment tu remplis les valeur de liste??

stroumpf · Answer

oui mais jai réparé ca
voila
void AfficherOrigineListe(char mot [150], Liste *p,int res)
{
    char s3[150];
    float support;
	float confiance;
    while (p!=NULL)
    {
        printf("les mots");
       // printf("%s
", p->mot);
            printf("%s
",mot);
         if(strcmp(p->mot, mot)!=0)

        {
            printf("%s", p->mot);
            printf("->");
            printf("%s
", mot);
            sprintf(s3, "%s %s", p->mot, mot);
  	        printf("%s
",s3);

                   printf("%d
", res);
        			printf("%d", frequence(s3));
        			support=frequence(s3)/(float)(res);     //ça serait pas n+1 le nbre ligne
        			confiance =(float)(support)/(float)((float)(p->freq)/(float)(res));
        			//printf(" / %i ",p->freq);
        			printf("	support : %f
", support);
        			printf("	Confiance : %f
",  confiance );







        }
         p=p->suivant;


    }
}

lami20j · Answer

Alors, essaie juste un seule appel, sans boucle

AfficherOrigineListe(p->mot, seq[0], res);

et affiche ici ce que tu obtiens (il ne suffit pas de dire que ça fonctionne nikel, comme je suis un peu parano ;-) j'ai besoin d'interpréter moi même le résultat )

lami20j · Answer

Essaie juste ça
printf("%s, %s
",seq[0]->mot,seq[1]->mot);
et affiche le résultat (il ne suffit pas de me dire qu'il plante)

stroumpf · Answer

ya rien
la liste des sequence :
  ca plante
il naffiche rien

lami20j · Answer

Alors affiche le résultat de AfficherListe(seq[0]);
 AfficherListe(seq[1]);

stroumpf · Answer

seq[0]et seq[1] sont vides donc rien à afficher mais des [4] et seq [8] sont pas vide donc l'affichage donne:

lami20j · Answer

Alors, faitAfficherOrigineListe(seq[4]->mot, seq[8], res);

stroumpf · Answer

voila les listes:
il est beau he is nice
bonjour les mais du monde good morning friends of world->il est beau he is nice
bonjour les mais du monde good morning friends of world il est beau he is nice
support: 0.1427
confiance : 0.333
voila
pas d'erreur de segmentation ^^

lami20j · Answer

Ok, donc il faut écrire la fonction d'une manière qu'il ne traite que les éléments du tableau seq qui pointe vers quelque chose.
Tu as en fait l'erreur de segmentation puisque seq[0] par exemple est NULL depuis le début.

Au moins on a localisé le problème ;-)
Je ne peux plus rester.

A demain si jamais il n'y a pas d'autre personne pour prendre le relais ;-)

lami20j · Answer

En fait je pense que tu dois écrire for(i=0; imot); for(j=0; jmot, seq[j], res); p=p->suivant; } } }

stroumpf · Answer

merci, mais le meme problem persiste encore

lami20j · Answer

Afficheprintf("%p
",seq[0]);

stroumpf · Answer

00000000
le probleme Lami n'ets pas dans la 1ere case qui est vide, mais je pense la derniere ou bien au milieu mais certainement pas la 1ere

lami20j · Answer

Les conditions if que j'ai mis c'est justement pour agir seulement pour les listes valides.

Utilise la fonction sans res pour voir.
Donc modifie le prototype et l'appel

stroumpf · Answer

oui une erreur de segmentation ici: printf("%s", p->mot);

[C]erreur de segmentation dans une fonction

41 réponses

Discussions similaires