Problem with JTextField

[waker3210]

I know swing decently well, but im having a weird problem with a JTextField.
Here's the code:

import javax.swing.*;
import java.awt.event.*;

public class Frame
{
	public static void main(String[] args)
	{
		new Frame();
	}
	
	JFrame checkerFrame;
	JButton logon;
	JTextField passwordField;
	
	String password = "tree";
	
	public Frame()
	{
		checkerFrame = new JFrame("Password");
		checkerFrame.setSize(200,200);
		checkerFrame.setResizable(false);
		checkerFrame.setLayout(null);
		checkerFrame.setLocationRelativeTo(null);
		checkerFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		checkerFrame.setVisible(true);
		
		logon = new JButton("Enter");
		logon.setBounds(65,140,75,25);
		
		checkerFrame.add(logon);
		
		passwordField = new JTextField();
		passwordField.setBounds(25,25,150,25);
		
		checkerFrame.add(passwordField);
		
		logon.addActionListener(new Handler());
	}
	
	private class Handler implements ActionListener
	{
		public void actionPerformed(ActionEvent e)
		{
			if(e.getSource() == logon)
			{
				if(passwordField.getText() == password)
				{
					checkerFrame.dispose();
				}
				else
				{
					JOptionPane.showMessageDialog(checkerFrame,"Incorrect Password. Please Try Again.");
				}
			}
		}
	}
}

It should dispose of the frame if the text in passwordField is the same as password (in this case "tree"), but even if it's the same it runs the else statement. Do i do something wrong? I really don't know what's wrong.

[waker3210]

Oh, wow. Nevermind, stupid mistake on my part. I got it.

[Fubarable]

Yeah, you've got problems in that code

• Using == to compare Strings
• Using Strings for passwords period
• Using null layout, absolute positioning and setSize(...) rather than preferredSizes and layout managers
• Giving your class the same name as a core Java class, one similar to what you are using.
• ...

You'll want to read the Java tutorials and especially the Swing tutorials to fix these things, especially the layouts.

[waker3210]

yea. thanks, i appreciate it.