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

ramsch in jskat-0.22.0-linux #172

Closed
Heiko1234567 opened this issue Dec 18, 2023 · 16 comments · Fixed by #176
Closed

ramsch in jskat-0.22.0-linux #172

Heiko1234567 opened this issue Dec 18, 2023 · 16 comments · Fixed by #176
Assignees
Labels
Milestone

Comments

@Heiko1234567
Copy link

Ist der Fehler bereits bekannt? Jedes Ramsch-Spiel ist in der Version skat-0.22.0-linux nicht möglich.
Fehler: Der Skat kann nicht weiter geschoben werden. Drückt man ein paar mal auf den Button: "schieben", stürzt das Programm ab.
Mein System: Ubuntu 22.04.3 LTS / Intel I7-7700HQ / Grafik: Intel HD 630 / NVidea RTX 1070 / 32 Gb Ram.

@b0n541
Copy link
Owner

b0n541 commented Dec 18, 2023

Vielen Dank für das Melden des Fehlers. Ich kann das Problem nachstellen.

@b0n541 b0n541 self-assigned this Dec 18, 2023
@b0n541 b0n541 added the bug label Dec 18, 2023
@b0n541
Copy link
Owner

b0n541 commented Dec 22, 2023

Ich habe den Fehler gefunden. Leider wird die Behebung einen etwas größeren Umbau nötig machen. Bitte solange die Version 0.21 verwenden.

@Heiko1234567
Copy link
Author

Ja, danke für die Info. Die 0.21 ist installiert.

@b0n541 b0n541 added this to the Version 0.23 milestone Jan 31, 2024
b0n541 added a commit that referenced this issue Jan 31, 2024
@b0n541 b0n541 linked a pull request Jan 31, 2024 that will close this issue
@b0n541
Copy link
Owner

b0n541 commented Jan 31, 2024

Unter https://github.com/b0n541/jskat/actions/runs/7732354462 ist eine gefixte Version verfügbar:

Artifacts -> ubuntu-latest

JSkat setzt jetzt ein JDK 21 voraus.

@Heiko1234567
Copy link
Author

Heiko1234567 commented Feb 1, 2024

Das Problrem besteht nach wie vor. Ich kann den Skat beim Ramsch-Spiel nicht weiter schieben. Die Funktion des Button "Schieben" ist nicht ausführbar. Zumindest stürzt das Programm nicht mehr ab, wenn auf Schieben gedrückt wird. Auch kann man jetzt die Karten im Skat tauschen, sogar drei oder vier Karten in den Skat legen (das wird aber vom Programm kommentiert). Es geht aber auch mit zwei Karten im Skat nicht weiter im Spiel.

@b0n541 b0n541 reopened this Feb 1, 2024
@b0n541
Copy link
Owner

b0n541 commented Feb 1, 2024

Hmm, dann werde ich noch mal schauen müssen.

@Heiko1234567
Copy link
Author

Wenn man nur Ramsch als Skatserie spielen möchte, funktioniert alles einwandfrei. Wenn in einer normalen Spielserie eine Ramschrunde anfällt (bspw, weil keiner Allein spielen will), dann stellt sich der Fehler ein.

@b0n541
Copy link
Owner

b0n541 commented Feb 5, 2024

Es scheint bei mir nur aufzutreten, wenn beim ersten Schiebeversuch entweder der Hinweis über eine falsche Anzahl von Karten im Skat kommt oder der Hinweis, dass noch Buben im Skat liegen, kommt. Danach funktioniert Schieben nicht mehr.

Während des Drückens ist es erlaubt weniger als zwei und bis zu vier Karten im Skat zu haben. Nur beim weiterschieben/drücken in den Skat wird geprüft, ob die Anzahl stimmt. Aber stimmt schon, im normalen Leben nimmt man die beiden Skatkarten ja auch erst auf die Hand und drückt wieder zwei Karten. Werde ich mal noch mit einem anderen Ticket ändern.

@b0n541
Copy link
Owner

b0n541 commented Feb 5, 2024

Ich habe mir gerade noch einmal https://de.wikipedia.org/wiki/Schieberamsch durchgelesen und bin der Meinung, dass die gesamte Oberfläche während der Drückphase beim Schieberamsch uneinheitlich und missverständlich ist. So sollte es nur einen Knopf geben der entweder mit "Schieben" oder mit "Drücken" beschriftet sein, je nachdem, ob der Spieler in den Skat geschaut hat oder nicht.

Auch der "Contra"-Knopf funktioniert nicht richtig (Fehler: keine Karte ausgespielt) und es gibt keine Möglichkeit "Jungfrau" anzusagen.

Ich befürchte, das läuft auf eine Neuimplementierung des Ramschspiel in JSkat hinaus, da es jetzt schon sehr aufwändig war, den ersten Fix zu erstellen, der aber leider nicht vollständig wirksam war.

@Heiko1234567
Copy link
Author

Oder Aufnehmen/Schieben/Drücken, Knopf. Ja, das klingt nach viel Arbeit.

@b0n541
Copy link
Owner

b0n541 commented Feb 24, 2024

Ich konnte das Problem zunächst ohne komplette Neuimplementierung lösen. Schieben sollte jetzt wieder sowohl in einer Pflichtramschrunde als auch bei einem Ramsch nach fehlenden Reizgeboten funktionieren. Nach den Fehlermeldungen über ungültige Karten im Skat funktioniert das Schieben auch wieder.

Das Ganze kann mit dem letzten erfolgreichen Build ausprobiert werden:

https://github.com/b0n541/jskat/actions/runs/8030903791

@b0n541 b0n541 closed this as completed Mar 5, 2024
@Heiko1234567
Copy link
Author

Ich hab's grad mal ausprobiert. Der Skat lässt sich nun aufnehmen, aber kommt als gedrückt vom letzten (der drückt in der Runde) zurück und wird angezeigt. Austauschen kann man nun ein zweites mal und per Button "schieben" l(einzige Option) lässt sich das Spiel nicht weiter spielen.

@b0n541
Copy link
Owner

b0n541 commented Mar 7, 2024

Hmm, ich habe die oben verlinkte version ausprobiert und tatsächlich ist die weiterhin kaputt. Die neuesten Builds sollten aber funktionieren. Habe es selbst gerade noch einmal getestet:

https://github.com/b0n541/jskat/actions/runs/8162177055

Tut mir leid für die Verwirrung. Ich werde zeitnah eine ordentliche Version bauen und freigeben.

@Heiko1234567
Copy link
Author

Jetzt ist es prima. Funktioniert - danke!

@b0n541
Copy link
Owner

b0n541 commented Mar 9, 2024

Vielen Dank für's Testen und die Geduld bis zur Lösung.

@b0n541
Copy link
Owner

b0n541 commented Mar 19, 2024

Ich habe gerade eine neue Version 0.23 von JSkat erstellt, welche auch noch weitere Bugfixes für Ramsch enthält:

https://github.com/b0n541/jskat/releases/tag/v0.23.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants