Stilllegung des Forums
Das Forum wurde am 05.06.2023 nach über 20 Jahren stillgelegt (weitere Informationen und ein kleiner Rückblick).
Registrierungen, Anmeldungen und Postings sind nicht mehr möglich. Öffentliche Inhalte sind weiterhin zugänglich.
Das Team von spieleprogrammierer.de bedankt sich bei der Community für die vielen schönen Jahre.
Wenn du eine deutschsprachige Spieleentwickler-Community suchst, schau doch mal im Discord und auf ZFX vorbei!
Werbeanzeige
Community-Fossil
Dieser Beitrag wurde bereits 3 mal editiert, zuletzt von »DeKugelschieber« (16.03.2015, 14:59)
Es ist ja keine Schande etwas falsch zu machen, als Programmierer tu ich das täglich, [...].
Community-Fossil
Bei Punkt 1 wären wir schon mal beim "Javaland" in dem es nur Nomen gibt. Mein C++ Gehirn hatte an der Stelle auch erstmal keine Klasse angelegt, sondern wie du schon sagst, einfach mit einer Nummer (echten Daten) gearbeitet. Die Klasse wurde dann auf Wunsch hin eingebaut.
- Es gibt eine Klasse für einen "Getränkewunsch". Das kann man (in diesem Fall) vereinfachen, da der "Getränkewunsch" nichts weiter als ein Getränk (beschrieben durch das Fach, in welchem es lagert) ist. Am Ende würde man also das Getränk/das Fach des Getränks (oder dessen ID/Nummer) verwenden, statt eine Klasse dafür anzulegen.
- Ein "Münzschacht" und ein "Getränkefach" sind meiner Meinung nach unterschiedliche Dinge. Ich bin mir nicht sicher, was mit diesen Beiden Dingen genau gemeint ist, allerdings kann ich mir anhand des Namens nicht vorstellen, dass sie an irgendeiner Stelle anhand ihrer gemeinsamen Basisklasse (also polymorph) verwendet werden würden.
- Diverse Exceptions erscheinen mir dem Namen nach nicht unbedingt sinnvoll. Wenn der Benutzer bspw. zu wenig Geld eingeworfen hat, um ein Getränk zu erwerben, dann schalten die meisten Automaten auf einen anderen Zustand um, statt die Eingabe (Getränkauswahl) zu verweigern.
- Diverse Setter erscheinen mir überflüssig zu sein, da diese sehr wahrscheinlich nur ein einziges Mal aufgerufen werden (für Getraenkefach.setPreis(int) würde ich das annehmen). Stattdessen wäre ein weiterer Parameter im Konstruktor und ein Verzicht auf den Setter besser.
- Getraenkewunsch.getraenkeFachNummer, Fach.inhaltLimit etc. könnte genausogut auf final und public gesetzt werden, wodurch der Getter nicht mehr erforderlich wäre.
- Allgemein wird scheinbar einfach probiert, eine Aktion durchzuführen und auf eine Exception zu warten, statt vorher zu prüfen, ob die Operation überhaupt möglich ist. Das ist nur an solchen Stellen nicht der Fall, an denen mit Hilfe des eigenen Code geprüft werden _muss_ (bspw. weil sonst das Fachlimit überschritten werden würde).
Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von »DeKugelschieber« (05.03.2015, 13:26)
Community-Fossil
Beruf: Teamleiter Mobile Applikationen & Senior Software Engineer
Community-Fossil
Oftmals stellt sich auch die Frage danach, wie lange die Technologie die man verwendet offiziell Support hat, solange die Software im Einsatz ist.
Community-Fossil
Dieser Beitrag wurde bereits 2 mal editiert, zuletzt von »DeKugelschieber« (05.03.2015, 14:55)
Community-Fossil
Beruf: Teamleiter Mobile Applikationen & Senior Software Engineer
Das ist dann ein Zeichen für eine schlechte Firma, wenn die Idioten hochgelobt werden. Sorry.Immer merken, dass im echten Leben Architekten und Projektleiter keine Ahnung haben. Sonst würden sie auch nicht so einen Job machen. Da sind meistens die Aussortierten.
Werbeanzeige