[Anfänger] Kann mal wer drüber gucken?

[GoodGuy]

Hallo JavaRaner,

ich habe vor einigen Tagen angefangen Java zu lernen und wollte mein Können an einem kleinen Projekt erproben; es läuft auch soweit. Allerdings fehlt mir das Wissen um "Schwachstellen" zu finden bzw Ansätze wie man es besser machen könnte.

Es geht darum X Würfel mit X Seiten zu würfeln und die Ergebnisse auszuwerfen, bzw sollte es die niedrigste oder höchste Zahl sein, diese zu makrieren.

http://pastebin.com/L0CRKz5U

Besonders unsicher bin ich mir bezüglich des "try" und "catch", womit ich eine Fehlerausgabe verhindert wollte bei Eingabe einer negativen Zahl (was bei einer von zwei Variablen auch klappt). Außerdem finde ich es komisch wieso ich keine "if else" setzen kann ohne einen Fehler zu erhalten, weshalb ich alles mit "if" gemacht habe.

Ich hoffe, dass jemand Zeit findet es sich anzugucken

PS: Gibt es eine bessere Möglichkeit um das "," zu setzen zwischen den Ergebnissen? Ich würde gerne alle Auswürfe in der Config editierbar machen und dafür müsste es irgendwie in einen String passen, oder?

 

[Wirago]

Hi,
also das try-catch benötigst du nicht, da reicht das if. Normals würdest du die Prüfung aber direkt nach der Eingabe machen, damit der User die Eingabe wiederholen muss.

Das mit dem "," verstehe ich nicht ganz. Wenn du das ein einer Variable haben willst, dass mach das doch und gib nur die Variable aus.

Was mir so ad hoc aufgefallen ist:

boolean schutz=false
return schutz
else
boolean schutz=true;
return schutz;

mach doch gleich return false bzw return true. Und statt

boolean schutz=schutz(würfel,seiten);
if(schutz==true)

kannst auch if(schutz(würfel,seiten)) machen

 

[Joose]

Kleiner Tipp: Du kannst Code auch direkt bei uns im Forum posten
[JAVA] /* dein Code */ [/JAVA]

 

[GoodGuy]

Vielen Dank für die Hinweise, ich werde morgen versuchen sie umzusetzen und melde mich dann nochmal