operator= is implicitly deleted, pętla for z iteratorami std::vector

Question

   for(auto it = m_islands.begin();
       it != m_islands.end();)
   {
      it->update();

      if(it->getPosition().x < 0)
         m_islands.erase(it);
      else it++;
   }

m_islands to vector obiektów klasy Island.

Klasa Island ma zdefiniowany konstruktor kopiujący.

Przy kompilacji wyrzuca error:
 

error: use of deleted function 'Island& Island::operator=(const Island&)'|
note: 'Island& Island::operator=(const Island&)' is implicitly deleted because the default definition would be ill-formed:|
error: non-static const member 'const size_t Island::m_squareNumber', can't use default assignment operator|

 

 Kompletnie nie ogarniam. Gdzie ja tu używam operatora =? I właściwie dlaczego miałby być usunięty?

Answer 1

Możesz pokazać 1. dokładnie gdzie wyskakuje błąd? (pewnie linia z .erase?) 2. Definicję klasy Island? 3. Jeśli klasa Island po czymś dziedziczy, pokaż też jej klasy bazowe.

Comments

Trudno mi powiedzieć, nic oczywistego nie widzę. Powinieneś zastąpić memcpy przez normalne przypisanie obiektów (ogólnie powinieneś unikać memcpy na C++owych obiektach), ale w tym przypadku nie wydaje mi się żeby to naprawiło program.

---

Masz racje, nie naprawiło. Najdziwniejsze, że kilka godzin temu ten sam kod działał. Wcześniej co prawda dziwne crashe były przy usuwaniu z vectora, ale tylko jeśli zawiera więcej niż 1 element. Ale teraz są w innym momencie, mimo że kod się nie zmienił. Może warto przeinstalować całe C::B razem z kompilatorem...

---

Wróciłem do punktu w którym założyłem temat i:

Ostatecznie użyłem shared_ptr i wszystko działa. Teoretycznie bardziej odpowiedni wydawałoby się unique_ptr, ale wydaje mi się, że w moim przypadku nie ma to znaczenia w kwestii zwalniania pamięci. Ale nie byłem pewien czy unique_ptr nie przeszkadzałby w kopiowaniu... W każdym razie jeśli użyje unique_ptr zamiast shared_ptr, kompilator się burzy znowu o operator=() potrzebny dla .erase(). Twierdzi, że nie jest w stanie go utworzyć - identyczny błąd jak przy const. Nie mam pojęcia dlaczego.

Także... jeśli wszystko jest jak należy, pamięć jest zwalniana poprawnie.

Dwa ciekawe linki nt. używania shared_ptr i tablic:
http://stackoverflow.com/questions/13061979/shared-ptr-to-an-array-should-it-be-used

http://stackoverflow.com/questions/30780262/accessing-array-of-shared-ptr

---

unique_ptr zamiast shared_ptr, kompilator się burzy znowu o operator=() potrzebny dla .erase(). Twierdzi, że nie jest w stanie go utworzyć - identyczny błąd jak przy const. Nie mam pojęcia dlaczego.

Ponieważ unique_ptr - jak sama nazwa sugeruje - cała jego idea i cel istnienia polega na tym, że nie można go kopiować.

---

Aha, czyli dobrze podejrzewałem. Dzięki za upewnienie.