Review

Calls JFrame.setVisible(true) without calling JFrame.pack(), such that the frame is not layouted before being shown.

Call JFrame.pack() before JFrame.setVisible(true).

• missing/call

NoPackBeforeVisible.java

misuse(JFrame)

class NoPackBeforeVisible {
	public void misuse(JFrame f) {
		f.setVisible(true);
	}

}

public class PackBeforeVisible {
	public void pattern(JFrame f, Dimension d) {
		f.pack();
		f.setVisible(true);
	}
}