2211945
/
WIZARD_PR2_DOP
2
0
Fork 1
Code Issues 8 Pull Requests 2 Packages Projects 1 Releases Wiki Activity

kts.block #42

Closed
2210892 wants to merge 2 commits from kts.block into main
pull from: kts.block
merge into: 2211945:main
Conversation 2 Commits 2 Files Changed 3
2210892 commented 2023-11-02 12:07:59 +01:00
Collaborator

BITTE einmal schauen, ob man eine Exception so eigentlich schreiben kann. Oder ob es einen besseren, typischen Stil gibt.
Bei der Klasse Spiel hab ich nur einen Buchstaben wegen Rechtschreibung hinzugefügt. :-)

BITTE einmal schauen, ob man eine Exception so eigentlich schreiben kann. Oder ob es einen besseren, typischen Stil gibt. Bei der Klasse Spiel hab ich nur einen Buchstaben wegen Rechtschreibung hinzugefügt. :-)
2210892 added 2 commits 2023-11-02 12:08:00 +01:00
2210892 requested review from 2211945 2023-11-02 12:08:08 +01:00
2210892 commented 2023-11-02 12:09:18 +01:00
Poster
Collaborator

Falls man das iwie anders machen muss....kurze Rückmeldung.

Falls man das iwie anders machen muss....kurze Rückmeldung.
2211945 commented 2023-11-02 12:16:19 +01:00
Owner

Hey hey,
also so funktioniert das nicht ganz. Kartensoritert.put wirft ja an sich keine Exception und catchen brauchst du da auch keine. Es geht ja darum, dass du einen Fehlerfall erstellst. Die Frage ist, welcher wäre das dort. MIt der Exception würd ich sagen, dass keine Karte von dem Kartenstapelsortiert gezogen werden kann. also if(kartenstapelsortiert.contains("Karte bliblablub")) -> dann mach irgendwas, ansonsten throw new EmptyKartenStapelException

Hey hey, also so funktioniert das nicht ganz. Kartensoritert.put wirft ja an sich keine Exception und catchen brauchst du da auch keine. Es geht ja darum, dass du einen Fehlerfall erstellst. Die Frage ist, welcher wäre das dort. MIt der Exception würd ich sagen, dass keine Karte von dem Kartenstapelsortiert gezogen werden kann. also `if(kartenstapelsortiert.contains("Karte bliblablub")) -> dann mach irgendwas, ansonsten throw new EmptyKartenStapelException`
2210892 closed this pull request 2023-11-07 15:54:49 +01:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
2211945
Labels
Clear labels   
bug

Something is not working   
duplicate

This issue or pull request already exists   
enhancement

New feature   
help wanted

Need some help   
invalid

Something is wrong   
question

More information is needed   
wontfix

This won't be fixed
No Label
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: 2211945/WIZARD_PR2_DOP#42
There is no content yet.
Delete Branch "kts.block"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?