Observer/Observableメソッドを使用して、Javaの基本MVCパターンを把握しました。今ではそれをきれいにして読みやすいようにすることに興味があります。私の前に、私のViewを整理する方法についていくつかの指針が欲しいと思います。学校では、クラスごとのファイルサイズを20 KB以下に抑えて、可読性を保ち、後で保守しやすくしています。ここで私のビュー(MVC)をきれいに整理するにはどうすればいいですか?
は、私の見解である:新しいクラスまたはどのように役立つコードを最小限にするために分離するものに
package view;
import model.*;
import helper.*;
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import java.util.Observable;
import java.util.Observer;
import net.miginfocom.swing.MigLayout;
public class View extends JFrame implements Observer
{
private Model model;
private JPanel left = new JPanel(new MigLayout());
private JPanel center = new JPanel(new MigLayout());
private JPanel right = new JPanel(new MigLayout());
private void setConstraints()
{
this.left.setMinimumSize(new Dimension(252, 540));
this.left.setMaximumSize(new Dimension(252, 37500));
this.center.setMinimumSize(new Dimension(298, 540));
this.right.setMinimumSize(new Dimension(250, 540));
this.right.setMaximumSize(new Dimension(250, 37500));
}
//Left panel contents
private Towers box = new Towers();
private Modules tree = new Modules();
private JPanel setupLeft()
{
this.left.add(this.box, "growx, pushx, wrap");
this.left.add(new JScrollPane(this.tree), "grow, push");
return this.left;
}
//Center panel contents
private Browser browser = new Browser();
private JPanel setupCenter()
{
this.center.add(new JScrollPane(this.browser), "grow, push");
return this.center;
}
//Right panel contents
private JLabel tower = new JLabel("No tower selected.");
private JLabel cap = new JLabel("Capacitor");
private JLabel cpu = new JLabel("CPU");
private JLabel shield = new JLabel("0");
private JLabel armor = new JLabel("0");
private JLabel em = new JLabel("0.0");
private JLabel th = new JLabel("0.0");
private JLabel kn = new JLabel("0.0");
private JLabel ex = new JLabel("0.0");
private JPanel setupRight()
{
this.right.add(this.tower, "span, wrap");
this.right.add(this.cap, "span, wrap");
this.right.add(this.cpu, "span, wrap");
this.right.add(this.shield, "span, wrap");
this.right.add(this.armor, "span, wrap");
this.right.add(this.em, "span, wrap");
this.right.add(this.th, "span, wrap");
this.right.add(this.kn, "span, wrap");
this.right.add(this.ex, "span, wrap");
return this.right;
}
public View(Model ui_model)
{
model = ui_model;
this.setTitle("MVC Experiment 6");
this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
this.setExtendedState(JFrame.MAXIMIZED_BOTH);
this.setMinimumSize(new Dimension(800, 600));
this.setLayout(new MigLayout());
this.setConstraints();
this.add(this.setupLeft(), "dock west");
this.add(this.setupCenter(), "dock center");
this.add(this.setupRight(), "dock east");
}
//Left panel contents - Listeners and methods for addressing JComponents
public void xTowersBrowser(ActionListener event)
{
this.box.addActionListener(event);
}
public void xModulesBrowser(MouseListener event)
{
this.tree.addMouseListener(event);
}
public Towers getTowersBrowser()
{
return this.box;
}
public Modules getModulesBrowser()
{
return this.tree;
}
//Left panel - END
//Center panel - components :: listeners and methods
public void xBrowser(MouseListener event)
{
this.browser.addMouseListener(event);
}
public Browser getBrowser()
{
return this.browser;
}
//Center panel - END
public void update(Observable o, Object arg)
{
}
}
任意の提案。これは私のメインのViewクラスから削除されたものです。JComponentsがたくさん残っているので、もっと面倒です。
これはおそらく[codereview](http://codereview.stackexchange.com)でより適切でしょう。あなたはどこでも 'this'を使う必要はありません。そして、Javaでは、改行の中にカッコを入れません。 – toto2
非常に長いファイルを生成するためのUIプログラミングは有名です。 – toto2
ああ、私はそれについてのcodereview感謝については知らなかった。私はこの部分を理解しています。それはPHPの日などの古い習慣です。 –