java - 这是正确的 MVC 设计吗？关于 MVC 设计和线程通信的建议

Question

最后一天我读了很多很多关于 MVC 模式的教程和博客。现在我已经基本理解了这个概念，但在我看来，每个教程都展示了在 Java 中实现此模式的另一个概念。所以我决定编写自己的应用程序，然后向技术更高的程序员寻求建议。 (边做边学是我的风格)。

所以我不想透露太多我最初的想法，但我会向你展示我的代码:

主类:

public class MainClass {enter code here
    public static void main(String[] args){

        MainController controller = new MainController();
        controller.initView();
    }
}

Controller 类:

public class MainController implements ActionListener{

private ExtractorStatics stat;
private MainView mainview;
private BusinessExtractor bExtractor;
private InfoboxTextPane infobox;
private BufferedImage logoGS;

public MainController(){
     stat = new Statics(); 
     model = new Model();
     mainview   = new MainView();
}
public void initView(){
    if(mainview!=null){
        mainview.setActionListener(this);
        mainview.setItemListener(new ComboBoxItemListener(this));
        mainview.setVisible(true);
    }
}

@Override
public void actionPerformed(ActionEvent event) {
    String command = event.getActionCommand();
    if(command.equalsIgnoreCase(stat.SCAN_ACTION_COMMAND)){
        this.quickScanButtonAction();
    }
}


private void quickScanButtonAction(){
    infobox = mainview.getInfobox();
    ProcessingInformation information = model.quickScan();
    InputStream informationStream = information.getInformationStream();
    BufferedReader infoReader = new BufferedReader(
                new InputStreamReader(informationStream));
    String line;
    try {
    while ((line = infoReader.readLine()) != null) {
                infobox.appendLine(line);
            }

            infoReader.close();
        } catch (IOException e) {
            e.printStackTrace();
        }
        System.out.println("End reached");
    }
    else
        infobox.appendLine("Bitte Eingabe überprüfen.");
}
public void comboBoxChanged() {
    //do some fancy stuff
}
   }

 class ComboBoxItemListener implements ItemListener{
        MainController mc;
    public ComboBoxItemListener(MainController mc){
        this.mc=mc;
    }
    @Override
    public void itemStateChanged(ItemEvent e) {
        mc.comboBoxChanged();
    }   
}

MainView Class:

    public class MainView extends JFrame {


    private static final long serialVersionUID = 559229524422932258L;
    private JPanel contentPane;
    private JTextField txt_stichwort,txt_loc;
    private JButton btn_quickscan;
    private JTable table;
    private JLabel label;
    public  InfoboxTextPane txtpn_infobox;
    private String lineSep;
    private final JComboBox<String> combobox;
    /**
     * Create the frame.
     */
    public MainView() {
        try {
            UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
        } catch (ClassNotFoundException | InstantiationException
                | IllegalAccessException | UnsupportedLookAndFeelException e3) {
            // TODO Auto-generated catch block
            e3.printStackTrace();
        }

        lineSep=System.lineSeparator();
        Statics stats=new Statics();



        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setBounds(100, 100, 800, 620);
        contentPane = new JPanel();
        contentPane.setBorder(new EtchedBorder(EtchedBorder.LOWERED, null, null));
        setContentPane(contentPane);
        contentPane.setLayout(null);


        txt_subject= new JTextField();
        txt_stichwort.setBounds(88, 47, 318, 20);
        pan_suche.add(txt_stichwort);
        txt_stichwort.setColumns(10);

        combobox = new JComboBox<String>();
        combobox.setBounds(88, 16, 318, 20);
        pan_suche.add(combobox);
        combobox.setModel(new DefaultComboBoxModel(new String[] {"Item1", "Item2"}));

        txt_loc = new JTextField();
        txt_loc.setBounds(88, 79, 318, 20);
        pan_suche.add(txt_loc);
        txt_loc.setColumns(10);



        btn_quickscan = new JButton("Quick Scan");
        btn_quickscan.setActionCommand(stats.SCAN_ACTION_COMMAND);
        btn_quickscan.setBounds(10, 23, 130, 30);
        pan_dos.add(btn_quickscan);

        //a few more buttons that have other action commands defined by statics
        //more labels and other GUI components


    }
    public InfoboxTextPane getInfobox(){
        return this.txtpn_infobox;
    }
    public String getSearchSubject(){
        return this.txt_stichwort.getText();
    }

    public String getSearchLocation(){
        return this.txt_loc.getText();
    }
    public String getSearchWebsite(){
        return (String)this.combobox.getSelectedItem();
    }
    public JComboBox<String> getComboBox(){
        return this.combobox;
    }
    public JLabel getLogoLabel(){
        return this.label;
    }

    public void setActionListener(ActionListener al){
        try {

            btn_quickscan.addActionListener(al);
        } catch (NullPointerException e) {
            e.printStackTrace();
        }
    }

    public void setItemListener(ItemListener il){
        this.combobox.addItemListener(il);
    }
    }

模型类:

public class Model {
private ExecutorService exeService;

public Model(){
    exeService =Executors.newCachedThreadPool();
}

public ProcessingInformation quickScan(){ 
    QuickScanRoutine qs = new QuickScanRoutine();
    Future<String> result = exeService.submit(qs);
    return qs.getProcessingInformation(); 
}

}

例程类(可调用):

public class QuickScanRoutine implements Callable<String> {
    private ProcessingInformation pi;
    private BufferedWriter writer;



public QuickScanRoutine(){
    pi = new ProcessingInformation();
    PipedOutputStream pos = new PipedOutputStream();
    PipedInputStream pis;
    writer = new BufferedWriter(new OutputStreamWriter(pos));

    try {
        pis = new PipedInputStream(pos);
        pi.setInformationStream(pis);
    } catch (IOException e) {
        e.printStackTrace();
    }

}

public ProcessingInformation getProcessingInformation(){
    return this.pi;
}

@Override
public String call() throws Exception {

    //ofcourse this isnt the real purpose of this Class but I want to     `demonstrate writing to the stream`

    for(int i =0; i<100;i++){
        writer.write("Hello");
        writer.newLine();
        writer.flush();
    }
    writer.close();
    return "Routine has been executed sucessfully!";
}

}

处理信息:

public class ProcessingInformation {
PipedInputStream informationStream,resultDataStream;

    public void setInformationStream(PipedInputStream info) {
        this.informationStream = info;
    }
    public void setResultDataStream(PipedInputStream data){ 
//I will need this stream for further tasks
        this.resultDataStream = data;
    }

    public InputStream getInformationStream(){
    if(informationStream!=null){
        return informationStream;
    }
    else
        return null;
}

public InputStream getResultDataStream(){
    if(informationStream!=null){
        return informationStream;
    }
    else
        return null;
}
}

我尝试尽可能缩短代码，但尝试将其放在上下文中，以便您可以理解整个结构。我的主要问题是:

• 这是正确的 MVC 设计吗？
• 这是从该模型线程到 View 进行通信的合适方式吗？ (管道流)

这些是我的主要问题，但我很感激并愿意接受您的所有建议。

Answer 1

我同意 Abhi Beckert 在某些方面的回答 - 通常如下:

1. 模型与模型以外的任何对象进行通信都是 Not Acceptable 。
2. View 应该只与其他 View 对象通信，而不了解 Controller 。

此外， Controller 必须了解有关 View 组成的任何信息，因为这会将 Controller 和 View 耦合起来。这种耦合意味着，如果您需要更改 View 表示，或者为此向用户提供各种数据表示的选择，您将必须为每个表示编写一个新的 Controller 。让我通过一个例子来具体说明这一点。

当您在 MS-Word 等文字处理器中打开 Word 文档时，页面可以采用各种布局，例如纵向、横向、打印和 Html。在每种布局中，可见的用户元素都不同。但数据是一样的。因此，解决该问题的最合乎逻辑的方法是:

• 让 Controller 向模型发送命令以更新相应的 View 。
• 让 Controller 查询模型以获取数据，并将获取的数据简单地传递给 View 进行渲染。

我浏览了您发布的代码并确定您的 Controller 知道 View 的内容，例如MainController 调用 MainView.getinfobox()。根据上述解释，您可能想要更改它。

关于与使用PipedStream发送数据相关的第二个问题，我没有发现其中有任何技术上的错误 - 因为流也是数据的容器。但是，到目前为止，我个人还没有看到或遇到过遵循这种模式的设计或代码片段。

希望这有帮助