Problems pausing a thread

Question

Hi, I'll try to explay my problem. I've got 3 classes (See code below):
1-PruebaThreadMainGui,
2-PruebaThread
3-PruebaThreadGui.
PruebaThread works as follows: When the method PruebaThread.execute is called , it writes "hello" , a new PruebaThreadGui (which extends JFrame) is created and then waits until the user presses the "Press" button in the PruebaThreadGui. After that, it writes "goodbye". This works OK if PruebaThread is executed directly.
However the problem arises when PruebaThread.execute is called from PruebaThreadMainGui (when I press the "Execute" button). PruebaThread.execute writes "hello" and then creates the PruebaThreadGui, but this window frozes and can't continue working. I guess the problem is related to my attemp to pause PruebaThread? But if it's that, why does it work if I execute it directly?
Well, thank you and sorry for my crappy English.
PruebaThreadMainGui:
package webServices;
import javax.swing.SwingUtilities;
import java.awt.BorderLayout;
import javax.swing.JPanel;
import java.awt.GraphicsConfiguration;
import java.awt.HeadlessException;
import javax.swing.JFrame;
import javax.swing.JButton;
public class PruebaThreadMainGui extends JFrame {
     private static final long serialVersionUID = 1L;
     private JPanel jContentPane = null;
     private JButton jButton1 = null;
       //  @jve:decl-index=0:
     public PruebaThreadMainGui() throws HeadlessException {
          // TODO Auto-generated constructor stub
          super();
          initialize();
     private JButton getJButton1() {
          if (jButton1 == null) {
               jButton1 = new JButton();
               jButton1.setText("Execute");
               jButton1.addActionListener(new java.awt.event.ActionListener() {
                    public void actionPerformed(java.awt.event.ActionEvent e) {
                         PruebaThread pt=new PruebaThread();
                         pt.execute();
          return jButton1;
     public static void main(String[] args) {
          // TODO Auto-generated method stub
          SwingUtilities.invokeLater(new Runnable() {
               public void run() {
                    PruebaThreadMainGui thisClass = new PruebaThreadMainGui();
                    thisClass.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                    thisClass.setVisible(true);
     private void initialize() {
          this.setSize(300, 200);
          this.setContentPane(getJContentPane());
          this.setTitle("JFrame");
     private JPanel getJContentPane() {
          if (jContentPane == null) {
               jContentPane = new JPanel();
               jContentPane.setLayout(new BorderLayout());
               jContentPane.add(getJButton1(), BorderLayout.CENTER);
          return jContentPane;
}PruebaThread:
package webServices;
public class PruebaThread {
     public PruebaThread() {
          // TODO Auto-generated constructor stub
public void execute(){
     System.out.println("hello");
     // Now I want to open a PruebaThreadGui and wait until "Press" is pressed in that PruebaThreadGui
     Thread t=Thread.currentThread();
     PruebaThreadGui ptg=new PruebaThreadGui(t);
     ptg.pack();
     ptg.setVisible(true);
     synchronized(t){try {
          t.wait();
     } catch (InterruptedException e) {
          // TODO Auto-generated catch block
          e.printStackTrace();
     //This doesn't have to execute until "Press" has been pressed.
     System.out.println("goodbye");
     public static void main(String[] args) {
          // TODO Auto-generated method stub
PruebaThread pt=new PruebaThread();
pt.execute();
}PruebaThreadGui:
package webServices;
import java.awt.BorderLayout;
import javax.swing.JPanel;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JButton;
public class PruebaThreadGui extends JFrame {
     private static final long serialVersionUID = 1L;
     private JPanel jContentPane = null;
private Thread t;
private JLabel jLabel1 = null;
private JButton jButton1 = null;
private PruebaThreadGui me=this;
     public PruebaThreadGui(Thread t) {
          super();
          this.t=t;
          initialize();
     private void initialize() {
          this.setSize(300, 200);
          this.setContentPane(getJContentPane());
          this.setTitle("JFrame");
     private JPanel getJContentPane() {
          if (jContentPane == null) {
               jLabel1 = new JLabel();
               jLabel1.setText("JLabel");
               jContentPane = new JPanel();
               jContentPane.setLayout(new BorderLayout());
               jContentPane.add(jLabel1, BorderLayout.NORTH);
               jContentPane.add(getJButton1(), BorderLayout.SOUTH);
          return jContentPane;
     private JButton getJButton1() {
          if (jButton1 == null) {
               jButton1 = new JButton();
               jButton1.setText("Press");
               jButton1.addActionListener(new java.awt.event.ActionListener() {
                    public void actionPerformed(java.awt.event.ActionEvent e) {
                         //Execution returns to PruebaThread
                         synchronized(t){t.notify();}
                    me.dispose();
          return jButton1;
}

Answer 1

Hi Saloa
Try the following... Ive updated youre code so it actually uses java.lang.Thread and overrides its run method.
I hope it is what you wanted.
package webServices;
public class PruebaThread extends Thread {
    public PruebaThread() {
        // TODO Auto-generated constructor stub
    public void run(){
        System.out.println("hello");
        // Now I want to open a PruebaThreadGui and wait until "Press" is pressed in that PruebaThreadGui
        PruebaThreadGui ptg=new PruebaThreadGui(this);
        ptg.pack();
        ptg.setVisible(true);
        synchronized(this){try {
            this.wait();
        } catch (InterruptedException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        //This doesn't have to execute until "Press" has been pressed.
        System.out.println("goodbye");
    public static void main(String[] args) {
        // TODO Auto-generated method stub
        PruebaThread pt=new PruebaThread();
        pt.start();
package webServices;
import java.awt.BorderLayout;
import javax.swing.JPanel;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JButton;
public class PruebaThreadGui extends JFrame {
    private static final long serialVersionUID = 1L;
    private JPanel jContentPane = null;
    private Thread t;
    private JLabel jLabel1 = null;
    private JButton jButton1 = null;
    private PruebaThreadGui me=this;
    public PruebaThreadGui(Thread t) {
        super();
        this.t=t;
        initialize();
    private void initialize() {
        this.setSize(300, 200);
        this.setContentPane(getJContentPane());
        this.setTitle("JFrame");
    private JPanel getJContentPane() {
        if (jContentPane == null) {
            jLabel1 = new JLabel();
            jLabel1.setText("JLabel");
            jContentPane = new JPanel();
            jContentPane.setLayout(new BorderLayout());
            jContentPane.add(jLabel1, BorderLayout.NORTH);
            jContentPane.add(getJButton1(), BorderLayout.SOUTH);
        return jContentPane;
    private JButton getJButton1() {
        if (jButton1 == null) {
            jButton1 = new JButton();
            jButton1.setText("Press");
            jButton1.addActionListener(new java.awt.event.ActionListener() {
                public void actionPerformed(java.awt.event.ActionEvent e) {
                    //Execution returns to PruebaThread
                    synchronized(t){t.notify();}
                    me.dispose();
        return jButton1;
package webServices;
import javax.swing.SwingUtilities;
import java.awt.BorderLayout;
import javax.swing.JPanel;
import java.awt.GraphicsConfiguration;
import java.awt.HeadlessException;
import javax.swing.JFrame;
import javax.swing.JButton;
public class PruebaThreadMainGui extends JFrame {
    private static final long serialVersionUID = 1L;
    private JPanel jContentPane = null;
    private JButton jButton1 = null;
    //  @jve:decl-index=0:
    public PruebaThreadMainGui() throws HeadlessException {
        // TODO Auto-generated constructor stub
        super();
        initialize();
    private JButton getJButton1() {
        if (jButton1 == null) {
            jButton1 = new JButton();
            jButton1.setText("Execute");
            jButton1.addActionListener(new java.awt.event.ActionListener() {
                public void actionPerformed(java.awt.event.ActionEvent e) {
                    PruebaThread pt=new PruebaThread();
                    pt.start();
        return jButton1;
    public static void main(String[] args) {
        // TODO Auto-generated method stub
        SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                PruebaThreadMainGui thisClass = new PruebaThreadMainGui();
                thisClass.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                thisClass.setVisible(true);
    private void initialize() {
        this.setSize(300, 200);
        this.setContentPane(getJContentPane());
        this.setTitle("JFrame");
    private JPanel getJContentPane() {
        if (jContentPane == null) {
            jContentPane = new JPanel();
            jContentPane.setLayout(new BorderLayout());
            jContentPane.add(getJButton1(), BorderLayout.CENTER);
        return jContentPane;
}