Mittelwertbildung C++

[Andre P. (stanleyku)]

Hallo Zusammen,

kann mir jemand auf die Sprünge helfen was ich hier falsch mache:

unsigned int cell_voltage[2][26] = {{0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}, {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}};
unsigned long cell_current[2][6] = {{0, 0, 0, 0, 0, 0}, {0, 0, 0, 0, 0, 0}};



                    cell_current[j][0] = ADC_Read(j+2);
                    cell_current[j][1] = ADC_Read(j+2);
                    cell_current[j][2] = ADC_Read(j+2);
                    cell_current[j][3] = ADC_Read(j+2);
                    cell_current[j][4] = ADC_Read(j+2);

                    cell_current[j][5] = cell_current[j][0] + cell_current[j][1] + cell_current[j][2] + cell_current[j][3] + cell_current[j][4];
                    cell_current[j][5] = (cell_current[j][5] / 4) & 0xFFFF;

                    cell_voltage[j][24] = cell_current[j][5] & 0xFF;                  //Module current lowbyte
                    cell_voltage[j][25] = (cell_current[j][5] >> 8) & 0xFF;           //Module current highbyte

Eigentlich sollte ich in cell_voltage[j][24-25] einen digitalen Rohwert 
um die 2050 bekommen, es kommen aber ca. 2560 raus. So schwer kann das 
doch nicht sein :/ ?

VG und Danke!

[Stefan (Gast)]

Teil durch 5 wenn du schon 5 Werte summierst

[Cube_S (Gast)]

Nicht durch 4 sondern durch 5 teilen?

[Daniel A. (daniel-a)]

Falls cell_voltage und cell_current static storage duration haben sind 
die Initialiser dort überflüssig. andernfalls würde ich memset 
verwenden.

unsigned int cell_voltage[2][26];
memset( cell_voltage, 0, sizeof(cell_voltage) );

[Marcel P. (Gast)]

Noch als Tipp. ich habe so etwas ähnliches mal gemacht. hab das 
folgendermassen gelösst:

#define N_SAMPLES 5  // geht natürlich auch mit einer Variable...

unsigned int cell_voltage[2][26] = {{0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}, {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}};
unsigned long cell_current[2] = {0, 0};

        for (int i = 0 ; i < N_SAMPLES; i++ ) {
                    cell_current[j] += ADC_Read(j+2);
        }
        
        cell_current[j] = ( cell_current[j] / N_SAMPLES )  & 0xFFFF;
               
                cell_voltage[j][24] = cell_current[j] & 0xFF;                  //Module current lowbyte
                cell_voltage[j][25] = (cell_current[j] >> 8) & 0xFF;           //Module current highbyte

Dabei muss beachtet werden, das die Variable cell_current so gross ist, 
das sie die Summe natürlich aufnehmen kann (stichwort int16, int32, etc)