Discussion :

[passant_]

Salut à tous,

Néophyte en informatique, je découvre maintenant les joies de la lecture et l'écriture dans les fichiers. Ici le fichier s'appelle valeurs.txt et contient 4 valeurs.
En fait, j'essaie de créer une fonction qui lit les 4 valeurs dans mon fichier et me retourne la valeur moyenne et la retourne au programme principal.
Pouvez-vous m'aider à trouver mon (mes?) erreurs ?

Merci d'avance.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
 
float MoyenneFichier(char nom[]);
 
int main()
{
    float resu;
    resu=MoyenneFichier("valeurs.txt");
    printf("La moyenne est %f\n", resu);
    return EXIT_SUCCESS;
}
 
float MoyenneFichier(char nom[])
{
    float a,b,c,d,resultat;
    FILE *MonFichier;
    MonFichier=fopen("nom","r");
    if (MonFichier=NULL) {
        printf("erreur! fichier valeurs.txt n'existe pas");
        return EXIT_FAILURE;
        }
    fscanf(MonFichier,"%f",&a);fscanf(MonFichier,"%f",&c);
    fscanf(MonFichier,"%f",&b);fscanf(MonFichier,"%f",&d);
    printf("Nous avons lu %f,%f,%f et %f\n",a,b,c,d);
    resultat=(a+b+c+d)/4;
    fclose(MonFichier);
    return resultat;
}

Je pense que l'erreur se situe au niveau de la ligne "MonFichier=fopen("nom","r");"


PS : J'ai pensé aux balises [code]cette fois...

[matafan]

if (MonFichier=NULL)

[passant_]

Quelle éloquence ! Je suis bluffé.

[souviron34]

ce que veut dire Matafan, c'est que un test d'égalité en C s'écrit avec 2 signes "=". Un seul et c'est une assignation.

Si tu écris :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if ( MonFichier = NULL)

cela signifie que tu affectes NULL à la variable MonFichier, ce qui n'est pas ce que tu veux

Deuxième point : ton calcul du résultat se fait par une moyenne de 4 nombres réels, dont le résultat est stocké dans un réel. MAIS tu divises par un nombre entier, tu auras donc une division entière.

Pour faire une vraie division réelle, il suffit de mettre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
    resultat=(a+b+c+d)/4.0;

[passant_]

Merci souviron 34.
Bonne journée.

[souviron34]

et tu peux aussi simplifier en écrivant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
    fscanf(MonFichier,"%f %f %f %f",&a,&b, &c, &d);

En admettant que tes nombres soient séparés par un espace dans le fichier.

[diogene]

souviron34 :

Deuxième point : ton calcul du résultat se fait par une moyenne de 4 nombres réels, dont le résultat est stocké dans un réel. MAIS tu divises par un nombre entier, tu auras donc une division entière.

Pour faire une vraie division réelle, il suffit de mettre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

    resultat=(a+b+c+d)/4.0;

Absolument pas :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
float a,b,c,d,resultat;
...
resultat=(a+b+c+d)/4;

le 4 sera automatiquement en float pour effectuer l'opérations entre float et obtenir un résultat du type float.

[Emmanuel Delahaye]

et tu peux aussi simplifier en écrivant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
    fscanf(MonFichier,"%f %f %f %f",&a,&b, &c, &d);

Dans tous les cas, vérifier la valeur retournée par fscanf() avant de continuer...