Простой калькулятор с использованием арифметических выражений Java правильно ли исполнен код?

Question

[PRAIT]

Было задание сделать простой калькулятор с выводом в консоль.


Вот сам код, много ли косяков? Если нет желания смотреть полностью, гляньте пожалуйста последнюю часть.
Я не совсем понимаю как исполнять "min" и "max"
min = 2
max = 7
Спасибо!

package dz;
import java.util.Scanner;
public class SimpleCalculator {
  public static void main(String[] args) {
    /* КАЛЬКУЛЯТОР С ВЫВОДОМ В КОНСОЛЬ */
    // ПРИБАВЛЕНИЕ
    Scanner upp = new Scanner(System.in);
    System.out.println("Пожалуйста введите цифру (Прибавление)");
    int a = upp.nextInt();
    System.out.println("Пожалуйста введите цифру(Прибавление)");
    int b = upp.nextInt();
    System.out.println("a = " + a);
    System.out.println("b = " +b);
    int c = a + b;
    System.out.println("a + b = " +c);
    
    //ВЫЧИТАНИЕ
    System.out.println("Пожалуйста введите цифру (Вычитание)");
    int q = upp.nextInt();
    System.out.println("Пожалуйста введите цифру (Вычитание)");
    int w = upp.nextInt();
    System.out.println("q = " + q);
    System.out.println("w = " + w);
    int e = q - w;
    System.out.println("q - w = " + e);
    
    //УМНОЖЕНИЕ
    System.out.println("Пожалуйста введите цифру (Умножение)");
    int r = upp.nextInt();
    System.out.println("Пожалуйста введите цифру (Умножение)");
    int t = upp.nextInt();
    System.out.println("r = " + r);
    System.out.println("t =" + t);
    int y = r * t;
    System.out.println("r * t = " +y);
    
    //ДЕЛЕНИЕ
    System.out.println("Пожалуйста введите цифру (Деление)");
    int u = upp.nextInt();
    System.out.println("Пожалуйста введите цифру (Деление)");
    int i = upp.nextInt();
    System.out.println("u = " + u);
    System.out.println("i = " + i);
    int o = u / i;
    System.out.println("u / i = " + o);
    
    //ЦИФРА С ПЛАВАЮЩЕЙ ТОЧКОЙ
    System.out.println("Пожалуйста введите цифру (Цифра с плавающей точкой)");
    double p = upp.nextInt();
    System.out.println("Пожалуйста введите цифру (Цифра с плавающей точкой)");
    double s = upp.nextInt();
    System.out.println("p = " + p);
    System.out.println("s = " + s);
    double d = p / s;
    System.out.println("p ./ s = " + d);
    
    //
    System.out.println("Пожалуйста введите цифру");
    int f = upp.nextInt();
    System.out.println("Пожалуйста введите цифру");
    int g = upp.nextInt();
    System.out.println("f = " +f);
    System.out.println("g = " +g);
    int h = f % g;
    System.out.println("h % g = " + h);
    
    //
    System.out.println("Пожалуйста введите цифру");
    int j = upp.nextInt();
    System.out.println("Пожалуйста введите цифру");
    int k = upp.nextInt();
    System.out.println("j = " + j);
    System.out.println("k = " + k);
    int l = j ^ k;
    System.out.println("j ^ k = " + l);
    
    System.out.println("Пожалуйста введите цифру");
    int min = upp.nextInt();
    System.out.println("Пожалуйста введите цифру");
    int max = upp.nextInt();
    System.out.println("min = " + min);
    System.out.println("max = " + max);
    
  }
}

Answer 1

[EVGENY T.]

Если задание для школы, то пойдёт. А так видно, что вы не профессиональный программист. Вашу программу дальше можно только выкинуть - её нельзя расширить, усложнить, использовать в другой программе. Учитесь.

P.S. По min и max сморите изучите https://docs.oracle.com/javase/7/docs/api/java/lan...

Comments

[PRAIT]

Спасибо вам, а не могли бы вы написать пример хотя бы двух арифметических выражений

Scanner upp = new Scanner(System.in);
    System.out.println("Пожалуйста введите цифру (Прибавление)");
    int a = upp.nextInt();
    System.out.println("Пожалуйста введите цифру(Прибавление)");
    int b = upp.nextInt();
    System.out.println("a = " + a);
    System.out.println("b = " +b);
    int c = a + b;
    System.out.println("a + b = " +c);
    
    //ВЫЧИТАНИЕ
    System.out.println("Пожалуйста введите цифру (Вычитание)");
    int q = upp.nextInt();
    System.out.println("Пожалуйста введите цифру (Вычитание)");
    int w = upp.nextInt();
    System.out.println("q = " + q);
    System.out.println("w = " + w);
    int e = q - w;
    System.out.println("q - w = " + e);

Как было бы правильно?

[Денис Загаевский]

PRAIT, для начала надо понять, в чём вообще смысл программы которая делает такие тупые действия.

[EVGENY T.]

PRAIT, попробуйте немного обобщить код. Для начала напишите функцию (например myCalc(int, int, char)), которая принимает два операнда и оператор. Сразу увидите как всё преобразится: вместо длинных кусков кода, останутся запросы пользователя и вызовы функции myCalc(), которую потом можно будет дополнять и использовать если вам вдруг предложат усложнить вашу программу.

У вас очень большой путь впереди, но главное всегда делайте так, чтобы после решения очередной задачи у вас в запасе появлялась новая функция, класс, библиотека, которую вы сможете использовать в будущем. Только там можно расти куда-то.

[PRAIT]

EVGENY T., Я бы был очень рад если бы вы показали на примере, так как для меня пока что это сложно

[PRAIT]

Денис Загаевский, Давай свою версию

[Денис Загаевский]

PRAIT, версию чего?

[PRAIT]

Денис Загаевский, Осмысленной программы которая не делает такие "тупые действия"

[PRAIT]

EVGENY T., А как вам такое исполнение? Есть недочеты?

package dz;
import java.util.Scanner;
public class BooleanCalculator {
  public static void main(String[] args) {
    try (Scanner upp = new Scanner(System.in)) {
      int additionA, additionB, subtractionA, subtractionB, equally;
      
      // ПРИБАВЛЕНИЕ
      System.out.println("Пожалуйста введите число (Прибавление)");
      additionA = upp.nextInt();
      System.out.println("Пожалуйста введите число (Прибавление)");
      additionB = upp.nextInt();
      equally = additionA + additionB;
      System.out.println(" = " + equally);
      
      // ВЫЧИТАНИЕ
      System.out.println("Пожалуйста введите число (Вычитание)");
      subtractionA = upp.nextInt();
      System.out.println("Пожалуйста введите число (Вычитание)");
      subtractionB = upp.nextInt();
      equally = subtractionA - subtractionB;
      System.out.println(" = " + equally);
    }
  }
}

[Денис Загаевский]

PRAIT, ну, например, написать нормальный калькулятор, которому даешь выражение (2+4-6*10) со скобками, а он считает значение этого выражения. А если не может распарсить выражение, пишет, где именно споткнулся.