Как я могу сделать этот код java более эффективным

Question

Не будьте критичным, потому что это моя первая в мире рабочая программа Я хочу знать, насколько она хороша и как я могу сделать это более эффективным как в производительности, так и в памяти.

вот мой код:

package leren; 
import java.util.Scanner; 

public class Calculator3 { 

static boolean again =true; 
static Scanner keyIn = new Scanner(System.in); 
public static void main(String args[]){ 

     double in1, in2; 
     char operator; 

    while(again = true){ 
    System.out.println("Type your first number:"); 
    in1 = keyIn.nextDouble(); 
    System.out.println("Type your second number:"); 
    in2 = keyIn.nextDouble(); 
    System.out.println("Type your operator"); 
    operator = keyIn.next().charAt(0); 

     switch(operator){ 
     case '+': 
      System.out.println("your result is:"+(in1 + in2)); 
      Timer(); 
      again(); 
      break; 

     case '-': 
      System.out.println("your result is:"+(in1 - in2)); 
      Timer(); 
      again(); 
      break; 

     case '/': 
      System.out.println("your result is:"+(in1/in2)); 
      Timer(); 
      again(); 
      break; 

     case '*': 
      System.out.println("your result is:"+(in1 * in2)); 
      Timer(); 
      again(); 
      break; 

     } 
    } 
} 
public static void Timer(){ 
    try{ 
     Thread.sleep(1500); 
    } 
    catch(Exception e){ 
    } 
} 
public static void again(){ 

    char YoN; 
    System.out.println("do you wanna do this again?[Y | N]"); 
    YoN = keyIn.next().charAt(0); 
    switch(YoN){ 
    case 'Y': 
     String[] args = {}; 
     Calculator3.main(args); 
     break; 
    case 'N': 
     System.out.println("well get out of here"); 
     Timer(); 
     System.exit(0); 
     } 
    } 
} 

Любая помощь поможет мне понять вещи лучше.

Answer 1

Эта версия не эффективна с точки зрения памяти и не быстрее.

Эта версия легче читать и понимать, включая себя через несколько месяцев.

Я поместил большую часть кода в ваш класс. Я избавился от статических методов, кроме основного. Я разбил ваш код на методы. Я также исправил ваше заявление while, поэтому вам не нужно запускать приложение снова и снова.

Вот код.

package com.ggl.testing; 

import java.util.Scanner; 

public class Calculator3 implements Runnable { 

    private boolean again = true; 
    private Scanner keyIn = new Scanner(System.in); 

    public static void main(String args[]) { 
     new Calculator3().run(); 
    } 

    @Override 
    public void run() { 
     while (again) { 
      double in1 = readDouble("Type your first number: "); 
      double in2 = readDouble("Type your second number: "); 
      char operator = readChar("Type your operator: "); 
      System.out.println("Your result is: " 
        + operation(in1, in2, operator)); 
      timer(); 
      char yOrN = readChar("Do you want to do this again? [Y | N] "); 
      if (Character.toLowerCase(yOrN) != 'y') { 
       System.out.println("Well, get out of here"); 
       again = false; 
      } 
     } 

     keyIn.close(); 
    } 

    public double readDouble(String s) { 
     System.out.print(s); 
     return keyIn.nextDouble(); 
    } 

    public char readChar(String s) { 
     System.out.print(s); 
     return keyIn.next().charAt(0); 
    } 

    public double operation(double in1, double in2, char operator) { 
     switch (operator) { 
     case '+': 
      return in1 + in2; 
     case '-': 
      return in1 - in2; 
     case '/': 
      return in1/in2; 
     case '*': 
      return in1 * in2; 
     default: 
      return 0D; 
     } 
    } 

    public void timer() { 
     try { 
      Thread.sleep(1500); 
     } catch (InterruptedException e) { 

     } 
    } 

}