Frame's setSize() method is not working

[Abhimanyus singh]

import java.awt.Container;  
import java.awt.Dimension;  
import javax.swing.JFrame;  
class MyFrame extends JFrame  
{  
MyFrame(String h)  
    {  
    super(h);  
    this.setSize(200, 200);  
    this.setResizable(false);  
    //setLocationRelativeTo(null);  
    Container content=this.getContentPane();  
    content.setLayout(null);  
     
      
    this.setVisible(true);  
    }  
}  
  
class Main1  
{  
    public static void main(String args[])   
    {  
     MyFrame frame=new MyFrame("This frame");  
      
    }  
}  

Everytime I run this code, it results into a maximised window, preventing the setSize() method from taking any effect !!! Please help

[m Korbel]

works, no issue

- JFrame returns Dimension(200, 200)

- ContentPane returns JFrames Dimansion minus Borders and toolbar (L&F sensitive)

- read Oracles tutorial about Initial Thread

- don't extends JFrame, create this Object as an local variable

[Campbell Ritchie]

That was correect, but a bit laconic.
This is where you find out about the single thread policy. As for not extending JFrame, you should do this:-

public class MyGUI(String h)
{
    public MyGUI(String h)
    [
        JFrame frame = new JFrame(h);  
        frame.setSize(200, 200);  
        frame.setResizable(false);  
        //setLocationRelativeTo(null);  
        Container content=frame.getContentPane();  
        content.setLayout(null);

        frame.setVisible(true);  
}

Don’t use null layout. You often do not need to get the content pane separately from the frame.

[Abhimanyus singh]

Campbell Ritchie wrote:That was correect, but a bit laconic.
This is where you find out about the single thread policy. As for not extending JFrame, you should do this:-

public class MyGUI(String h)
{
    public MyGUI(String h)
    [
        JFrame frame = new JFrame(h);  
        frame.setSize(200, 200);  
        frame.setResizable(false);  
        //setLocationRelativeTo(null);  
        Container content=frame.getContentPane();  
        content.setLayout(null);

        frame.setVisible(true);  
}

Don’t use null layout. You often do not need to get the content pane separately from the frame.

It's still not working...let me explain it....the moment I run the program, it opens the frame in MAXIMIZED mode.!!! Earlier it was not happening.

[Campbell Ritchie]

Abhimanyus singh wrote: . . .....the moment I run the program, it opens the frame in MAXIMIZED mode.!!! Earlier it was not happening.

It didn’t appear maximised when I tried it.

[Darryl Burke]

Abhimanyus, how, exactly are you executing that code?

[Randall Twede]

not sure if this will help, just a couple points. i also don't extend JFrame anymore but i used to.
first, you shouldn't have to call super() in the constructor. at least i didn't have to using no argument constructor and calling setTitle(String) in it.
second you don't have to say "this" all the time.
setSize(200, 200);
and
setVisible(true);
is sufficient.

try getting rid of the call to super(h)

[Maneesh Godbole]

Randall Twede wrote:
first, you shouldn't have to call super() in the constructor. at least i didn't have to using no argument constructor and calling setTitle(String) in it.
try getting rid of the call to super(h)

The OP doesn't have a super() but rather a super(h) call which is akin to setTitle(String title).

Randall Twede wrote:
second you don't have to say "this" all the time.

Technically correct. But it is a good practice to use this.xxx and super.xxx as much as possible.

I suspect the OP is executing some old code and getting the strange results. Like Darryl said, once we know exactly, how it is being executed, we will know