Verwenden von MEMSET auf Strukturen in C++

Question

Hey Leute. Ich arbeite daran, älteren Code für meinen Job zu reparieren. Es ist derzeit in C++ geschrieben. Sie haben die statische Zuweisung in dynamic konvertiert, aber die memsets/memcmp/memcpy nicht bearbeitet. Dies ist mein erstes Programmierpraktikum, so leer mit meiner Newbe-ähnlichen Frage.

Der folgende Code ist in C, aber ich möchte es in C++ haben (Ich habe gelesen, dass malloc ist keine gute Praxis in C++). Ich habe zwei Szenarien: Zuerst haben wir erstellt. Dann verwenden Sie & f, um mit Null zu füllen. Der zweite ist ein Zeiger * pf. Ich bin nicht sicher, wie pf auf alle 0 wie das vorherige Beispiel in C++ festgelegt werden.

Konnten Sie einfach pf = new foo anstelle von malloc tun und dann memset(pf, 0, sizeof(foo)) anrufen?

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

Ja, aber nur wenn foo ein POD ist. Wenn es virtuelle Funktionen oder irgendetwas anderes C++ ish hat, benutze kein memset, da es über die Interna der struct/class stampfen wird.

Was Sie wahrscheinlich anstelle von memset tun möchten, ist foo ein Konstruktor, um seine Mitglieder explizit zu initialisieren.

Wenn Sie neu verwenden möchten, vergessen Sie nicht das entsprechende Löschen. Noch besser wäre es Shared_ptr zu verwenden :)

Answer 2

Ja, das würde funktionieren. Ich denke jedoch nicht, dass Malloc notwendigerweise eine schlechte Übung ist, und ich würde es nicht ändern, nur um es zu ändern. Natürlich sollten Sie sicherstellen, dass Sie die Zuordnungsmechanismen immer richtig zuordnen (new-> delete, malloc-> free usw.).

Sie könnten der Struktur auch einen Konstruktor hinzufügen und diesen zum Initialisieren der Felder verwenden.

Answer 3

Können Sie? Ja möglicherweise. Sollten Sie?

Während es wahrscheinlich funktioniert, verlieren Sie den Zustand, den der Konstruktor für Sie erstellt hat. Was passiert, wenn Sie sich entscheiden, eine Unterklasse dieser Struktur zu implementieren? Dann verlieren Sie den Vorteil von wiederverwendbarem Code, den C++ OOP bietet.

Sie sollten stattdessen einen Konstruktor erstellen, der die Mitglieder für Sie initialisiert. Auf diese Weise verwenden Sie diesen Konstruktor, wenn Sie diese Struktur später in der Zeile untergliedern, einfach, um Ihnen beim Erstellen der Unterklassen zu helfen. Dies ist ein kostenloser, sicherer Code! benutze es!

Edit: Der Vorbehalt zu diesem ist, dass, wenn Sie bereits eine riesige Code-Basis haben, ändern Sie es nicht, bis Sie beginnen, die Strukturen subclassing. Es funktioniert so wie es jetzt ist.

Answer 4

Sie könnten neue foo (wie der normale Weg in C++) und einen Konstruktor implementieren, die foo initialisiert anstatt Memset.

z.

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

    } 

    int m_nInt; 
}; 

Vergessen Sie auch nicht, wenn Sie neue verwenden löschen rufen, wenn Sie mit dem Objekt fertig sind sonst Sie mit Speicherlecks enden.