This week's book giveaway is in the Java 8 forum.
We're giving away four copies of Java 8 in Action and have Raoul-Gabriel Urma, Mario Fusco, and Alan Mycroft on-line!
See this thread for details.
The moose likes Beginning Java and the fly likes What is wrong with my code? Big Moose Saloon
  Search | Java FAQ | Recent Topics | Flagged Topics | Hot Topics | Zero Replies
Register / Login


Win a copy of Java 8 in Action this week in the Java 8 forum!
JavaRanch » Java Forums » Java » Beginning Java
Bookmark "What is wrong with my code?" Watch "What is wrong with my code?" New topic
Author

What is wrong with my code?

Kev So
Greenhorn

Joined: Jul 08, 2006
Posts: 2
When I enter the correct username and password into the textfields it still displays "Wrong username/password!" Could someone please tell what I am doing wrong in my code! Thanks!



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

public class LoginFrame extends JFrame {
private JFrame loginFrame;
private JPanel northPanel, southPanel;
private JLabel username, password;
private JTextField user, pass;
private JButton login;

public LoginFrame() {
super("Login...");

loginFrame = new JFrame();

northPanel = new JPanel();
northPanel.setLayout(new GridLayout(2, 2));

username = new JLabel("Username:", username.CENTER);
password = new JLabel("Password:", password.CENTER);

user = new JTextField(15);
pass = new JTextField(15);

northPanel.add(username);
northPanel.add(user);
northPanel.add(password);
northPanel.add(pass);

southPanel = new JPanel();

ButtonHandler buttonHandler = new ButtonHandler();

login = new JButton("Login");
login.addActionListener(buttonHandler);

southPanel.add(login);

getContentPane().add("North", northPanel);
getContentPane().add("South", southPanel);

setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
setSize(175, 110);
setVisible(true);
setResizable(false);
}

public static void main(String[] args) {
LoginFrame login = new LoginFrame();
}

private class ButtonHandler implements ActionListener {
public void actionPerformed(ActionEvent event) {

if(user.getText() == "username" && pass.getText() == "password")
JOptionPane.showMessageDialog(null, "Welcome!");
else
JOptionPane.showMessageDialog(null, "Wrong username/password!");

}
}

}
Jesus Angeles
Ranch Hand

Joined: Feb 26, 2005
Posts: 2046
Instead of

if(user.getText() == .............

you can use

user.getText().equals(.......) ..............

== and .equals work differently

you can look here
Kev So
Greenhorn

Joined: Jul 08, 2006
Posts: 2
Thanks for the help! The .equals() worked perfectly!
 
I agree. Here's the link: http://aspose.com/file-tools
 
subject: What is wrong with my code?
 
Similar Threads
Need help on login frame in java.
java.net.SocketException: Error
run-time error HELP!
Removing a contentPane- HELP!
Java uses or overrides a deprecated API?