jPasswordField problem

[MichaelKl]

Der Code:

System.out.println(jPasswordField.getPassword().toString());

ergibt: [C@1f20eeb

wollte eigentlich ein passwort abgleichen....aber das if hat nie erfolg....

User tmpUser = new User(jTextField.getText(), jPasswordField.getPassword().toString());

legt mir einen temporären Benutzer an, der die Werte der Eingbe maske hat.
Könnte hier bei der .toString Methode der Fehler liegen ?
und eine Methode userAvailable

if (myAuth.userAvailable(tmpUser)== true){
						System.out.println("Benutzer vorhanden!");
}

prüft ober der benutzer bereits vorhanden ist.

 

[faetzminator]

Du solltest nie, und wirklich nie ein Passwort plaintext als String speichern. Warum dass das so ist? Google liefert viele Antworten darauf, da will ich mich nicht wiederholen. Auf alle Fälle eine Frage der Sicherheit. Mach aus dem zu überprüfenden Passwort ebenfalls ein [c]char[][/c].

 

[Spacerat]

[c]new String(jPasswordField.getPassword());[/c] wurde schon probiert? Schliesslich gibt [c]getPassword();[/c] ja [c]char[][/c] zurück 😉.
@Edit: ...und äh... ja ja... die Sicherheit...
@Edit2: Bei erneuter überlegung wäre eigentlich auch [c]String.valueOf(jPasswordField.getPassword());[/c] um einiges eleganter. Von wegen Stringpool und so...

 

[SegFault]

ich glaube getPassword gibt ein Char[] zurück. Die toString Methode von Byte[] liefert dann nur die Adresse. Du musst
new String(jPasswordField.getPassword()) machen.

Mist war jemand schneller 😉

 

[MichaelKl]

der fehler liegt in dieser Methode

public boolean userAvailable(User user){
		boolean available = false;
		User operator = new User();
				for (int i = 0; i < operator.getAllUser().size(); i++){
					if(operator.getAllUser().get(i).getName().equals(user.getName())){
						available = true;
					}
				}
		return available;
	}

kann mir jemand helfen, was da falsch läuft...?

 

[faetzminator]

Wieso ist [c]getAllUser()[/c] keine statische Methode ???:L ?

 

[MichaelKl]

gute Frage, aber macht für die funktionalität kein Unterschied....meine Wissens jedenfalls

funktioniert die abfrage mit

... getPasswort().toString

oder könnte hier der Fehler liegen...
die Ausgabe sieht immerhin sehr kryptisch aus : [C@107077e

So funktioniert es leider auch nicht:

	public boolean userAvailable(User user) {
		boolean available = false;
		User operator = new User();
		if (operator.getAllUser().contains(user)){
			available = true;
		}
		return available;
	}

 

[Michael...]

Lass doch das

toString()

weg.

getPassword()

liefert Dir ein char-Array.
Vgl.

char[] c= {'a', 'b', 'c'};
System.out.println(c.toString());
System.out.println(c);

 

[MichaelKl]

die Methode soll ja prüfen ob der Benutzer vorhanden ist, und dies anhand von dem Benutzername.
Wenn das der Fall ist, soll sie true zurückgeben...

 

[Schandro]

@Michael
da muss aber noch z.b. ein

Arrays.toString(...)

drumherum, sonst machts keinen Unterschied (println ruft ja selber toString auf das Object auf...)

 

[eRaaaa]

Also wenn du das Passwort eh zu einem String umwandelst, kannst du auch gleich getText() auf dem Passwortfeld aufrufen?!

Ich würde aber lieber die Arrays vergleichen und hinterher "nullen" so wie hier beschrieben:

How to Use Password Fields (The Java™ Tutorials > Creating a GUI With JFC/Swing > Using Swing Components)

 

[Michael...]

@Schandro: Ich kenne die Implementierung nicht, aber beii mir ist

println(char[] c);

so schlau
@MichaelKl:
char-Arrays könnte man so mit einem String vergleichen:

System.out.println(String.valueOf(c).equals("abc"));

 

[MichaelKl]

nebenbei ... die Funktion funktioniert nicht wegen dem Array Problem sondern aus anderen Gründen...

ich frag ja das pw gar nicht ab...

 

[Michael...]

nebenbei ... die Funktion funktioniert nicht wegen dem Array Problem sondern aus anderen Gründen...

ich frag ja das pw gar nicht ab...

?? Hab Dich jetzt nicht verstanden ??
In dem oben geposteten Code steht doch in zwei von fünf Zeilen

jPasswordField.getPassword().toString(

und das was da rauskommt ist sicher nicht das was Du willst.
getText() würde funktionieren - ist aber deprecated

 

[U2nt]

Unten drunter hat er gefragt was an der Methode falsch ist, den User zu checken.

 

[Spacerat]

Wenn das PW nicht relevant ist, sollte [c]userAvailable()[/c] bereits funktionieren. Ich würd's aber ein wenig anders schreiben:

public boolean userAvailable(User user) {
  String name = user.getName();
  Collection<User> users = new ArrayList<User>(operator.getAllUser());
  for(User user : users) {
    if(name.equals(user.getName()) {
      return true;
    }
  }
  return false;
}