JLabel Visible setzen in KeyListener

[jalako]

Hallo,

Ich versuche momentan über einen KeyListener ein JLabel in einer anderen Klasse Visible bzw. UnVisible zu setzen. Dies funktioniert aber einfach nicht wie es soll.

Teil des KeyListener Codes:

case KeyEvent.VK_E:
    Inventory.inv2.setVisible(true);
break;

JLabel Code in anderer Klasse:

inv2 = new JLabel(new ImageIcon(getClass().getResource("/resource/inventory.png")));
inv2.setLayout(null);
inv2.setBounds(0,0,650, 400);

Wenn ich nun E drücke kommt die zum testen dahingeschriebene "System.out.println("");" zwar, jedoch wird das Label nicht visible. Habe ich vielleicht irgendwo einen Denkfehler?

 

[thet1983]

zeig mal deine komplette switch

 

[keram]

Kann es sein das du deine Klasse zwei mal initialisiert hast?

 

[Thallius]

Ist zwar wie Kaffeeesatzlesen hier aber ich würde mal tippen er hat das Label gar nicht geadded, da es sonst ja schon direkt angezeigt würde....

 

[jalako]

Das Label wird direkt angezeigt aber anschließend direkt visible(false) gesetzt und der andere Code bei dem "case KeyEvent.VK_E:" wird auch ausgeführt. nur das Label wird durch visible(true) einfach nicht visible

 

[Thallius]

Dann siehe Post #3

 

[jalako]

Es ist halt die Klasse die heißt Inventory also

"Public class Inventory"

und dann ne funktion public Inventory()

die erstellen wir dann "Inventory inv2 = new Inventory();"

und "Inventory.inv2.setVisible(true);" tut dann nicht was es soll

 

[Thallius]

Kann es auch nicht- Siehe Post #3

Objecte und Instanzen sind zwei vollkommen unterschiedliche Paar Schuhe. Du solltest Dich erst noch einmal mit den Grundlagen der OOP auseinander setzen bevor du anfägst irgendwelche UI Programme zu schreiben.

 

[jalako]

Das bringt mich bei diesem spezifischen Problem jetzt aber nicht direkt weiter...

 

[Thallius]

Doch, wenn Du Dich erstmal mit Grundlagen beschäftigst statt irgendwelchen, für dich, viel zu komplexen Code zu versuchen zu einem Programm zusammen zu klicken, dann hilft dir das auch weiter. Wenn Du daran kein Interesse hast, dann ist das eventuell das falsche Hobby, denn lesen und lernen gehört zur Informatik ein Leben lang dazu.

 

[jalako]

Vielen Dank für diesen Tipp. Ich lese mich noch mal ein, wenn ich wieder Zeit habe. Könntest du aber vielleicht trotzdem noch mal gucken, ob du meinen Fehler nicht doch findest... Das Programm kommt ja in alle Methoden rein, was ich mit "System.out.println("");" getestet habe, aber das setVisible funktioniert einfach nicht.

 

[Flown]

Du hast kein Programm gepostet. Dein Snippet ist wertlos. Entweder du postest den ganzen Code oder musst dich selbst herumquälen.

 

[keram]

Das Programm kommt ja in alle Methoden rein, was ich mit "System.out.println("");" getestet habe, aber das setVisible funktioniert einfach nicht.

System.out kannst du von überall anrufen da es sich auf die Konsole bezieht und diese ein und die selbe bleibt.

Es kann sein das du deine JLabel falsch initalisiert hast. Du musst eine Object referenz aufbauen. Solltest du aus anderen Klassen versuchen das Label anzusprechen dann musst du deine Object referenz mit übergeben und es nicht neu initalisieren. Es wird zwar ein Label erstellt es verschwindet jedoch in einen nicht Sichtbaren berreich, sprich du hast zwei Labels. Stoff zum lesen

 

[Harry Kane]

Inventory.inv2.setVisible(true);

Die Syntax legt nahe, das inv2 eine statische Variable der Inventory-Klasse ist, und bei statischen Variablen habe ich sofort den Verdacht, das jemand den Unterschied zwischen Klasse und Instanz nicht verstanden hat und/oder einen Workaround brauchte, um "von überall auf die Variable zugreifen zu können".

JLabel Code in anderer Klasse:
inv2 = new JLabel(new ImageIcon(getClass().getResource("/resource/inventory.png")));
inv2.setLayout(null);
inv2.setBounds(0,0,650, 400);

Was ist die "andere Klasse"?
Ein JLabel braucht kein Layout, weil es im allgemeinen nicht als Container verwendet wird.
Die Deklaration von inv2 ist nicht gezeigt. Damit ist unklar, ob die beiden inv2 wirklich auf dasselbe Objekt zeigen.

Wenn ich nun E drücke kommt die zum testen dahingeschriebene "System.out.println("");"

In deinem Codesnippet gibt es kein Sysout. Und als Test einen Leerstring auszugeben, ist... nunja... ungewöhnlich. Warum nicht etwas spezifisches wie "Ich bin in KeyEvent.VK_E" ?

und dann ne funktion public Inventory()

Deine "funktion" ist der Konstruktor.

"Inventory inv2 = new Inventory();"
und "Inventory.inv2.setVisible(true);" tut dann nicht was es soll

Full stop. Ist inv2 eine Instanz von JLabel (s. erstes Snippet) oder von Inventory?
Und wenn inv2 eine Instanz von Inventory ist, dann ist Inventory.inv2 so etwas wie ein gewolltundnichtgekonnt-Singleton.
Kann den anderen Postern nur zustimmen: Grundlagen lernen. Mehr Code posten. Wenn der Code den du hast größer ist als 200 Zeilen, dann mit was kleinerem anfangen.