Utiliser memset sur les structures en C++

Question

Hé les gars. Je travaille sur la réparation du code plus ancien pour mon travail. Il est actuellement écrit en C++. Ils ont converti l'allocation statique en dynamique mais n'ont pas édité les memsets/memcmp/memcpy. C'est mon premier stage de programmation si vide avec ma question de newbe.

Le code suivant est en C, mais je veux avoir en C++ (j'ai lu que malloc est pas une bonne pratique en C++). J'ai deux scénarios: Premièrement, nous avons créé f. Ensuite, vous utilisez & f afin de remplir avec zéro. La seconde est un pointeur * pf. Je ne suis pas sûr de savoir comment définir pf à tous les 0 comme l'exemple précédent en C++.

Pourriez-vous simplement faire pf = new foo au lieu de malloc et ensuite appeler memset(pf, 0, sizeof(foo))?

struct foo { ... } f; 
memset(&f, 0, sizeof(f)); 

//or 

struct foo { ... } *pf; 
pf = (struct foo*) malloc(sizeof(*pf)); 
memset(pf, 0, sizeof(*pf)); 

Answer 1

Oui, mais seulement si foo est un POD. S'il a des fonctions virtuelles ou n'importe quoi d'autre à distance C++ ish, n'utilisez pas memset car il écrasera tous les internes de la structure/classe.

Ce que vous voulez probablement faire au lieu de memset est de donner foo un constructeur pour initialiser explicitement ses membres.

Si vous souhaitez utiliser nouveau, n'oubliez pas la suppression correspondante. Mieux encore serait d'utiliser shared_ptr :)

Answer 2

Oui, cela fonctionnerait. Cependant, je ne pense pas que malloc soit nécessairement une mauvaise pratique, et je ne le changerais pas juste pour le changer. Bien sûr, vous devriez vous assurer de toujours faire correspondre les mécanismes d'allocation correctement (nouveau-> supprimer, malloc-> gratuit, etc.).

Vous pouvez également ajouter un constructeur à l'struct et l'utiliser pour initialiser les champs.

Answer 3

Pouvez-vous? Oui probablement. Devrais-tu? Non

Bien qu'il ne fonctionnera probablement, que vous perdez l'état que le constructeur a construit pour vous. Ajoutant à cela, que se passe-t-il lorsque vous décidez d'implémenter une sous-classe de cette structure? Ensuite, vous perdez l'avantage du code réutilisable que C++ OOP offre.

Qu'est-ce que vous devez faire est de créer à la place un constructeur qui initialise les membres pour vous. De cette façon, lorsque vous sublimez cette structure plus tard sur la ligne, vous utilisez simplement ce constructeur pour vous aider à construire les sous-classes. C'est gratuit, code sécurisé! utilise le!

Edit: La mise en garde à ce que si vous avez une énorme base de code déjà, ne changent pas jusqu'à ce que vous commencez à Dérivation de l'struct. Cela fonctionne comme maintenant.

Answer 4

Vous pourriez nouvelle foo (comme la manière standard en C++) et mettre en œuvre un constructeur qui initialise foo plutôt que d'utiliser memset.

E.g.

struct Something 
{ 
    Something() 
     : m_nInt(5) 
    { 

    } 

    int m_nInt; 
}; 

Aussi ne pas oublier si vous utilisez nouveau pour appeler supprimer lorsque vous avez terminé avec l'objet, sinon vous finirez avec des fuites de mémoire.