Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Spielfeldgrößeneingabe wird nicht korrekt umgesetzt #182

Merged
merged 3 commits into from
Jun 18, 2021

Conversation

ValentinDamjantschitsch
Copy link
Contributor

Mir ist da ein Bug aufgefallen: Wenn man die Feldgröße im Menu eingibt, wird das auf eine Instanz angewendet, die beim Spielstarten gar nicht aufgerufen wird. Probiert mal aus die Größe einzustellen und startet danach ein Spiel, dann wisst ihr sofort was ich meine.

Deswegen meine Idee hier, direkt nachdem man die Größe eingegeben hat, das Spiel mit der entsprechenden Spielfeldgröße zu starten. Was dabei aber noch fehlt, ist das der Outputstreamplayer dann die richtigen Werte von der game Instanz bekommt, das hab ich aber leider nicht hinbekommen...

Eine andere Lösung wäre natürlich, die Eingaben der Spielfeldgröße zu speichern und bei jedem erneuten Aufruf eines Games mitzunehmen, aber auch das habe ich nicht hingekriegt.

Welche der beiden Lösungsansätze gefällt euch besser? Und noch viel wichtiger: habt ihr dann dafür auch eine konkrete Idee zur Implementierung?

@ValentinDamjantschitsch ValentinDamjantschitsch added bug Something isn't working In Arbeit User Story bzw. Task noch nicht abgeschlossen Iteration 3 für Iteration 3 geplant labels Jun 17, 2021
@michaelgrantner
Copy link
Contributor

Gut, dass es dir aufgefallen ist! Ich bin für die zweite Variante, da dies sonst nicht konsistent mit den anderen Einstellungen wäre, da diese auch nicht sofort ein Spiel starten.
Was meint ihr @alablz @KLichtenauer @Sen66 ?

# Conflicts:
#	src/main/java/de/uni_passau/fim/se/memory/controller/InputStreamMainMenue.java
@ValentinDamjantschitsch
Copy link
Contributor Author

Perfekt gelöst!

@michaelgrantner michaelgrantner self-requested a review June 18, 2021 08:31
@Sen66 Sen66 self-requested a review June 18, 2021 08:31
@ValentinDamjantschitsch ValentinDamjantschitsch merged commit 586b6a0 into master Jun 18, 2021
@michaelgrantner michaelgrantner added Gemerged Pull request wurde bereits in master gemerged and removed In Arbeit User Story bzw. Task noch nicht abgeschlossen labels Jun 18, 2021
alablz pushed a commit that referenced this pull request Jul 19, 2021
alablz pushed a commit that referenced this pull request Jul 19, 2021
[BUG] Spielfeldgrößeneingabe wird nicht korrekt umgesetzt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Gemerged Pull request wurde bereits in master gemerged Iteration 3 für Iteration 3 geplant
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants