c++ vector neuen löschen

Wie kann ich freien Speicher im nächsten Beispiel.
vt.pop_back() löscht element in der vt, aber es ist nicht der freie Speicher.
löschen vt[i] nicht funktioniert, und es mir "segmentation fault".

#include <vector>
#include <unistd.h>

using namespace std;

const int SIZE = 1000000;

class Test
{
public:
    Test();
    ~Test();    
private:
    int *arr;    
};


int main()
{
    vector<Test *> vt;
    for(int i = 0; i < 100; i++)
        vt.push_back(new Test());
    sleep(10);
    return 0;    
}


Test::Test()
{
    arr = new int[SIZE];
}


Test::~Test()
{
    delete[] arr;
}
  • Warum auch du mit neues?
  • Warum delete v[i] funktioniert nicht? Es gibt keine delete v[i] in diesem code.
  • da der Konstruktor kann Argumente bestehen
  • Sie können vector für arr zu.
  • Ich würde nicht empfehlen, dies zu tun. Wenn man einen Vektor von Zeigern, versucht zu kopieren, der Vektor wird nicht tun, was Sie erwarten, pops wird gelöscht, und es wird in der Regel führt zu Schmerzen und leiden, wie Sie wickeln jedes vector-Betrieb machen ihn tun die Rechte Sache mit den Pointern und Speicher. Erhöhung der ptr_vector wäre ein Beispiel für eine viel bessere Möglichkeit, dies zu tun.
  • mögliche Duplikate von Tut vector::erase() zerstören das entfernte Objekt?

InformationsquelleAutor k_zaur_k | 2012-04-09
  1. 6

    Sind Sie nicht die Speicherung von "Test" Objekte, die Sie speichern Zeiger auf diese Objekte. So löschen bringt nichts, aber löschen der Zeiger.

    Wenn Sie möchten, dass Ihre Objekte werden gespeichert in dem Vektor sollte die Art 

    vector<Test>

    Diese Weise die delete-Aufrufe ausführen, tatsächlich das Destruktor der Objekte.

    Ist der resultierende code wird:

    int main() 
{
    vector<Test> vt; 
    for(int i = 0; i < 100; i++) 
        vt.push_back(Test());
    sleep(10);
    return 0;
}

    Als kommentierte dies erfordert auch ein neues copy-Konstruktor:

    Test::Test(const Test& t)
{
    arr = new int[SIZE];
    for(int i = 0; i != SIZE; i++)
        arr[i] = t.arr[i];
}
    • Keine Sorge, ich bin nicht etwas zu stehlen 😉 Passiert die ganze Zeit: ich habe zu lange post eine Antwort und am Ende löschen, da es eine ähnliche Frage schon 🙂 Hast ein +1.
    • Ich weiß, vor allem auf Fragen wie diese. viele Leute Antworten in Minuten -- und die meisten von Ihnen haben im Grunde die gleiche Antwort 🙂
    • aber vector<Test> vt; for(int i = 0; i < 100; i++) vt.push_back(Test()); funktioniert nicht
    • Ihre Antwort war mehr abgeschlossen, und ich wollte upvote, weil der, dass. Und dann ist es einfach verschwunden...
    • wenn ich Schreibe Vektor<Test> vt; for(int i = 0; i < 100; i++) vt.push_back(Test()); g++ main.cpp -o main && ./main gibt mir "segmentation fault";
    • Das ist, weil Ihr Test Klasse bricht in der in der Regel aus Drei und kann nicht sicher kopiert. push_back Kopien ein temporäres Objekt in dem Vektor, und zerstört das original, das löschen des array zugewiesen.
    • du hast Recht. Ich habe ein Kopie-Konstruktor handle, das problem

    InformationsquelleAutor Martin Kristiansen
  2. 4

    Können Sie delete vt[i] wie dieser, ist es den Speicher frei zugeordnet Test Objekte:

    for(size_t i = 0; i < vt.size(); ++i)
{
   delete vt[i];
}
vt.clear();
    InformationsquelleAutor Naveen
  3. 1

    Den Sie speichern Zeiger und nicht die Objekte, dafür ist der "pointer-Objekt" wird zerstört, wenn Sie zerstören Ihre Vektor, aber nicht das Objekt der Zeiger zeigt

    InformationsquelleAutor Sebastian Hoffmann
  4. 0

    Wenn Sie wollen, um das löschen von bestimmten Elementen, Sie könnten versuchen, mit der Funktion erase von std::vector.

    InformationsquelleAutor eSedano
Schreibe einen Kommentar