Exceptions (try/catch)

[hüteüberhüte]

@maki: Ist es ok für Dich, Personen ADHS zu unterstellen? Für mich jedenfalls nicht. Das gehört nicht zum Thema, das gehört nicht hierher

Die Anzahl an Zeilen hat mal nix mit Performance zu tun

Na, das ist ja mal eine ganz neue Erkenntnis... [/IRONIE] Hab ich auch nicht behauptet. Viel besser (schneller) als das Ziehen der Lottokugeln oben kann es aber eig. gar nicht gehen...

Code der von oben nach unten durchläuft ist nicht lesbar sondern eher Spaghetticode, daher lass es mal gut sein.

Code, der also zwischen Methoden immer hin und her springt, oder Programmlogik ausschließlich in der main-Methode ist also deiner Meinung nach kein "Spaghetticode"? Imho bist du dir über den Begriff noch nicht ganz im Klaren.

Bei einer Auswahl von 1 bis 7 wird man um eine Fallunterscheidung gar nicht drumrum kommen...

Für jede Zahl ne eigene Methode zu machen..sorry.. epic fail!

Überleg doch mal, was du da schreibst *kopfschüttel*. Andernfalls würde es bedeuten, du schreibst alles in eine Methode. Dann wirds erst recht unübersichltich...

@all: Afaics hat von euch bis jetzt noch niemand einen besseren Vorschlag gemacht... (außer einer Klasse für das Einlesen, die aber auch darauf beruht, im Ausnahmefall ein Default-Value zurückzugeben...)

Würde mich freuen, wenn der TO zurückkehrt und auch noch etwas dazu schreibt, was er nehmen würde.

 

[maki]

@maki: Ist es ok für Dich, Personen ADHS zu unterstellen? Für mich jedenfalls nicht. Das gehört nicht zum Thema, das gehört nicht hierher

Habe mich selber schon gefragt was mit dir nicht stimmt...

Es steht dir nicht zu Leuten Sperren anzudrohen PUNKT.
Da entscheidest du nicht im geringsten, empfehle dir dringend mehr Zurückhaltung.
Beim nächsten mal hast du wieder eine Sperre.

Ansonsten solltest du dir nicht mal anmassen überhaupt Code zu bewerten, da du offensichtlich von der Materie (SW Entwicklung, Java, sauberer Code, etc. pp.) keine Ahnung hast.

 

[hüteüberhüte]

@maki: Naja, dann sperr mich doch gleich. Ich halte mich jedenfalls nicht zurück, wenn irgendjemand mir gegenüber beleidigend wird, und das vom Team einfach ignoriert wird. Dann ist es für mich auch egal, hier weiter angemeldet zu sein... Also dann zeig mal, was du für ein genialer Moderator bist.

 

[ThreadPool]

[...]
Ansonsten solltest du dir nicht mal anmassen überhaupt Code zu bewerten, da du offensichtlich von der Materie (SW Entwicklung, Java, sauberer Code, etc. pp.) keine Ahnung hast.

Es steht dir frei besseren Code zu posten. Dann trägst du wenigstens was zur Diskussionsgrundlage bei. Latente Beleidigungen gehören nicht dazu. Und klar hier im Forum herrscht keine Demokratie aber genausowenig wünsche ich mir eine makitatur.

Davon abgesehen finde ich das Thema interessant, da nur sehr wenige Code gepostet haben der sich teilweise auch recht ähnlich ist, bis auf Fancy dessen Teillösung man schon wieder fast in die Ecke "over-engineering" stopfen könnten. Ich warte gespannt auf weitere gehaltvollere Beiträge.

 

[Spacerat]

@Hüte: Das war keine Unterstellung, sondern schlicht 'ne Frage und diese war durchaus ernst gemeint. Aber egal, von nun an bekomme ich deine Beiträge ohnehin nur noch in Zitaten anderer mit, sofern es welche gibt. Mir erspart das jede Menge unnütze Flames und dir möglicherweise die Sperrung. Evtl. beherzigst du das mal, wenn es sich erneut ergibt, dass ein Schuss nach hinten los geht.

 

[maki]

Es steht dir frei besseren Code zu posten. Dann trägst du wenigstens was zur Diskussionsgrundlage bei. Latente Beleidigungen gehören nicht dazu. Und klar hier im Forum herrscht keine Demokratie aber genausowenig wünsche ich mir eine makitatur.

Dir steht es frei deine Meinung zu äussern, diese muss aber nicht notwendigerweise Sinn machen..

Deine einzige bisherige Meinungsäusserung in diesem Thread kam anscheinend erst durch meinen Post(?), also erspar mir deine "trägt zur Diskussionsgrundlage bei" gerede wenn du das nicht ernst meinst.

Grund warum ich hier gepostet habe war dass gleich mehrere Meldungen von hth bezüglich einer angeblichen Beleidung kamen, zus. droht er anderen Usern mit sperren und dann bewertet er auch noch..

Dir muss nicht gefalen was ich zu sagen habe, kannst du das gerne "makitatur" oder sonstwie nennen, Tatsache ist aber dass deine Meinung zu bestimmten Themen irrelevant ist.

 

[ThreadPool]

[...]also erspar mir deine "trägt zur Diskussionsgrundlage bei" gerede wenn du das nicht ernst meinst.[...]

Maki, Kritik üben ist einfach, kritikfähig sein nicht. Und natürlich meinte ich das in Bezug auf die zitierte Passage von dir ernst. Du übst fachliche Kritik ohne konstruktive, anderslautende Vorschläge, wie so viele Andere auch (scheint heutzutage Gewohnheit zu sein). Aber Man lernt ja nur von Besseren, und wir wollen hier alle lernen (zumindest ich). Und du hast dich in deiner Aussage über jmden gestellt und ich
frage nun dich nach einer besseren Lösung.

 

[schalentier]

Koennt ihr das bitte direkt per PM klaeren oder woanders. Das stoert hier doch nur.

Weiss gar nich was das Problem ist, der Code huete ist doch nicht schlecht. Einige Anmerkungen dazu hab ich aber:

- Ich wuerde die Methoden nicht durchnummerieren, sondern sinnvolle Namen geben.
- Ich wuerde die IOException schnell fangen (in der readInt) und in eine RuntimeException gewrappt weiterwerfen ([c]...catch(IOException e) { throw new RuntimeException(e); }[/c]).
- Mach das komische Label weg. Einfach wegmachen und direkt vergessen. Imho is das dort sogar nen Bug, weil man das Programm so nicht beenden kann... ;-)

 

[Spacerat]

Maki, Kritik üben ist einfach, kritikfähig sein nicht. Und natürlich meinte ich das in Bezug auf die zitierte Passage von dir ernst. Du übst fachliche Kritik ohne konstruktive, anderslautende Vorschläge, wie so viele Andere auch (scheint heutzutage Gewohnheit zu sein). Aber Man lernt ja nur von Besseren, und wir wollen hier alle lernen (zumindest ich). Und du hast dich in deiner Aussage über jmden gestellt und ich frage nun dich nach einer besseren Lösung.

Jetzt kriegt ihr euch nicht auch noch in die Haare... ist ja nicht auszuhalten.

Ich bin zwar nicht gefragt, aber wenn es so wär' kann ich dir zumindest einen Grund nennen, warum "konstruktive, anderslautende Vorschläge" des öfteren von dem ein oder anderen ausbleiben, nämlich wenn die Lösung schlicht "ändere dies!" lautet. Muss dass etwa immer betont werden?

 

[Noctarius]

- Mach das komische Label weg. Einfach wegmachen und direkt vergessen. Imho is das dort sogar nen Bug, weil man das Programm so nicht beenden kann... ;-)

Das war auch das Erste was ich heute morgen gesehen habe, bevor ich mit gutem Gewissen die Meldung ignorierte

 

[hüte]

Dann sollte man vllt. anstatt des Labels eine boolean-Variable einführen. Mir ist bekannt dass Sprungmarken evil sind und Dijkstra sie ablehnte (Go To Statement Considered Harmful), aber in diesem Fall würde das mMn den Code unnötig aufblähen. Aber vielen Dank für ernst gemeinten Vorschläge / konstruktive Kritik. Ich denke ich mache einfach mal ein paar Tage Pause, nachdem hier so ein Flame entstanden ist. Byebye

 

[Guest2]

@Schalentier: Sein Code ist insofern nicht schlecht, als dass er vermutlich funktioniert (ich habe es nicht ausprobiert). Allerdings ist es auch ein rein prozeduraler Ansatz. Mit leicht anderer Syntax könnte sein Code genauso plain C oder Pascal sein.

Der TO zeigt in seinem Code jedoch das er zumindest erste objektorientierte Ansätze verfolgt. Und da Java weder C noch Pascal ist, halte ich es auch für sinnvoll diese Ansätze weiter auszubauen. Natürlich kann Clean Code und OOP schnell übertrieben wirken, gerade hier im Anfängerbereich muss das aber imho nicht schlecht sein. Es später hinhunzen, wenn die Zeit drängt, geht immer noch. (Mach ich auch )

Viele Grüße,
Fancy

 

[schalentier]

Also das vom TO ist doch fast genau der gleiche Code, nur dass dort extra Klassen fuer eine Liste von Tipps und eine Liste von Ziehungen erstellt wurden (was ich fuer ungut halte - zumindest hab ich soeben deutlich laenger gebraucht, bis ich das begriffen hab). Zu OOP gehoert auch Wiederverwenden von Klassen oder Sprachkonstrukten, nicht nur das Erzeugen moeglichst vieler Klassen und wilder Klassenhierarchien.

Eventuell koennte man tatsaechlich fuer Tipp und Ziehung Klassen einfuehren, find aber auch das hier eher uebertrieben.

Am Ende haengt es aber von der Aufgabenstellung ab.

 

[Noctarius]

Wieso nicht einfach mit return aus der Methode springen wenn das Programm beendet werden soll?