Win a copy of Re-engineering Legacy Software this week in the Refactoring forum
or Docker in Action in the Cloud/Virtualization forum!
  • Post Reply
  • Bookmark Topic Watch Topic
  • New Topic

What is wrong with my code?

 
Kev So
Greenhorn
Posts: 2
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
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
Posts: 2068
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
Instead of

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

you can use

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

== and .equals work differently

you can look here
 
Kev So
Greenhorn
Posts: 2
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
Thanks for the help! The .equals() worked perfectly!
 
  • Post Reply
  • Bookmark Topic Watch Topic
  • New Topic