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...

[souviron34]

souviron34 :


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.

je sais bien cela, simplement je conseille systématiquement, chaque fois qu'un nombre apparaît dans une opération flottante, de l'écrire en flottant.

Plus de questions et de recherches dans la norme. On est consistent : on fait des opérations sur des réels, on utilise des réels.

Je trouve (et cela se voit au nombre de posts sur ce forum traitant de cela) que ne pas insister sur ce point amène régulièrement des erreurs.

Donc, plutôt que de dire dans ce cas ça marche, mais si (a,b,c,d) étaient entiers ça ne marcherait pas, je préfère dire que toute valeur numérique dans une opération réelle doit être réelle.

[Emmanuel Delahaye]

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 [converti ]en float pour effectuer l'opération entre float et obtenir un résultat du type float.

Euh ça dépend. En C90, toutes les valeurs seront converties en double, le calcul effectué en double, puis le résultat converti en float (avec perte de précision)...

Je recommande que les données réelles servant à faire des calculs soient de type double.

[diogene]

souviron34 :
Mais ta réponse telle que rédigée :

MAIS tu divises par un nombre entier, tu auras donc une division entière.

est erronnée

ED :
Pourtant n1256 nous dit :

6.3.1.8 Usual arithmetic conversions

1...
Otherwise, if the corresponding real type of either operand is double, the other operand is converted, without change of type domain, to a type whose corresponding real type is double.
Otherwise, if the corresponding real type of either operand is float, the other operand is converted, without change of type domain, to a type whose corresponding real type is float.51)
...
2 The values of floating operands and of the results of floating expressions may be represented in greater precision and range than that required by the type; the types are not changed thereby.52)

[Emmanuel Delahaye]

ED :
Pourtant n1256 nous dit :

Il est possible que les choses aient changé en C99. (apparition des fonctions maths de type float).