array pointer an funktion problem

[Jan (Gast)]

Ich habe ein array, welches an einer Funktion übergeben wird.
Dort sollen die Werte bearbeitet werden und nach dem Funktionsaufruf die 
neuen werte zur Verfügung sein.
Dies funktioniert bei mir aber irgendwie nicht

void funktion1(uint16_t* arr1, uint16_t* arr2, uint8_t length);
{
  for(uint8_t x=0; x<length; x++)
  {
    arr2[x] = arr1[x] / 2;
    printf("arr1[%d]: %d     arr2[%d]: %d\r\n", x, arr1[x], x, arr2[x]);
  }
}

Die Ausgabe ist dann

arr1[0]: 100    arr2[0]: 50
arr1[1]: 100    arr2[1]: 50
arr1[2]: 100    arr2[2]: 50
... ... ...

Die Funktion rufe ich wie folgt auf

void funktion(void)
{
  uint16_t arr1[10] = {0};
  uint16_t arr2[10] = {0};
  for(uint8_t x=0; x<10; x++)
  {
    arr1[x] = 100;
  }
  funktion1(&arr1[0], &arr2[0], 10);

  for(uint8_t x=0; x<10; x++)
  {
    printf("arr1[%d]: %d     arr2[%d]: %d\r\n", x, arr1[x], x, arr2[x]);
  }
}

Die Ausgabe ist dann

arr1[0]: 100    arr2[0]: 0
arr1[1]: 100    arr2[1]: 0
arr1[2]: 100    arr2[2]: 0
... ... ...

warum ist arr2 immernoch 0? es wird doch eigentlich in funktion1 
geändert?

[Klaus W. (mfgkw)]

Wie groß ist eine int bei dir?

%d ist die Formatierung, um eine int dezimal auszugeben.
Du versuchst es aber mit uint8_t und uint16_t, dann findet printf() 
nicht die Werte auf dem Stack.

Die richtige Formatierung wäre:

#include <inttypes.h>
#include <stdint.h>
...
  printf( "arr1[%"PRIu8"]: %"PRIu16"     arr2[%"PRIu8"]: %"PRIu16"\r\n", x, arr1[x], x, arr2[x] );

[gamp (Gast)]

Hallo,

ist das C? Ich wusste garnicht, dass man die Definition des Iterators so 
in die for-Schleife einbringen kann -- Naja.
Ich habe leider keinen Fehler gefunden, nur eine Anmerkungen

Jan schrieb:

>   uint16_t arr1[10] = {0};
>   uint16_t arr2[10] = {0};

Du initialisierst nur das erste Feld des Arrays? Wozu ist ist das gut?

[Klaus W. (mfgkw)]

gamp schrieb:
> Du initialisierst nur das erste Feld des Arrays? Wozu ist ist das gut?

Der Rest wird automatisch mit 0 aufgefüllt.

gamp schrieb:
> ist das C? Ich wusste garnicht, dass man die Definition des Iterators so
> in die for-Schleife einbringen kann

Das geht schon seit dem letzten Jahrtausend so :-)

[Dirk B. (dirkb2)]

Klaus W. schrieb:
> Wie groß ist eine int bei dir?

Mindestens 16 Bit.

Bei Funktionen mit variabler Parameterliste (so wie printf)
werden Ganzahlen auf mindestens int aufgewertet.
Fließkommazahlen auf double.
Darum gibt es eigentlich auch kein %lf, da %f schon alles macht.

[Mikro 7. (mikro77)]

Funktioniert bei mir.

Zeit doch mal das Programm was bei dir tatsächlich compiliert und 
ausgeführt wird.

> gcc -o test test.c
> ./test
arr1[0]: 100     arr2[0]: 50
arr1[1]: 100     arr2[1]: 50
arr1[2]: 100     arr2[2]: 50
arr1[3]: 100     arr2[3]: 50
arr1[4]: 100     arr2[4]: 50
arr1[5]: 100     arr2[5]: 50
arr1[6]: 100     arr2[6]: 50
arr1[7]: 100     arr2[7]: 50
arr1[8]: 100     arr2[8]: 50
arr1[9]: 100     arr2[9]: 50
arr1[0]: 100     arr2[0]: 50
arr1[1]: 100     arr2[1]: 50
arr1[2]: 100     arr2[2]: 50
arr1[3]: 100     arr2[3]: 50
arr1[4]: 100     arr2[4]: 50
arr1[5]: 100     arr2[5]: 50
arr1[6]: 100     arr2[6]: 50
arr1[7]: 100     arr2[7]: 50
arr1[8]: 100     arr2[8]: 50
arr1[9]: 100     arr2[9]: 50

[XYZ (Gast)]

Hi,
Dein Funktionsheader ist fehlerhaft:

void funktion1(uint16_t* arr1, uint16_t* arr2, uint8_t length);

Das Semikolon am Ende muss weg, dann sollte es gehen.

VG
XYZ