Closed M4lph closed 7 months ago
Sollte das so bleiben, oder haben wir den Issue dafür vergessen zu erstellen?
Kicher - der Wert im Tooltip ist der theoretische Wert. Der Wert im Panel ist der effektive Wert und je nach Höhe feuert der Turm weiter, daher ergibt sich die Kommazahl. :) Wenn Du ihn "unten" baust, sollte er auch bei 4 sein.
Naming von Tests inkonsistent:
Gut aufgefallen, welches Naming gefällt Dir besser?
Welchen Editor/Funktionen nutzt du um nach sowas zu suchen?
Auch nur das, was Rider so kann und so viele Stellen gibt's noch nicht, Du solltest alle getroffen haben :)
Sollte das so bleiben, oder haben wir den Issue dafür vergessen zu erstellen?
Kicher - der Wert im Tooltip ist der theoretische Wert. Der Wert im Panel ist der effektive Wert und je nach Höhe feuert der Turm weiter, daher ergibt sich die Kommazahl. :) Wenn Du ihn "unten" baust, sollte er auch bei 4 sein.
Wenn man selbst nicht an die Features denkt an denen man mitgewirkt hat... Und da merkt man auch, dass ich nicht anständig geschaut hat, wo die Ursache liegt :)
Gut aufgefallen, welches Naming gefällt Dir besser?
Sowas wie Methode_ErwartetesVerhalten_Bedingung finde ich persönlich ansprechender, in der Weise sind auch die meisten bisher benannt. Aber du hast die meisten Tests geschrieben, wirst wohl auch noch die meisten schreiben, und bist Project Owner :) Also ich passe mich da einfach an dann 👍
Sollte das so bleiben, oder haben wir den Issue dafür vergessen zu erstellen?
Kicher - der Wert im Tooltip ist der theoretische Wert. Der Wert im Panel ist der effektive Wert und je nach Höhe feuert der Turm weiter, daher ergibt sich die Kommazahl. :) Wenn Du ihn "unten" baust, sollte er auch bei 4 sein.
Wenn man selbst nicht an die Features denkt an denen man mitgewirkt hat... Und da merkt man auch, dass ich nicht anständig geschaut hat, wo die Ursache liegt :)
Gut aufgefallen, welches Naming gefällt Dir besser?
Sowas wie Methode_ErwartetesVerhalten_Bedingung finde ich persönlich ansprechender, in der Weise sind auch die meisten bisher benannt. Aber du hast die meisten Tests geschrieben, wirst wohl auch noch die meisten schreiben, und bist Project Owner :) Also ich passe mich da einfach an dann 👍
Passt für mich aber auch, wenn wir die Underscore-Variant wählen, das dürfte besser lesbar sein. Ich mach ein Ticket, um die Name anzupassen. :)
Habe die Funktion sowohl als eigene Klasse, als auch als Extensionmethode implementiert, da ich persönlich an solchen Stellen die Extensions lieber nutze. Kann ich an nutzenden Stellen aber auch gerne ändern. Apropos - Ich habe mit Rider nach sowas wie "0.00" und "0.##" im Code gesucht, bin mir aber nicht 100% sicher ob ich so alle Stellen erwischt habe. Welchen Editor/Funktionen nutzt du um nach sowas zu suchen? Zurzeit konvertiere ich auch nur floats, hab nicht die Notwendigkeit für double oder decimal gesehen, kann aber auch gerne nachgeholt werden :) Was mir noch aufgefallen ist:
Anzeige von Reichweite eines Turmes ist unterschiedlich auf ausgewähltem Turm der platziert ist, und einem der noch nicht platziert wurde- einmal ohne Nachkommastellen, einmal mit. Kann sogar sein, dass ich dich da schonmal nach gefragt hatte im Gespräch, leider aber keine Notizen dazu gemacht, deswegen sorry :) Sollte das so bleiben, oder haben wir den Issue dafür vergessen zu erstellen?![grafik](https://github.com/BoundfoxStudios/fairy-tale-defender/assets/18291863/af7964dd-c8eb-41a7-990b-ec934b5aa5a3)
Naming von Tests inkonsistent:
resolves #404