ВитаминКО, ты с типами накосячил. У тебя operator - String, результат считается в double, а присваивается в String.

Перебирать операции или создавать ассоциативный массив тебе всё равно придётся, потому что нельзя просто так задать соответствие между "+" и Operations.PLUS.

http://annimon.com/forum/?act=post&id=458493
Вроде понял, чего не хватает, тестирую..

aNNiMON, ща проверю и типы. Не, исключение кидает, No enum const class. Сейчас попробую метод тот модифицировать и уже от этого буду исходить.
Да, перебор как раз хотел добавить.

ВитаминКО, через Enum тогда можно так:
Вместо:
так:

в общем, как-то так

Run.java

Run.java

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

1. import java.util.Scanner;
2.  
3. public class Run {
4.  
5.     private static Scanner sc = new Scanner(System.in);
6.  
7.     // ...
8.  
9.     public void runApp() {
10.         // ...
11.         EOperations operator = getOperator();
12.         double result = operator.getResult(argument1, argument2);
13.         // ...
14.     }
15.  
16.     private EOperations getOperator() {
17.         System.out.println("Enter operation: ");
18.         while (true) {
19.             EOperations operator = EOperations.getResult(sc.next());
20.             if (operator != EOperations.DEFAULT_OPERATION) {
21.                 return operator;
22.             }
23.         }
24.     }
25. }

EOperations.java

EOperations.java

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

1. public enum EOperations {
2.  
3.     OP_ADDITION("+") {
4.         @Override
5.         public double getResult(double arg1, double arg2) {
6.             return arg1 + arg2;
7.         }
8.     },
9.     // ...
10.     DEFAULT_OPERATION(null) {
11.         @Override
12.         public double getResult(double arg1, double arg2) {
13.             throw new UnsupportedOperationException("Unsupported operation!");
14.         }
15.     };
16.  
17.     public String op;
18.  
19.     private EOperations(String op) {
20.         this.op = op;
21.     }
22.  
23.     public static EOperations getResult(String op) {
24.         for (EOperations operator : EOperations.values()) {
25.             if (op.equals(operator.op)) {
26.                 return operator;
27.             }
28.         }
29.         return DEFAULT_OPERATION;
30.     }
31.  
32.     public abstract double getResult(double arg1, double arg2);
33.  
34. }

Работает, остается только допилить проверки на нуль и т.д., работу в цикле и дополнительные исключения выбрасывать.

Сяп за помощь

Потом попробую с интерфейсами еще

ВитаминКО, с интерфейсами всё-таки меньше кода. А здесь ты Enum используешь не по назначению. К тому же, перебор в цикле не есть хорошо. Гораздо быстрее будет работать, если воспользуешься HashMap.

aNNiMON, я не настолько продвинут, чтобы так далеко заходить. Нам дали задание - заюзайте перечисления вместо ваших дрянных свитчей.
ООП только началось, а до коллекций и тп еще далеко.

Да, скопировать - скопирую, только я все равно пока не пойму, не хочу нагружаться. У нас цель - написать не столько быстрый, сколько читабельный, легко расширяемый и понятный код. Придерживаясь Java Conventions.

скучаю по паскалю

Цитата ВитаминКО:

---

Нам дали задание - заюзайте перечисленияC этого бы и начинал.
Цитата ВитаминКО:

---

У нас цель - написать не столько быстрый, сколько читабельный, легко расширяемый и понятный код.Насчёт понятности ладно, каждому своё, а вот расширяемости в Enum'ах нет.

Sweety, Этот код не работает на me, не знаю на се как работает.
Вообщем в вектор добавляются элементы типы которых не известны, нужно например сложить два элемента, но нужно определить какому типу относятся эти элементы с максимальной производительностью.

ВитаминКО, энамки не наследуются

mcdevil, visitor в помощь