javaのSwingにて背景に画像を表示したいがされない問題

Question

###前提・実現したいこと
javaのswingでGUIについて勉強しています。今、背景に画像を使ったパネルを作り、さらにその上に一回り小さいデフォルトのパネルを作成して表示したいと考えています。
###発生している問題・エラーメッセージ
エラーメッセージはでていませんが、Mainクラスを実行すると大小二つのタブが生成されます。どちらも何も表示されていません。

###該当のソースコード

package スタート画面;

import java.io.IOException;

import javax.swing.JFrame;

public class Main extends JFrame {

	public Main(){
		super("test");
		setResizable(false);
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		pack();
		setVisible(true);
	}
	public static void main(String[] args) throws IOException {
		Main main = new Main();
		StartScreen jp = new StartScreen();
		main.getContentPane().add(jp);

	}
}

package スタート画面;

//import java.awt.CardLayout;
import java.io.IOException;

import javax.swing.ImageIcon;
import javax.swing.JPanel;

public class StartScreen extends JPanel {
	private ImageIcon img;
	//private CardLayout layout = new CardLayout();
	private JPanel mainpane;

	public StartScreen() throws IOException {
		//背景
        BufferedImage img;
		try {
			  img = ImageIO.read(new File("p0131_l.jpg"));
			} catch (Exception e) {
			  e.printStackTrace();
			  img = null;
			}
		//img = new ImageIcon("p0131_l.jpg");
		ImagePanel backpane = new ImagePanel(img);
		backpane.setVisible(true);

		//メインパネル
		mainpane = new JPanel();
		mainpane.setLayout(null);
		mainpane.setBounds(400, 400, 900, 500);
		mainpane.setVisible(true);
		backpane.add(mainpane);
	}
}

package スタート画面;

import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Image;

import javax.swing.JPanel;

public class ImagePanel extends JPanel{
	private BufferedImage image;
	/*
	 * コンストラクタ引用（Java Swing Hacks,Joshua Marinacci著）
	*/
	public ImagePanel(BUfferedImage image){
		this.image = image;
		Dimension size = new Dimension(image.getWidth(null),image.getHeight(null));
		setPreferredSize(size);
		setMaximumSize(size);
		setMinimumSize(size);
		setLayout(null);
	}
	public void paintComponent(Graphics g){
		g.drawImage(image, 0, 0, null);
	}
}

StartScreenのフィールドでカードレイアウトがインスタンス化されてるのはいずれデフォルトのパネルに適応しパネルを追加して画面遷移を行いたいと考えているからです。
###補足情報(言語/FW/ツール等のバージョンなど)
作業環境はEclipseです。画像はインポートで取り込み同じプロジェクトファイル内にあります。画像の大きさは1200×1035です。

Answer 1

なんというか、いろいろと不可解なコードです。(修正確認しました)

• 命名がおかしい

MainとImagePanelがJFrame?SmartScreenがJPanel?

• 画像の取り扱いがおかしい

1200x1035の画像はアイコンとは呼べないと思います。
また、どうしてJFrameにべた張りしようとしているのでしょうか。

• コンポーネントの自己主張が激しい

特にSmartScreen。自身をaddする前にsetVisible(true)にしてしまっています。
あとMainのコンストラクタは完全に無駄だと思います。

• setLayout(null)

せっかくlayout用意したのに使わないんですか？

もっと簡潔に書けるはずです。
重複して書いているからフレームも二つ出て、処理の順序が前後しているからまっしろです。

追記

デバッグしてみましたが、それなりの量書き換える必要がありました。
要点は次のとおりです。

• pack()とsetVisible(true)の呼ばれるタイミングを適切にする

これらのメソッドはその性格上、コンポーネントの配置がすべて終わった時に呼ばれるべきです。

• SmartScreenがパネルとして機能していない

フィールドのmainpaneに対して操作をしているので、SmartScreenのインスタンスjpは真っ白です。

• パネルの親子関係に意識を向ける

下記のコードを見る限り、mainpaneはbackpaneの子なのですよね？

backpane.add(mainpane);

メインとなるフレームから子を直接呼び出すのはおかしな設計です。

• フレームに与えるのは親のパネル

直前の項と重複しますが、子をaddしてどうするんです？

StartScreen jp = new StartScreen();
main.getContentPane().add(jp);

• 必要のないsetVisibleを削る

しっかりとGUIを組めていれば、フレームを可視化すればそれで充分です。

---

設計的に改善するとしたら、次のどっちかですかね。

• MainクラスからImagePanelを呼び出し、ImagePanelからStartPanelを呼び出す。
• Mainクラスから両方を呼び出し、ImagePanelに子としてStartScreenを与える。

後者の方が柔軟な気もします。どんな機能を組み込むかにもよりますが。
ともかく、インタフェースを意識して作る必要はあるでしょう。

不明な点がありましたらコメントにてお知らせ下さい。
コードを見せちゃった方が早いんですが、質問者様のデバッグの機会を奪いたくないので。

Comments

[tarutarupop]

処理の順序について教えてもらってもいいでしょうか？

[LouiS0616]

追記しました。setVisibleやpackを呼び出す順序を特に気にしてください。