<![CDATA[Statement has no effect]]>void TimerController::update( const u32 delta, const u32 ticks ) { m_counter += delta; NLTimerList::iterator it = m_timers.begin(); for ( it; it != m_timers.end(); ++it ) // WARNUNG { Timer* t = it->second; u32 delay = m_counter - t->getStartingTime(); if ( delay >= t->getInterval() ) { t->setStartingTime(m_counter); if ( t->isRunning() ) { t->fire(delay); if ( t->isOneShot() ) { delete t; it = m_timers.erase(it); // FEHLER if ( it == m_timers.end() ) { break; } } } } } // Fire global update event m_logic_event(delta, ticks); }

Das meldet mir MinGW als Warnung beim Beginn der for-schleife.
Ausserdem gibts auch eine Fehlermeldung:

no match for 'operator=' in 'it = ((NightLight::TimerController*)this)->NightLight::TimerController::m_timers.std::map<_Key, _Tp, _Compare, _Alloc>::erase<std::basic_string<char>, NightLight::Timer*, std::less<std::basic_string<char> >, std::allocator<std::pair<const std::basic_string<char>, NightLight::Timer*> > >(it)'

Die Warnung bekomme ich nen paar mal, jeddesmal bei andere for-schleifen die gleich gestrickt sind, aber ich kann hier nicht sehen wo der Fehler liegen soll.
Unter VC++ 2008 kompiliert das Warnungs- und Fehlerfrei.

Definition:

typedef std::map<std::string, Timer*> TimerList;
TimerList m_timers;

Kann jemand weiterhelfen?

]]>https://www.c-plusplus.net/forum/topic/306384/statement-has-no-effectRSS for NodeSat, 08 Aug 2026 07:36:39 GMTThu, 26 Jul 2012 17:04:54 GMT60<![CDATA[Reply to Statement has no effect on Thu, 26 Jul 2012 17:07:53 GMT]]>void TimerController::update( const u32 delta, const u32 ticks ) { m_counter += delta; NLTimerList::iterator it = m_timers.begin(); for ( it; it != m_timers.end(); ++it ) // WARNUNG { Timer* t = it->second; u32 delay = m_counter - t->getStartingTime(); if ( delay >= t->getInterval() ) { t->setStartingTime(m_counter); if ( t->isRunning() ) { t->fire(delay); if ( t->isOneShot() ) { delete t; it = m_timers.erase(it); // FEHLER if ( it == m_timers.end() ) { break; } } } } } // Fire global update event m_logic_event(delta, ticks); }

Das meldet mir MinGW als Warnung beim Beginn der for-schleife.
Ausserdem gibts auch eine Fehlermeldung:

no match for 'operator=' in 'it = ((NightLight::TimerController*)this)->NightLight::TimerController::m_timers.std::map<_Key, _Tp, _Compare, _Alloc>::erase<std::basic_string<char>, NightLight::Timer*, std::less<std::basic_string<char> >, std::allocator<std::pair<const std::basic_string<char>, NightLight::Timer*> > >(it)'

Die Warnung bekomme ich nen paar mal, jeddesmal bei andere for-schleifen die gleich gestrickt sind, aber ich kann hier nicht sehen wo der Fehler liegen soll.
Unter VC++ 2008 kompiliert das Warnungs- und Fehlerfrei.

Definition:

typedef std::map<std::string, Timer*> TimerList;
TimerList m_timers;

Kann jemand weiterhelfen?

]]>https://www.c-plusplus.net/forum/post/2236007https://www.c-plusplus.net/forum/post/2236007Thu, 26 Jul 2012 17:07:53 GMT<![CDATA[Reply to Statement has no effect on Thu, 26 Jul 2012 17:08:57 GMT]]>Was soll denn das erste it auch bewirken?

]]>https://www.c-plusplus.net/forum/post/2236008https://www.c-plusplus.net/forum/post/2236008Thu, 26 Jul 2012 17:08:57 GMT<![CDATA[Reply to Statement has no effect on Thu, 26 Jul 2012 17:10:48 GMT]]>Es bewirkt nichts, nur ob ich da jetzt nen Semikolon setze oder das it; hinschreibe, sollte keinen Unterschied machen.
Die Fehlermeldung gibt mir aber wirklich Rätsel auf.
Ich kann da keinen Fehler erkennen.
Ich hab den neuesten MINGW(4.70).

]]>https://www.c-plusplus.net/forum/post/2236009https://www.c-plusplus.net/forum/post/2236009Thu, 26 Jul 2012 17:10:48 GMT<![CDATA[Reply to Statement has no effect on Thu, 26 Jul 2012 17:13:55 GMT]]>
  • der Compiler warnt dich, dass das Statement überflüssig ist. Wenn dich das stört, lass es weg
  • was liefert erase einer Map zurück?
    • ]]>    https://www.c-plusplus.net/forum/post/2236011https://www.c-plusplus.net/forum/post/2236011Thu, 26 Jul 2012 17:13:55 GMT    <![CDATA[Reply to Statement has no effect on Thu, 26 Jul 2012 17:16:35 GMT]]>Danke. VC++ liefert hier einen iterator auf das nächste Element zurück.
    Aber in der offiziellen stl scheint das nicht so zu sein.

    http://msdn.microsoft.com/en-us/library/z2f3cb7h(v=vs.71).aspx
    http://www.cplusplus.com/reference/stl/map/erase/

    ]]>    https://www.c-plusplus.net/forum/post/2236012https://www.c-plusplus.net/forum/post/2236012Thu, 26 Jul 2012 17:16:35 GMT    <![CDATA[Reply to Statement has no effect on Thu, 26 Jul 2012 17:34:54 GMT]]>Das hat mglw. nichts mit dem VC++, sondern mit dem neuen Standard zu tun. In C++98 gibt die erase-Methode eines assoziativen Containers void zurück, in C++11 einen Iterator.

    ]]>    https://www.c-plusplus.net/forum/post/2236017https://www.c-plusplus.net/forum/post/2236017Thu, 26 Jul 2012 17:34:54 GMT    <![CDATA[Reply to Statement has no effect on Thu, 26 Jul 2012 17:37:53 GMT]]>VC2008 hat afaik kein C++11 Support, zumindest hab ich nichts aktiviert und auch nicht installiert in die Richtung.
    Ist einfach das Microsoft erase() :D.
    Habs jetzt anders gelöst, indem ich den iterator kopiere, inkrementiere und dann lösche per erase.

    ]]>    https://www.c-plusplus.net/forum/post/2236018https://www.c-plusplus.net/forum/post/2236018Thu, 26 Jul 2012 17:37:53 GMT