Additionner une array

Bonjour,

Je n'arrive pas a additionner une array lue a partir d'un fichier. Voici le programme:
Code:

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 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 # include <stdio.h> # include <stdlib.h> # include <math.h> # define SQUARE(x) (x)*(x) int main ( void ) { double filearray1 [ 14 ], filearray2 [ 14 ]; int r = 0; FILE * fp; char fname [ ] = "rent2.dat"; if ( ( fp = fopen ( fname, "r" ) ) == NULL ) { printf ( "Unable to open %s.\n", fname ); exit ( 1 ); } printf("\tRent\tIncome\n"); printf("\t X\t Y\t\t XX\t\t YY\t\t XY\n"); while ( fscanf ( fp, "%lf%lf\n", & filearray1 [ r ], & filearray2 [ r ] ) != EOF ) { printf ( "\t%.1f\t%.1f\t\t%.2f\t%.2f\t%.2f\n", filearray1 [ r ], filearray2 [ r ], SQUARE(filearray1 [r]), SQUARE(filearray2[r]), filearray1[r] * filearray2[r]); } printf("\t-----\t-------\t ---------\t-------------\t-----------\n"); printf ( "\t%d", SumArray ( filearray1, r ) ); return 0; } SumArray ( double a [ ], int n ) { int c; int sum = 0; for ( c = 0; c < n; c ++ ) sum += a [ c ]; return 0; }
Le resultat

Citation:

Rent Income
X Y XX YY XY
320.0 18000.0 102400.00 324000000.00 5760000.00
292.0 20000.0 85264.00 400000000.00 5840000.00
368.0 21600.0 135424.00 466560000.00 7948800.00
324.0 28400.0 104976.00 806560000.00 9201600.00
396.0 24800.0 156816.00 615040000.00 9820800.00
440.0 43600.0 193600.00 1900960000.00 19184000.00
568.0 48000.0 322624.00 2304000000.00 27264000.00
408.0 34800.0 166464.00 1211040000.00 14198400.00
500.0 34000.0 250000.00 1156000000.00 17000000.00
520.0 35600.0 270400.00 1267360000.00 18512000.00
360.0 30000.0 129600.00 900000000.00 10800000.00
572.0 56000.0 327184.00 3136000000.00 32032000.00
440.0 32000.0 193600.00 1024000000.00 14080000.00
512.0 53200.0 262144.00 2830240000.00 27238400.00
----- ------- --------- ------------- -----------
0>

Je comprend pas pourquoi ca affiche un zero a la place de 6020? Je pense bien que la fonction est correcte? est ce que c'est probleme de data type?

Merci de votre aide.

ta fonction sumArray est mal définie
là elle retourne 0 tout le temps
il faut que tu la déclare en int sumArray(...,...) et que tu mette return(sum):mrgreen:

Citation:

Envoyé par virtuadrack

Je n'arrive pas a additionner une array lue a partir d'un fichier. Voici le programme:

Il y a des bugs dans ce code. Ton compilateur devrait te le dire :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `main':
main.c:32: warning: implicit declaration of function `SumArray'
main.c: At top level:
main.c:38: warning: return type defaults to `int'
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 1 seconds)
0 errors, 2 warnings

Si il ne dit rien, il est probablement ma configuré :

http://emmanuel-delahaye.developpez....tm#cfg_compilo

Code:

#define SQUARE(x) (x)*(x)

Comportement indéfini. On ne sait pas comment la macro est 'appelée'. Il faut des parenthèses autour de l'expression pour garantir l'ordre d'évaluation :

Code:

#define SQUARE(x) ((x)*(x))

De plus, il subsiste le problème de l'évaluation multiple qui fait qu'un appel avec un ++

Code:

SQUARE (x++);

engendrerait aussi un CI. Mais qui ferait ça...

Je recommande l'usage de pow(), même si c'est un peu plus lent.

Code:

printf ("\t%.1f\t%.1f\t\t%.2f\t%.2f\t%.2f\n", <...>

On utilise jamais TAB ('\t') pour faire de la mise en page, mais les formateurs de printf() qui sont faits pour ça.

Le vrai problème est que r ne varie pas. Il faut mettre un r++ dans la boucle de lecture. Argh, grillaid...

Ton code corrigé :

Code:

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
 
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
 
#define SQUARE(x)  pow ((x), 2)
 
#define W "10"
#define S "%"W"s"
#define F1 "%"W".1f"
#define F2 "%"W".2f"
 
double SumArray (double a[], int n)
{
   int c;
   double sum = 0;
 
   for (c = 0; c < n; c++)
   {
      sum += a[c];
   }
   return sum;
}
 
int main (void)
{
   double filearray1[14], filearray2[14];
   size_t r = 0;
   FILE *fp;
   char fname[] = "rent2.dat";
 
   if ((fp = fopen (fname, "r")) == NULL)
   {
      printf ("Unable to open %s.\n", fname);
      exit (EXIT_FAILURE);
   }
 
   printf (S S "\n", "Rent","Income");
   printf (S S S S S "\n", "X","Y","XX","YY","XY");
 
   while (r < (sizeof filearray1/sizeof *filearray1 )
   &&   fscanf (fp, "%lf%lf\n", &filearray1[r], &filearray2[r]) != EOF)
   {
 
      printf (F1 F1 F2 F2 F2 "\n",
              filearray1[r], filearray2[r], SQUARE (filearray1[r]),
              SQUARE (filearray2[r]), filearray1[r] * filearray2[r]);
              r++;
 
   }
   printf (S S S S S"\n" ,"---------","---------","---------","---------","---------");
   printf (F1 F1"\n", SumArray (filearray1, r), SumArray (filearray2, r));
 
   return 0;
}

Avec :

Code:

Code:

1
2
3
4
5
6
7
8
9
10
11
 
      Rent    Income
         X         Y        XX        YY        XY
       1.0       2.0      1.00      4.00      2.00
       3.0       4.0      9.00     16.00     12.00
       5.0       6.0     25.00     36.00     30.00
       7.0       8.0     49.00     64.00     56.00
 --------- --------- --------- --------- ---------
      16.0      20.0
 
Press ENTER to continue.

Tu as également oublié d'incrémenter l'indice r dans la boucle de saisie. Tu ne remplis donc pas ton tableau...

Thierry

Bonsoir,

Je vous remercie pour l'aide et les explications approfondies, ca m'a beaucoup aide.
Code:

1 2 Press ENTER to continue.
Comment on fait une pause entre deux ecrans deja ?

Merci,

Citation:

Envoyé par virtuadrack

Comment on fait une pause entre deux ecrans deja ?

C'est quoi un 'écran' ?

Sinon, pour suspendre l'exécution de manière portable, il y a les fonctions de saisie (fgetc(), fgets() ...). Ne pas oublier de nettoyer le flux après usage.