Всем привет, у меня вопрос, оператор instanceof насколько медленный?
P.S. Если он очень медленный, есть ли альтернатива этому оператору?

mcdevil, альтернатива - шаблон проектирования Visitor.

Ginosaji, А что это?

mcdevil, Visitor не совсем об этом Здесь просто перегрузка метода.
Вот такой код:

Можно переписать так:

А Visitor уже про объекты. Есть набор объектов, над которыми нужно выполнять некоторые операции. При это, если нужно определить новую, то нельзя изменять структуру этих классов (чем меньше мы трогаем её, тем лучше).

Преимущество такого шаблона в том, что если мы вдруг захотим выводить не в консоль, а в файл, да ещё и в каком-нибудь JSON, то нам придётся просто создать ещё одного «посетителя» (а создание класса дешевле изменения существующего)

Короче, я уже запарился разбираться с Enum.. пишу какой-то сс..ый калькулятор уже второй день..

Есть два класса, мейн и класс операций. по задумке, чтобы уйти от использования свитчей, нужно в перечислениях операций реализовать метод getResult.

В основном классе я считываю сканером два значения и оператор (тип String, с char не хочу извращаться, да и много лишнего кода из-за него будет). Далее мне как-то надо вызвать тот метод getResult, чтобы вывести консоль. Текущая реализация с ошибкой. Не знаю, что делать. Пытался привести к типу Operations строковое значение оператора - ошибка, писал похожую конструкцию Operations o = o.getResult(arg1, arg2); - никак.

Примеры гуглил, было похожее что-то, но там просто по-очереди вызывался метод типа моего getResult, но мне не подходит, т.к. оператор заранее не известен.. А то и вообще какие-то громоздкие конструкции.

Run.java

[копировать] [скачать]

1. import java.util.Scanner;
2.  
3. public class Run {
4.  
5.     private static double argument1;
6.     private static double argument2;
7.     private String operator;
8.     private static String result;
9.  
10.     public static void main(String[] args) {
11.         Run app = new Run();
12.         app.runApp();
13.     }
14.  
15.     public void runApp() {
16.  
17.         Scanner sc = new Scanner(System.in);
18.  
19.         System.out.println("Enter your first value:");
20.         argument1 = sc.nextDouble();
21.  
22.         System.out.println("Enter your second value:");
23.         argument2 = sc.nextDouble();
24.  
25.         System.out.println("Enter operation: ");
26.         operator = sc.next();
27.         sc.close();
28.         // Здесь трабла, хз как правильно вызвать метод
29.         result = operator.getResult(argument1, argument2);
30.  
31.         System.out.println("Result: " + result);
32.     }
33. }

Operations.java

[копировать] [скачать]

1. public enum Operations {
2.  
3.     PLUS("+") {
4.         @Override
5.         public double getResult(double arg1, double arg2) {
6.             return arg1 + arg2;
7.         }
8.     },
9.     MINUS("-") {
10.         @Override
11.         public double getResult(double arg1, double arg2) {
12.             return arg1 - arg2;
13.         }
14.     },
15.     DIVIDE("/") {
16.         @Override
17.         public double getResult(double arg1, double arg2) {
18.             return arg1 / arg2;
19.         }
20.     },
21.     MULTIPLY("*") {
22.         @Override
23.         public double getResult(double arg1, double arg2) {
24.             return arg1 * arg2;
25.         }
26.     };
27.  
28.     public String operator;
29.  
30.     private Operations(String operator) {
31.         this.operator = operator;
32.     }
33.  
34.     public abstract double getResult(double arg1, double arg2);
35.  
36. }

И еще по поводу @Override: где-то он используется, где-то нет (в разных реализациях одного и того же кода) - здесь оно надо? До этого у меня было проще, но ошибка все равно оставалась, проблемы с типами. Пока пробовал разные варианты кода, допогромировал до такого состояния

ВитаминКО, предлагаю такой вариант:
1. Делаем интерфейс IBinaryOperator.
Пишем его реализации под каждый оператор:
В классе Operations заводим ассоциативный массив:
В главном классе, как обычно, читаешь сканнером оператор и аргументы и передаёшь их в Operators.getResult(String, double, double);

Sweety, при этом производительность повышается или нет никакой разницы?

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

Ginosaji, оно, конечно, интересно, но мне надо именно с помощью Enum. И у тебя изврат какой-то, зачем так.. сложно, что ли..

С помощью Enum можно сделать, я знаю. Мне просто надо понять, где у меня ошибка.