Java 別プログラムのclassを呼び出し、GUIを表示したい

Question

現在javaを用いて、音楽ゲームを作成しています。
その中で、以下のような動作をする2つのjavaファイルを用意しています。

top.java
プログラム起動後のトップ画面の表示‐＞曲選択画面の表示

play.java
音楽再生、譜面描画、判定など音楽ゲームを動かす。

そこで、top.javaからplay.java に飛ばして、音楽ゲームの動作をするプログラムと、それ以外の処理を行うプログラムで分けて動作させたいと思い
play.javaをインスタンス化してtop.javaから呼び出したのですが、
play.java内の音楽再生、譜面読み込み等の他の処理は動いているのですが、画面描画だけ上手く動かず、ただ白い画面（デフォルトの薄いグレーではない）が表示されるのみです。
別プログラムのGUIを表示させるとき、何か特別な処理等が必要なのでしょうか。

top.java

import 省略

public class top extends JFrame implements ActionListener{


	static top frame = new top("*  。ｖｉｏｌｉｎ ｍａｎｉａ。 *");
	static JFrame frames = new JFrame("* 。 ｍｕｓｉｃ ｓｅｌｅｃｔ。 *");//曲選択
	public static int[] sec = new int[1000];
	public static  int[][] fumen = new int[1000][2];
	static play startgame = new play();
	
	

  public static void main(String args[]){
    frame.setVisible(true);
  }	
    	
	top(String title){
   	 	setTitle(title);
    	setBounds(600,10,600,1000);
   	 	setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		
   	 	JPanel p = new JPanel();
   	 		p.setLayout(null);
   	 		p.setOpaque(true);
   	 		
   	 		
   	 	ImageIcon top_bg = new ImageIcon("top.png");
   	 		JLabel bg1 = new JLabel(top_bg);
   	 		bg1.setBounds(0,0,600,1000);  
		
   		ImageIcon icon2 = new ImageIcon("option.png");
   		JButton fileButton = new JButton(icon2);
    		fileButton.setBounds(370,750,200,50);
   			fileButton.addActionListener(this);
   		
    	ImageIcon icon1 = new ImageIcon("start.png");
    	JButton button = new JButton(icon1);
    		button.setBounds(370,600,200,50);    	
    		button.addActionListener(this);
    		button.setActionCommand("start");
    	
    	p.add(bg1);
    	p.add(button);
    	p.add(fileButton);
    

		
		Container contentPane = getContentPane();
    	contentPane.add(p, BorderLayout.CENTER);

	}
	public void actionPerformed(ActionEvent e){
		String cmd = e.getActionCommand();
		AudioClip audio = null;
	if (cmd.equals("start")){
		frame.setVisible(false);
		frames.setBounds(600,10,600,1000);
		frames.getContentPane().setBackground(Color.black);
		frames.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		JPanel p_select = new JPanel();
   	 		p_select.setLayout(null);
   	 		p_select.setOpaque(false);
			
		ImageIcon concon1 = new ImageIcon("./music/concon/concon_out.png");
		ImageIcon concon2 = new ImageIcon("./music/concon/concon_over.png");
		JButton music3 = new JButton(concon1);
			music3.setBounds(100,500,410,116);
			music3.setRolloverIcon(concon2);
			music3.setPressedIcon(concon2);
			music3.addActionListener(this);
			music3.setActionCommand("./music/concon/concon");
				
		p_select.add(music3);
			
		
		Container contentPane2 = frames.getContentPane();
		contentPane2.add(p_select, BorderLayout.CENTER);
		    	
		frames.setVisible(true);

 	}else if (cmd.equals("./music/concon/concon")){
 		frames.setVisible(false);
 		startgame.game();//ここでplay.javaを呼び出す。


        audio.stop();
 	}
 	}
	
	  
}

import 省略

public class play {

	public static void game(){
		vimani action = new vimani();
		action.viner();
	}
}

class vimani extends JPanel implements KeyListener{

	private JFrame frameC;
	public lanePanel violin;
	public static int[] sec = new int[1000];
	public static int[][] fumen = new int[1000][2];
	public static long gets = 0;
	static int muchmicros = 0;
	static int y1ic=0;static int y2ic=0;static int y3ic=0;static int y4ic=0;
	static int[] y1 = new int[300];
	static int[] y2 = new int[300];
	static int[] y3 = new int[300];
	static int[] y4 = new int[300];
	static int speed = 7;
	static String filen = "concon";
	static int startcounter=0;
	static Sounder sounder = new Sounder();
	public void viner(){
		window();//ウィンドウを生成
		for(int i=0;i<300;i++){
			y1[i] = 0;
			y2[i] = 0;
			y3[i] = 0;
			y4[i] = 0;
		}
		read_fumen(filen);
		runi();
	}
				
	public void window(){
	
		frameC = new JFrame();
		frameC.setTitle("Violinist");
		frameC.setBounds(600,10,600,1000);//x=600,y=10,横600,縦1000のフレームを準備
		frameC.setLayout(null);//レイアウトマネージャ使用しない
		frameC.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);//一緒に消す
	
		violin = new lanePanel();
		violin.setOpaque(false);
		violin.setLayout(null);
		
		violin.setSize(600,1000);
		violin.setBackground(new Color(31,38,56,255));
		
		Container contentPane3 = frameC.getContentPane();
		contentPane3.add(violin, BorderLayout.CENTER);
		
	
		//frame.add(violin);

		frameC.setVisible(true);

		frameC.addKeyListener(this);
		violin.addKeyListener(this);
		
	}
	
	public void runi(){//指定msでrepaint()を行うメソッド
				violin.repaint();//repaintを呼び出す。
		}
		
public static void fumeneed(){
	//譜面データを読む処理
}
	    
public static void musicstart(String selectMusic){
	//音楽再生処理
}
    
    public static void read_fumen(String fileName){
		//譜面ファイルをオープンする処理
}

	private static boolean checkBeforeReadfile(File file){
    		//譜面ファイルを読むのに必要な処理
  	}

	public void keyPressed(KeyEvent e){
  	}
    	public void keyReleased(KeyEvent e){
    	}
   	public void keyTyped(KeyEvent e){
   		char key = e.getKeyChar();
		System.out.println(key);
    		searchHitNotes(key);//キー判定
    	}	
public void searchHitNotes(char hitKey){
	//キーの押したタイミングで点数をつける処理
}
}

class Sounder implements LineListener{
	//音楽再生準備、再生中かどうかの判定等をするクラス
}
	class lanePanel extends JPanel{//譜面描画メソッド
			
		@Override
		public void paintComponent(Graphics g) {
				super.paintComponent(g);
				
				// レーン
				g.setColor(new Color(153,51,51,255));
				g.fillRect(220,10, 30, 650);
				g.fillRect(255,10, 30, 650);
				g.fillRect(290,10, 30, 650);
				g.fillRect(325,10, 30, 650);
				
				g.setColor(new Color(255,255,255,255));
				g.fillRect(250,10, 5, 650);
				g.fillRect(285,10, 5, 650);
				g.fillRect(320,10, 5, 650);
				
				g.setColor(new Color(255,180,0,255));
				g.fillRect(220,590,135,20);				
				//ノート
				g.setColor(new Color(102,255,204,255));//ノーツの色
				for(int i=vimani.one+1;i<vimani.y1ic;i++){
					if(vimani.y1[i] <640){
						g.fillRoundRect(220, vimani.y1[i], 30, 20, 5, 10);
						vimani.y1[i]+=vimani.speed;
						}else{
							vimani.y1[i] = 990;
							vimani.one = i;
						}
				}
				for(int i=vimani.two+1;i<vimani.y2ic;i++){
					if(vimani.y2[i] <640){
						g.fillRoundRect(255, vimani.y2[i], 30, 20, 5, 10);
						vimani.y2[i]+=vimani.speed;
					}else{
						vimani.y2[i] = 990;
						vimani.two = i;
					}
				}
				for(int i=vimani.three+1;i<vimani.y3ic;i++){
					if(vimani.y3[i] <640){
					g.fillRoundRect(290, vimani.y3[i], 30, 20, 5, 10);
					vimani.y3[i]+=vimani.speed;
					}else{
					vimani.y3[i] = 990;
					vimani.three = i;
					}
				}
				for(int i=vimani.four+1;i<vimani.y4ic;i++){
					if(vimani.y4[i] <640){
						g.fillRoundRect(325, vimani.y4[i], 30, 20, 5, 10);
						vimani.y4[i]+=vimani.speed;
					}else{
						vimani.y4[i] = 990;
						vimani.four=i;
					}
				}

		}
		
		public void addObj(){
		}
	}

Comments

[argius]

質問に記載されている状態だとコンパイルエラーになりますね。それを補完した場合でも、白い画面は出てきませんでした。できれば再現時の正確な情報を記載してください。エラーメッセージが出ていたらそれも記載してください。

[Puhu]

文字数制限から描画処理に関係のないメソッドの中身を省略したため、エラーが出たのかと思います。こちらのアップローダー様にzipファイルごと置かせていただきましたので宜しければこちらの方でお願いいたします。

http://whitecats.dip.jp/up/download/1452487504.zip

解凍後、直下に置いてある.java、classファイルのみで動作します。中のtopファイル、playファイルは手違いでそのままにしてしまいましたので気にしないでください。
PW 11223

Answer 1

runi()メソッドが、イベントディスパッチスレッド（描画を行うスレッド）で実行されているのが問題です。

runi()の処理がイベントディスパッチスレッドを占有しているため、いつまでも描画処理が終わらず、画面が白くなっているのです。
タイミングを処理するスレッドは、描画スレッドとは別にしてください。

これで少なくとも画面が白くなる問題は解決するはずです。

デバッグ用に、

System.out.println("実行中のスレッド: " + Thread.currentThread());

を入れてみれば、処理がどのスレッドで実行されているのかが分かります。
AWT-EventQueueというスレッド名なら、それがイベントディスパッチスレッドです。

Comments

[Puhu]

ありがとうございます、調べながら対応してみようと思います。
質問なのですが、プログラムを統合する前(play.java)単体ではちゃんと動いていたのですが、統合するとイベントディスパッチスレッドというものは変わってしまうのでしょうか？

[argius]

> 統合するとイベントディスパッチスレッドというものは変わってしまうのでしょうか？

統合したことが原因ではなく、呼び出すスレッドが変わったのが原因です。

統合前は（おそらく）mainスレッドからplayクラスを起動していたのを、
統合後はボタンのアクションからplayクラスを起動するように変えたことで、ボタンのアクションを処理しているイベントディスパッチスレッドで実行されるようになっています。

[Puhu]

なるほど！　ありがとうございます！　わかりやすい説明で助かりました；

[Puhu]

runi()の中身を新しくThreadを作り、その中で動かすようにしたら動きました、ありがとうございました!