travaillant avec memcpy pour les tampons

Question

Je suis en train de créer un tampon de caractères comme indiqué:

#include<stdio.h> 
#include<stdlib.h> 
#include<string.h> 
struct gvk{ 
    char a ; 
    char b ; 
    char c ; 
}; 

int main(){ 
    char *buf; 
    struct gvk k = {'A','B','C'}; 

    memcpy(buf, &k , 3*sizeof(char)); 
    printf("values in buf is %s\n",buf); 
} 

Mais mon programme est sortie en disant « avec le statut non sorti zéro » Où ai-je fait de mal?

Answer 1

Collègue supérieur vous dit d'allouer de la mémoire dynamique, mais depuis sa plus sujette aux erreurs, plus difficile de travailler avec (besoin de libre mémoire) puis avec tableaux automatiques + vous connaissez la taille dans la compilation, il n'y a aucune raison d'utiliser l'allocation dynamique. Je vous recommande d'utiliser un tableau automatique à la place. N'oubliez pas non plus un octet pour le caractère de terminaison nulle.

char buf[4]; 

Nous utilisons l'allocation dynamique juste s'il n'y a pas d'autre moyen. Même si vous ne connaissez pas la taille du tableau au moment de la compilation, puisque c99 vous pouvez utiliser VLA.

---

compilateur également peut ajouter un rembourrage entre/derrière des éléments de struct. Rembourrage aligns structure des membres aux frontières d'adresses "naturelles".

Dans votre cas, il probablement ressembler à ce

struct gvk{ 
    char a ; // 1B 
    char b ; // 1B 
    char c ; // 1B 
      // 1B padding 
}; 

Ce qui signifie que dans ce cas, il devrait bien fonctionner, de toute façon, vous devriez éviter d'utiliser memcpy(buf, &k , 3*sizeof(char)); sur des structures comme celle-ci, parce que vous pouvez copier le remplissage au lieu de véritable élément de structure.

---

De plus il n'y a pas besoin de taper explicitement sizeof(char) qui, par la norme, a une taille 1 (B).

Answer 2

Vous devez allouer de la mémoire avant d'utiliser buf sinon il s'agit d'un comportement non défini.

Vous devez le déclarer comme celui-ci (Dynamiquement):

char *buf = malloc(sizeof(char)*4);