Sinn des Ganzen:

ich habe ein Objekt (TElement), dieses besitzt eine Liste (vector) von anderen Objekten (TConnection).
beim Erstellen von TElement werden diverse TConnection erzeugt und dabei der this-pointer des TElement in den TConnection's gespeichert.

wenn ich dieses TElement in einem Vector speichere, passen scheinbar die TElement-Pointer in den TConnections nicht mehr.

Wie kann ich dieses Problem lösen?

Gruß Frank

Oder du schaust, dass die Connections beim Kopieren des Objekts ebenfalls angepasst werden.

wie kann ich letzteres abfangen?

Gruß Frank

dann müsste ich das doch so definieren:
typedef std::vector<std::unique_ptr<TElement>> TElementList;

aber ich bekomme die Fehlermeldung
‘unique_ptr’ is not a member of ‘std’

was mache ich falsch?

http://gcc.gnu.org/projects/cxx0x.html
Important: GCC's support for C++11 is still experimental.

trotzdem hab ichs mal probiert:
g++ -Wall -std=c++11 platine.cpp -o platine `pkg-config --cflags gtk+-2.0` `pkg-config --libs gtk+-2.0`

cc1plus: error: unrecognized command line option "-std=c++11"

in der man-page finde ich den 11er nicht nur 0x

gibts nix vergleichbares zu unique_ptr im c++ standard?

aber scheinbar ändert sich da auch mein aufruf...

Fehlermeldung:

error: no matching function for call to ‘std::vector<std::unique_ptr<TConnection, std::default_delete<TConnection> >, std::allocator<std::unique_ptr<TConnection, std::default_delete<TConnection> > > >::push_back(TConnection&)’
/usr/include/c++/4.4/bits/stl_vector.h:733: note: candidates are: void std::vector<_Tp, _Alloc>::push_back(const _Tp&) [with _Tp = std::unique_ptr<TConnection, std::default_delete<TConnection> >, _Alloc = std::allocator<std::unique_ptr<TConnection, std::default_delete<TConnection> > >]
/usr/include/c++/4.4/bits/stl_vector.h:746: note: void std::vector<_Tp, _Alloc>::push_back(_Tp&&) [with _Tp = std::unique_ptr<TConnection, std::default_delete<TConnection> >, _Alloc = std::allocator<std::unique_ptr<TConnection, std::default_delete<TConnection> > >]

code:

TElement::TElement(...)
{
...
    TConnection c(fPlatine,this,id,s);
    Connections.push_back(c); //diese zeile wird bemängelt

}

also push_back(&c)?

ggf. mit cast auf unique pointer oder wie läuft das dann?

wie bekomme ich mein Objekt wieder raus und wie ist das mit der Speicherfreigabe?

std::unique_ptr<TConnection> c(new TConnection(fPlatine,this,id,s));
Connections.push_back(c);

habe überlegt, obs nicht sinnvoller wäre die ganze pointer-verarbeitung aus dem constructor rauszuverlagern (init-prozedure), die ich auf das listen-element ausführe.

Pseudocode:

std::vector<TConnection> TConnectionList;
TConnectionList clist;

TConnection c();
clist.push_back(c);
clist[clist.size()-1].init(...)

Danke trotzdem an alle, die mitgeholfen haben

probiere schon eine Weile rum, aber komme nicht wirklich weiter mit dem unique_ptr...

typedef std::vector<std::unique_ptr<TElement>> pElementList;
pElementList pElements;

unique_ptr<TElement> pE(new TElement());
pE->InitElement(this,Template,left,top,4,4);
pElements.push_back(pE);

bei der letzten Zeile wirft mir der cimpiler eine 10-seitige Fehlermeldung um die Ohren aus der ich nicht wirklich was herausnehmen kann...

In file included from /usr/include/c++/4.4/i486-linux-gnu/bits/c++allocator.h:34,
from /usr/include/c++/4.4/bits/allocator.h:48,
from /usr/include/c++/4.4/string:43,
from /usr/include/c++/4.4/bits/locale_classes.h:42,
from /usr/include/c++/4.4/bits/ios_base.h:43,
from /usr/include/c++/4.4/ios:43,
from /usr/include/c++/4.4/ostream:40,
from /usr/include/c++/4.4/iostream:40,
from platine.h:1,
from platine.cpp:1:
/usr/include/c++/4.4/bits/unique_ptr.h: In member function ‘void __gnu_cxx::new_allocator<_Tp>::construct(_Tp*, const _Tp&) [with _Tp = std::unique_ptr<TElement, std::default_delete<TElement> >]’:
/usr/include/c++/4.4/bits/stl_vector.h:737: instantiated from ‘void std::vector<_Tp, _Alloc>::push_back(const _Tp&) [with _Tp = std::unique_ptr<TElement, std::default_delete<TElement> >, _Alloc = std::allocator<std::unique_ptr<TElement, std::default_delete<TElement> > >]’
platine.cpp:499: instantiated from here
/usr/include/c++/4.4/bits/unique_ptr.h:214: error: deleted function ‘std::unique_ptr<_Tp, _Tp_Deleter>::unique_ptr(const std::unique_ptr<_Tp, _Tp_Deleter>&) [with _Tp = TElement, _Tp_Deleter = std::default_delete<TElement>]’
/usr/include/c++/4.4/ext/new_allocator.h:105: error: used here
In file included from /usr/include/c++/4.4/vector:69,
from platine.h:3,
from platine.cpp:1:
/usr/include/c++/4.4/bits/unique_ptr.h: In member function ‘void std::vector<_Tp, _Alloc>::_M_insert_aux(__gnu_cxx::__normal_iterator<typename std::_Vector_base<_Tp, _Alloc>::_Tp_alloc_type::pointer, std::vector<_Tp, _Alloc> >, _Args&& ...) [with _Args = const std::unique_ptr<TElement, std::default_delete<TElement> >&, _Tp = std::unique_ptr<TElement, std::default_delete<TElement> >, _Alloc = std::allocator<std::unique_ptr<TElement, std::default_delete<TElement> > >]’:
/usr/include/c++/4.4/bits/stl_vector.h:741: instantiated from ‘void std::vector<_Tp, _Alloc>::push_back(const _Tp&) [with _Tp = std::unique_ptr<TElement, std::default_delete<TElement> >, _Alloc = std::allocator<std::unique_ptr<TElement, std::default_delete<TElement> > >]’
platine.cpp:499: instantiated from here
/usr/include/c++/4.4/bits/unique_ptr.h:214: error: deleted function ‘std::unique_ptr<_Tp, _Tp_Deleter>::unique_ptr(const std::unique_ptr<_Tp, _Tp_Deleter>&) [with _Tp = TElement, _Tp_Deleter = std::default_delete<TElement>]’
/usr/include/c++/4.4/bits/vector.tcc:314: error: used here

pElements.push_back(std::move(pE));

heißen. Dies hat auch Nexus falsch gemacht.

kann ich da nicht auf unique_ptr verzichten und std::move beim normalen vector anwenden (damit keine Kopie erstellt wird)?

TElement e;
fElements.push_back(std::move(e));

der compiler frisst es zumindest

Wenn der Default-Move-Ctor das richtige tut, oder du deinen eigenen hast, schon.

ich habe einen leeren constructor, da ich die Klasse erst mit der InitElement initialisiere...
ich weis ja nicht, ob std::move das tut, was ich will...er soll das erstellte objekt in den Vector "moven" (also nicht kopieren, sondern verschieben)

typedef std::vector<std::unique_ptr<TElement>> pElementList;
  pElementList pElements;

  unique_ptr<TElement> pE(new TElement());
  pE->InitElement(this,Template,left,top,4,4);//init-code in constructor verlagern
  pElements.push_back(std::move(pE));

stimmt der zugriff? freigegeben wird das TElement ja, wenn die liste zerstört wird, oder? bzw. wenn ich erase aufrufe...kann ich irgendwie elemente ohne iterator löschen? also per index-zugriff (z.b. remove(4) soll das 5.element löschen)?

vec.erase(vec.begin() + n);

Bei vector könnte sowas jedoch langsam sein, da hier alle Elemente hinter dem gelöschten um eine Stelle vorgeschoben werden müssen.

Bei vector könnte sowas jedoch langsam sein, da hier alle Elemente hinter dem gelöschten um eine Stelle vorgeschoben werden müssen.

welche Alternativen gäbe es denn? prinzipiell brauche ich ja ein dynamisches array mit Klassen-Elementen.

314159265358979 schrieb:

Bei vector könnte sowas jedoch langsam sein, da hier alle Elemente hinter dem gelöschten um eine Stelle vorgeschoben werden müssen.

welche Alternativen gäbe es denn? prinzipiell brauche ich ja ein dynamisches array mit Klassen-Elementen.

Wenn die Reihenfolge im Vektor egal ist, such mal das erase-remove Idiom.