Effektsystem #17

r-zander · 2019-10-24T21:23:40Z

Details: https://trello.com/c/HSEQ3wJ5/433-effektsystem

… speed modification.

…lags

…of effects

r-zander · 2019-10-24T21:24:55Z

api/effects.json

+[
+  {
+    "id": 1,
+    "name": "QuickEscape",


Erster, super simpler Effekt. Wird genutzt um Dodos einen kleinen Sprint zu geben, wenn sie geschlagen werden.

r-zander · 2019-10-24T21:32:53Z

berryhunterd/items/itemdefinition.go

+	// Parse effects
+	var effectsByEvent = &EffectsByEvent{}
+	var err error
+	if effectsByEvent.WhileCarried, err = effects.MapAndValidateEffects(r, "WhileCarried", i.Effects.WhileCarried); err != nil {


Gibt es eine Variante um das schöner zu lösen statt 3 Zeilen pro Event-Parsing zu haben?

Ich würde das ganze ein wenig flexibler gestalten, wenn du jedoch deinen fixen struct befüllst kommst du wohl kaum um die Zeilen rum. Ev. könntest to ein bisschen reflect nutzen.

chieftaind/client/client.go

r-zander · 2019-10-24T21:44:40Z

frontend/src/ts/Events.ts

@@ -1,4 +1,4 @@
-import {isDefined, removeElement} from "./Utils";
+import {isDefined, isUndefined, removeElement} from "./Utils";


Kannst du ignorieren, das war nur ein Fehler den ich im Frontend gefixt habe

r-zander · 2019-10-24T21:50:18Z

Meine Hauptfragen wären:

Was hältst du von der Package-Struktur wie ich sie jetzt für die Effekte genutzt habe?
Siehst du irgendwelche Änderungen, die extrem zulasten der Performance gehen?
Es gibt einen weirden Bug, dass das EffectSystem irgendwie dafür sorgt, dass sys.Update und sys.State nicht mehr richtig herum ausgeführt werden. Das führt sichtbar dazu, dass cheat: KILL nicht mehr funktioniert (da der Player leben regeneriert bevor er als tot aus dem Spiel entfernt wird), aber evtl gibt es noch weitere üble Nebenwirkungen. Ich kann mir leider absolut nicht erklären, warum - immerhin zählt das system nur die effekt durations herunter.

trichner · 2019-10-25T08:01:47Z

uiii, sexy! Ich führ mir das mal zu Gemüte am Wochenende 👍

trichner · 2019-10-27T21:15:03Z

api/_definitions/effects.json

+        "type": "integer",
+        "description": "Number of times this effect can be applied simultaneously to the same entity. Omitting this property means no limit to the stack size.",
+        "minimum": 1,
+        "default": "infinity"


sollte nicht 0 als 'infinity' gelten? typisierte Sprachen haben für int kaum ein infinity definiert ;)

Da habe ich eine Weile überlegt. Ich habe es so implementiert, dass "Wert nicht definiert im JSON" als nil herumgereicht wird, und dann maxStacks ignoriert wird. Natürlich könnte man das auch als 0 annehmen, aber dann würde der Default mit dem Minimum kollidieren, daher meine Angabe "infinity" (die natürlich nicht als Integer.INFINITY gelöst ist, sondern eben als nil) an den User adressiert, von wegen "wenn du nichts angibt, kann der Effekt beliebig oft stacken"

trichner

Cooles Ding!

Ich würde unbedingt das addieren/subtrahieren unterlassen, du hast eine gute Wahrscheinlichkeit dass a * x / x != a da die Operationen im Allgemeinen lossy sind. Ich würde die Effekte für jeden Tick wieder frisch aufaddieren, wenns zu hart ist für die performance liesse sich das dann auch einfach cachen...

Im Allgemeinen scheint mir die momentane Lösung skaliert relativ schlecht, wie du bereits selber gemerkt hast repetiert sich einiges relativ oft.

trichner · 2019-10-27T21:19:26Z

api/_definitions/effects.json

@@ -0,0 +1,188 @@
+{
+  "$schema": "http://json-schema.org/draft-07/schema#",
+  "$id": "https://berryhunter.io/schema/effects.json",


api/effects.json

berryhunterd/effects/effect.go

trichner · 2019-10-27T23:33:00Z

berryhunterd/items/inventory.go

 	return Inventory{
-		items: make([]*ItemStack, 0, 10),
-		cap:   DEFAULT_INVENTORY_CAP,
+		effectEntity: p,


Warum sind effekte teil des inventars?!

Weil das Inventar sonst nicht weiß, zu wem es gehört und wohin die Effekte der Items im Inventar zu applyen sind. Bin offen für eine bessere Lösung, aber sehe es nicht so recht, da diese Effect entity tatsächlich relativ oft gebraucht wird:

trichner · 2019-10-27T23:33:23Z

berryhunterd/items/inventory.go

 // Cap returns the maximum amount of item slots
 func (i *Inventory) Cap() int {
-	return i.cap
+	return i.cap + i.effectEntity.EffectStack().Factors().InventoryCap


Kannst du das nicht generischer lösen?

trichner · 2019-10-27T23:35:57Z

berryhunterd/items/itemdefinition.go

+	// Parse effects
+	var effectsByEvent = &EffectsByEvent{}
+	var err error
+	if effectsByEvent.WhileCarried, err = effects.MapAndValidateEffects(r, "WhileCarried", i.Effects.WhileCarried); err != nil {


Ich würde das ganze ein wenig flexibler gestalten, wenn du jedoch deinen fixen struct befüllst kommst du wohl kaum um die Zeilen rum. Ev. könntest to ein bisschen reflect nutzen.

trichner · 2019-10-27T23:50:59Z

berryhunterd/sys/state.go

 	for _, p := range s.players {

-		if p.VitalSigns().Health == 0 {
+		if p.VitalSigns().Health <= 0 {


warum? Ziemlich sicher sind die VitalSigns gegen over- und underflow geschützt

Da habe ich wohl rumgespielt, um den angesprochenen Bug zu beheben

Es gibt einen weirden Bug, dass das EffectSystem irgendwie dafür sorgt, dass sys.Update und sys.State nicht mehr richtig herum ausgeführt werden. Das führt sichtbar dazu, dass cheat: KILL nicht mehr funktioniert (da der Player leben regeneriert bevor er als tot aus dem Spiel entfernt wird), aber evtl gibt es noch weitere üble Nebenwirkungen. Ich kann mir leider absolut nicht erklären, warum - immerhin zählt das system nur die effekt durations herunter.

chieftaind/client/client.go

r-zander added 12 commits October 13, 2019 16:27

Effect system: read definitions

d147b29

Backend: Don't exit when there's no chieftaind

85dab0d

Effect system: added effect stack to player entity

e52541e

Init effect stack on player and try to use it in EquipAction

19f95ca

Effect system: apply effects while equipped

8d5f233

Effect system: While carried and while equipped work now for movement…

a49b1cb

… speed modification.

Effect system: Implement more events

ebf0e4c

Effect system: implement all remaining events

6a9da90

Effect system: implement all remaining factors

7a53eec

Effect system: implement maxStacks, duration and different behavior f…

bc4693f

…lags

Effect system: created JSON schema to document configurable features …

91dabaa

…of effects

Effect system: validations and code clean-up

2fe799a

r-zander commented Oct 24, 2019

View reviewed changes

chieftaind/client/client.go Outdated Show resolved Hide resolved

r-zander commented Oct 24, 2019

View reviewed changes

Code cleanup

d26195e

r-zander requested a review from trichner October 24, 2019 21:50

trichner reviewed Oct 27, 2019

View reviewed changes

Changes after code review

3a6f910

		@@ -1,4 +1,4 @@
		import {isDefined, removeElement} from "./Utils";
		import {isDefined, isUndefined, removeElement} from "./Utils";

Effektsystem #17

Are you sure you want to change the base?

Effektsystem #17

Uh oh!

Conversation

r-zander commented Oct 24, 2019

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

r-zander commented Oct 24, 2019

Uh oh!

trichner commented Oct 25, 2019

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

trichner left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants