JPasswordField klassenübergreifend auslesen->static Probl

[Benny85]

Hallo alle zusammen,

ich bin gerade im Rahmen eines Projektes für mein Wirtschaftsinformatikstudium etwas mit Java rumzuprogrammieren...

Dabei bin ich jetzt auf folgendes Problem gestossen.
Das Szenario ist wie folgt:
Es dreht sich um den Loginscreen des Programmes, es gibt also ein JField für den Benutzernamen und ein JPasswordField für das Passwort sowie einen Button zum einloggen.
Das ganze steckt in einer Methode, die ein JPanel zurückliefert, welches in einer anderen Klasse in ein JTabbedPane eingefügt wird.

public class Login extends GUI {
	static JPasswordField enterpw = new JPasswordField(20);
	
	public static JPanel loginpanel() {
        [...]
JPasswordField enterpw = new JPasswordField(20);
		enterpw.setFont(new Font("Verdana", Font.BOLD, 13));
		c.gridx = 3;
		loginpanel.add(enterpw,c);
		Loginbutton login = new Loginbutton();
		login.setText("  Login!");
		URL loginbuttonurl = Login.class.getResource( "/images/login.gif" ); 
	    final Icon loginbuttonicon = new ImageIcon(loginbuttonurl);
		login.setIcon(loginbuttonicon);
		login.setFont(new Font("Verdana", Font.BOLD, 13));
		login.addActionListener(login);
       [...]
       }
	public static void readPassword() {
		System.out.println(enterpw.getText());	
	}

Der Button zum einloggen ist als eigene Klasse definiert:

class Loginbutton extends JButton implements ActionListener {
	
	public void actionPerformed(ActionEvent AE) {
		Login.readPassword();
	}

	static String verschluesseln(String passwordToEncrypt) {
		[...]
	}
	
	public static String bytesToHex(byte[] b) { 
	      [...]
	   } 
}

Leider kann ich jetzt mit der Methode readPassword (aufgerufen durch die methode actionPerfomed in der Klasse Loginbutton) nicht das Passwort auslesen bzw nicht den aktuellen Wert auslesen. Wenn ich das JPasswordField bereits mit einem Wert initialisiere, liest die Methode immer den initialisierten Wert aus, auch wenn was andres im Feld steht.
Wahrscheinlich liegt das daran, dass ich das JPasswordField mit static deklariert habe.
Wenn ich static jedoch entferne, bekomme ich den kompilierungsfehler "Cannot make a static reference to the non-static dield enterpw"

Was kann ich tun?! Ich bin momentan wirklich mit meinem Latein am Ende.. kann mir jemand von euch weiterhelfen? Zumindest mit einer Idee?

Danke schoneinmal im Vorraus,
viele Grüsse,
Benny

 

[Murray]

Dur erzeugst in der Methode loginpanel() eine neue lokale Variable mit dem gleichen Namen wie die statische Referenz. Dadurch fragst du immer das initial angelegt Feld ab, aber nicht das Feld, dass der Benutzer ausgefüllt ab.

Schnelle Lösung:

public static JPanel loginpanel() {
        [...]
      enterpw = new JPasswordField(20); //--- so wird keine neue Variable angelegt
      enterpw.setFont(new Font("Verdana", Font.BOLD, 13));

Grundsätzlich deutet die Notwendigkeit eines statischen Widgets aber auf eine gewisse Designschwäche hin.

Und noch ein Tip am Rande: es ist gängige Praxis, Passwörter im Programm nicht als Strings, sondern als char[] zu behandeln, damit sie nicht ganz so offensichtlich zu erkennen sind.

 

[Benny85]

Hi Murray,

danke für deine Antwort!
Mit deinem Hinweis auf das doppelt deklarierte JPasswordField hast du natürlich recht! Habe vor meinem ersten Posting nochmal verschiedene Varianten mit meinem Code ausprobiert und dabei scheinbar vergessen, die zweite Initialisierung wieder zu entfernen...
Wenn ich den Code jetzt allerdings deinem Vorschlag entsprechend anpasse, kommt in Eclipse der Compilerfehler "Cannot make a static reference to the non-static field enterpw" in der zeile

enterpw = new JPasswordField(20);

Das mein Code gewisse Designschwächen hat (sehr nett ausgedrückt ;-) ) habe ich befürchtet. Du hast nicht zufällig nen guten Link parat, in dem das Chaos mit static und non-static etc mal anschaulich erklärt wird (oder selber die Motivation, etwas zu schreiben? gerne poste ich den vollständigen Code wenn das hilft...)

Danke für den best-practice hinweiss zu der passwortbehandlung - das werde ich anpassen!

Viele Grüsse,
Benny

 

[Murray]

Zumindest im geposteten Code ist enterpw doch static deklariert ?!?

 

[Benny85]

ich bekenne mich in allen anklagepunkten schuldig... ;-)
Ich hab keine Ahnung warum ich das übersehen habe.. jetzt funktioniert alles.. ich habe wohl zu viel rumprobiert und dabei irgendwann den überblick verloren und diese kombination nie überprüft...
Danke trotzdem für deine Hilfe!!!

...und wenn trotzdem noch jemand mehr Infos über static und non-static hätte, wäre das das i-tüpfelchen ;-)

Danke!!
Benny