Bug: Denken() im Tick() mit freundlicher? Übernahme

[halloamt]

Hi Tom,

kann es sein, dass ich eure Ameisen übernehme? Ich lasse sie in Tick() Ihre Kaste ausgeben. Irgendwann, und nicht jedesmal, fangen auch andere Ameisen an zu denken.
Am Anfang machen sie das nicht, aber irgendwann, ich glaub, wenn die ersten gestorben sind, fangen sie damit an. Das passiert bei mir in fast jedem Durchlauf, wenn sie in Tick() denken. Aber nur dann, wenn man mehrere Duchläufe hintereinander laufen lässt.
Die Version ist aktuell.

Viele Grüße

Justus
(ja, der aus dem Student Program, ich finde die Ameisen toll für unsere Azubis :))

[halloamt]

public override void Tick()
{
	/// Da denken sie für mich, der Rest ist dann Zielsuche in einer statischen Liste.
	Denke("Justus seine Ameise");
}

[halloamt]

Weiter rumprobiert, man kann die anderen Ameisen nicht übernehmen :(
Sie behalten nur das alte Label. Werden die intern recycelt?

[tomwendel]

Hi Justus,

schön von dir zu lesen! Auch wenn du mir hier nur Arbeit machst 😁

Zum Glück scheint der Bug nicht ganz so kritisch zu sein wie ursprünglich erwartet. Allerdings stimmt da trotzdem was nicht. Soweit ich mich erinnern kann, sind die Ids pro Volk unique. Ich könnte mir vorstellen, dass ich damals bei der Implementierung der Labels aber von global eindeutigen Ids ausgegangen bin und die UI deshalb die Labels der anderen Völker mit ausgibt. Lass mich das mal genauer anschauen...

Cheers
Tom

[tomwendel]

Kleiner Zwischenstand: Das Problem ist nicht ganz offensichtlich. Wir haben hier kein Recycling der Objekte, die Denknachricht wird im Objekt gespeichert und die Ids sind über die ganze Simulation eindeutig. Die UI speichert diese Nachrichten in einem Id-basierten Dictionary. Bin also auf den ersten Blick ratlos.

Würde jetzt mal die Serialisierung der Daten zwischen Simulation und UI genau anschauen.

[tomwendel]

Habs! Wer die Repro-Schritte richtig lesen kann,... Der wichtige Punkt: Mehrere Runden.
Das Problem liegt im Cache für die Debug Nachrichten auf UI Seite: Dort ist ein Dictionary das die Nachricht basierend auf der Ameisen-ID speichert. Die Nachricht wird anhand ihrer Lebenszeit entfernt. Aber nur, wenn es die Ameise noch gibt.

Was also passiert: Die letzten Ameisen von Runde n haben ihre Nachricht noch im Cache. In der folgenden Runde verweilen diese Nachrichten noch dort, weil es keinen Mechanismus gibt diese über die Lebenszeit hinaus zu löschen. Sobald also die Ameisen Id der nächsten Runde in den bereich der zuletzt gelebten aus Runde n kommt, trauchen die "alten" Nachrichten wieder auf.

Mein Fix: Cache clear bei einer neuen Runde. PR wird geöffnet