Code kürzen

[Jonsered]

Hallo Leute, ich habe ein Programm ( Hausaufgabe ) erstellt und es macht, was es soll

Aber ...

Kann man den Code nicht irgendwie kürzen? Ich wiederhole mich sehr oft, zB oft den gleichen System.out.printl Text...

Fällt euch sonst noch was auf, wie man das ganze "optimieren" könnte? Super wäre es, wenn ihr den Code gleich mit rein schreibt, ich lerne einfacher und besser, wenn ich es sehe und dann auf ein andere Bsp anwende.

Vielen Dank

public class Bsp03 {

   static double flaeche;
   static final double PI = 3.141592653589793;
   static boolean leapYear;

   public static void main(String[] args) {
   
      int inMenue;
      int year;
      int n;
      double inRadius;
      
      System.out.println ("Please choose calculation");
      System.out.println ("1. Calculate circle area");
      System.out.println ("2. Leap year calculation");
      System.out.println ("3. Calculate next N leap years");
      System.out.println ("0. Exit");
   
      inMenue = SavitchIn.readLineInt();
   
      while ( inMenue != 0 ) {
      
         if ( inMenue < 0 || inMenue > 3 ) { // Eingabe wird ueberprueft
            System.out.println ("Invalid choice!");
            System.out.println ();
            System.out.println ("Please choose calculation");
            System.out.println ("1. Calculate circle area");
            System.out.println ("2. Leap year calculation");
            System.out.println ("3. Calculate next N leap years");
            System.out.println ("0. Exit");
         
            inMenue = SavitchIn.readLineInt();     
         }
      
         if ( inMenue == 1 ) {
            System.out.println ("Please enter circle radius:");
            inRadius = SavitchIn.readLineDouble();
            getCircleArea(inRadius);
            System.out.println ("Circle area is: " + flaeche);
            System.out.println ();
            System.out.println ("Please choose calculation");
            System.out.println ("1. Calculate circle area");
            System.out.println ("2. Leap year calculation");
            System.out.println ("3. Calculate next N leap years");
            System.out.println ("0. Exit");
         
            inMenue = SavitchIn.readLineInt();
         }
      
         if ( inMenue == 2 ) {
            System.out.println ("Please enter year:");
            year = SavitchIn.readLineInt();
            isLeapYear(year);
         
            if ( leapYear == true ) {
               System.out.println ( year + " is a leap year");
            }
            
            else {
               System.out.println ( year + " is not a leap year");
            }
         
            System.out.println ();
            System.out.println ("Please choose calculation");
            System.out.println ("1. Calculate circle area");
            System.out.println ("2. Leap year calculation");
            System.out.println ("3. Calculate next N leap years");
            System.out.println ("0. Exit");
         
            inMenue = SavitchIn.readLineInt();                 
         }
      
         if ( inMenue == 3 ) {
            System.out.println ("Please enter year:");
            year = SavitchIn.readLineInt();
            System.out.println ("How many leap years should be calculated:"); 
            n = SavitchIn.readLineInt();
            getNextLeapYears(year, n);
            System.out.println ();
            System.out.println ();
            System.out.println ("Please choose calculation");
            System.out.println ("1. Calculate circle area");
            System.out.println ("2. Leap year calculation");
            System.out.println ("3. Calculate next N leap years");
            System.out.println ("0. Exit");
            
            inMenue = SavitchIn.readLineInt();
         }
      }   
   
   }

// Methoden

   public static double  getCircleArea (double radius) {
      flaeche = PI * radius * radius;
      return flaeche;
   }
   
   public static boolean isLeapYear(int year) {
      if (((year % 4 == 0) && (year % 100 != 0)) || (year % 400 == 0)) {
         leapYear = true;
      }
      
      else {
         leapYear = false;
      }   
   
      return leapYear;
   }   
   
   public static void getNextLeapYears(int year, int n) {
      int i = 0;
      
      while (i < n) {
         isLeapYear(year);
         
         if (leapYear == true) {
            System.out.print ( year + " ");
            i++;
         }
            
         year++;
      }
   }
     
}

 

[Joose]

         isLeapYear(year);
         if (leapYear == true) {

Die Methode "isLeapYear" gibt dir ein boolean zurück, dieses wird von dir aber nirgends gespeichert.
Entweder entfernst du den Rückgabewert oder du speicherst den Rückgabewert und verwendest diesen.

Ansonsten ist anzumerken: Eine if-Abfrage erwartet einen boolschen Wert -> true oder false. leapYear ist boolscher Wert, dieser kann nur true oder false sein.
Eine extra Prüfung auf "leapYear == true" ist unnötig. Willst du auf "false" prüfen verwende "!leapYear".

if(leapYear == true) {
}
....
// Die Prüfung auf == true ist hier unnötig
if(leapYear) {
}
....
// Die Methode "isLeapYear()" gibt dir ein boolean zurück dieses kannst du auch gleich zur auswertung verwenden
if(isLeapYear(year)) {
}

In jeder if-Abfrage fragst du den User ob er nochmal etwas berechnen will. Das könntest du auch ans Ende der Schleife verschieben und statt 4x nur 1x stehen haben

 

[chuxXo]

Anstatt deinen ganzen "if (inMenue == ***) könntest du die kontrollstruktur "switch" verwenden.

if (inMenue != 0){
     switch (inMenue){
     case 1: break;
     ..........
     }
}

 

[Joose]

Anstatt deinen ganzen "if (inMenue == ***) könntest du die kontrollstruktur "switch" verwenden.

Worin besteht der Vorteil eines switch gegenüber der if/else Anweisungen? Bzw. inwiefern würde dies den Code kürzen?
Ein switch hat schon seine Berechtigung, aber ich würde ein if/else einen switch immer vorziehen.


EDIT:
Wenn du den Code auch besser formtierst sparst du ebenfalls Zeilen:

if (Anweisung) {
 ....
} else if (...) {
 ....
} else {
 ....
}

 

[Thallius]

Alles was man öfter als zwei mal benötigt sollte man in eigene Methoden auslagern. In Deinem Beispiel kannst du z.B. die ganze Eingabe der Zahl mit all Ihren print() in eine eigeneMethode stecken und diese aufrufen.

Gruß

Claus

 

[chuxXo]

Worin besteht der Vorteil eines switch gegenüber der if/else Anweisungen? Bzw. inwiefern würde dies den Code kürzen?
Ein switch hat schon seine Berechtigung, aber ich würde ein if/else einen switch immer vorziehen.
[/code]

Bin davon ausgegangen, dass es switch nicht umsonst gibt und eigentlich für solche Fälle dann ja perfekt zu verwenden ist. Och Joose, du hast immer ein Gegenargument

Das was mir noch auffällt, ist, dass du deine Variablen zusammen erstellen könntest. Durch Kommas getrennt. Und statt den ganzen println könntest du ein printf verwenden.

EDIT:
Aber warum sollte man if vorziehen ?

 

[Joose]

Bin davon ausgegangen, dass es switch nicht umsonst

Stimmt, switch gibt es auch nicht umsonst

Das was mir noch auffällt, ist, dass du deine Variablen zusammen erstellen könntest.

Wäre natürlich auch möglich, aber wozu? Hier sparst du wirklich nur "Platz" aber machst vielleicht den Code unübersichtlicher.
Also immer abwegen ob man es machen sollte oder nicht.

Aber warum sollte man if vorziehen ?

Der Vorteil eines if liegt darin das leichter Fehler vermieden werden
Vor allem bei Anfänger wird beim "switch" immer wieder mal das "break;" vergessen und plötzlich führt der Code etwas aus was er nicht tun sollte.

In diesem Fall macht es keinen großen Unterschied ob switch oder if/else , aber persönlich ziehe ich ein if/else switch vor

 

[Flown]

So ich habe das Bedürfnis meinen Senf auch dazuzugeben.

Also die switch vs. if-else Konstrukt Diskussion ist Geschmacksache. Aber ich würde hier auch ein switch vorziehen, denn in den Tutorials steht:

Deciding whether to use if-then-else statements or a switch statement is based on readability and the expression that the statement is testing. An if-then-else statement can test expressions based on ranges of values or conditions, whereas a switch statement tests expressions based only on a single integer, enumerated value, or String object.

Also nachdem ich keine Ranges prüfen will, ist switch die richtige Wahl.

Spoiler: Verbesserungen

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

public class Bsp03 {
  
  public static void main(String[] args) {
    
    try (Scanner scanner = new Scanner(System.in)) {
      int inMenue;
      do {
        printMenu();
        inMenue = scanner.nextInt();
        switch (inMenue) {
          case 0:
            break;
          case 1:
            handleCircleArea(scanner);
            break;
          case 2:
            handleLeapYear(scanner);
            break;
          case 3:
            handleLeapYearList(scanner);
            break;
          default:
            System.out.println("Invalid choice!");
            System.out.println();
            break;
        }
      } while (inMenue != 0);
    }
  }
  
  private static void printMenu() {
    System.out.println("Please choose calculation");
    System.out.println("1. Calculate circle area");
    System.out.println("2. Leap year calculation");
    System.out.println("3. Calculate next N leap years");
    System.out.println("0. Exit");
  }
  
  private static void handleCircleArea(Scanner scanner) {
    double inRadius;
    System.out.println("Please enter circle radius:");
    inRadius = scanner.nextDouble();
    System.out.println("Circle area is: " + getCircleArea(inRadius));
    System.out.println();
  }
  
  private static void handleLeapYear(Scanner scanner) {
    System.out.println("Please enter year:");
    int year = scanner.nextInt();
    
    System.out.println(year + " is " + (isLeapYear(year) ? "" : "not ") + "a leap year");
    System.out.println();
  }
  
  private static void handleLeapYearList(Scanner scanner) {
    System.out.println("Please enter year:");
    int year = scanner.nextInt();
    System.out.println("How many leap years should be calculated:");
    int n = scanner.nextInt();
    System.out.println(Arrays.toString(getNextLeapYears(year, n)));
    System.out.println();
  }
  
  // Methoden
  public static double getCircleArea(double radius) {
    return Math.PI * radius * radius;
  }
  
  public static boolean isLeapYear(int year) {
    return year % 4 == 0 && (year % 100 != 0 || year % 400 == 0);
  }
  
  public static int[] getNextLeapYears(int year, int n) {
    int[] leapYears = new int[n];
    int i = 0;
    while (i < n) {
      if (isLeapYear(year)) {
        leapYears[i++] = year;
      }
      year++;
    }
    return leapYears;
  }
}

 

[chuxXo]

Wäre natürlich auch möglich, aber wozu? Hier sparst du wirklich nur "Platz" aber machst vielleicht den Code unübersichtlicher.
Also immer abwegen ob man es machen sollte oder nicht.

Das stimmt. Aber wenn ich es so mach sieht das ganze dann folgendermaßen aus:

int variable1 = 0,
    variable2 = 1,
    variable3 = 2;

Wie sieht es da Perfomancetechnisch aus ? Bin mir sicher, dass es nicht spürbar sein wird, aber eigentlich müsste es so schneller sein, oder? Er hat ja so einen Schritt weniger.

Und switch und if ist in dem Fall Geschmackssache.
Immer gut, neue Dinge zu erfahren

 

[Jonsered]

Hey Leute, vielen Dank!

Also das mit switch möchte ich noch nicht machen, da wir das auch noch nicht um Unterricht hatten

@ Joose: Wie würdest du den Code dann schreiben, bin gerade ein wenig verwirrt , speziell mit Anfang und Ende

Oder besser als Methode, nur wie macht man da eine gaze Textausgabe?

Ich danke euch!!!

 

[arilou]

Zusätzliche Methode

void showOptions() {
  System.out.println ("Please choose calculation");
  System.out.println ("1. Calculate circle area");
  System.out.println ("2. Leap year calculation");
  System.out.println ("3. Calculate next N leap years");
  System.out.println ("0. Exit");
}

Und die rufst du auf, überall dort, wo die obigen Zeilen dafür wegfallen können.

 

[Jonsered]

Ach mit void... Danke