Frame ändert Farbe nicht

[Kotelettklopfer]

Hallo zusammen,

jemand eine Idee warum mein Frame nicht die Farbe anpasst?
Was mache ich hier falsch?

[CODE lang="java" highlight="24,25"]import javax.swing.*;
import java.awt.*;

public class InfoFrame extends JFrame {


static JLabel text;



public InfoFrame() {
setVisible(true);
setSize(300, 100);
setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
setLocationRelativeTo(null);
setTitle("Boss Info");
setResizable(false);
setLayout(null);




text = new JLabel();
setForeground(Color.BLUE);
setBackground(Color.BLACK);
text.setBounds(0, 15, 300, 20);
add(text);


}




public static void setStatus(String status) {
text.setText(status);
}
}

[/CODE]

So sieht das ganze dann aus

 

[LimDul]

Wie soll es denn aussehen? Du änderst die Farbe vom Frame, das hat keinerlei Auswirkungen auf die Farbe von angezeigten Komponenten. Wenn du den Text geändert haben willst, muss du die Farbe des Textes ändern. Wenn den Hintergrund ändern willst, musst du entweder den Hintergrund des Labels ändern (und es auf nicht transparent setzen) oder vermutlich von der Rootpane die Farbe ändern.

 

[kneitzel]

Das JFrame ist so eigentlich nicht sichtbar, daher siehst Du die Farben nicht. Der Hintergrund ist nicht der background vom JFrame sondern con der ContentPane. Den Background müsstest Du also per getContentPane().setBackground(...) setzen können.
Und das beeinflusst aber nicht den Label - wie @LimDul bereits gesagt hat.

Und dann solltest Du mit dem static aufpassen. Warum ist das Jabel text bei dir static?

 

[Kotelettklopfer]

Das Programm soll nach dem öffnen immer zu bestimmten Uhrzeiten einen Text ausgeben. Gesteuert wird das durch "switch Case". Es kann auch vorkommen, das mal zwei Case gleichzeitig angezeigt werden müssen. Da ich es nicht hinbekommen habe, dass der Frame die Zeilen nicht gegenseitig überschreibt, habe ich mich für eine Variante entschieden, die für jeden Text ein neues Frame generiert.

Beim erstellen der neuen Klasse, schlug intelij "static" vor, da ich ansonsten Fehler bekomme.

 public static void setStatus(String status) {
        text.setText(status);
    }

"Non-static field 'text' cannot be referenced from a static context"


Ich hoffe man versteht was ich meine, mir fällt das erklären sehr schwer, weil mir die ganzen Begrifflichkeiten noch nicht ganz klar sind.
Denke aber ich lerne besser, wenn ich versuche eigene Projekte umzusetzen.

Sorry falls mal was durcheinander geworfen wird

Edit:
Das Programm läuft so auch, aber die Farbe macht Ärger

 

[Kotelettklopfer]

Ok habe es hinbekommen..
Vielen Dank.


[CODE lang="java" highlight="19,25"]import javax.swing.*;
import java.awt.*;

public class InfoFrame extends JFrame {


static JLabel text;



public InfoFrame() {
setVisible(true);
setSize(370, 100);
setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
setLocationRelativeTo(null);
setTitle("Boss Info");
setResizable(false);
setLayout(null);
getContentPane().setBackground(Color.red);



text = new JLabel();
text.setBounds(10, 15, 370, 20);
text.setForeground(Color.white);
add(text);


}




public static void setStatus(String status) {
text.setText(status);
}
}

[/CODE]

 

[mihe7]

Zwei Dinge:

1. static raus!
2. setVisible immer ganz am Ende (hier also z. B. in Zeile 27) aufrufen.

 

[Kotelettklopfer]

Zwei Dinge:

1. static raus!
2. setVisible immer ganz am Ende (hier also z. B. in Zeile 27) aufrufen.

dann haut er mir unten bei text.setText "Non-static field 'text' cannot be referenced from a static context" um die Ohren.


Habe dann das static auch unten entfernt bei "public static void SetStatus", dann meckert er in der Klasse "Applikation"
Non-static method 'setStatus(java.lang.String)' cannot be referenced from a static context

[CODE lang="java" highlight="39"]private void synth() throws MidiUnavailableException, InvalidMidiDataException, InterruptedException {

Synthesizer synth = MidiSystem.getSynthesizer();
synth.open();
Receiver rcvr = synth.getReceiver();
ShortMessage msg = new ShortMessage();
msg.setMessage(ShortMessage.NOTE_ON, 0, 30, 30);
rcvr.send(msg, -1);
Thread.sleep(100);
msg.setMessage(ShortMessage.NOTE_OFF, 0, 40, 0);
msg.setMessage(ShortMessage.NOTE_ON, 0, 40, 40);
rcvr.send(msg, -1);
Thread.sleep(200);
msg.setMessage(ShortMessage.NOTE_OFF, 0, 40, 0);
msg.setMessage(ShortMessage.NOTE_ON, 0, 50, 50);
rcvr.send(msg, -1);
Thread.sleep(300);
msg.setMessage(ShortMessage.NOTE_OFF, 0, 40, 0);
msg.setMessage(ShortMessage.NOTE_ON, 0, 60, 60);
rcvr.send(msg, -1);
Thread.sleep(400);
msg.setMessage(ShortMessage.NOTE_OFF, 0, 40, 0);
msg.setMessage(ShortMessage.NOTE_ON, 0, 80, 80);
rcvr.send(msg, -1);
Thread.sleep(500);
msg.setMessage(ShortMessage.NOTE_OFF, 0, 40, 0);
msg.setMessage(ShortMessage.NOTE_ON, 0, 90, 90);
rcvr.send(msg, -1);
Thread.sleep(4000);
msg.setMessage(ShortMessage.NOTE_OFF, 0, 40, 0);
rcvr.send(msg, -1);
synth.close();
}

private void updateStatus(String status) {
SwingUtilities.invokeLater(new Runnable() {
@Override
public void run() {
InfoFrame.setStatus(status);
}
});
}
[/CODE]

 

[kneitzel]

Du erzeugst irgendwann eine Instanz von InfoFrame und speicherst diese als Instanzvariable. Auf dieser Instanzvariable rufst Du dann die Methode auf.

Die Instanz von InfoFrame erzeugst Du ja jetzt auch schon irgendwo - nur eben merkst Du Dir die Instanz nicht oder machst es an einer Stelle, die zu optimieren wäre ...

 

[Kotelettklopfer]

Ich kann versuchen was ich will, ich bekomme das static nicht sauber rauscodiert.
Er wirft immer diesen einen Fehler raus.
Seltsamerweise läuft die APP aber.

Warum genau ist static hier denn falsch?

 

[kneitzel]

Falsch ist relativ. Es ist nicht im Sinne einer objektorientierten Entwicklung.

Bei der objektorientierten Entwicklung hast du Objekte mit Attributen In Deinem Fall ist das Textfeld ja ein klares Attribut vom Fenster. Durch das static ist es aber kein Attribut des Fensters mehr sondern von der Klasse des Fensters.
Problematisch wird es, wenn Du z.B. mehrere Fenster darstellen willst. Dann hast Du mehrere Instanzen Deiner Klasse, aber diese Variable "text" gibt es nur einmal und verweist auf das Textfeld des letzten erzeugten Fensters.

 

[Kotelettklopfer]

Glaube ich habe es geschafft.
Zumindest sind die Statics raus (im Bezug zum Text)

Ich habe dazu Zeile 1 angepasst

[CODE lang="java" highlight="1,"]JLabel text = new JLabel();



public BossInfo() {
setSize(370, 100);
setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
setLocationRelativeTo(null);
setTitle("Boss Info");
setResizable(false);
setLayout(null);
getContentPane().setBackground(Color.red);



text = new JLabel();
text.setBounds(10, 15, 370, 20);
text.setForeground(Color.white);
add(text);

setVisible(true);
}




public void setStatus(String status) {
text.setText(status);
}
}
[/CODE]



und der Fehler lag dann in der Applikation Zeile 6. Hier stand BossInfo und nicht bossInfo.

[CODE lang="java" highlight="6"]
private void updateStatus(String status) {
SwingUtilities.invokeLater(new Runnable() {
@Override
public void run() {
bossInfo.setStatus(status);
}
});
}[/CODE]

So ist es "richtig" oder ?

 

[kneitzel]

So ist es "richtig" oder ?

So sieht es richtig aus (haben natürlich nur einen kleinen Einblick bekommen). Aber so war es gemeint.

 

[Kotelettklopfer]

Danke Euch !!!!
Jetzt heißt es Code studieren und verstehen was da passiert.

Noch viel zu lernen ich habe