JAVAの画像とスクロールバーの表示がうまくいかない

Question

実現したいこと

ウィンドウを作成したと同時に画像とスクロールバーを表示させる。

前提

eclipceでJAVAで大きい画像の表示とスクロール機能の実装をしたいのです。そこで以下のようなコードを実行しました。すると初めに作成されるウィンドウは真っ白で、そのウィンドウの大きさを変えた(最大表示など)したとたんに画像の表示とスクロールバーが表示されます。そうではなく実行した際に作られるウィンドウに画像の表示とスクロールバーを表示させたいです。

該当のソースコード

public class mine {

public static void main(String[] args) {

	Window test = new Window();
	test.window("test", 600, 600);			
}

}

//以下ウィンドウ作成の作成と画像表示のためのクラス
import java.awt.BorderLayout;
import java.awt.Container;

import javax.swing.ImageIcon;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JScrollPane;

public class Window extends JFrame {

private String name;//ウィンドウの名前を入れる為の変数
private int x;//ウィンドウの大きさの横幅を入れる為の変数
private int y;//ウィンドウの大きさの立幅を入れる為の変数

 public void window(String name,int x,int y){
	 this.name = name;
	 JFrame window = new JFrame(name);
	 window.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
     window.setSize(x,y);
     window.setVisible(true);
    
 
    
     JLabel label = new JLabel();//ラベルの作成
     //画像の読み込み
     ImageIcon icon = new ImageIcon("C:\\Users\\yuu19\\Downloads\\eclipce\\workspace\\pokecchi\\image\\とある壁紙.jpg");
    
     label.setIcon(icon);//ラベルに画像を貼り付け
    JScrollPane scrollpane = new JScrollPane(label);
   
 
    Container contentPane = window.getContentPane();
    contentPane.add(scrollpane, BorderLayout.CENTER);//パネルをウィンドウに貼り付け
     }
	 

}

試したこと

ラベルやパネルの作成の順番を変えてみるなどは試してみたのですがうまくいきませんでした。しかし、このプログラムを書くにあたって参考にしたプログラムをコピペして試したところ上手くいきました。そのプラグラムと見比べてみたりしたのですが原因はよくわからなかったです。

プログラミング自体初心者で独学の為、もしかすると簡単なミスなのかもしれないですがわかる方いたらご教授願いたいです。
よろしくお願いします。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[jimbe]

とりあえず、ソースコードは「コードのマークダウン」を用いてファイル毎にご提示ください。

```java
public class mine {
:
}
```
```java
import java.awt.BorderLayout;
:
public class Window extends JFrame {
:
}
```

Answer 1

問題の動作は、コマンドラインのスレッドで JFrame を表示( setVisivle(true); ) 後にもまだ Swing のコンポーネントを弄っているためと思われます。
Swing で一度何かを表示させたら、コマンドラインのスレッド上でそれ以上処理をしてはいけないことになっています。

MainFrame.java

import java.awt.BorderLayout;

import javax.swing.*;

public class MainFrame extends JFrame {
  public static void main(String[] args) {
    SwingUtilities.invokeLater(() -> new MainFrame("test",600,600).setVisible(true));
  }

  public MainFrame(String name, int width, int height) {
    super(name);
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setSize(width, height);

    //String filename = "C:\\Users\\Jimbe\\Pictures\\艦これ\\E5Colorado.jpg";
    String filename = "C:\\Users\\yuu19\\Downloads\\eclipce\\workspace\\pokecchi\\image\\とある壁紙.jpg";

    JLabel label = new JLabel();
    ImageIcon icon = new ImageIcon(filename);
    label.setIcon(icon);

    JScrollPane scrollpane = new JScrollPane(label);
    add(scrollpane, BorderLayout.CENTER);
  }
}

ウインドウシステムはコマンドラインより格段に難しいです。
考え方を180度切り替える必要があるかもしれませんのでご留意ください。

Comments

[YUUdesu]

ご指摘のようにsetVisible(true)を最後に記述することで解決しました！
ちなみに載せてくださったプログラムの「SwingUtilities.invokeLater()」についても調べてみたら便利なメソッドみたいですね。勉強になりました。しかし、まだAPIなどを見ても具体的な使い方が分からなかったのでこれからまたいろいろ漁って勉強したいと思います。
ありがとうございました。

[jimbe]

元々私は SwingUtilities.invokeLater は使わず
public static void main(String[] args) {
new MainFrame("test",600,600).setVisible(true);
}
のような形で書いていました。
setVisible がここにあるのは、呼び出しが確実に最後になるように(コピペ等で setVisible の呼び出しが変な所に行かないように)です。
のですが、公式には SwingUtilities.invokeLater を使うようにとなっていたり lamda 式が出来て インターフェースの無名定義が簡単になりましたので、今は回答のような main メソッドを書いています。
SwingUtilities.invokeLater を使ってフレームを作るなら setVisible は最後で無くても大丈夫かもしれませんが…まぁ、全て準備が出来てから表示とした方が何かスッキリしているかもしれません。

[YUUdesu]

とりあえずJAVAの基礎が終わったところで、とりあえず何か一つアプリケーションを作ろうと思ってるところなので、今後画面の移り替わりなどについてもやっていこうと思うのでまた参考にしたいと思います！