Usuwanie dynamicznych obiektów z wektora

Question

Zastanawiam się jak odbywa się usuwanie elementów wektora. U mnie to wygląda tak:

vector<Obiekt> v;
Obiekt *w;
// tutaj dodanie czegos do wektora
w = new Obiekt();
v.push_back(*w);
//tutaj wyrzucenie z wektora
w= &v[choice];
delete w;
v.erase(v.begin()+choice);

w - wskaźnik

v- wektor

Dla próby zrobiłem w klasie zmienną statyczną ilosc, konstruktor ilosc++, destruktor ilsoc--

Przy usuwaniu zmienna ilosc zaczyna wariować, nawet nie że zmniejsza się np o 2 za każdym razem, po prostu odejmowane są randomowe liczby. W samym działaniu programu to niby nie przeszkadza, ale zależy mi zeby pamięć była czysta i faktycznie usuwany obiekt nie tylko wyleciał z wektora, ale też zniknął z pamięci.

Answer 1

A nie myślałeś może o stworzeniu vectora wskaźników ? :P vector<Obiekt*> wtedy wrzucasz v.push_back(new Obiekt), a usuwasz np. delete *it; gdzie it to jest iterator do tego vectora tylko musisz pamietać że trzeba każdy element wektora ręcznie usuwać zanim użyje się pop_back albo clear albo erase :D

Comments

delete v[cyferka] by nie działało? Z tym *it nei bardzo rozumiem. Co do samego pomysłu jest na tyle podobny w działaniu co teraz mam, że chyba nie ma sensu, przynajmniej tak mi się wydaje. W każdym razie chce się dowiedzieć jak przebiega erase(), że tak mi ta ilosc wariuje.

---

poczytaj o iteratorach bo w vectorze możesz użyć [] ale np. w takiej liście albo mapie już nie da rady :) jeżeli chcesz tworzyć obiekty dynamicznie to dlaczego nie chcesz przechowywać wskaźników na nie ? tylko chcesz przechowywać statyczne obiekty ? sprawdź metode emplace która potrafi sama stworzyć obiekt i go dodać do vectora :P

---

Okej, zacząłem ogarniać. Dla próby zeby zobaczyć jak to działa napisałem coś takiego:

class Obiekt{
      public:
      string name;
      void show()
      {
           cout<<this->name<<endl;
           }
      Obiekt(){ name="nazwa"; }
      };
      

int main()
{
vector<Obiekt*> v;
v.push_back(new Obiekt());

while(true){
for( vector<Obiekt*>::iterator it = v.begin(); it != v.end(); it++ )
{
    (*it)->show();
}
char choice=getch();
if(choice=='q') v.push_back(new Obiekt());
}

Problem w tym, że po każdym kliknięciu 'q', wyświetlają się kolene 2 "nazwy". Co zrobiłem źle?

//EDIT: nieważne, zapomniałem czyścić ekran : D

Answer 2

class Obiekt{
      public:
      string name;
      void show()
      {
           cout<<this->name<<endl;
           }
      Obiekt(){ name="nazwa"; }
      };
      

int main()
{
vector<Obiekt*> v;
v.push_back(new Obiekt());
int poz=0;

while(true){
for( vector<Obiekt*>::iterator it = v.begin(); it != v.end(); it++ )
{
    cout<<(*it)->name;
//tutaj:
    if(it=v.begin()+poz) cout<<"<-"<<endl;
    else cout<<endl;
}
char choice=getch();
if(choice=='q') v.push_back(new Obiekt());
system("cls");
}
}

Chce zeby pojawiała mi sie taka strzałka w zalezności od zmiennej poz. Chciałem to zrobić w taki sposób jak widać. Wyrzuca mi taki error: 

could not convert `((*&it) <unknown operator> ((const __gnu_cxx::__normal_iterator<Obiekt**, std::vector<Obiekt*, std::allocator<Obiekt*> > >&)((const __gnu_cxx::__normal_iterator<Obiekt**, std::vector<Obiekt*, std::allocator<Obiekt*> > >*)(&(&(&v)->std::vector<_Tp, _Alloc>::begin [with _Tp = Obiekt*, _Alloc = std::allocator<Obiekt*>]())->__gnu_cxx::__normal_iterator<_Iterator, _Container>::operator+ [with _Iterator = Obiekt**, _Container = std::vector<Obiekt*, std::allocator<Obiekt*> >](((const ptrdiff_t&)((const ptrdiff_t*)(&poz))))))))' to `bool' 

Dosc ładny, nie powiem :| Domyslam sie dlaczego tak nie moge zrobic, ale w takim razie jak inaczej?