[ C ] Erreur de segmentation (core dumped)

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [ C ] Erreur de segmentation (core dumped)

AigleRoyal

Bonjour à tous,

Je suis débutant en C et je me mêle les pinceaux avec les pointeurs en ce moment. Je vous expose mon problème.
Je souhaite :

1 - Aller chercher dans un fichier une ligne qui contient par exemple : "etat1 etat2 etat3 e4 e5#" (espaces compris)

2 - Maintenant, je voudrais créer un tableau de 5 pointeurs. Chaque pointeur, pointera sur chacune des chaines de caractères. Autrement dit, si on appelle, le tableau : tab_pointeurs, on aura :

tab_pointeurs[0] qui pointe sur "etat1"
tab_pointeurs[1] qui pointe sur "etat2"
......
tab_pointeurs[4] qui pointe sur "e5"

SEULEMENT, on ne sait pas à l'avance combien d'états on aura dans le fichier, et donc de combien de pointeurs on aura besoin. Et on ne sait pas non plus quelle taille feront chaque chaine de caractères. D'où les allocations mémoires.

Je ne sais pas si le problème est assez clair .

Code :

typedef char* t_etats;
'etats' est une variable globale. Un tableau de 500 caractères maxi qui contiendra toute la ligne du fichier (espaces compris).
i est la variable qui permet de se déplacer dans ce tableau.
//////////////////////////////////////// FONCTION RECHERCHER_ETATS ///////////////////////////////////////////////
/*
* Recherche les états de la machine dans le fichier 'machine'.
* Préconditions : Le fichier existe
* Postconditions : Renvoie un tableau de chaîne de caractères (les états).
*/
//////////////////////////////////////////////////////////////////////////////////////////////////////////////////
t_etats* rechercher_etats(FILE* fichier){
char marqueur = '$'; t_etats* etats_ret=NULL;
/* Parcours du fichier ligne par ligne. On sort de la boucle une fois le mot trouvé.*/
while (!feof(fichier)){
fgets(etats,sizeof(etats),fichier);
if(etats[0] != marqueur){
break;
}
}
printf("Etats recherches : %s",etats);
printf("Taille : %d\n",strlen(etats)-2); // on enlève le '#' et le marqueur de fin '\0'
int unsigned i=0; int j=0; int unsigned k=0; int maxetats=0; int taille_nom_etat=0; int t=0;
// On compte le nombre total d'états présents pour la machine. En comptant le nombre d'espace entre leurs noms et en ajoutant 1.
while(k<strlen(etats)-2){
if(etats[k]==' '){
maxetats++;
}
k++;
}
maxetats++;
printf("nbre d'etats : %d\n",maxetats);
/* tab_etat est un tableau de pointeurs. Sa taille (maxetats) n'est pas connue à l'avance. Mais, on en aura besoin un peu plus haut dans la fonction
* "indice_etat". Sa taille réelle est contenue dans la variable maxetats et on ajoute une case de plus qui contiendra toujours NULL. Cette case jouera le rôle
* de marqueur de fin et nous permettra à l'aide d'une boucle de déterminer la taille du tableau. Les indices vont de 0 à maxetats-1. Donc, si on ajoute une
* colonne de plus, elle aura pour indice 'maxetats'.
*/
t_etats tab_etat[maxetats+1]; tab_etat[maxetats]=NULL;
for(j=0;j<maxetats;j++){
/* On compte la taille en caracteres de chaque etat.
* strlen(mot)-2 pour éviter de prendre le dernier caractère de la chaine # et le marqueur de fin : '\0'.
*/
while(etats[i]!=' ' && i<strlen(etats)-2){
i++;
taille_nom_etat++;
}
printf("taille du nom de l'etat = %d\n",taille_nom_etat);
/*
* Pour chaque état, on alloue une zone mémoire de taille variable qui contiendra son nom (chaine de caractère qui le représente). Ceci, en partant du
* principe que les noms (des états) i.e les chaines de caractère sont de tailles différentes.
*
* Si par exemple, nous avons les états : 'etat1' 'etat2', 'e3', 'q4', 'eta5'
* On allouera :
* Une zone mémoire de 5 caractères pour chacun des états 1 et 2.
* Une zone de 2 caractères pour chacun des états 3 et 4 et
* Une zone de 4 caractères pour l'état 5.
* tab_etat est un tableau de pointeurs sur ces caractères. Donc,
* tab_etat[0] pointera sur l'état 1 et donc sur la chaîne : 'etat1'
* tab_etat[1] pointera sur l'état 2 et donc sur la chaîne : 'etat2'
* tab_etat[2] pointera sur l'état 3 et donc sur la chaîne : 'e2'
* tab_etat[3] pointera sur l'état 4 et donc sur la chaîne : 'q4'
* tab_etat[4] pointera sur l'état 5 et donc sur la chaîne : 'eta5'
*/
tab_etat[j] = malloc(sizeof(char)*taille_nom_etat);
if(tab_etat[j]==NULL){
printf("\n Allocation Impossible \n" );
exit(EXIT_FAILURE);
}
int temp; temp = taille_nom_etat; int taille; taille = strlen(etats)-2;
if(i <= taille){
for(t=0;t<taille_nom_etat;t++){
tab_etat[j][t]=etats[i-temp];
temp--;
}
}
i++;
taille_nom_etat=0;
}
printf("Les etats sont : \n" );
for(j=0;j<maxetats;j++){
printf("%s\n",tab_etat[j]);
}
printf("\n" );
etats_ret = tab_etat;
return etats_ret;
}
// Fin de la fonction rechercher_etats

Merci d'avance

Message édité par AigleRoyal le 10-11-2008 à 13:07:12

Publicité

O'Gure

Multi grognon de B_L

Ce sujet a été déplacé de la catégorie OS Alternatifs vers la categorie Programmation par O'gure

---------------
Relax. Take a deep breath !

Joel F

Real men use unique_ptr

Ton code est fouilli. Decoupes ca en fonction a but restreint (lire le fichier, creer les tableaux, les remplir) tu y verras deja plus clair.

Taz

bisounours-codeur

n'utilise pas feof (ou alors lis sa documentation).

while(fgets(...) != NULL) est la bonne méthode.

olivthill

Ici, dans ce programme, c'est un peu fouillis, surtout parce que l'indentation est mauvaise, mais sinon la longueur n'est pas exagérée, il y a des commentaires, et l'absence de sous-fonctions n'est pas la cause du problème. Ici, dans ce programme, feof() n'est pas non plus la cause du problème.

Le cause est la mauvaise utilisation de la variable "i".

el muchacho

Comfortably Numb

Si tu cherchais au moins où ça plante... la valeur de i, etc. C'est le B.A.BA du débogage.

---------------
Les aéroports où il fait bon attendre, voila un topic qu'il est bien

AigleRoyal

Bonjour,

J'ai essayé de débugger avec gdb et je n'ai rien vu. La valeur de i ne pose pas de problèmes. Pour tester, j'ai rajouté 2 instructions au sortir de la fonction. Je rappelle qu'elle renvoit un tableau de pointeurs.

tableau_etats = rechercher_etats(p_fichierM); // où p_fichierM est un pointeur sur un fichier contenant la ligne "e1 e2 e3 e4 e5#"

printf("Avant : tableau_etats[0] = %s\n",tableau_etats[0]);
printf("Tout est OK\n" );
printf("Apres : tableau_etats[0] = %s\n",tableau_etats[0]);

Il affiche :

"Avant : tableau_etats[0] = e1
Tout est Ok
Erreur de segmentation (core dumped)" Au lieu d'afficher "Après : tableau_etats[0] = e1"

Autrement dit, j'ai l'impression qu'entre deux instructions, la valeur du pointeur est supprimée ?!!! Deux instructions identiques séparées par l'affichage de "Tout est Ok" et l'une donne la bonne valeur et après, PLUS RIEN. Une erreur de segmentation.

J'essaie de débogger avec valgrind s'il en y a qui connaissent et je vous tiens au courant.

A tout à l'heure

AR

Message édité par AigleRoyal le 10-11-2008 à 13:35:27

philippe06

1. typedef char* t_etats;

45. t_etats tab_etat[maxetats+1]; tab_etat[maxetats]=NULL;

en fait l'erreur c'est ligne 45. Je suis surpris que ca compile en fait. Faut faire un malloc à la place d'une déclaration.

---------------
Aimer les femmes intelligentes est un plaisir de pédéraste. (Charles Baudelaire) - Vous vulgarisez :o (Jean-Kevin Dubois)

AigleRoyal

Ok, merci Philippe. Je fais ça et je te met au parfum

Mais, quelle est réellement la différence entre un malloc d'une taille fixe et la déclaration d'un tableau de la même taille fixe ? Y a-t-il une différence en mémoire ?

AR

AigleRoyal

Putain, ça marche NIIIIIIIIIIIIKKKKKKKKKKELL. Ca fait des jours que je cherche... Avec différents déboggeurs.

Tu m'expliques Philipe stp ?

Je crois qu'une fois que j'aurais compris ça, ça restera gravé à TOUT JAMAIS. Bon, alors pourquoi faire un malloc plutôt que de déclarer un tableau ?
Je suis prêt à apprendre ma leçon...

AR

Publicité

philippe06

Oui, il y a une différence en mémoire, c'est tout à fait ça:
malloc -> tas
déclaration -> pile ou "zone statique"

pour une déclaration, il faut connaitre la taille avant l'execution. Si tu veux faire une zone mémoire de taille variable, il faut faire des mallocs.

---------------
Aimer les femmes intelligentes est un plaisir de pédéraste. (Charles Baudelaire) - Vous vulgarisez :o (Jean-Kevin Dubois)

morph

int foo(int n)
{
char tab[n];
...
}

c'est tout à fait possible en C99

Message cité 1 fois

philippe06

Je m'en souviendrais ...

---------------
Aimer les femmes intelligentes est un plaisir de pédéraste. (Charles Baudelaire) - Vous vulgarisez :o (Jean-Kevin Dubois)

Elmoricq

Modérateur

morph a écrit :

int foo(int n)
{
char tab[n];
...
}

c'est tout à fait possible en C99

Possible, mais je ne sais pas ce qui leur est passé par la tête pour nous pondre les VLA. Probablement une grosse pierre.

Mise en situation :
- n est négatif > plantage
- n est abusivement grand > plantage
- problème d'allocation > impossible de le savoir

Bref, les VLA, c'est pas une bonne idée je trouve.

Joel F

Real men use unique_ptr

Sachant que 'abusivement grand' ca commence à 1024 double :E

Amonchakai

Juste pour ma culture VLA, c'est l'acronyme de quoi ? (le 'L' et 'A', j'imagine que c'est Local Allocation)

Elmoricq

Modérateur

Variable-Length Array.

Amonchakai

ok, merci

FORUM HardWare.fr

Programmation

[ C ] Erreur de segmentation (core dumped)

Sujets relatifs
Programmation Threads en C++	utiliser une dll compilée en C# dans un projet VisualC++
Exercices programmation C++	pb erreur dans mon code
Erreur d’exécution 7, mémoire insuffisante	VBA - C++ - DLL
Récupération de temps de réponse (PING) en C	[DRUPAL] C'est içi?
probleme de programmation en C	erreur " objet requis "
Plus de sujets relatifs à : [ C ] Erreur de segmentation (core dumped)

Page générée en 0.148 secondes