BottunのsetEnabledの切り替え

Question

#作ろうとしているシステムの流れ
0. ユーザに問題を提示し、回答してもらう
（回答するごとにカウンタ変数ｉをカウントしていき、ｉがある数字に到達したら回答終了とみなす）
0. 全て回答を終えたら、CardPanelによる画面の切り替えが可能な「結果表示ボタン」を表示し、結果表示画面に移行する

#実現したいこと
カウンタ変数ｉが一定数カウントされたら、自動的にsetEnabledをtrueに変更し、ボタンが有効になるようにしたい
if文でｉの数値による動作を書いているが、カウンタ変数ｉがある数に到達してもボタンが有効にならない

#疑問・予想
エラーは出ていないためif文は機能していると思うが、cardPanelの再描画がうまくいっていないためボタンが有効になっていても表示されない？

#プログラム

package screen;

import //（省略）
　・
　・
　・

public class Sc extends JFrame implements ActionListener{

    JPanel cardPanel;
    CardLayout CardLayout;
    SpringLayout SpringLayout;
    int i = 0; //カウンタ変数

    public static void main(String[] args) {
        Sc frame = new Sc();
        frame.setTitle("画面遷移テスト");
        frame.setSize(1450, 900);
        frame.setDefaultCloseOperation(EXIT_ON_CLOSE);
        frame.setVisible(true);
    }

    public Sc() {
    	// panel01（省略）

    	// panel02
        JPanel p2 = new JPanel();
        p2.setLayout(null);

　　　　 JButton btn02 = new JButton("C"); //ｉを増やすイベントを持ったボタン
      	btn02.addActionListener(new Code());
        p2.add(btn02)
   	    
        // panel03
        JPanel p3 = new JPanel();
        p2.setLayout(null);
        JButton btn03 = new JButton("panel03");
        p3.add(btn03);

        // CardLayout用パネル
        cardPanel = new JPanel();
        CardLayout = new CardLayout();
        cardPanel.setLayout(CardLayout);

        cardPanel.add(p1, "panel01");
        cardPanel.add(p2, "panel02");
        cardPanel.add(p3, "panel03");

        // カード移動用ボタン
        JButton btn_2to3 = new JButton("結果に進む");
        btn_2to3.addActionListener(this);
        btn_2to3.setActionCommand("panel03");

        JPanel btnPanel = new JPanel();
        btnPanel.add(btn_1to2);
        btnPanel.add(btn_2to3);
        btn_2to3.setEnabled(false);

        if(i==11){
        	cardPanel.validate();
        	cardPanel.repaint();
            btn_2to3.setEnabled(true);
        }

        // cardPanelとカード移動用ボタンをJFrameに配置
        Container contentPane = getContentPane();
        contentPane.add(cardPanel, BorderLayout.CENTER);
        contentPane.add(btnPanel, BorderLayout.PAGE_END);
    }

    public void actionPerformed(ActionEvent e) {
        String cmd = e.getActionCommand();
        CardLayout.show(cardPanel, cmd);
    }
    class Code implements ActionListener{
    		public void actionPerformed(ActionEvent e) {
    			Clip clip = null;
    			AudioInputStream audioInputStream;
    			try{
    			　　//省略
    		        i++;
    		    } catch (UnsupportedAudioFileException e1) {
    		    	e1.printStackTrace();
    		    } catch (LineUnavailableException e1) {
    		        e1.printStackTrace();
    		    } catch (IOException e1) {
    		        e1.printStackTrace();
    		    }
    		}
	}
}

質問を見やすくするために何箇所か省略させていただいておりますが、
その関係でプログラムが変になっている箇所がありましたら申し訳ありません。
ご回答よろしくお願い致します。

#追記
修正前

public class Sc extends JFrame implements ActionListener{
    public static void main(String[] args) {
        Sc frame = new Sc();
        frame.setTitle("画面遷移テスト");
        frame.setSize(1450, 900);
        frame.setDefaultCloseOperation(EXIT_ON_CLOSE);
        frame.setVisible(true);
    }
    public Sc(){
        ・
        ・
        ・
    }
    ・
    ・
    ・
}

修正後

public class Sc extends JFrame implements ActionListener{
    public static void main(String[] args) {
        new Sc().setVisible(true);
    }
    public Sc(){
        setSize(1450, 900);
        setLocationRelativeTo(null);
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        ・
        ・
        ・
    }
    ・
    ・
    ・
}

Comments

[shiketa]

> 質問を見やすくするために何箇所か省略させていただいておりますが、

もし、わたしが回答できるかどうかを検討するとしたら、質問内容が読みやすく理解可能であることはもちろんですが、余計な部分を省いてあり、コピペしただけで問題や疑問を再現できるソースが、提示されているかどうか。問題を抱えたヒトは「読みやすく」するために省略しているのかもしれませんが、その原因の所在がわかっていないのに、省略できるものでしょうか？省略した部分に原因がある、かもしれない、とは考えませんか。
最低限のソースで問題を再現できるところまでソースを簡潔にしてゆけば、その過程で原因が浮き上がってくることは、ままあります。浮き上がればそれで解決。解決できなくても、疑問を提示するソースとしては役に立つものになることでしょう。

Answer 1

3 枚のパネルと、それが入る CardLayout のパネル及びカードを切り替えるボタンのパネルをそれぞれ(JPanel を継承した)クラスとしてみました。(フレームクラスの内部クラスにはしていません。)

"i" は二枚目のクラスにあり、カウントオーバー時にナビゲーションのメソッドを呼ぶことで三枚目へのボタンを有効にしています。
"i" の値の確認用に JTextField を付けています。

一枚目と二枚目の間のナビゲーションが分からないので二枚目に行ったらもう一枚目に戻れない状態になっていますが、"ふぁいなるあんさー" を押した時まだ三枚目に移動する状態で無ければ一枚目に戻る(次の問題？)のでしょうか。

package teratail_java.q358814;

import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.*;

public class Q358814 extends JFrame {
  public static void main(String[] args) {
    new Q358814().setVisible(true);
  }

  public Q358814() {
    super("画面遷移テスト");
    setSize(450, 300);
    setDefaultCloseOperation(EXIT_ON_CLOSE);
    setLocationRelativeTo(null);

    //コンテンツ
    QuestionPanel p1 = new QuestionPanel();
    AnswerPanel p2 = new AnswerPanel();
    ResultPanel p3 = new ResultPanel();

    ContentsPanel cardPanel = new ContentsPanel(p1, p2, p3);

    Container contentPane = getContentPane();
    contentPane.add(cardPanel, BorderLayout.CENTER);
    contentPane.add(cardPanel.getNavigationPanel(), BorderLayout.SOUTH);

    p2.setCallback(new AnswerPanel.Callback() {
      @Override
      public void countOver() {
        cardPanel.getNavigationPanel().setToResultButtonEnabled(true);
      }
    });
  }
}

//一枚目(panel01)
class QuestionPanel extends JPanel {
  QuestionPanel() {
    super();
    JLabel questionLabel = new JLabel("問題？");
    add(questionLabel);
  }
}

//二枚目(panel02)
class AnswerPanel extends JPanel {
  interface Callback {
    void countOver(); //"ふぁいなるあんさー"ボタンを10回押すと呼ばれる.
  }
  private Callback callback = null;
  private int i = 0;
  AnswerPanel() {
    super();

    JTextField countText = new JTextField(10); //i確認用
    countText.setText(""+i);
    add(countText);

    JButton finalAnswer = new JButton("ふぁいなるあんさー？"); //ｉを増やすイベントを持ったボタン
    finalAnswer.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent e) {
        //Code.XXX(); //音を鳴らす
        i++;
        countText.setText(""+i);
        if(i >= 10 && callback != null) {
          callback.countOver();
        }
      }
    });
    add(finalAnswer);
  }
  void setCallback(Callback callback) {
    this.callback = callback;
  }
}

//三枚目(panel03)
class ResultPanel extends JPanel {
  ResultPanel() {
    super();
    JLabel resultLabel = new JLabel("結果？");
    add(resultLabel);
  }
}

//メインパネル
class ContentsPanel extends JPanel {
  private enum Card {
    PANEL01, PANEL02,PANEL03;
  }

  private CardLayout cardLayout = new CardLayout();
  private NavigationPanel navigationPanel = null;

  ContentsPanel(JPanel question, JPanel answer, JPanel result) {
    super(null);
    setLayout(cardLayout);

    add(question, Card.PANEL01.name());
    add(answer, Card.PANEL02.name());
    add(result, Card.PANEL03.name());
  }

  private void showCard(Card card) {
    cardLayout.show(this, card.name());
  }

  NavigationPanel getNavigationPanel() {
    if(navigationPanel == null) navigationPanel = new NavigationPanel();
    return navigationPanel;
  }

  //ナビゲーション
  class NavigationPanel extends JPanel {
    private JButton toResultButton;
    private NavigationPanel() {
      super();

      JButton toAnswerButton  = new JButton("回答に進む"); //btn_1to2
      toAnswerButton.addActionListener(new ActionListener() {
        @Override
        public void actionPerformed(ActionEvent e) {
          toAnswerButton.setEnabled(false);
          showCard(Card.PANEL02);
        }
      });

      toResultButton = new JButton("結果に進む"); //btn_2to3
      toResultButton.setEnabled(false);
      toResultButton.addActionListener(new ActionListener() {
        @Override
        public void actionPerformed(ActionEvent e) {
          toResultButton.setEnabled(false);
          showCard(Card.PANEL03);
        }
      });

      add(toAnswerButton);
      add(toResultButton);
    }
    void setToResultButtonEnabled(boolean b) {
      toResultButton.setEnabled(b);
    }
  }
}

Answer 2

前回の質問の続きですよね？
Java - 親クラスの変数を子クラスで変更し、親クラスに返す｜teratail

前回の質問をどう解決されたかわからないので複数クラスに分けましたが、内部クラス等でいいならもっと簡単な話になります。

カウンタ変数ｉが一定数カウントされたら、自動的にsetEnabledをtrueに変更し、ボタンが有効になるようにしたい

これはかなり本質を突いた問いでして、そういうことができれば便利なのですが、ただのint変数にはそういう機能はありません。
Scの外でiを変えてもScは知りようがないし、逆にScがiを変えてもほかのクラスからはわかりません。

もちろんiが変わった後にiを見れば増えてはいます（いつ増えたのかはわからない）

簡単な解決策は直接iを変えずに、iを変えるメソッドを用意することです。
そうすればScはいつiが変わるのかを把握できます。
こういうメソッドをセッター（setter）といいます。

import java.awt.BorderLayout;
import java.awt.CardLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;


public class Sc extends JFrame implements ActionListener {
    public static void main(String[] args) {
        new Sc().setVisible(true);
    }

    private final JPanel cardPanel;
    private final CardLayout CardLayout;
    private final JButton toResult;

    private int i = 0;

    public Sc() {
        setSize(400, 300);
        setLocationRelativeTo(null);
        setDefaultCloseOperation(EXIT_ON_CLOSE);

        cardPanel = new JPanel();
        CardLayout = new CardLayout();
        cardPanel.setLayout(CardLayout);

        cardPanel.add(new QuestionPanel(this), "question");

        JPanel result = new JPanel();
        result.add(new JButton("result"));
        cardPanel.add(result, "result");

        JPanel btnPanel = new JPanel();
        toResult = new JButton("結果に進む");
        toResult.addActionListener(this);
        toResult.setActionCommand("result");
        toResult.setEnabled(false);
        btnPanel.add(toResult);

        add(cardPanel, BorderLayout.CENTER);
        add(btnPanel, BorderLayout.PAGE_END);
    }

    // public int getI() {return i;} // ゲッター（getter）
    // public void setI(int i) {this.i = i;} // セッター（setter）
    // ゲッターセッターでもいいのだが、「1ずつ増やす」しかしないのでより明確に
    public void incrementI() {
        i++;
        if (11 <= i) {
            toResult.setEnabled(true);
        }
    }

   @Override public void actionPerformed(ActionEvent e) {
        CardLayout.show(cardPanel, e.getActionCommand());
    }
}

class QuestionPanel extends JPanel implements ActionListener {
    private final Sc sc;

    QuestionPanel(Sc sc) {
        this.sc = sc;

        JButton yes = new JButton("〇");
        yes.addActionListener(this);
        yes.setActionCommand("yes");
        add(yes);

        JButton no = new JButton("×");
        no.addActionListener(this);
        no.setActionCommand("no");
        add(no);
    }

    @Override public void actionPerformed(ActionEvent e) {
        sc.incrementI();
    }
}

---

ただのint変数にはそういう機能はありません。

と言いましたが、実はそういうことができる仕組みは用意されています（Swingで活用している例はほとんど見ないのですが^^;
PropertyChangeListener (Java Platform SE 8 )
PropertyChangeSupport (Java Platform SE 8 )

import java.awt.BorderLayout;
import java.awt.CardLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.beans.PropertyChangeEvent;
import java.beans.PropertyChangeListener;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.event.SwingPropertyChangeSupport;


public class Sc extends JFrame implements ActionListener, PropertyChangeListener {
    public static void main(String[] args) {
        new Sc().setVisible(true);
    }

    private final JPanel cardPanel;
    private final CardLayout CardLayout;
    private final JButton toResult;
    private final QuestionModel model;


    public Sc() {
        setSize(400, 300);
        setLocationRelativeTo(null);
        setDefaultCloseOperation(EXIT_ON_CLOSE);

        model = new QuestionModel();
        model.addPropertyChangeListener(this);

        cardPanel = new JPanel();
        CardLayout = new CardLayout();
        cardPanel.setLayout(CardLayout);

        cardPanel.add(new QuestionPanel(model), "question");

        JPanel result = new JPanel();
        result.add(new JButton("result"));
        cardPanel.add(result, "result");

        JPanel btnPanel = new JPanel();
        toResult = new JButton("結果に進む");
        toResult.addActionListener(this);
        toResult.setActionCommand("result");
        toResult.setEnabled(false);
        btnPanel.add(toResult);

        add(cardPanel, BorderLayout.CENTER);
        add(btnPanel, BorderLayout.PAGE_END);
    }

    @Override public void actionPerformed(ActionEvent e) {
        CardLayout.show(cardPanel, e.getActionCommand());
    }

    @Override public void propertyChange(PropertyChangeEvent evt) {
        if (11 <= model.getI()) {
            toResult.setEnabled(true);
        }
    }
}

class QuestionPanel extends JPanel implements ActionListener {
    private final QuestionModel model;

    QuestionPanel(QuestionModel model) {
        this.model = model;

        JButton yes = new JButton("〇");
        yes.addActionListener(this);
        yes.setActionCommand("yes");
        add(yes);

        JButton no = new JButton("×");
        no.addActionListener(this);
        no.setActionCommand("no");
        add(no);
    }

    @Override public void actionPerformed(ActionEvent e) {
        model.incrementI();
    }
}

class QuestionModel {
    private final SwingPropertyChangeSupport pcs = new SwingPropertyChangeSupport(this);

    public void addPropertyChangeListener(PropertyChangeListener listener) {
        pcs.addPropertyChangeListener(listener);
    }

//    public void removePropertyChangeListener(PropertyChangeListener listener) {
//        pcs.removePropertyChangeListener(listener);
//    }

    private int i;

    public int getI() { return i; }

//    public void setI(int newValue) {
//        int oldValue = i;
//        i = newValue;
//        pcs.firePropertyChange("i", oldValue, newValue);
//    }

    public void incrementI() {
        pcs.firePropertyChange("i", i++, i);
    }
}

あまり変わり映えしませんが、変更を知れるクラスをいくつでも登録できます。

---

追記

内部クラスで変数を扱うことにしております。

Codeは内部クラスになっていませんが、書き間違いだとして^^;

内部クラスでいいなら話は簡単です。
外（Sc）の変数がそのまま使えるので、i++したついでにチェックするだけです。

import java.awt.BorderLayout;
import java.awt.CardLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class Sc extends JFrame implements ActionListener {
    public static void main(String[] args) {
        new Sc().setVisible(true);
    }

    private final JPanel cardPanel;
    private final CardLayout cardLayout;
    private final JButton btn_2to3;

    private int i = 0;

    public Sc() {
        setSize(400, 300);
        setLocationRelativeTo(null);
        setDefaultCloseOperation(EXIT_ON_CLOSE);

        JPanel p2 = new JPanel();
        JButton btn02 = new JButton("C");
        btn02.addActionListener(new Code());
        p2.add(btn02);

        JPanel p3 = new JPanel();
        JButton btn03 = new JButton("panel03");
        p3.add(btn03);

        cardPanel = new JPanel();
        cardLayout = new CardLayout();
        cardPanel.setLayout(cardLayout);

        cardPanel.add(p2, "panel02");
        cardPanel.add(p3, "panel03");

        btn_2to3 = new JButton("結果に進む");
        btn_2to3.addActionListener(this);
        btn_2to3.setActionCommand("panel03");

        JPanel btnPanel = new JPanel();
        btnPanel.add(btn_2to3);
        btn_2to3.setEnabled(false);

        add(cardPanel, BorderLayout.CENTER);
        add(btnPanel, BorderLayout.PAGE_END);
    }

    public void actionPerformed(ActionEvent e) {
        String cmd = e.getActionCommand();
        cardLayout.show(cardPanel, cmd);
    }


    class Code implements ActionListener {
        public void actionPerformed(ActionEvent e) {
            i++;
            if (i == 11) {
                btn_2to3.setEnabled(true);
            }
        }
    }
}

Comments

[退会済みユーザー]

前回に引き続き、ご回答ありがとうございます。
内部クラスで変数を扱うことにしております。
ｉを増やすイベントを記載していなかったのでプログラムを修正させていただいたのですが（下方のCodeクラスでｉの変更を行います）、ｉの変更はScクラスの中で行う必要がある、という認識で間違っていないでしょうか？

[TN8001]

内部クラス例 追記

[退会済みユーザー]

ありがとうございます。
ご指摘いただいた内容で修正したところ、うまく作動しました。
追記欄に修正前と修正後のmainメソッドを記載させていただいたのですが、
どちらもframeの設定をしているように見えるのですが、これらはどのような違いがあるのでしょうか？
重ねての質問になってしまい申し訳ありません。

[TN8001]

> これらはどのような違いがあるのでしょうか？

そんなに深い理由はありません^^; 慣れた書き方や好みの問題です^^

どちらも意味は同じなのですが、個人的に
* mainは一番上にあってほしい（IDEで実行するときに行番号のあたりに出る実行ボタンを押しやすい）
* mainはできるだけ短くしたい（mainのあとにフィールドが来ることになるので）

といったことが理由です。実行結果に違いはありません。

jimbeさんも同じようにされていますが、同様の理由かはちょっとわかりません。

znrさんに直せと言っているわけではないです。好きに書いてください^^;

[退会済みユーザー]

参考になりました。ありがとうございました！

[jimbe]

> jimbeさんも同じようにされていますが
あ、ハイ。

私の場合は main はあくまで起動するだけということにしています。
プログラムとしてのメインは JFrame を実装しているクラスというつもりです。…まぁ、"画面中央に表示する" を起動する側が決めるべきか起動される側が決めるべきかとか微妙ではあるのですが (=_=;

公式というか正式というか、Swing の起動は本当は setVisible だけでは無く、EDT 経由で実行しないといけないのですね。
ですが覚えてなくて面倒なので、とりあえず一行で書いておいて、ちゃんと書く場合にはそこだけ置き換える感じにするというか ^^;;;

Answer 3

public class Sc extends JFrame implements ActionListener{
    ...
    int i = 0; //カウンタ変数
    ...

    public Sc() {
        ...
        if(i==11){
            cardPanel.validate();
            cardPanel.repaint();
            btn_2to3.setEnabled(true);
        }

1. Sc()はScクラスのコンストラクタ。
2. mainメソッドで、Sc frame = new Sc(); としたときほんの一回しか実行されない。
3. コンストラクタの中で、if(i==11){としても、iは、初期値の1のまま。だからbtn_2to3.setEnabled(true);にたどり着くことは、絶対にない。

すべてを提示していないので、iがどのタイミングで更新されるのかわらないが、そのあたりにif(i==11){...をもっていく必要がある。おそらく。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
ｉを増やすイベントを記載していなかったのでプログラムを修正させていただきました（下方のCodeクラスでｉの変更を行います）。
Sc()は一度しか呼び出されていないためｉが変化してもSc()内のif文が機能しない、ということでしょうか？

[shiketa]

TN8001さんの回答を参考にしてください。

[shiketa]

\# ちょっと(かなりかな)方向がずれますが。
ご存知かとおもいますが、SwingではなくJavaFXというGUIフレームワークがあります。Java9以降はJDKにバンドルされなくなったので自力で手間をかけてインストール/セットアップする必要がありますが、今回のようなケースを一行で解決する手段があります。
ButtonのdisablePropertyと、iを保持するIntegerPropertyとをBindingsで関連付けする。
`button.disableProperty.bind(Bindings.greaterThan(11, iProperty));`
これで、iPropertyが11より大きくなると、ボタンが無効になる。
https://docs.oracle.com/javase/jp/8/javafx/api/javafx/beans/binding/Bindings.html#greaterThan-int-javafx.beans.value.ObservableNumberValue-
余計な情報でしょうが、参考まで。

[TN8001]

私はC#でWPFが得意なのですが、JavaFXも同じような思想ですよね。

しかしいまだにSwingやWinFormsのほうが多数派な感じで残念です^^;