Ringpuffer so richtig?

[Gerd (Gast)]

Hallo,
ich möchte gern einen Ringpuffer benutzen um einen gleitenden Mittelwert 
zu bilden. Mach ich das so richtig?

Prototyp:

unsigned int calc_floating_average (unsigned int new_value);

Puffer:

volatile unsigned int floating_average[16];

Auswertung Puffer:

unsigned int calc_floating_average (unsigned int new_value)
{
  unsigned long average_sum=0;
  /* rotate value storage */
  for(unsigned char i=15; i>1;i--)
  {
    floating_average[i]=floating_average[i-1];
  }
  
  /* apply new value into buffer */
  floating_average[0] = new_value;
  
  /* sum all values */
  for (unsigned char j=0;j<15;j++)
  {
    average_sum+= floating_average[j]; 
  }
  
  /* calc middle */
  average_sum/=16;
  return (unsigned int)average_sum;
}

Besten Dank,
Gerd

[Helmut L. (helmi1)]

Gerd schrieb:
> ich möchte gern einen Ringpuffer benutzen um einen gleitenden Mittelwert
> zu bilden. Mach ich das so richtig?

Etwas umstaendlich.

Das schieben hier kannst du dir sparen:

>  for(unsigned char i=15; i>1;i--)
>  {
>    floating_average[i]=floating_average[i-1];
>  }
>
>  /* apply new value into buffer */
>  floating_average[0] = new_value;

Was du brauchst ist ein Pointer der immer auf das aelteste Element 
zeigt.

Also so in etwa:

floating_average[n++] = new_value;
if (n >= MAXBUFFER) n = 0;

MAXBUFFER ist die Groesse deines Ringbuffers.

[Gerd (Gast)]

Helmut Lenzen schrieb:
> Was du brauchst ist ein Pointer der immer auf das aelteste Element
> zeigt.

Stimmt, hab ich so noch garnicht gesehen! Geht so natürlich deutlcih 
schneller! Ein FIFO ist hier auch garnicht nötig, stimmt.


Grüße,
Gerd