C: Fragen zu memset, memcpy, malloc

**Mxt** · 28.06.2016, 16:15

Zitat von HaWe

jap, das mit free ist klar

aber wenn man's nicht vergisst...?

Spätestens wenn mehrere an einem Programm arbeiten, wird es irgendwann wer vergessen.

Deshalb sieht es in modernem C++ jetzt so aus, statt

Code:

void f()
{
    int* pi = new int;

     // ...

    delete pi;
}

nur noch

Code:

void f()
{
     auto pi = std::make_unique<int>();

     // ...
}

gelöscht wird automatisch, wenn irgendwo (auch durch eine exception) die Klammern verlassen werden.

Die Zusatzfrage überlasse ich dann mal den sicher bald erscheinenden C-Experten.

HaWe · 28.06.2016, 16:21

C++ ist für mich definitiv keine Option, ich hasse Objekte wie der Teufel das Weihwasser...

a-Bär:

wenn eins hiervon richtig ist, wäre das schon mal gut zu wissen
(ich tippe aufs zweite...)

buf = (int16_t *) malloc(arrlen);
oder:
buf = (int16_t *) malloc(arrlen*sizeof(int16_t) );

**Mxt** · 28.06.2016, 16:25

Zitat von HaWe

(ich tippe aufs zweite...)

Würde ich auch vermuten, aber wie gesagt ich bin kein C-Experte.

**botty** · 28.06.2016, 16:51

Hallo HaWe,

wie Du getippt hast:

Code:

size_t len = 8;

int16_t *ptr = (int16_t*)malloc(arrlen * sizeof(int16_t));

/* Oder wenn's mit 0 initialisiert sein soll: *)
ptr = (int16_t*)calloc(arrlen, sizeof(int16_t));

ist die richtige Antwort.

Allerdings ist es dann üblich mittels Pointer Arithemetik und nicht mit dem Array-Operator auf das Feld zuzugreifen also drittes Element im Feld;

Code:

*(ptr + 2) = 32 * 27 + *(ptr + 1);

ist aber Geschmackssache.

HaWe · 28.06.2016, 16:58

ok, super, danke - hatte ich beide getestet, Fehler besteht fort, dann liegt es nicht an malloc.

calloc kannte ich noch gar nicht, und es beantwortet eigtl sogar schon die nächste Frage, trotzdem, der Vollständigkeit halber:

wenn ich mit
buf = (int16_t *) malloc(arrlen*sizeof(int16_t) );
den array angelegt habe, wie kann ich ihn manuell (nachträglich) mit Null initialisieren?

Jetzt geht es also um memset...
ist das hier korrekt:
memset(buf, 0, sizeof(buf) );
...

**botty** · 28.06.2016, 17:20

Nein, so geht's nicht, genau darin unterscheiden sich Arrays mit vorgegebener Größe gegenüber Speicherblöcken die mittels malloc & Co angelegt werden.
sizeof(buf) gibt die Bytes für die Größe des Zeigers buf zurück. Der weiß nix über das was malloc & free so treiben.

Code:

memset(buf, 0, arrlen * sizeof(int16_t));

wäre korrekt.

Du mußt hier zwischen der Feldlänge die Du vorgibst und der Anzahl der Bytes, die sich immer aus 'Feldlänge mal Datenentypgröße' ergibt, unterscheiden.

HaWe · 28.06.2016, 17:35

ahaaa... da könnte der seltsame Fehler begründet sein, das muss ich jetzt mal testen.

aber zu guter letzt, auch hier wieder die sizeof-Sache....:
Thema memcpy:

wenn ich einen array auf einen anderen kopieren will, habe ich bei statischen arrays verwendet
memcpy(array1, array2, sizeof(array2) );

nun ist array1 statisch und array2 ist dynamisch alloziert (mein buf).
geht das nun stattdessen so...
memcpy(array1, buf, arrlen * sizeof(int16_t) );
...