Лучший способ элементов в JComboBox

Вы можете создать интерфейс (например, Task), представляющий задачу, которую необходимо выполнить. Затем создайте реализацию этого интерфейса для каждого из значений в поле со списком (ContrastTask, MeanTask, ...). Наконец, в вашем actionPerformed напишите фабричный метод, возвращающий правильную реализацию - это будет в основном "переключатель", возвращающий правильный экземпляр Task. Тогда вы можете просто запустить эту задачу...

Task может выглядеть так:

public interface Task {
  [whatever result it should return] execute([whatever parameters it needs]);
}

Одна из реализаций может выглядеть так:

public class MeanTask implements Task {
  public int execute(int a, int b) {
    return (a + b) / 2;
  }
}

Ваша фабрика будет выглядеть так:

private Task create(String option) {
  if (option.toLowerCase().compareTo("mean") == 0) {
    return new MeanTask();
  }
  else if ....
}

Преимущество этого в том, что логика каждой задачи прекрасно инкапсулирована в свой собственный класс.

Это может помочь вам:

public class Main {
  private Map<String, BaseStrategy> strategyMap = new HashMap<String, BaseStrategy>();

  {
    strategyMap.put("case1", new Strategy1());
    strategyMap.put("case2", new Strategy2());
    strategyMap.put("case3", new Strategy3());
  }

  //...

  public void actionPerformed(ActionEvent e) {
    strategyMap.get(option.toLowerCase()).processEvent();
  }

  abstract class BaseStrategy {

    public abstract void processEvent();

  }

  class Strategy1 extends BaseStrategy {

    @Override
    public void processEvent() {
      //case 1
    }
  }

  class Strategy2 extends BaseStrategy {

    @Override
    public void processEvent() {
      //case 2
    }
  }

  class Strategy3 extends BaseStrategy {

    @Override
    public void processEvent() {
      //case 3
    }
  }
}

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

На самом деле, если у вас слишком много случаев - Map намного лучше, чем if-else-if-... Карта найдет правильную стратегию намного быстрее - O(ln n) вместо O(n) .

Как сообщили вектор и Юджин, я бы объединил оба решения примерно так:

public interface Task {
  [whatever result it should return] execute([whatever parameters it needs]);
}

//call this method in your constructor
private void initTasks() {
    this.tasks.put("option1", new Task() {
                public int execute(int a, int b) {
                   return (a + b) / 2;
                }
          });
    //init more tasks
    //this.task.put(..., ...);
}

protected Task getTask(String option) {
     return this.task(option);
}