Wie KeyListener adden?

[Anto]

class A {

public A {
}


}


class B implements KeyListener{

private A a;

private B(A x){

this.a = x;
a.addKeyListener(this);

}


public static void main(final String[] args) {
            
                
		A x = new A();
                B b = new B(x);
                
                
	}

}

Der Grundgedanke: Ich erzeuge in B, wo meine Main steht, ein Objekt der Klasse A und möchte diesem Objekt einen KeyListener verpassen - und zwar von B aus, da a auf B keinen Zugriff hat.
Der addBefehl wirft keinen Fehler, allerdings eine Checkstylewarnung: Leaking this in Construktor.
Wie korrigiere ich das, bzw. ist meine Lösung von Anfang an grundfalsch?

Grüße

Anto

 

[Michael...]

[Der addBefehl wirft keinen Fehler, allerdings eine Checkstylewarnung: Leaking this in Construktor.

Ist ja "nur" eine Warnung und sollte in dem Fall - sofern der PseudoCode hier mit dem echten Code übereinstimmt - keine Probleme verursachen.
Wenn ich es richtig deute, empfiehlt Deine IDE hier

this.a.add...

 

[SlaterB]

nein, es bedeutet (gerade selber nachgeschlagen, alles im Internet zu finden ) dass this nicht im Konstruktor an wer weiß wen übergeben werden sollte, das Objekt ist noch nicht voll initialisiert, besonders schlimm wird das ja mit weiterer Vererbung,

in diesem Fall immer noch ungefährlich, allgemein durchaus eine schlechte Variante,
bei mir würde diese Meldung aber ziemlich oft auftauchen..

die Alternative wäre, nach Ende des Konstruktors, z.B. in der main-Methode, a.addKeyListener(b); aufzurufen,
das spart dann auch die Übergabe an B, sofern nicht aus anderem Grund nötig

 

[bygones]

taucht auch auf, wenn man eine vererbare klasse hat, die im Konstruktor eine vererbare Methode aufruft.

Wie schon gesagt, das Problem ist, dass die Gefahr droht, dass jemand etwas in der Methode macht mit einem Objekt, was noch nicht vollstaendig initalisiert ist.

Daher sollte man das immer vermeiden....

 

[Anto]

Slater was Du meintest habe ich probiert, dann springt allerdings der KeyListener gar nicht an, also auch ein println("Hallo") würde nicht funktionieren.

 

[SlaterB]

und ging es vorher mit this?
allgemein kann das viele Ursachen haben, ein vollständiges Testprogramm oder jede Zeile mehr Code wäre hilfreich

 

[Anto]

Das einzige was funktioniert ist IN A reinzuschreiben: this.addKeyListener(new B (this));

Das ist die einzige Methode die ich bisher kenne die anspringt und das ist wohl kaum sinnvoll.

 

[Anto]

Ich habe es jetzt nochmal umgeschrieben habe aber immer noch dasselbe Problem:

Ich habe eine Klasse 1 die die mainMethode enthält.
In der mainMethode wird ein Objekt der Klasse 2 erzeugt.
Klasse 2 implementiert KeyListener.
Klasse zwei erzeugt außerdem ein Objekt der Klasse 3.

Mit welchem Befehl kann ich nun an das Objekt der Klasse 3 einen KeyListener dranhängen? Ich habe jetzt schon bestimmt 15 mögliche Kombinationen probiert, ein paar davon kompilieren, aber bei keinem springt der Listener dann auch an.

 

[SlaterB]

wenn du in dermain bist, dann in etwa
b.get3().addKeyListener(b);
abr nach wie vor wäre Code hilfreich

 

[Anto]

Den KeyListener adden will ich aber doch aus Klasse 2!

class Main {public static void main(final String[] args) {

Con c = new Con():


}


}


class Con implements KeyListener {

public Con () {}

Fr f = new Fr();


}


Class Fr extends JFrame{

public Fr () {}


}

Warum soll ich da mehr Code bringen, es existiert überhaupt noch nicht mehr Code (naja, bis auf die DummyMethoden KeyTyped usw...und darin halt ein Printbefehl.

Was ich nun möchte, ist, dass Con selbst an f einen Listener dranhängt, denn nach unserer Vorgabe muss es der Controller sein, der den Listener an den Frame hängt. Und ich finde den Befehl dafür nicht.

 

[Anto]

So, dann nochmal vollständiger Code, so wie er bei mir im Programm (Testprojekt) steht:

package testkey;

/**
 *
 * @author admin
 */
public class Testkey {

    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        Lis lis = new Lis();
    }
}
package testkey;

import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

/**
 *
 * @author admin
 */
class Lis implements KeyListener{
    
    Feld f = new Feld();
    
    
    
    public Lis(){
    }

    @Override
    public void keyTyped(KeyEvent e) {
        System.out.println("Hallo");
    }

    @Override
    public void keyPressed(KeyEvent e) {
          System.out.println("Hallo");
    }

    @Override
    public void keyReleased(KeyEvent e) {
         System.out.println("Hallo");
    }
    
    
    
}
package testkey;

import javax.swing.JFrame;

/**
 *
 * @author admin
 */
class Feld extends JFrame{
    
    public Feld() {}
    
    
    
}

Mit welchem Befehl hänge ich jetzt an f einen KeyListener? Ist das Problem wirklich so unlösbar bzw. die Implementation so schlecht?

 

[Kevin94]

Feld f;
    public Lis()
    {
         f=new Feld();
         f.addKeyListener(this);
    }

Du musst allerdings beachten, das blos weil du an nem Frame einen KeyListener angemeldest hast, dessen Methoden nicht bei jedem Tastendruck aufgerufen werden. Das Frame muss den Fokus haben.

Wenn das nicht klappt probiers mal mit nem Textfeld. Wenn dann in dem Textfeld Text erscheint und deinen Methoden nicht aufgerufen werden solltest du dir Sorgen machen.

 

[Anto]

Das war meine ursprüngliche Lösung, das ist aber "verboten" weil ich im Konstruktor nicht auf this referenzieren darf, es kommt auch die Warnung "leaking this in constructor".

Das heisst ich muss das irgendwo anders als im Konstruktor von Lis machen. Wo?

 

[Kevin94]

das ist nicht "verboten", sondern du wirst blow gerwarnt das du nicht gedankenlos this an irgendwelche Methoden übergeben sollst. Die alternative währe, wie schon gesagt beide Objecte in der main zu erstellen:

Feld f=new Feld();
Lis lis=new Lis();
f.addKeyListener(lis);

 

[Anto]

Nun ist aber Lis mein Kontroller, ist es bei Model View Control nicht eigentlich üblich, dass der Controller die View erzeugt?

Und: bei beiden Varianten muss doch noch irgendetwas anderes Fehlen, denn der Listener springt jeweils nicht an?

 

[Kevin94]

Wie geseagt: versuch mit nem JTextField oder ähnlichem.

und bei dir fehlt noch der "Model" teil, weil du den noch nicht implementiert hast.