2

数独プログラムの GUI を書いています。MVC フレームワークを使用しようとしています。次の投稿は、私の現在のコードの多くの基礎でした: Building a GUI for a Sudoku Solver (Complete with ASCII Example) . 私の質問は、アクションリスナーはどこに行くのですか? 私はそれらがコントローラーに入ると思いますが、それは単一のクラスですか、それとも複数のクラスですか? 私は現在、単一のアクションリスナーと、イベントを発生させることができる各スイングコンポーネントを処理する if/else ロジックを持つ単一のクラスを持っています。ただし、このコードは私には悪臭がします。コードを構造化するための好ましい方法はありますか? かなりの数のボタンとコンポーネントを追加することを期待しており、私が行っているパスが合理的であることを確認したいと考えています。かなりの数の投稿と Web チュートリアルを読みましたが、この回答はまだ見つかりません。

ありがとうございました

public class SudokuView extends JFrame{
SudokuController controller;
//SudokuEngine sudokuEngine;

private static final int yMar = 100;
private static final int xMar = 25;
private static final int cellSize = 40;
private static final int gridSpace = 3;
private static final int secSpace = 9;
private static final int width = 2*xMar+9*cellSize+2*secSpace+6*gridSpace+10;
private static final int height = 650;

public JButton[][] gridButtons;

public SudokuView() {
    controller = new SudokuController();
    //sudokuEngine = new SudokuEngine();
    gridButtons = new JButton[9][9];
    setTitle("Sudoku Ninja 0.1");
    setSize(width, height);
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setLayout(null);
    setBackground(Color.GRAY);
    setResizable(false);

    DrawBlankGrid();

    setVisible(true);
}

public void setSudokuImplementation(SudokuEngineInterface listener) {
    controller.setListener(listener);
}

public final void DrawBlankGrid(){
    int y=yMar;
    for (int r=0;r<9;r++){
        int x=xMar;
        for (int c=0;c<9;c++){
            gridButtons[r][c]=new JButton("");
            gridButtons[r][c].setBounds(x,y,cellSize,cellSize);
            gridButtons[r][c].putClientProperty("caller", 1);
            gridButtons[r][c].putClientProperty("column", c);
            gridButtons[r][c].putClientProperty("row", r);
            gridButtons[r][c].addActionListener(new SudokuController());
            add(gridButtons[r][c]);
            if (c==2||c==5){
                x = x+cellSize+secSpace;
            }else{
                x = x+cellSize+gridSpace;
            }
        }
        if (r==2||r==5){
            y = y+cellSize+secSpace;
        }else{
            y = y+cellSize+gridSpace;
        }
    }
}

public class SudokuController implements ActionListener{
SudokuEngineInterface listener;

public void setListener(SudokuEngineInterface listener) {
    this.listener = listener;
}

@Override
public void actionPerformed(ActionEvent e) {
    if (e.getSource() instanceof JButton){
        JButton gridButtons = (JButton) e.getSource();
        System.out.println("clicked column " + gridButtons.getClientProperty("column") + ", row " + gridButtons.getClientProperty("row"));
    }
}
}

public class RunSudokuNinja implements Runnable{

@Override
public void run() {
    //SudokuEngineInterface sudokuEngine = new SudokuEngine();
    SudokuView sudokuView = new SudokuView();
    //sudokuView.setSudokuImplementation(sudokuEngine);
    sudokuView.setVisible(true);
}


public static void main(String[] args) {
    EventQueue.invokeLater(new RunSudokuNinja());
}
}
4

1 に答える 1

1

わかりました、http://leepoint.net/notes-java/GUI/structure/40mvc.htmlはとても役に立ちました。基本的に、コントローラーの内部クラスとビューの addListener メソッド。この方法ではるかに優れた組織

于 2013-03-30T11:46:47.873 に答える