JFrame setsizeが働かない

Question

javaのJFrameでsetsizeが働きません。

import javax.swing.*;

public class MainFrame extends JFrame{
  public MainFrame(){
    this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    this.setLocationRelativeTo(null);
    this.setSize(320, 240);    
    this.setVisible(true);
  }
}

このようなコードを書いて他のクラスからインスタンス化させる事によってフレームを作ろうとしているのですががサイズが0のフレームしか表示されません。
何故でしょうか。
回答お願いします。

Comments

[LouiS0616]

MainFrameにmainメソッドを追加して実行してみましたが、再現できませんでした。

Answer 1

二点原因がありそうです。

以下、チュートリアルから引用:

Window#setLocationRelativeTo(...)
Position the window so that it is centered over the specified component. If the argument is null, the window is centered onscreen. To properly center the window, you should invoke this method after the window size has been set.
指定したコンポーネントの中央になるようにウィンドウを配置します。 引数がnullの場合、ウィンドウは画面の中央に配置されます。 ウィンドウを適切に中央に配置するには、ウィンドウサイズが設定された後にこのメソッドを呼び出す必要があります。

とのことなので、まずthis.setSize(320, 240);したあとでthis.setLocationRelativeTo(null);を呼ぶように変更すると良いのではないでしょうか。

もう一点は、JFrameなどのSwingコンポーネントを呼び出す場合はSwingUtilities.invokeLater(...)などを使ってEvent Dispatch Thread上で行う必要があることです。

The Event Dispatch Thread (The Java™ Tutorials > Creating a GUI With JFC/Swing > Concurrency in Swing)
Programs that ignore this rule may function correctly most of the time, but are subject to unpredictable errors that are difficult to reproduce.
このルールを無視するプログラムは、ほとんどの場合正しく機能しますが、再現するのが困難な予測不可能なエラーが発生する可能性があります。

• 参考1: FrameDemo.java
• 参考2: Initial Threads (The Java™ Tutorials > Creating a GUI With JFC/Swing > Concurrency in Swing)

サンプルコード

import javax.swing.*;

public class MainFrame extends JFrame{
  public MainFrame() {
    this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    //this.setLocationRelativeTo(null);
    this.setSize(320, 240);
    this.setLocationRelativeTo(null);
    this.setVisible(true);
  }
  public static void main(String[] args) {
    SwingUtilities.invokeLater(() -> new MainFrame());
  }
}

Comments

[KSwordOfHaste]

「ウィンドウサイズが設定された後にこのメソッドを呼び出す必要があります」の自分の解釈は「サイズが確定していないと正確に中央位置を決められないから」でした。実際のところははっきりわかりませんが・・・。
なお自分はsetSizeは比較的低水準なAPIで、peerの生成タイミングなどに影響されるAPIと思っています。そのためsetSizeをなるべく使わずにsetPreferredSizeなどのレイアウト制約プログラミングをお勧めしたいという意見です。いかがでしょうか？

[0603nanakusa]

setPreferredSizeにした所無事解決することができました。
ありがとうございました。

[KSwordOfHaste]

MainFrameのコンストラクターの最後にpack()を呼び出すのをお忘れなく・・・と蛇足ながら補足させていただきます。

[aterai]

@KSwordOfHaste さん、「正確に中央位置を決められないから」で正しいと思います。ただ何故か`setLocationRelativeTo(null)`が`Event Dispatch Thread`ルールとの合わせ技で`JFrame`のサイズに影響する場合(環境)があるみたいなんですよね(EDTの方が重要な気もしますが)。
「レイアウト制約...」、同意します。チュートリアルのサンプルでも`JFrame#setSize(...)`より`JFrame#pack()`を使用するものが多いようです。

[aterai]

@0603nanakusa さん、どういたしまして。同じく蛇足ですが`MainFrame`のコンストラクタの最初で`super();`を呼ぶのも良さそうです。

[0603nanakusa]

ありがとうございます

[KSwordOfHaste]

> ateraiさん
なるほど。想像ですがsetLocationRelativeTo(null)はpeer(例えばWindowsであればOSのWindow Handle)を必要とするといったことに関係する気がします。EDTにからむということもあるかも知れません。おっしゃるとおり、「正確に中央位置を決められないから」だけではないかも知れませんね。