3013379
/
OnlineShop
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Warenkorb-Klasse enthält jede Menge UI #1

New Issue
Open
opened 2024-10-22 17:12:49 +02:00 by hummel · 2 comments
hummel commented 2024-10-22 17:12:49 +02:00
Collaborator

bd313a53d3/src/de/hs_mannheim/informatik/domain/ShopVerwaltung.java (L11)

Bitte darauf achten, dass KEINE Scanner und KEINE Sysouts in Domänenklassen gemacht werden!

https://gitty.informatik.hs-mannheim.de/3013379/OnlineShop/src/commit/bd313a53d3ef4e82a5e29a4690d6aa4c57d48d4f/src/de/hs_mannheim/informatik/domain/ShopVerwaltung.java#L11 Bitte darauf achten, dass KEINE Scanner und KEINE Sysouts in Domänenklassen gemacht werden!
3013379 commented 2024-10-22 20:17:13 +02:00
Owner

Scanner und Sysouts wurden aus der Domänenklasse entfernt und zur main Methode hinzugefügt.

Scanner und Sysouts wurden aus der Domänenklasse entfernt und zur main Methode hinzugefügt.
hummel commented 2024-10-23 18:00:53 +02:00
Poster
Collaborator

Hallo Herr Heil,

danke schön, bitte achten Sie darauf künftig sauberen Code zu erstellen. Ich würde bpsw. auch empfehlen, Ihre UI in eine entsprechende Klasse mit einzelnen Methoden pro "Screen" zu packen und nicht alles in der Main-Methode zu haben.

Ich werde Ihnen das Testat als bestanden eintragen, darf Sie aber gleichzeitig dringend bitten, weiter am Ball zu bleiben und künftig sorgfältiger zu arbeiten. Noch einmal werden wir nicht so großzügig sein.

Freundliche Grüße

Oliver Hummel

Hallo Herr Heil, danke schön, bitte achten Sie darauf künftig sauberen Code zu erstellen. Ich würde bpsw. auch empfehlen, Ihre UI in eine entsprechende Klasse mit einzelnen Methoden pro "Screen" zu packen und nicht alles in der Main-Methode zu haben. Ich werde Ihnen das Testat als bestanden eintragen, darf Sie aber gleichzeitig dringend bitten, weiter am Ball zu bleiben und künftig sorgfältiger zu arbeiten. Noch einmal werden wir nicht so großzügig sein. Freundliche Grüße Oliver Hummel
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
master
main
Labels
Clear labels
No items
No Label
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: 3013379/OnlineShop#1
There is no content yet.
Delete Branch "%!s(<nil>)"

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?