setVisibleについて

Question

import java.awt.BorderLayout;
import java.awt.Container;
import java.awt.FlowLayout;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JTextField;

public class Ki {
	public static void main(String[] args) {
	 Game a = new Game();
	 new SSample6_1();
		 }
	}
class Game extends JFrame{
	Game(){
		   super("aa");
		   this.setBounds(600,500,500,500);
		   this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		   this.setVisible(true);
		   setLayout(new FlowLayout());
		   JPanel p = new JPanel();
		   JTextField ko = new JTextField("住所5", 20);
		   p.add(ko);
		   Container contentPane = getContentPane();
		   contentPane.add(p, BorderLayout.CENTER); 
		   }
}
class SSample6_1 extends JFrame {
	   SSample6_1(){
		   super("aa");
		   this.setBounds(600,500,500,500);
		   this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		   this.setVisible(true);
		   setLayout(new FlowLayout());
		   JPanel p = new JPanel();
		   JTextField kotae = new JTextField("住所1", 20);
		   p.add(kotae );
		   Container contentPane = getContentPane();
		   contentPane.add(p, BorderLayout.CENTER);

		  }
	}

上記のようなまったく同じクラスがあります。
これの場合は最初にnewをしたほうがテキストフィールドがなくなります。
３、４つに増やしても最初のだけです。

 Game a = new Game();
	 new SSample6_1();
	 a.setVisible(true);

とすると普通に動くのですが、これだとnewしただけのもおかしくなるはずなのに、おかしいのは最初にnewしたものだけです。
classの中のsetVisibleの効力がなくなってるのも不思議です。
これは何が原因なのでしょうか？

Answer 1

swingコンポーネントを初期化(コンポーネントを生成したり、生成の過程でアクセス)する処理をmainスレッドで行っていることに根本的な原因があると言えます。

swingコンポーネントは本来EDT(Event Dispatch Thread)でのみアクセスされることを意図して設計されています。一方、Javaのmainメソッドはmainスレッド上で起動されます。mainスレッドでswingコンポーネントの初期化処理を書くこともできるのですが、setVisible(true)とした途端にswingに関するGUI処理がEDT上で開始されてしまうため、そのままmainスレッド上でGUIコンポーネントの初期化処理を継続すると、「複数のスレッドからswingのコンポーネントが同時にアクセスされる」ことになるため、動作が保証されなくなります。

対処としては次のようにするのが簡単です。invokeLaterに指定したRunnable#runメソッドはEDTスレッドで動くことが保証されるため期待通りになります。

public class Ki {
  public static void main(String[] args) {
    SwingUtilities.invokeLater(new Runnable() {
      @Override public void run() {
        Game a = new Game();
        new SSample6_1();
      }
    });
  }
}

とはいえ、質問者さんの混乱は無理もないと自分は思います。世の中にswingの初期化コードとして次のようなものを数多く見かけます。

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

  MyFrame() {
    super("title");
    ...
    setDefaultCloseOperation(EXIT_ON_CLOSE);
    pack();
  }
}

上のコードではswingの初期化をmainスレッドでやってますね。しかし、前述の事情があるため、このようなコードで期待どおりに動くためには「setVisibleした以降は一切swingコンポーネントに触ってはいけない」という点を守る必要があります。
必然的に複数のWindowをmainスレッドから表示状態にすることも(本来は)許されないことになります。

例外が発生してくれるわけでもないので、非常に分かりにくいですね

Comments

[toutou]

つまり同時に動作してしまうとクラッシュみたいなことが起こって動作しないということですね。なんとなくわかりました。ありがとうございます。

[KSwordOfHaste]

swingコンポーネントの内部の情報が矛盾した状態になっているといえると思います。