Ist das schlimm wenn man ein Script/Code komplizierter schreibt als es sein könnte?

Hallo,

Ich bin Programmieranfänger und habe eine Frage bezüglich dem Titel und zwar habe ich um mehr Praxiserfahrung zu sammeln ein simplen Kalkulator für die Konsole programmiert (in Java). Dieser ist jedoch spürbar komplizierter geworden als es sein müsste.

import java.util.Objects;
import java.util.Scanner;

public class Main {

    public static void main(String[] args) {
        calculate();
    }
    public static void calculate() {

        System.out.println("Rechenart:");
        Scanner scanner = new Scanner(System.in);
        String arithmeticMethod = scanner.next();

        if (Objects.equals(arithmeticMethod, "addition") || Objects.equals(arithmeticMethod, "+")) {
            addition();
        } else if (Objects.equals(arithmeticMethod, "subtraction") || Objects.equals(arithmeticMethod, "-")) {
            subtraction();
        } else if (Objects.equals(arithmeticMethod, "multiplication") || Objects.equals(arithmeticMethod, "*") || Objects.equals(arithmeticMethod, "x")) {
            multiplication();
        } else if (Objects.equals(arithmeticMethod, "division") || Objects.equals(arithmeticMethod, ":") || Objects.equals(arithmeticMethod, "/")) {
            division();
        } else if (Objects.equals(arithmeticMethod, "stop")) {
            System.out.println("Program ended.");
        }
    }
    public static void addition() {

        Scanner scanner = new Scanner(System.in);

        int firstSummand;
        int secondSummand;

        System.out.println("1. Summand: ");
        firstSummand = scanner.nextInt();
        System.out.println("2. Summand: ");
        secondSummand = scanner.nextInt();
        System.out.println("Ergebnis: ");
        System.out.println(firstSummand + secondSummand);
        calculate();
    }
    public static void subtraction() {

        Scanner scanner = new Scanner(System.in);

        int minuend;
        int subtrahend;

        System.out.println("Minuend: ");
        minuend = scanner.nextInt();
        System.out.println("Subtrahend: ");
        subtrahend = scanner.nextInt();
        System.out.println("Ergebnis: ");
        System.out.println(minuend - subtrahend);
        calculate();
    }
    public static void multiplication() {

        Scanner scanner = new Scanner(System.in);

        int firstFactor;
        int secondFactor;

        System.out.println("1. Factor: ");
        firstFactor = scanner.nextInt();
        System.out.println("2. Factor: ");
        secondFactor = scanner.nextInt();
        System.out.println("Ergebnis: ");
        System.out.println(firstFactor * secondFactor);
        calculate();
    }
    public static void division() {

        Scanner scanner = new Scanner(System.in);

        int dividend;
        int divisor;

        System.out.println("Dividend: ");
        dividend = scanner.nextInt();
        System.out.println("Divisor: ");
        divisor = scanner.nextInt();
        System.out.println("Ergebnis: ");
        System.out.println(dividend / divisor);
        calculate();
    }
}

(Entschuldigung dass ich es nicht über ein Bild zeigen kann da nicht alles auf ein Bild passt.) Ist das schlimm? Sollte ich mir das schnell abgewöhnen?

Schonmal Danke im voraus.

2 Antworten

Babelfish

Von gutefrage auf Grund seines Wissens auf einem Fachgebiet ausgezeichneter Nutzer

Programmieren & Softwareentwicklung, Software & Apps

13.10.2023, 13:45

Code sollte vor allem lesbar sein – und das auch noch in ein paar Wochen, wenn du nicht mehr genau weißt, was du da gemacht hast. Faustregel: So kurz wie möglich und so lang wie nötig. Sinnvolle Teilbereiche sollten in Funktionen oder Klassen ausgelagert werden, damit diese einfacher wiederverwendet oder geändert werden können.

Dein Code ist jetzt nicht zu lang und die einzelnen Berechnungen in Methoden auszulagern ist absolut in Ordnung. Persönlich hätte ich jetzt noch eine (oder mehrere) extra Methode(n) für die Dateneingabe gemacht aber das ist hier auch nicht unbedingt nötig. Beim Dividieren musst du aber noch die 0 abfangen.

Woher ich das weiß:Berufserfahrung – Entwickle Anwendungen für iOS, iPadOS und macOS beruflich.

Destranix

Von gutefrage auf Grund seines Wissens auf einem Fachgebiet ausgezeichneter Nutzer

Java, Programmiersprache, Programmieren & Softwareentwicklung

13.10.2023, 13:21

Dein Code ist an sich nicht sonderlich kompliziert, sondern sogar recht strukturiert und ordentlich.

Nur wirst du wegen der Endlosrekursion einen Stackoverflow bekommen wenn du zu oft Eingaben tätigst. Statt "calculate" in den Unterfunktionen aufzurufen bau einfach eine Schleife in der "calculate"-Funktion.

Weitere Anmerkungen:

Den Scanner solltest du einmal erstellen und wiederverwenden.
Die Eingaben sollten verifiziert werden.
Zum Parsen von Input bieten sich Parser und Parsergeneratoren an. (Für Fortgeschrittene.)
Kommentare hinzufügen.

Ähnliche Fragen

Alter Berechnen Java?

Hallo, also ich möchte das Alter berechnen in dem man in die Konsole sein Geburtstag eingibt und das mit der Jetzt genommenen Zeit abzieht. Davon sollte man die Differenz Berechnen.

ich weiß wo der Fehler in mein Code ist, aber kein Plan wie ich das ändern soll.

import java.util.Scanner;
import java.time.LocalDate;
import java.time.Duration;

public class Main {
    static Scanner scanner = new Scanner(System.in);

    public static double alter;

    public static void main(String[] args) {
        System.out.println("Hallo du hast dein Alter vergessen kein Problem.");
        System.out.println("Bitte schreib dein Geburtstag in xx-xx-xxxx Format in den Chat");
       double alter = scanner.nextDouble(); // DIe person schreibt sein alter in den chat
        LocalDate datum = LocalDate.now();  // Die Jetzt Zeit wird genommen
        Duration diff = Duration.between(datum, alter);
        System.out.println("Dein Geburtsdatum ist am " + diff);

    }
}

...zur Frage

Wie mache ich daraus ein Strucktogramm?

import java.util.Scanner;
public class ChristmasTreeMain {
  Scanner scanner = new Scanner(System.in);
  int Tage;
  public static void main(String args[]) {
    ChristmasTreeMain tannenbaum = new ChristmasTreeMain();
    tannenbaum.inputTage();
    tannenbaum.drawTop(tannenbaum.Tage);
    tannenbaum.drawMiddle(tannenbaum.Tage);
    tannenbaum.drawBottom(tannenbaum.Tage);
  }
  public void inputTage() {
    boolean isInputValid = false;
    do {
      System.out.println("Bitte geben Sie einen Tage zwischen 1 und 24 ein:");
      while (!scanner.hasNextInt()) {
        System.out.println("Ungueltige Eingabe, geben Sie eine Zahl zwischen 1 und 24 ein.");
        scanner.next();
      }
      Tage = scanner.nextInt();
      if (Tage >= 1 && Tage <= 24) {
        isInputValid = true;
      } else {
        System.out.println("Ungueltige Eingabe, geben Sie eine Zahl zwischen 1 und 24 ein.");
      }
    } while (!isInputValid);
    scanner.close();
  }
  public void drawTop(int Tage) {
    for (int i = 2; i <= Tage; i++) {
      System.out.print(" ");
    }
    System.out.print("ii");
    System.out.println();
  }
  public void drawMiddle(int Tage) {
    int Dots = 0;
    for (int Tag = 1; Tag <= Tage; Tag++) {
       
      if (Tag > 2) {
        Dots += 2;
      }
   
      if (Tag > 1) {
   
        for (int i = 0; i < Tage - Tag; i++) {
          System.out.print(" ");
        }
   
        System.out.print("i/");
        for (int i = 0; i < Dots; i++) {
          System.out.print(".");
        }
        System.out.print("\\i");
   
        System.out.println();
      }
    }
  }
  public void drawBottom(int Tage) {
    for (int r = 1; r <= Tage; r++) {
      System.out.print("--");
    }
    System.out.println();
    for (int l = 2; l <= Tage; l++) {
      System.out.print(" ");
    }
    System.out.print("||");
  }
}

...zur Frage

Java: Wie kann der Preis berechnet werden?

Der Kunde kann die Anzahl Kugeln wählen und ob er diese in einem Becher oder einem Biscuit möchte.

Der Code sieht folgendermaßen aus:

import java.util.Scanner;

public class Main {
  public static void main(String[] args) {
    Scanner keyScan = new Scanner(System.in);
    System.out.print("Becher oder Biscuit: ");
    String typ = keyScan.nextLine();
    System.out.print("Anzahl Kugeln: ");
    int kugen = keyScan.nextInt();

    // TODO: Preis berechnen und in der Variablen preis speichern.
    double preis = 0.0;
    System.out.println("Preis: " + preis);
    keyScan.close();
  }
}

...zur Frage

2D Array- Zeilensummen und Spaltensummen müssen die selben Summen haben. Wie prüfen?

Die Aufgabe sieht folgendermassen aus und ich komme einfach nicht weiter.

Wenn alle Spalten- und Zeilensummen gleich sind, ist es magisch.

Was muss im Code verändert werden?

Anbei der Code:
import java.util.Scanner;

public class Main {

	public static void main(String[] args) {
		int [][] quadrat = new int[3][3];
		Scanner keyScan = new Scanner(System.in);

		// Quadrat einlesen
		System.out.println("Bitte 9 Zahlen eingeben und mit Enter abschliessen");
		for (int i=0; i<3; i++) {
			for (int j=0; j<3; j++) {
				quadrat[i][j] = keyScan.nextInt();
			}
		}

		boolean magisch = true;
		int magischeZahl = 0;
		
		
		// TODO: Quadrat prüfen und die Variablen magisch und magischeZahl auf den richtigen Wert setzen. 

		
		// Ausgabe des Ergebnisses
		if (magisch) {
			System.out.println("Magisch: "+magischeZahl);
		} else {
			System.out.println("Nicht magisch");
		}
		keyScan.close();
	}
}

...zur Frage

Array jeden wert addieren?

package com.companyenelogy;

import java.lang.reflect.Array;
import java.util.Scanner;

public class Main {

    public static void main(String[] args){
        Scanner scanner = new Scanner(System.in);

        System.out.println("Geben sie ihr Array ein:");

        int[] array = new int[scanner.nextInt() ];

                System.out.print("Das Array beinhaltet folgende Zahlen: ");
        for (int i = 0; i < array.length; i++) {
            array[i] = i + 1;

        }
        for (int i = 0; i < array.length; i++) {
         
            System.out.print(array[i] + ",");
;
        }

        /*
        *Schreibe folgende Funktion: Jedem Wert im Array soll die 13 addiert werden (Verschlüsslung der Zahlen

ich kriege diese aufgäbe nicht hin ich muss jede Zahl in einen array mit 13 addieren aber wen ich was versuche wird nur die erste Zahl mit 13 addiert

...zur Frage

Java / Taschenrechner: Wie können die Variablen zusammen ausgegeben werden?

Hallo, kurz gesagt fehlt nur noch die Ausgabe der Antwort.

Hat jemand eine Idee, wie wir die Variablen alle zusammen ausgeben können?

Variablen:

erste_Zahl
zweite_Zahl
operator

import java.util.Scanner;
import javax.swing.JOptionPane;

public class Taschenrechner {
  public static void main(String[] args) {
    Scanner benutzereingabe = new Scanner(System.in);

    int operator;
    String ersteZahl = JOptionPane.showInputDialog("Bitte geben sie ihre erste Zahl ein: ");
    String zweiteZahl = JOptionPane.showInputDialog("Bitte geben sie ihre zweite Zahl ein: ");
    double erste_Zahl = Double.parseDouble(ersteZahl);
    double zweite_Zahl = Double.parseDouble(zweiteZahl);

    JOptionPane.showInputDialog("Bitte geben sie Ihren Rechenoperator ein: ");
    operator = benutzereingabe.next().charAt(0);

    if (operator == '+') {
      System.out.println(erste_Zahl + zweite_Zahl);
    }
    else if (operator == '-') {
      System.out.println(erste_Zahl - zweite_Zahl);
    }
    else if (operator == '*') {
      System.out.println(erste_Zahl * zweite_Zahl);
    }
    else if (operator == '/') {
      System.out.println(erste_Zahl / zweite_Zahl);
    }
  }
}

...zur Frage

Java : Warum wartet der Scanner nicht auf meine Eingabe?

Hallo,

Gebt mal folgenden Code bei euch ein und startet :

package Lernen;

import java.util.Arrays;
import java.util.Scanner;

public class CarSeller {

    int rColor = (int)(Math.random()*9);
    int color;
    int ps;
    int mark;
    String color2;
    String ps2;
    String mark2;
    String[] colorArray = new String[10];

    public void price(int color, int ps, int mark) {
        this.color = color;
        this.ps = ps;
        this.mark = mark;
    }

    public int RandomColor() {
        System.out.println(colorArray[rColor]);
        return 1;
    }

    public static void main(String[] args) {
        CarSeller acess = new CarSeller();
        acess.colorArray[0] = "Black";
        acess.colorArray[1] = "Green";
        acess.colorArray[2] = "Red";
        acess.colorArray[3] = "Orange";
        acess.colorArray[4] = "Blue";
        acess.colorArray[5] = "Cyan";
        acess.colorArray[6] = "White";
        acess.colorArray[7] = "Brown";
        acess.colorArray[8] = "Yellow";
        acess.colorArray[9] = "Grey";

        System.out.println("Hey, what do you want to do ? | BUYCAR | DRIVE | GOINHOUSE | BUYPLOTS | REFUEL | QUIT |");
        Scanner start = new Scanner(System.in);

        if (start.equals("BUYCAR")) {
            System.out.println("What car do you want to buy? | Lamborghini | Porsche | VW | Mercedes | BMW  | Tesla | CANCEL|");
            Scanner a = new Scanner(System.in);
            
            if (a.equals("Lamborghini")) {
                
                  CarSeller Lamborghini = new CarSeller();
                  System.out.println("Which color do you want ? | Choose one ");
                  System.out.println(Arrays.toString(acess.colorArray));
                  System.out.println("Or do you want a random color ? | Random");
                  Scanner colorchoose = new Scanner(System.in);
                  
                  if (colorchoose.equals("Random")) {
                      System.out.println(acess.RandomColor());
                  }


            }

        }
    }

}

Bei mir wartet der irgendwie nicht auf eine Eingabe... Wie behebe ich diesen Fehler ?

...zur Frage

Wie kann ich input.close() verwenden?

Hallo,

ich habe einen Rechner in Java programmiert, der (fünfmal) "wiederverwendbar" ist, ohne den Code neu zu starten. Nach diesen fünf Berechnungen, möchte ich, dass ein weiteres Programm ausgeführt wird, was aber nur möglich ist, nachdem ich

input.close()

verwende. Aber egal, wo ich

input.close()

platziere, gibt es einen Fehler.

Nun meine Frage: Wo muss ich

input.close()

hinschreiben, damit der Code funktioniert?

Ich bedanke mich schon einmal im Voraus.

import java.util.Scanner;

class Main {
  public static void main(String[] args) {
    int z = 0;

    while (z < 5) {
      z = z + 1;
      Scanner input = new Scanner(System.in);
      System.out.println("1. Zahl: ");
      double Zahl1 = input.nextDouble();
      System.out.println("Operator: ");
      char Operator = input.next().charAt(0);
      System.out.println("2. Zahl: ");
      double Zahl2 = input.nextDouble();

      switch (Operator) {
        case '+':
          double A1 = Zahl1 + Zahl2;
          System.out.println("=" + A1);
          break;
        case '-':
          double A2 = Zahl1 - Zahl2;
          System.out.println("=" + A2);
          break;
        case '*':
          double A3 = Zahl1 * Zahl2;
          System.out.println("=" + A3);
          break;
        case '/':
          double A4 = Zahl1 / Zahl2;
          System.out.println("=" + A4);
          break;
        default:
          System.out.println("Fehler");
      }
    }
  }
}

...zur Frage

Findet ihr den Taschenrechner gut?

Hey ich habe gerade einfach spontan nach paar Videos Java lernen zum ersten mal mein eigenes Programm erstellen wollen und hab mal versucht einen Taschenrechner der + - / * ausführen kann zu machen, findet ihr den Code ok?

import java.util.Scanner;
public class Calc {
    public static void main (String[] args){
        Scanner scan = new Scanner(System.in);
        int zahl;
        int operation;
        int zahl2;
        int ergebnis = 0;
        int wiederholung = 1;

        while (wiederholung == 1) {

            System.out.println("Tippen sie eine Zahl ein");
            zahl = scan.nextInt();
            System.out.println("Welche Operation möchten sie durchführen? - (0) + (1) / (2) * (3)");
            operation = scan.nextInt();
            if (operation == 0) {
                System.out.println("Mit welcher Zahl möchten sie subtrahieren?");
            } else if (operation == 1) {
                System.out.println("Mit welcher Zahl möchten sie addieren?");
            } else if (operation == 2) {
                System.out.println("Mit welcher Zahl möchten sie dividieren?");
            } else if (operation == 3) {
                System.out.println("Mit welcher Zahl möchten sie multiplizieren?");
            }
            zahl2 = scan.nextInt();
            if (operation == 0) {
                ergebnis = zahl - zahl2;
            } else if (operation == 1) {
                ergebnis = zahl + zahl2;
            } else if (operation == 2) {
                ergebnis = zahl / zahl2;
            } else if (operation == 3) {
                ergebnis = zahl * zahl2;
            }
            System.out.println("Hier ist ihr Ergebnis: " + ergebnis + " ...möchten sie weitere Berechnungen durchführen? ja (1) nein (2)");
            wiederholung = scan.nextInt();
            if (wiederholung == 2) {
                System.out.println("Programm wird beendet");
                break;
            } else if (wiederholung == 1) {
                System.out.println("Super!");
            }
        }

...zur Frage

Java: Wie kann ich bei einer Integer-Scanner-Eingabe whitespaces verbieten?

Hallo,

ich habe ein Primzahlberechnungs-programm geschrieben, bei der man eine Zahl eingibt und es sagen soll, ob es eine Primzahl ist. jedoch möchte bei eingaben wie "3 4 23 3" oder " 23 4 1" bei der whitespaces enthalten sind "ungültige Eingabe" angezeigt bekommen. Haben Sie eine Idee wie wie man prüft, ob die Integer-Eingabe whitespaces enthält?

Code:

import java.util.Scanner;

public class Primzahl {
        public static void main(String[] args)
        {
                System.out.println("Gib ne Zahl");
                Scanner scanner = new Scanner(System.in);
                Integer Zahl = scanner.nextInt();

                for (int i = 2; i < Zahl; i++)
                {
                        if (Zahl % i == 0)
                        {
                                    System.out.println(Zahl + " ist keine Primzahl");
                                    break;
                            }
                            else
                            {
                                    System.out.println(Zahl + " ist eine Primzahl");
                                    break;
                            }
                    }
            }
}

...zur Frage

Java: Methode wird nicht erkannt?

Hallo, ich arbeite gerade an einem Text Adventure. Intellij erkennt meine Methoden nicht. Hat jemand Lösungsvorschläge?

Code:

public class main {
    public static void main (String[] args){
        player();
        enemies();
        cases();
    }
    public static void player() {
        String name = "Shadow";
        int health = 50;
        int damage = 30;
    }
    public static void enemies() {
            public static void dragon() {
                String name = "Barbaton";
                int health = 100;
                int damage = 40;
            }
            public static void lion() {
                String name = "Eon";
                int health = 85;
                int damage = 55;
            }
            public static void pig() {
                String name = "Memba";
                int health = 20;
                int damage = 5;
            }
            public static void frog() {
                String name = "Zee";
                int health = 5;
                int damage = 1;
            }
            public static void zebra() {
                String name = "Zoy";
                int health = 28;
                int damage = 18;
            }
            public static void monkey() {
                String name = "Bart";
                int health = 45;
                int damage = 38;
            }
            public static void fox() {
                String name = "Foxie";
                int health = 35;
                int damage = 24;
            }
        }
    }
    public static void cases() {
        int min = 1;
        int max = 8;
        int random_int = (int)Math.floor(Math.random()*(max-min+1)+min);
        if(random_int == 1){
            System.out.println("Du hast Barbaton(Drache) als Gegner bekommen.");
        }
        else if (random_int == 2){
            System.out.println("Du hast Eon(Löwe) als Gegner bekommen.");
        }
        else if (random_int == 3){
            System.out.println("Du hast Memba(Schwein) als Gegner bekommen.");
        }
        else if (random_int == 4){
            System.out.println("Du hast Zee(Frosch) als Gegner bekommen.");
        }
        else if (random_int == 5){
            System.out.println("Du hast Zoy(Zebra) als Gegner bekommen.");
        }
        else if (random_int == 6){
            System.out.println("Du hast Bart(Affe) als Gegner bekommen.");
        }
        else if (random_int == 7){
            System.out.println("Du hast Foxie(Fuchs) als Gegner bekommen.");
        }
        else {
            System.out.println("");
        }
    }
}

...zur Frage

Java Scanner Buchstaben einlesen, wie?

Hallo zusammen!!!

Ich bin gerade an einem Programm, welches mir einen Buchstabencode einliest und mir dann eine Zahl ausdruckt.

Bsp: Ich gebe abc ein und das Proramm druckt mir die Zahl 0 aus.

Folgenden Code habe ich ausprobiert, jedoch irgendwie liesst er mir die Buchstaben nicht ein...

public class Zahlenerkennung {
	public static void main(String[] args) {
		Scanner myConsole = new Scanner(System.in);
		int adder = myConsole.nextInt();
		char abc = 0 ;
    if (adder == abc)
        System.out.println(0);

Muss man da anderst vorgehen, respektieve mit einem anderen Scanner?

Vielen Dank!!! :DDD

...zur Frage

Wie kann ichWerte einer Schleife in einem Array speichern?

Ich versuche verzweifelt raus zu finden, wie man Werte von einer Schleife in einem Array speichert. Ich hoffe mir kann jemand helfen.

import java.util.Scanner;

public class Aufgabe51 {

public static void main(String[] args) {
    Scanner scn= new Scanner(System.in);
   
    int zaehler = 0, ergebnis;
   
   
   
    for(int i= 0; i<10; i++)
    {
    System.out.println("Geben Sie die Zahl ein");
    zaehler = scn.nextInt(); 

    }
   
    int [] ergebnis1 = {zaehler};

...zur Frage

Java Rechenaufgabe?

import java.util.Scanner;
public class Zusammenrechnen {
  public static void main(String[] args) {
    Scanner scan = new Scanner(System.in); int zahl = scan.nextInt();
  while(zahl > 0) {
    zahl = scan.nextInt();
  }
  if(zahl < 0) { System.out.println("Programm beendet"); }
  scan.close();
  }
}

Frage: Was muss ich ändern, damit das Programm die Summe der eingegebenen Zahlen (ohne die letzte, negative Zahl) angibt?😕

...zur Frage

Was möchtest Du wissen?

Deine Frage stellen