Additionner une array

**virtuadrack** · 10/09/2007, 03h37

Bonjour,

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

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

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.

**Laughing Man** · 10/09/2007, 08h19

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)

**Emmanuel Delahaye** · 10/09/2007, 09h13

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 :

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

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 :

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

#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 :

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

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

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

Code :

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

   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 :

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

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 :

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

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

Code :

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

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.

**Thierry Chappuis** · 10/09/2007, 09h48

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

Thierry

**virtuadrack** · 10/09/2007, 23h54

Bonsoir,

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

Code :

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

1
2
 
Press ENTER to continue.

Comment on fait une pause entre deux ecrans deja ?

Merci,

**Emmanuel Delahaye** · 11/09/2007, 08h33

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.

Additionner une array

C

Vue hybride

Discussions similaires

Partager

Partager