Bonjour à tous,

je viens aujourd'hui sur ce forum à la recherche de quelqu'un qui pourrait m'aider sur un problème en C.

Pour exposer mon problème, j'ai un fichier test.txt disponible où sont répertoriées des infos sur des œuvres (nom de l'œuvre, artiste, valeur et 0 ou 1 suivant qu'elle soit prêtée ou non) présenté comme ceci:

tableau1 peintre1 valeur1 0
tableau2 peintre2 valeur2 0
tableau3 peintre3 valeur3 0
tableau4 peintre4 valeur4 0

A partir de ce fichier, j'ai écrit une fonction qui doit permettre (normalement ) d'afficher la valeur du tableau que l'on veut. Voici la fonction:

void Valeur_oeuvre()
{
char nom[20]={""};
char art[20]={""};
int val=0;
int pret=0;
char nom_rech[20]={""};

FILE* fichier = NULL;
fichier = fopen("test.txt", "r");

if (fichier != NULL)
{
printf("Entrer le nom de l'oeuvre dont vous recherchez le prix : ");
scanf("%s", &nom_rech);

while (!EOF)
{
fscanf(fichier, "%s %s %d %d", &nom, &art, &val, &pret);

if (nom == nom_rech)
{
printf("%d", val);
}
}
fclose(fichier);
}
else
{
printf("Problème d'ouverture du fichier !");
}
}

Mais avec cette fonction, après avoir entré le nom de l'œuvre le programme se termine et rien ne s'affiche.

Voila mon problème. Si quelqu'un pouvait m'indiquer ce qui ne va pas. Je vous remercie d'avance.

On ne compare pas des chaine de caractere avec == mais avec strncmp. regarde la manuel de la fonction pour plus de details.

Je rajouterai que :
« if (nom == nom_rech) » ce test sera toujours faux, quoi que tu fasses.

Je te laisse le soin de trouver pourquoi.

Ah oui, effectivement ca marche avec strcmp . La je compare des pointeurs en fait, si j'ai bien compris.

En tout cas, merci à vous deux, je vais pouvoir avancer .

On t'a dis str***n***cmp.

Ne viens pas pleurer dans 2 jours parce que ton code rame et/ou segfaulte…

C'est bon j'ai fait une erreur de frappe, pas besoin d'être agressif comme ça

Chris_27 est peut-etre un tantinet agressif mais tu t'es toi-meme apercu que des fois il fallait etre un peu plus agressif que le compilateur pour avoir un programme qui fonctionne

Strcmp marcherait tres bien dans ce cas la, puisque comparerait les deux chaines en entier, non ?

Vous lui consseillez de faire strNcmp pour quelle raison ?

Quand tu fais des comparaisons de chaine de caracteres et que tu ne sais pas trop ce que tu fais (comme OP). Tu utilises toujours strncmp: "Better safe than sorry".

En l'occurence la chaine qu'il recoit de scanf n'est deja pas borne puisque scanf n'est pas une fonction safe de lecture de chaine de caractere.