Discussion :

[lcoulon]

Bonjour,

Je cherche à calculer la moyenne de 10 valeurs contenus dans un tableau de chiffre de 'unsigned short int'

Code :

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

1
2
3
4
 
unsigned short long average_ADC_int[10] = 0;
unsigned short long sum_ADC_int = 0;
unsigned long averaged_ADC_int = 0;

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
    sum_ADC_int = 0;
 
    for (int i=0; i<10; i++)                      
        {
        ADC_int = ADC_read();                 // Collecte la valeur ADC et stockage dans le tableau
        average_ADC_int[i] = ADC_int;
 
        sum_ADC_int += average_ADC_int[i];
        __delay_ms(20);                        // Prochaine mesure dans 20ms
        }
    averaged_ADC_int = (sum_ADC_int/10);

En procédant ainsi la somme des 10 valeurs est erronée, et de ce fait la moyenne aussi.
Pouvez vous me dire ce qui ne va pas svp ?

Merci à vous,

[Invité]

Bonsoir,

Code :

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

1
2
3
4
 
unsigned short long average_ADC_int[10] = 0;
unsigned short long sum_ADC_int = 0;
unsigned long averaged_ADC_int = 0;

Heuuu.... unsigned short long ..... ??? Tu es sur de toi là ?
Soit c'est l'un, soit c'est l'autre, mais ce n'est pas les deux ! Ce n'est même pas censé pouvoir compiler.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
    sum_ADC_int = 0;
 
    for (int i=0; i<10; i++)                      
        {
        ADC_int = ADC_read();                 // Collecte la valeur ADC et stockage dans le tableau
        average_ADC_int[i] = ADC_int;
 
        sum_ADC_int += average_ADC_int[i];
        __delay_ms(20);                        // Prochaine mesure dans 20ms
        }
    averaged_ADC_int = (sum_ADC_int/10);

Tu pourrais te passer de la variable ADC_int, et mettre directement ADC_read() dans ton average_ADC_int[i]. (dans le code présenté, en tout cas)

(Je pars du principe que tu as consciemment mis le résultat de ta moyenne dans un long. Si ce n'est pas le cas, attention à la perte d'information sur la division !)
Comme ça, cela me semble correct. As-tu déjà fait un coup de debog ? A coup de printf(), voir de débogueur intégré (ou non) à un ide, pour voir l'état du tableau à chaque tour de boucle ?

[Bktero]

Heuuu.... unsigned short long ..... ??? Tu es sur de toi là ?
Soit c'est l'un, soit c'est l'autre, mais ce n'est pas les deux ! Ce n'est même pas censé pouvoir compiler.

Et pourtant ça existe Ce n'est pas standard, certes, mais certains compilateurs le proposent. Par exemple, C18 de Microchip propose short long et long short. Les deux types sont équivalents et sont sur 24 bits.

@lcoulon : pourquoi stocker la somme dans un short long et la moyenne dans un long ?? Tu devrais au moins faire l'inverse, tu as moins de chances d'avoir un overflow lors de la somme.

[lcoulon]

En effet, j'utilise XC8 de Microchip, l'evolution de C18.

Mon tableau "average_ADC_int" se rempli bien a tour de boucle avec des valeurs:

1701098
1701099
1701102
1701098
1701130
1701116
1701140
1701099
1701103
1701110

Cependant, le resultat "sum_ADC_int" est : 11907783
alors qu'il etre : 17011095

le result de "averaged_ADC_int" n'est également pas celui attendu : 23388

Je ne sais pas pourquoi les résultats sont faux.

[lcoulon]

En déclarant les variables ainsi, le résultat s'approche de la réalité à +- 4 près

Code :

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

1
2
3
unsigned short long average_ADC_int[10] = 0;
unsigned long sum_ADC_int = 0;
unsigned short long averaged_ADC_int = 0;

Le fait de déclarer le résultat en short long plutôt qu'en float évite un résultat à virgule, mais cela peut-il générer un mauvais résultat ou bien le résultat est automatiquement arrondi puisque stocké dans un short long ?

[johnlobs]

Bonjour,pourrais ton connaitre le type retourné par la fonction ADC_read() ainsi que celui de ADC_int (où tu stock la valeur retournée par la fonction)? Tu utilises des variables sans les déclarer?
Tu peux commencer par te débarrasser de la variable et vérifier le type retourné,ce serait un bon début!
Ensuite passer aux types de tes variables sum et avg.
Bon courage!

[lcoulon]

Code :

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

unsigned short long ADC_read(void);

Code :

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

unsigned short long ADC_int;                  // ADC 24 bits value from ADC converter

En fait si je ne passe pas par une variable intermédiaire, et procède ainsi, le compileur refuse la ligne :

Code :

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

 average_ADC_int[i] = ADC_read();

error: (712) can't generate code for this expression