mouseClicked event for the "Cancel button"

[Shikha Upadhyaya]

I am trying to write the mouseClicked event for the "Cancel button" where the entire window has to close on pressing the Cancel button. I wrote the following code:

package my.initPckg;
import com.jcraft.jsch.*;

import java.awt.*;
import javax.swing.*;
import java.awt.BorderLayout;
import java.awt.EventQueue;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.border.EmptyBorder;
import javax.swing.JLabel;
import java.awt.Font;

import javax.swing.JOptionPane;
import javax.swing.SwingConstants;
import javax.swing.JTextField;
import javax.swing.JPasswordField;
import javax.swing.JButton;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;

public class PmtGui extends JFrame {

	private JPanel contentPane;
	private JTextField textFieldHN;
	private JTextField textFieldPW;

	/**
	 * Launch the application.
	 */
	public static void main(String[] args) {
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					PmtGui frame = new PmtGui();
					frame.setVisible(true);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}
		});
	}

	/**
	 * Create the frame.
	 */
	public PmtGui() {
		setTitle("Performance Monitor Tool for 8950 SAM");
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(100, 100, 450, 300);
		contentPane = new JPanel();
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		setContentPane(contentPane);
		contentPane.setLayout(null);
		
		JLabel lblHostName = new JLabel("Host Name:");
		lblHostName.setFont(new Font("Tahoma", Font.BOLD, 12));
		lblHostName.setBounds(117, 83, 83, 14);
		contentPane.add(lblHostName);
		
		JLabel lblPassword = new JLabel("Password:");
		lblPassword.setFont(new Font("Tahoma", Font.BOLD, 12));
		lblPassword.setBounds(117, 127, 67, 14);
		contentPane.add(lblPassword);
		
		textFieldHN = new JTextField();
		textFieldHN.setBounds(210, 81, 122, 20);
		contentPane.add(textFieldHN);
		textFieldHN.setColumns(10);
		
		JButton btnConnect = new JButton("Connect");
		btnConnect.addMouseListener(new MouseAdapter() {
			@Override
			public void mouseClicked(MouseEvent arg0) {
				String host = textFieldHN.getText();			
				String passwd = textFieldPW.getText();
				String user = host.substring(0, host.indexOf('@'));
				host = host.substring(host.indexOf('@')+1);
				try {
					JSch jsch=new JSch();
					Session session=jsch.getSession(user, host, 22);
					java.util.Properties config = new java.util.Properties(); config.put("StrictHostKeyChecking", "no");
					session.setConfig(config);
					session.setHost(host);
					session.setPassword(passwd);
					session.connect();
					
					Channel channel = session.openChannel("shell");
					channel.setInputStream(System.in);
				    channel.setOutputStream(System.out);

				    channel.connect();
				}
				catch (Exception e){
					System.out.println(e);
				}
				
			    
			}
		});
		btnConnect.setBounds(117, 193, 89, 23);
		contentPane.add(btnConnect);
		
		JButton btnCancel = new JButton("Cancel");
		btnCancel.addMouseListener(new MouseAdapter() {
			@Override
			public void mouseClicked(MouseEvent arg0) {
				contentPane.setVisible(false);
			}
		});
		btnCancel.setBounds(234, 193, 89, 23);
		contentPane.add(btnCancel);
		
		JLabel lblConnectToServer = new JLabel("Connect to Server");
		lblConnectToServer.setFont(new Font("Tahoma", Font.BOLD, 13));
		lblConnectToServer.setBounds(148, 29, 146, 14);
		contentPane.add(lblConnectToServer);
		
		textFieldPW = new JPasswordField(20);
		textFieldPW.setBounds(210, 125, 122, 20);
		contentPane.add(textFieldPW);
		textFieldPW.setColumns(10);
	}


}

But, when I run this code, only the contents of the window get erased and the window remains unclosed. Please let me know what has to be written between lines 104 and 110.

[Riaan Nel]

Look at line 108. You want to close the frame, but all you do is to make the components on your frame invisible. You need to dispose of the frame.

Something like this.

PmtGui.this.dispose();

The PmtGui.this is used to tell your anonymous inner class (MouseAdapter) that you are referring to the outer class (PmtGui).

On a side note, don't use MouseListeners on buttons. If you disable the button, the MouseListener will still register events. Rather use an ActionListener.

[Shikha Upadhyaya]

Riaan Nel wrote:Look at line 108. You want to close the frame, but all you do is to make the components on your frame invisible. You need to dispose of the frame.

Something like this.

PmtGui.this.dispose();

The PmtGui.this is used to tell your anonymous inner class (MouseAdapter) that you are referring to the outer class (PmtGui).

On a side note, don't use MouseListeners on buttons. If you disable the button, the MouseListener will still register events. Rather use an ActionListener.

Ok. Thanks It's working now