7

ボタンをクリックしたときに、ボタンを「開始」から「停止」に変更しようとしています。これを行うための私の試みは以下のとおりです。それを調べてガイドをコピーしようとしましたが、何が間違っているのかわかりません。無関係なコードをたくさん省略したため、「}」が欠落している可能性があります。誰かが私が間違っていることを見ることができますか？ 

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.event.*;

public class PipeGameApp extends JFrame implements ActionListener {

    private static int BOARD_SIZE = 11;
    private PipeGame game;      // The model
    private PipeGameView view;      // The view

    // This constructor builds the window
    public PipeGameApp(String title) {
        super(title);

        game = new PipeGame(BOARD_SIZE);
        view = new PipeGameView(game);

        //THE TOP BAR
        JPanel topBar = new JPanel();
        JButton startButton = new JButton("Start");
        startButton.addActionListener(this);



        ButtonGroup bg1 = new ButtonGroup();
        JRadioButton rb1 = new JRadioButton("2 minutes", true);
        rb1.addActionListener(this);


        JRadioButton rb2 = new JRadioButton("10 minutes", false);
        JRadioButton rb3 = new JRadioButton("No Time Limit", false);
        bg1.add(rb1);
        bg1.add(rb2);
        bg1.add(rb3);





        topBar.add(startButton);
        topBar.add(rb1);
        topBar.add(rb2);
        topBar.add(rb3);
        //END OF TOP BAR

        //THE BOTTOM BAR
        JPanel bottomBar = new JPanel();
        JLabel timeLeft = new JLabel("Time Left: ");
        JProgressBar bar = new JProgressBar();
        bottomBar.add(timeLeft);
        bottomBar.add(bar);
        bottomBar.setVisible(false);
        //end of bottom

        /*
         //bottom 2
         int fscore=10;
         JPanel bottomBar2 = new JPanel();
         JLabel score = new JLabel("Final score:" + fscore);
         bottomBar2.add(score);
         bottomBar2.setVisible(false);
         */


        getContentPane().add(view); //CHANGE LOCATION OF BOARD GAME HERE BorderLayout.SOUTH

        getContentPane().add(topBar, BorderLayout.NORTH);
        getContentPane().add(bottomBar, BorderLayout.SOUTH);

        //getContentPane().add(bottomBar2, BorderLayout.SOUTH);



        // Add the listeners to the view's buttons
        for (int r = 0; r < BOARD_SIZE; r++) {
            for (int c = 0; c < BOARD_SIZE; c++) {
                view.getButton(r, c).addActionListener(new ActionListener() {
                    public void actionPerformed(ActionEvent e) {
                        handleTileSelection(e);
                    }
                });
            }
        }

        setDefaultCloseOperation(EXIT_ON_CLOSE);
        //setSize(446,466);
        setSize(446, 530);
        setResizable(false);
    }

    // Handle a Tile Selection.   Just change the model and update the view.
    private void handleTileSelection(ActionEvent e) {
        // Find the row and column of the pressed button, then make the change
        int r = 0, c = 0;
        for (int i = 0; i < BOARD_SIZE; i++) {
            for (int j = 0; j < BOARD_SIZE; j++) {
                if (e.getSource() == view.getButton(i, j)) {
                    if (game.placePipe(i, j)) {
                        view.update();
                    }
                    return;
                }
            }
        }
    }

    // This is where it all begins
    public static void main(String[] args) {
        new PipeGameApp("The Frantic Pipe Layer").setVisible(true);
    }

    @Override
    public void actionPerformed(ActionEvent ae) {
        startButton.setText("asdf");
    }
}
4

2 に答える 2

5

あなたの問題は、限られた変数の範囲の1つです。startButton 変数はコンストラクターで宣言されているため、コンストラクターでのみ表示されます。コンストラクターで再宣言するのではなく、クラスで宣言する必要があります。これにより、クラスの残りの部分が変数を「参照」して使用できるようになります。

つまり、次のように変更します。

public class PipeGameApp extends JFrame implements ActionListener {

    private static int BOARD_SIZE = 11;
    private PipeGame game;      // The model
    private PipeGameView view;      // The view

    public PipeGameApp(String title) {

        JButton startButton = new JButton("Start");
        startButton.addActionListener(this);
        // etc...

これに：

public class PipeGameApp extends JFrame implements ActionListener {

    private static int BOARD_SIZE = 11;
    private PipeGame game;      // The model
    private PipeGameView view;      // The view
    private JButton startButton; // *** note change ***

    public PipeGameApp(String title) {

        startButton = new JButton("Start"); // *** note change ***
        startButton.addActionListener(this);
        // etc...

または:

  • JToggleButton を使用する
  • または、ActionEvent のメソッドによって返されたオブジェクトを使用し、現在のgetSource()状態に基づいて新しい状態を設定します。

例えば、

@Override
public void actionPerformed(ActionEvent ae) {
    Object source = ae.getSource();
    if (source instanceof JButton) {
       if (ae.getText().equals("Start")) {
          ae.setText("Stop");
          // do other stuff
       } else if (ae.getText().equals("Stop")) {
          ae.setText("Start");
          // do more stuff
       }
    }
}

について"I might be missing some "}" because I left out a lot of the code that's irrelevant."このようなことにならないようご尽力ください。この欠落している「}」は欠落してはならず、コードを理解して支援することが難しくなります。他の人に、暇なときにあなたを助けるために努力するように頼んでいるのであれば、ジャンク コードを投稿しないように求めすぎているわけではありません。

于 2013-03-08T03:05:19.753 に答える
2

mainメソッドの後にコンストラクターの終了ブレースを保持しました。これが修正されたコードです。

public PipeGameApp(String title) {
    super(title);

    JPanel topBar = new JPanel();
    JButton startButton = new JButton("Start");
    startButton.addActionListener(this);
}

public static void main(String[] args) {
    new PipeGameApp("The Frantic Pipe Layer").setVisible(true);
}

@Override
public void actionPerformed(ActionEvent ae) {
    startButton.setText("asdf");
}
于 2013-03-08T03:05:13.713 に答える