Hello,

Je bloque sur une segmentation fault à l'appel de cette fonction, ça fait deux heures que je test et je n'arrive pas à voir clair. Est ce que quelqu'un peut m'apporter son regard extérieur ?

1. define TMAX 100000

int search_by_title(char *titre, s_livre **tab)
{
int i;

/* Parcours du tableau de structure tab. */
for (i = 0; i < TMAX; i++) {
/* Arrêt si le titre de l'ouvrage est trouvé. */
if (strcmp(tab[i]->titre, titre) == 0) {
printf("Ouvrage de titre %s trouvé, numero %d, auteur %s.\n", titre, tab[i]->num, tab[i]->auteur);

return 0;
}
}

/* Sinon titre non trouvé car sortie de boucle. */
printf("Ouvrage de titre %s non trouve.\n", titre);

return 1;
}

Je pense qu'il y a un soucis avec la ligne :
strcmp(tab[i]->titre, titre) == 0

tab est un tableau de pointeur sur structure. titre sera récupérer dans le main via scanf et mis en paramètre à l'appel de cette fonction.
tab est bien crée avec un double malloc, initialise a NULL, puis chaque case est rempli par lecture dans un fichier. De ce coter le test et l'affichage tournent. Dès l'appel de cette fonction, j'ai une segmentation fault

hersage,

je n'ai pas le temps de jetter un oeil maintenant. Je ne sais pas quel compilateur et systeme d'exploitation tu utilises. Sous linux, il existe un utilitaire appelle valgrind qui sert a traquer les erreurs de memoire. L'utiliser pourrait t'amener plus d'information.

Je suis sous Mac OS X.
Je n'arrive pas à voir l'erreur.

La fonction parcours le tableau de structure, regard pour chaque itération si le titre stocké dans la structure à la case i du tableau (tab[i]->titre) correspond au titre fourni en argument via strcmp.
S'il le titre correspond, il affiche le titre, numero et l'auteur, c'est a dire la structure stockée dans la case i. Et on sort de la fonction.
Sinon on continu, jusqu'à TMAX, taille du tableau et on sort si on le trouve pas.

Ou ai je fais une erreur ?

comment est ce que ta structure est initialise ?

fournit un code complet.

s_livre **creer_tab()
{
s_livre **tab;
int i, j;

/* Allocation du tableau tab de taille TMAX. */
tab = (s_livre **)malloc(sizeof(s_livre *)*TMAX);
if (tab == NULL) {
printf("Probleme d'allocation.\n");
exit(1);
}

/* Allocation du tableau de structure s_livre. */
for (i = 0; i < TMAX; i++) {
tab[i] = (s_livre *)malloc(sizeof(s_livre));
if (tab[i] == NULL) {
printf("Probleme d'allocation.\n");
exit(1);
}
}

/* Initialisation a NULL du tableau. */
for (j = 0; j < TMAX; j++) {
tab[j]->num = 0;
tab[j]->titre = NULL;
tab[j]->auteur = NULL;
}

return tab;
}

.h
typedef struct livre
{
char *titre;
char *auteur;
int num;
} s_livre;

L'ensemble du code a pas mal de lignes

http://pastebin.com/

strcmp(NULL, "foo"); devrait faire une segfault

Ca veut dire que mon tableau est vide ?
Pourtant la fonction creer et la fonction stock qui rempli mon tableau le fait bien :

void stock(s_livre **t)
{
int i, numero; //numero, stock le numero entré par l'utilisateur.
char titre[TAILLE_MAX], auteur[TAILLE_MAX]; //titre et auteur, deux tableaux qui stokent le titre et l'auteur entrés par l'utilisateur.

/* Ouverture du fichier fichier.txt */
FILE *f = fopen("fichier.txt", "r");
if (f == NULL) {
printf("Erreur lors de l'ouverture de fichier.txt\n");
exit(1);
}

/* Stock toutes les lignes du ficher fichier.txt dans le tableau de structure. */
for (i = 0; i < TMAX; i++) {
fscanf(f, "%d %s %s\n", &numero, titre, auteur);
t[i]->num = numero;
t[i]->titre = strdup(titre); //strdup alloue et stock le titre dans t[i]->titre
t[i]->auteur = strdup(auteur);
}

/* Fermeture du fichier. */
fclose(f);
}

Le fichier contient a chaque ligne : num titre auteur
L'affichage du tableau affiche correctement. Pourquoi le tableau serait vide

je ne sais pas bien...
Que se passe t'il si le fichier n'a pas TMAX ligne ? ou si le titre ou l'auteur contienne un espace ?

De facon general, tu devrais utiliser les fonctions "n" des fonctions sur les string (strncmp, strndup, strncpy, ...)

BTW, valgrind fonctionne sur macosX.

Si tu ne trouve pas, fais une archive du code et des donnees et met le tout en ligne quelquepart.

Normalement si TMAX est plus grand que le nombre de ligne dans le fichier, les cases en trop du tableau sont NULL.
Dans le fichier, chaque lignes contient : numero titre auteur
Il n'y a pas d'espace dans le titre ou auteur.

J'ai une question, quand je demande à l'utilisateur d'entrer un titre par exemple et que je récupère sa saisi via scanf, est ce que ca pose un problème si l'utilise en paramètre d'une fonction, par exemple pour une recherche de titre?

si elle sont null, alors tu vas appeller strcmp(NULL, "foo");

Tu as passer le code dans un debugger pour voir ce qui se passe ? ou dans valgrind ?

J'ai bien recu ton code par MP.
La prochaine fois, fais une archive ou met le code sur un wall.

Tu as des pointeur non initialise dans le code. Recompile ton code avec tous les warning!

Ok je ferrai ça pour la prochaine fois
Par contre pour moi le problème vient du scanf, je vais re-vérifier pour les pointeurs...

il vient du scanf... compile avec les warning et execute le code dans valgrind. L'erreur apparait immediatement

Je pense que le probleme vient du fait que tu vas parcourir ton tableau jusqu'a TMAX structures, mais que si il fait moins de TMAX, tu vas deborder. Je te conseille de mettre un pointeur NULL a la fin de ton tableau de structure, et de mettre comme condition d'arret tab[i].