3

У меня сразу несколько вопросов по mvc. Спрашивать буду на примере игрушечной программки, которую специально для этого написал:

public class View implements Observer {

private Model model;
private IController controller;

private JLabel firstNumber = new JLabel();
private JLabel secondNumber = new JLabel();
private JLabel resultLabel = new JLabel();

public static void main(String[] args) {
    SwingUtilities.invokeLater(new Runnable() {
        @Override
        public void run() {
            Model model = new Model();
            IController controller = new Controller();
            View view = new View();
            view.setModel(model);
            view.setController(controller);
            view.createAndShowGUI();
        }
    });
}

public void setModel(Model model) {
    this.model = model;
    model.addObserver(this);
}

public void setController(IController controller) {
    this.controller = controller;
}

public void createAndShowGUI() {
    JFrame frame = new JFrame("mvc train");
    frame.setSize(400, 400);
    frame.getContentPane().add(createMainPanel());
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setVisible(true);
}

private Component createMainPanel() {
    JPanel mainPanel = new JPanel();
    mainPanel.add(firstNumber);
    mainPanel.add(secondNumber);
    mainPanel.add(resultLabel);
    JButton inc1 = new JButton("increment first");
    inc1.addActionListener(new ActionListener() {

        @Override
        public void actionPerformed(ActionEvent e) {
            controller.execute("inc1", model);
        }
    });
    JButton dec1 = new JButton("decrement first");
    dec1.addActionListener(new ActionListener() {

        @Override
        public void actionPerformed(ActionEvent e) {
            controller.execute("dec1", model);
        }
    });
    mainPanel.add(inc1);
    mainPanel.add(dec1);
    JButton inc2 = new JButton("increment second");
    inc2.addActionListener(new ActionListener() {

        @Override
        public void actionPerformed(ActionEvent e) {
            controller.execute("inc2", model);
        }
    });
    JButton dec2 = new JButton("decrement second");
    dec2.addActionListener(new ActionListener() {

        @Override
        public void actionPerformed(ActionEvent e) {
            controller.execute("dec2", model);
        }
    });
    mainPanel.add(inc2);
    mainPanel.add(dec2);
    return mainPanel;
}

@Override
public void update(Observable arg0, Object arg1) {
    firstNumber.setText("" + model.getA());
    secondNumber.setText("" + model.getB());
    resultLabel.setText("" + (model.getA() + model.getB()));
}

}

Это класс представления. Получается такая картинка: введите сюда описание изображения

Кнопка increment first увеличивает на единицу первое число, increment second - второе, decrement first уменьшает первое число на единицу, decrement secomd - второе. В общем, после двух нажатий на increment first и одного нажатия на increment second получаем такую картинку:

введите сюда описание изображения

Два числа хранятся в модели. Вот она:

public class Model extends Observable {

private int a;
private int b;

public void incrementA() {
    a++;
    setChanged();
    notifyObservers();
}

public void incrementB() {
    b++;
    setChanged();
    notifyObservers();
}

public void decrementA() {
    a--;
    setChanged();
    notifyObservers();
}

public void decrementB() {
    b--;
    setChanged();
    notifyObservers();
}

public int getA() {
    return a;
}

public int getB() {
    return b;
}

@Override
public void addObserver(Observer observer) {
    super.addObserver(observer);
    setChanged();
    notifyObservers();
}

}

Мой контроллер:

public class Controller implements IController {

@Override
public void execute(String action, Model model) {
    switch (action) {
    case "inc1":
        model.incrementA();
        break;
    case "inc2":
        model.incrementB();
        break;
    case "dec1":
        model.decrementA();
        break;
    case "dec2":
        model.decrementB();
    }
}

}

Теперь вопросы. Самый главный вопрос собственно в названии.

  1. Чем нам помог контроллер? Код из веток case оператора switch в контроллере мог бы с таким же успехом находится в слушателях. На каждую ветку по слушателю. В итоге слушателей все равно приходится создавать, что бы вызвать метод контроллера. А потом в этом методе в операторе switch прописывать тот же самой код, что мог бы быть в слушателях. Может быть я вообще неправильно реализовал mvc? Подозреваю что именно так. Причем именно в части контроллера. Потому что по нему в сети мало информации. Если это так то прошу подсказать как правильно. Слышал что можно создавать контроллеры для каждого элемента (в данном случае для каждой кнопки). Но ситуация не сильно поменяется.
  2. В моем примере все методы, которые вызывается в ветках case не имеют параметров. В реальных программах это скорее всего будет не так. Некоторые кнопки могли бы требовать вызовов методов с параметрами, причем с разным количеством параметров. Что делать в такой ситуации?
  3. Где лучше вычислять самое правое число? В моем примере я делаю это в функции update(). Не лучше ли это делать в модели?

1 ответ 1

4
  1. Контроллер помогает структурировать код. Весь код, реагирующий на пользовательский ввод, принадлежит контроллеру.

    Можно было бы всё слить в один класс, и делать всю логику в OnClick, и код получился бы наверное даже короче. Но вашей целью должен быть не самый короткий, а самый понятный и легко поддерживаемый код.

  2. Мне кажется, ваша диспетчеризация команд по строкам — неправильна. Команды должны быть не строками, а объектами. (У нас же ООП, в конце-концов, да?) И вместо длинного свитча у вас должен быть всего лишь вызов Invoke. [Но это практика из MVVM, общепринятого паттерна в WPF, а традиции в MVC могут быть и другими.]

  3. Вычисления принадлежат модели и только ей. Представление должно не выполнять работу модели (а сложение в реальной программе превратится в серьёзное, длинное вычисление), а лишь только отображать её.

14
  • спасибо большое, очень интересно. Можно по второму пункту уточнить? Нам нужно вызывать функции. Но мы можем обернуть их в классы? Они будут реализовать интерфейс MyListener, в котором будет один метод invoke(). То есть будет четыре класса, реализующих этот интерфейс: FirstIncrementer, SecondIncrementer и т.д. А метод execute в контроллере будет принимать параметр типа MyListener. И просто вызывать его метод invoke(). Тогда каждый слушатель подпихнет контроллеру нужную реализацию MyListener. Но что прописать в методах invoke() конкретных классов? По идее там должна быть работа с моделью 16 апр 2016 в 19:47
  • Но работа с моделью должна быть в контроллере. 16 апр 2016 в 19:47
  • получается, что в контроллере будет всего один вызовов, а весь обрабатывающий код опять же не в нем.. Хотя мы стремились как раз к переносы обрабатывающего ввод кода в контроллер. Не понимаю( 16 апр 2016 в 20:00
  • @АлександрЕлизаров: Значит, эти классы (команды) должны быть частью контроллера. Так что весь обрабатывающий код всё равно в контроллере.
    – VladD
    16 апр 2016 в 20:10
  • 1
    @АлександрЕлизаров: Ну, во время создания UI нужно привязывать команды. То есть кнопка будет знать свою команду, но не знать, что там внутри. И будет дёргать за неё по нажатию. Или не так, контроллер будет иметь Map кнопок в команды, и по кнопке находить нужную команду и запускать её. Не знаю пока, как правильнее. [Ну и наверное скорее ICommand, а не IController.]
    – VladD
    16 апр 2016 в 20:40

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.