Hier ein paar kleine Tipps für dein Programm:
Versuche dich an Konventionen einer Programmiersprache zu halten, wenn du eines Tages einen solchen Job ausführt wird das von dir erwartet und es gestaltet den Code übersichtlicher für dich und jeden der ihn liest.
- Klassen in Java fangen mit Großbuchstaben an
- Variablen in Java nutzen lowerCamelCase, d.h. ein kleiner Anfangsbuchszabe, alle weiteren zusammengesetzten Wörter werden mit einem Großbuchstaben begonnen.
- Konstanten werden in Caps geschrieben
- Versuche aussagekräftige Variablennamen zu nutzen, ich habe z.B. keine Ahnung was die Variable lol macht wenn ich ihren Namen lese
Dein switch(wasmachensettings) kannst du simplifizieren zu
if(wasmachensettings.toLowerCase().equals("kindersicherung an"){
firstGame.Settings.kindersicherung = "An";}
Mir ist nicht bewusst warum du do while Loops nutzt? Die Ausgabe soll doch nur einmal erfolgen.
Ich habe mal deinen Code ein bisschen überarbeitet, das ist nach wie vor kein guter Code, aber ich habe einiges an Redundanz entfernt. Wenn du dazu Fragen hast, nur zu und viel Spaß beim Programmieren. :)
package org.example;
import java.util.Scanner;
public class MainGamev2 {
public static void main(String[] args) {
Scanner scan = new Scanner(System.in);
String action;
String aktposi = "startmenu";
System.out.println("Which language do you want to use?");
Settings.sprache = scan.next();
switch (Settings.sprache.toLowerCase()) {
case "deutsch":
do {
System.out.println("Start");
System.out.println("Einstellungen");
System.out.println("Mitwirkende");
System.out.println("Ende");
action = scan.next();
if ("start".equals(action.toLowerCase())) {
aktposi = "game";
System.out.println("Okay dann los gehts");
action = scan.next().toLowerCase();
if (action.equals("back") || action.equals("zurück")) {
aktposi = "startmenu";
}
} else if ("einstellungen".equals(action.toLowerCase())) {
aktposi = "settings";
System.out.println("|Einstellungen|");
System.out.println("Kindersicherung: " + Settings.kindersicherung);
System.out.println("Sprache: " + Settings.sprache);
System.out.println("zurück");
action = scan.nextLine();
if(action.toLowerCase().equals("kindersicherung an"))
Settings.kindersicherung = "An";
}//switch menu zu
}//do while zu
while (aktposi.equals("startmenu") || aktposi.equals("settings") || aktposi.equals("game") || aktposi.equals("mitwirkende"));
System.out.println("Spielende...");
break;
}//switch sprache zu
}//public static void zu
}//public class zu