java - equals 関数が機能しないのはなぜですか?

Question

以下のコードをコンパイルすると、(string) username と myList.get(0) が等しいことが示されますが、 equals 関数が false を返すのは、 password に対しても同じことが起こる理由です。

btnLogIn.addActionListener(new ActionListener()
 {
   public void actionPerformed(ActionEvent e) {

    String Username=(String) textField.getText();   //fatch the user name from text field
    String Password=(String) textField_1.getText();  //fatch password frof text field
    databaseconnection connect = new databaseconnection(); // databaseconnection class object to connect to data base
    ArrayList myList = connect.search(Username,Password);  //serch the username and password in data base
            System.out.println((String)myList.get(0));  //for testing 
            System.out.println((String)myList.get(1));  //for testing
            System.out.println(Username);               //for testing
            System.out.println(Password);               //for testing
            System.out.println(Username.equals(myList.get(0)));  //for testing 
            System.out.println(Password.equals((String)myList.get(1)));   //for testing
            if(Username.equals(myList.get(0))&&Password.equals((String)myList.get(1))){
            System.out.println("Hello"+Username);
            }
        }
    });

ここに画像の説明を入力

これは私のデータベース接続クラスです

   import java.sql.*;
   import java.util.ArrayList; 

 public class databaseconnection{
Statement stmt  ;
ResultSet rs ;
Connection conn;
ArrayList<String> temp = new ArrayList<String>();

public void getconnection(){

try{


    Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");

    Connection conn = DriverManager.getConnection("jdbc:odbc:Database1","","");

    stmt = conn.createStatement();


}
catch(Exception e){
    System.out.println("connection error");
}
}
public ArrayList  search(String Username,String Password){

    getconnection();
    try{
    rs = stmt.executeQuery("select username,password from login where username = \'"+Username+"\'");
    if(rs.next()){
    String tempString=rs.getString("username");
    temp.add(tempString);
    tempString= rs.getString("password");
    temp.add(tempString);
    }
    }
    catch(Exception e){
    System.out.println("search error");
    }
    return temp;
  }

  }

score 1 · Accepted Answer

私はあなたのコードを次のように書き直しました：

String Username = "user";
String Password = "pass";

List<String> myList = new ArrayList<>(); 
myList.add("user");
myList.add("pass");

System.out.println(Username);
System.out.println(Password);
System.out.println(myList.get(0));
System.out.println(myList.get(1));
System.out.println(Username.equals(myList.get(0)));
System.out.println(Password.equals(myList.get(1)));

if (Username.equals(myList.get(0)) && Password.equals(myList.get(1))) {
    System.out.println("Hello, " + Username);
}

そして今、それは機能します。<String>List 宣言の後に注意してください。これはジェネリックStringsと呼ばれ、リストに入れることしかできないと言っています。また、エラーが発生しやすい明示的なキャストの必要性がなくなり、安全に実行できることが確実な場合にのみ実行する必要があります。

これに従ってコードを変更しても正しい結果が得られない場合は、接続からの値が正しく、目に見えない文字が含まれていないことを確認してください。

また、Java では、変数名は通常、lowerCamelCaseです。クラス名はUpperCamelCaseにする必要があります。Java 命名規則を参照してください。

接続を閉じることを忘れないでください。 Java 7 の自動リソース管理は、それを支援します。一般的なケースでは、接続エラーも処理する必要があります。

また、変数の名前を変数の動作に基づいて付けることをお勧めします。したがって、コードは次のようになります。

public void actionPerformed(ActionEvent e) {
    String username = "user";
    String password = "pass";

    List<String> dtbSearchResults;
    try (DatabaseConnection connection = new DatabaseConnection()) {
        dtbSearchResults = connection.search(username, password); 
    } catch (SomeExceptionYouReallyShouldHandle e) {
        // seriously, handle it here
    }

    System.out.println(username);
    System.out.println(password);
    System.out.println(dtbSearchResults.get(0));
    System.out.println(dtbSearchResults.get(1));
    System.out.println(username.equals(dtbSearchResults.get(0)));
    System.out.println(password.equals(dtbSearchResults.get(1)));

    if (username.equals(dtbSearchResults.get(0)) && password.equals(dtbSearchResults.get(1))) {
        System.out.println("Hello, " + username);
    }
}

メソッドは、正確に何をすべきかに基づいて、タイプのインスタンスまたはのsearch()インスタンスを返す必要があるため、まだ完全ではありません。次に、次のようになります。UserCredentialsList<UserCredentials>UserCredentials

public class UserCredentials {

    private final String username;
    private final String password;

    public UserCredentials(String username, String password) {
        // maybe some validity checks
        this.username = username;
        this.password = password;
    }

    public String getUsername() {
        return username;
    }

    public String getPassword() {
        return password;
    }

}

このクラスを実装すると、コードは次のようになります (search()結果が 1 つだけ返されると仮定します)。

public void actionPerformed(ActionEvent e) {
    String username = "user";
    String password = "pass";

    UserCredentials user;
    try (DatabaseConnection connection = new DatabaseConnection()) {
        user = connection.search(username, password); 
    } catch (SomeExceptionYouReallyShouldHandle e) {
        // seriously, handle it here
    }

    System.out.println(username);
    System.out.println(password);
    System.out.println(user.getUsername());
    System.out.println(user.getPassword());
    System.out.println(username.equals(user.getUsername()));
    System.out.println(password.equals(user.getPassword()));

    if (username.equals(user.getUsername())
            && password.equals(user.getPassword())) {
        System.out.println("Hello, " + username);
    }
}

追加の質問があればお尋ねください。

score 0 · Accepted Answer

0

myList を型として定義するArrayList<String>

于 2013-02-03T15:05:39.067 に答える

java - equals 関数が機能しないのはなぜですか?

2 に答える 2

Related

Reference