2013-05-08 119 views
2

我有两个相同的md5哈希密码,并应使用MessageDigest.isEqual方法返回true。但是,比较返回错误。比较两个消解的md5字符串返回false

使用Array.toString打印字节数组时,它们是相同的。它仍然不起作用,无论是Arrays.euqals还是MessageDigest.isEqual。

public boolean verifyUserCredentials(String username, MessageDigest password) { 

     ListIterator<User> iterator = userList.listIterator(); 
     while (iterator.hasNext()) { 
      User user = iterator.next(); 
      byte md1[] = user.getPassword().digest(); 
      byte md2[] = password.digest(); 
      if (user.getUsername() == username && MessageDigest.isEqual(md1, md2)) { 
       return true; 
      } 
     } 
     return false; 
    } 
+0

我不认为你的问题与比较md5s有任何关系。这是你笨拙的用户名。 – 2013-05-08 23:55:45

+0

我一直在做这么多次相同的错误。感谢您的意见。 – mgerstner 2013-05-09 06:31:39

回答

5

您已经使用==,但对于像String对象是比较对象引用,看看他们是否指向同一个对象。

要比较String的值,请改为使用String#equals

0

或者你可以采取一个邪恶的做法和滥用串实习:

user.getUsername().intern() == username.intern() 

下面的代码打印:

String s1 = "asdf"; 
String s2 = "as"; // break this into two parts to keep the compiler from 
s2 = s2 + "df"; // automatically interning the constant and having the strings 
        // be the same. 

System.out.println(s1 == s2 ? "true" : "false"); 
System.out.println(s1.intern() == s2.intern() ? "true" : "false"); 

输出:

false 
true 

的+ “DF” 是需要保持编译器不会自动插入常量字符串。

注意:这是更多的Java琐事和语言滥用,而不是我建议你实际做的事情。