Jonathan/Babushka
2
0
Fork 0
Code Pull Requests 1 Releases Activity

🐛 Fixed fighters keep fighting at 0 hp #24

Merged
Jonathan merged 1 commits from bug/enemy_die into develop 2025-11-11 10:14:20 +01:00
Conversation 1 Commits 1 Files Changed 2
+42 -3
Jonathan commented 2025-11-04 14:01:28 +01:00
Owner
Copy Link
Copy Source

Fixed a bug where fighters keep attacking despite being dead at 0 HP.

Fixed a bug where fighters keep attacking despite being dead at 0 HP.
kziolkowski was assigned by Jonathan 2025-11-04 14:01:28 +01:00
Jonathan added 1 commit 2025-11-04 14:01:28 +01:00
🐛 Fixed fighters keep fighting at 0 hp 50b2c04cb4
Jonathan requested review from kziolkowski 2025-11-04 21:57:59 +01:00
kziolkowski approved these changes 2025-11-05 14:55:14 +01:00
kziolkowski left a comment
Owner
Copy Link
Copy Source

Zwei kleinere Nachfragen, sonst alles juti 👍

Zwei kleinere Nachfragen, sonst alles juti 👍
scripts/CSharp/Common/Fight/FightHappening.cs
@@ -348,0 +354,4 @@
{
if (f.IsDead())
{
HappeningData.fighterTurn.Remove(f);
kziolkowski commented 2025-11-05 14:48:14 +01:00
Owner
Copy Link
Copy Source

Ich gehe zwar davon aus, dass du das getestet hast und es keine Fehler wirft, aber zur Sicherheit, weil ich mich gewundert habe: Wenn fighterTurn ein IEnumerable ist und du in der foreach-Schleife etwas daraus entfernst, kann das nicht zu exceptions führen? Hätte ich jetzt erwartet.

Ich gehe zwar davon aus, dass du das getestet hast und es keine Fehler wirft, aber zur Sicherheit, weil ich mich gewundert habe: Wenn fighterTurn ein IEnumerable ist und du in der foreach-Schleife etwas daraus entfernst, kann das nicht zu exceptions führen? Hätte ich jetzt erwartet.
Jonathan commented 2025-11-05 15:11:18 +01:00
Author
Owner
Copy Link
Copy Source

Ist auch oft so. Deshalb habe ich GetEnumerator so implementiert, dass er erst eine List aus den Einträgen erstellt und dann den Enumerator der Liste zurückgibt

Ist auch oft so. Deshalb habe ich `GetEnumerator` so implementiert, dass er erst eine `List` aus den Einträgen erstellt und dann den Enumerator der Liste zurückgibt
👍 1
scripts/CSharp/Common/Fight/FighterTurn.cs
@@ -97,0 +101,4 @@
public IEnumerator<FightWorld.Fighter> GetEnumerator()
{
if (_currentNode == null) return Enumerable.Empty<FightWorld.Fighter>().GetEnumerator();
kziolkowski commented 2025-11-05 14:53:32 +01:00
Owner
Copy Link
Copy Source

Sehe ich da was falsch oder ist das ein Loop?

Wenn currentnode == null, dann wird Enumerable.GetEnumerator ( Zeile 119) und darin dann wieder GetEnumerator (Zeile 102) aufgerufen und so weiter. Oder ist das noch ein anderer Overload, der da aufgerufen wird?

Again, ich gehe schon davon aus, dass du das getestet hast und ich da irgendwas übersehe, aber ich frage lieber zur Sicherheit immer nach.

Sehe ich da was falsch oder ist das ein Loop? Wenn currentnode == null, dann wird Enumerable.GetEnumerator ( Zeile 119) und darin dann wieder GetEnumerator (Zeile 102) aufgerufen und so weiter. Oder ist das noch ein anderer Overload, der da aufgerufen wird? Again, ich gehe schon davon aus, dass du das getestet hast und ich da irgendwas übersehe, aber ich frage lieber zur Sicherheit immer nach.
Jonathan commented 2025-11-05 15:07:39 +01:00
Author
Owner
Copy Link
Copy Source

Hier wird Enumerable.Empty<...>.GetEnumerator() aufgerufen. Das Empty ist nen Statischer member von der Enumerable Klasse. (Nicht IEnumerable)

Hier wird `Enumerable.`**`Empty<...>`**`.GetEnumerator()` aufgerufen. Das `Empty` ist nen Statischer member von der `Enumerable` Klasse. (Nicht `IEnumerable`)
👍 1
Jonathan merged commit a9446ec770 into develop 2025-11-11 10:14:20 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
kziolkowski
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
ascheck (Anna Scheck) jlink kziolkowski (Katharina Ziolkowski) lkrause Nao Palurdas saderako Sanelschnitte Jonathan
No Assignees kziolkowski
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Jonathan/Babushka#24
Delete Branch "bug/enemy_die"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?