std::vector::iterator muckt...

Reply to std::vector::iterator muckt... on Wed, 14 Nov 2007 21:40:53 GMT

piXelshooter — Wed, 14 Nov 2007 21:40:53 GMT

Hi,
folgeder code:

std::vector v;
...
for (std::vector::iterator i = reservations.begin(); i
    != reservations.end(); i++)

folgende fehlerlendung:

**** Build of configuration rel for project pXai ****

mingw32-make all
'Building file: ../src/AIcalendar.cpp'
'Invoking: GCC C++ Compiler'
g++ -O2 -g -Wall -c -fmessage-length=0 -MMD -MP -MF"src/AIcalendar.d" -MT"src/AIcalendar.d" -o"src/AIcalendar.o" "../src/AIcalendar.cpp"
C:/MinGW/bin/../lib/gcc/mingw32/3.4.2/../../../../include/c++/3.4.2/bits/stl_iterator.h: In constructor \_\_gnu\_cxx::\_\_normal\_iterator<\_Iterator, \_Container>::\_\_normal\_iterator(const \_\_gnu\_cxx::\_\_normal\_iterator<\_Iter, \_Container>&) \[with \_Iter = const AItask*, \_Iterator = AItask*, \_Container = std::vector >\]': ../src/AIcalendar.cpp:116: instantiated from here C:/MinGW/bin/../lib/gcc/mingw32/3.4.2/../../../../include/c++/3.4.2/bits/stl_iterator.h:609: error: invalid conversion from \const AItask* const' to `AItask*'
mingw32-make: *** [src/AIcalendar.o] Error 1

das sieht schon ma sehr aufgeräumt aus... zu info: AIcalendar.cpp ist das oben gezeigte file. besagte zeile 116 ist dort, wo der for-loop anfängt.

es hängt definitiv zusammen mit der deklaration

std::vector::iterator i;

nur, was geht daran denn nit?!

Reply to std::vector::iterator muckt... on Wed, 14 Nov 2007 22:18:10 GMT

camper — Wed, 14 Nov 2007 22:18:10 GMT

was ist mit reservations? zufällig ein std::vector const& ?

Reply to std::vector::iterator muckt... on Wed, 14 Nov 2007 23:02:43 GMT

piXelshooter — Wed, 14 Nov 2007 23:02:43 GMT

wie meinst du das?
Naja, wie der Vector deklariert ist, steht ja oben, nix const oä. und auch schon die deklaration std::vector::iterator i; macht das problem...

Reply to std::vector::iterator muckt... on Thu, 15 Nov 2007 04:47:01 GMT

MFK — Thu, 15 Nov 2007 04:47:01 GMT

piXelshooter schrieb:

Naja, wie der Vector deklariert ist, steht ja oben, nix const oä.

Da steht nur, was v ist. Camper hatte nach reservations gefragt. Es deutet nämlich alles darauf hin, dass dieser vector const ist.

Reply to std::vector::iterator muckt... on Thu, 15 Nov 2007 07:12:02 GMT

piXelshooter — Thu, 15 Nov 2007 07:12:02 GMT

sry.
was meint ihr mit reservations?

Reply to std::vector::iterator muckt... on Thu, 15 Nov 2007 07:15:29 GMT

MFK — Thu, 15 Nov 2007 07:15:29 GMT

piXelshooter schrieb:

sry.
was meint ihr mit reservations?

Den vector, dessen begin- und end-Methode du da aufrufst. Der Bezeichner stammt aus deinem Code.

Reply to std::vector::iterator muckt... on Thu, 15 Nov 2007 08:28:08 GMT

Simon2 — Thu, 15 Nov 2007 08:28:08 GMT

piXelshooter schrieb:

sry.
was meint ihr mit reservations?

piXelshooter schrieb:

... = reservations....

Gruß,

Simon2.

Reply to std::vector::iterator muckt... on Thu, 15 Nov 2007 11:06:14 GMT

piXelshooter — Thu, 15 Nov 2007 11:06:14 GMT

omg sry... dachte das hätte iwas mit dem speichermanagement im vektor zu tun, völlig verpeilt, dass mein vektor ja reservations HEIßT -__-°

std::vector reservations;
...
for (std::vector::iterator i = reservations.begin(); i
    != reservations.end(); i++)

das sind die relevanten codestücke.
aber auch wenn ich nur:

std::vector::iterator i;

schreibe, erhalte ich besagte fehlermeldung.

Reply to std::vector::iterator muckt... on Thu, 15 Nov 2007 11:08:32 GMT

camper — Thu, 15 Nov 2007 11:08:32 GMT

wie sieht die Definition von AItask aus?

Reply to std::vector::iterator muckt... on Thu, 15 Nov 2007 16:36:11 GMT

piXelshooter — Thu, 15 Nov 2007 16:36:11 GMT

class AItask
{
    public:
        AItask();
        ~AItask();
        bool operator <(AItask& cmp);
        AItimeInterval interval; // TODO private
    private:
        unsigned char priority;
        AIabstractAction* toDo;
};

Welche Eigenschaft könnte den verantwortlich dafür sein?

Reply to std::vector::iterator muckt... on Fri, 16 Nov 2007 08:37:49 GMT

Simon2 — Fri, 16 Nov 2007 08:37:49 GMT

Hi,

ich weiß nicht, ob's daran liegt, aber irgendwie würde ich den operator<() "doppelt const" ( operator<( .... const&) const; ) deklarieren...
Ansonsten gilt halt: Das Teil muss kopierbar sein - und damit auch alle seine Elemente (z.B. AItimeInterval).
Worauf zeigt eigentlich AIabstractAction* toDo ... nachdem ein Objekt kopiert wurde ? Auf ein "externes Objekt" - dann wäre das in Ordnung. Wenn es aber "irgendwie" auf "irgendetwas" innerhalb des AItask-Objekts zeigt, braucht's einen entsprechend intelligenten CopyConstr/operator=().
Gilt auch anders herum: Wenn irgendein Objekt "außerhalb der AItask" auf ein AItask-Objekt zeigt, verliert dieser Verweis seine Gültigkeit, sobald dieses Objekt verschoben wird.... und das tut std::vector halt bisweilen.

Gruß,

Simon2.