Pointer als parameter von Funktion zurück

[Johannes (Gast)]

Hallo,
ich möchte gerne in einer Funktion ein Array erzeugen und diesen als 
Parameter wieder zurückgeben.

static void work(uint8_t* a, uint8_t numOfA, uint8_t* resArray, uint8_t* doubled);

int main(void) {

  static uint8_t a[4] = {0};
  static uint8_t* c = NULL;
  static uint8_t d;
  static uint8_t numOfDoubled = 0;

  for(uint8_t x=0; x<4; x++)
  {
    a[x] = 0x10;
  }

  work(&a[0], sizeof(a), c, &numOfDoubled);

  for(uint8_t x=0; x<numOfDoubled; x++)
  {
    printf("%02X ", c[x]);
  }

  return EXIT_SUCCESS;
}

static void work(uint8_t* a, uint8_t numOfA, uint8_t* resArray, uint8_t* doubled)
{
  uint8_t* res;
  uint8_t counter = 0;

  res = malloc(numOfA * sizeof(*res));

  for(uint8_t x=x; x<numOfA; x++)
  {
    if(a[x] <= 100)
    {
      res[x] = a[x] * 2;
      counter++;
    }
    else
    {
      res[x] = a[x];
    }
  }

  *doubled= counter;
  resArray = res;
}

Allerdings ist c nach aufruf der Funktion immer noch NULL.

Wenn ich die Funktion allerdings wie folgt definiere

static uint8_t* work(uint8_t* a, uint8_t numOfA, uint8_t* doubled)

und wie folgt aufrufe

c = addition(&a[0], sizeof(a), &numOfDoubled);

ist c danach als array vorhanden.
Gerne würde ich aber später als returnwerte parameter übergeben, wo man 
ermitteln kann, ob die Funktion richtig durchgelaufen ist.
Die funktion work wird dann noch erweitert. Aber erst wenn ich dieses 
Problem gelöst habe.

[Oliver S. (oliverso)]

Parameter werden in C grundsätzlich und ohne Ausnahme als Kopie 
übergeben.

Willst du einen übergebene Variable in einer Funktion ändern, musst du 
also einen Pointer drauf übergeben, und über den auf die Variable 
außerhalb der Funktion zugreifen. Dabei ist es egal, was für ein Typ das 
ist.

Oliver

[Teo D. (teoderix)]

Johannes schrieb:
> work(&a[0], sizeof(a), c, &numOfDoubled);

&a[0] ist quatsch, da a[0] bereits die Adresse ist.

[A. S. (Gast)]

Teo D. schrieb:
> &a[0] ist quatsch, da a[0] bereits die Adresse ist.

&a[0] ist quatsch, da a (ohne & und [0]) bereits die Adresse ist.

[Oliver S. (oliverso)]

A. S. schrieb:
> &a[0]

ist unnötig, da a (ohne & und [0]) bereits die Adresse ist, aber kein 
Quatsch.

Oliver

[🐧 DPA 🐧 (Gast)]

https://en.wikipedia.org/wiki/Fundamental_theorem_of_software_engineering

[xy (Gast)]

Das

 for(uint8_t x=x; x<numOfA; x++)

müsste

 for(uint8_t x=0; x<numOfA; x++)

sein.

[Johannes (Gast)]

Habe es jetzt doch lösen können

static void work(uint8_t* a, uint8_t numOfA, uint8_t** resArray, uint8_t* doubled);

int main(void) {

  static uint8_t a[4] = {0};
  static uint8_t* c = NULL;
//  static uint8_t d;
  static uint8_t numOfDoubled = 0;

  for(uint8_t x=0; x<4; x++)
  {
    a[x] = 0x10;
  }

  work(&a[0], sizeof(a), &c, &numOfDoubled);

  for(uint8_t x=0; x<numOfDoubled; x++)
  {
    printf("%02X ", c[x]);
  }

  return EXIT_SUCCESS;
}

static void work(uint8_t* a, uint8_t numOfA, uint8_t** resArray, uint8_t* numOfElRes)
{
  uint8_t counter = 0;

  *resArray = malloc(numOfA * sizeof(resArray));


  for(uint8_t x=0; x<numOfA; x++)
  {
    if(a[x] <= 100)
    {
      (*resArray)[x] = a[x] * 2;
      counter++;
    }
    else
    {
      (*resArray)[x] = a[x];
    }
  }

  *numOfElRes = counter;
}

wichtig ist, dass

(*resArray)[x] = a[x] * 2;
..
(*resArray)[x] = a[x];

eine Klammer um *resArray ist.