Swing Übergabe von String an JLabel

[Call_He_Tempi]

Hallo zusammen, ich komme aktuell einfach nicht weiter. Aufgabe ist es, Informationen von einem Server auszulesen, diese aufzubereiten und zu sortieren und diese Werte dann in einer GUI-Anwendung darzustellen. Ich bekomme es aber einfach nicht hin, den aufbereiteten String "Headline" so zu übergeben, das ich ihn in

headline = new JLabel(Headline);

hiermit ausgeben kann.
Der String "Headline" ist immer "null", obwohl ich hier eigentlich übergebe:

setHeadline.setHeadline(Headline);

package Version_1;

import Version_1.GUI.Client;

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

        ReaderPerList object = new ReaderPerList();
        object.readStream();

        Client client = new Client();
        client.client();
    }
}

package Version_1;
import java.util.*;

public class ReaderPerList {

    public void readStream() {

        GetSet setHeadline = new GetSet();
        HTTPClientConnection connection = new HTTPClientConnection();
        String str = connection.sendRequest("http://localhost/Protokoll.csv");

        ArrayList<String> Protokoll = new ArrayList<>(Arrays.asList(str.split("\n")));
        String Headline = Protokoll.get(0).replace(";", "  ");

        setHeadline.setHeadline(Headline);
        Protokoll.remove(0);

        cleanUp(Protokoll);
    }

    public void cleanUp(ArrayList<String> protokoll) {

        Remover removerObj = new Remover();
        ArrayList<String> CleanProtokoll = new ArrayList<>();


        for (int i = 0; i < protokoll.size(); i++) {
            String[] Coordinates = protokoll.get(i).split(";");
            String TimeStamp = Coordinates[0] + " " + Coordinates[1];

            Coordinates = removerObj.removeElement(Coordinates, TimeStamp.substring(0, 8));
            Coordinates = removerObj.removeElement(Coordinates, TimeStamp.substring(9, 17));

            for (int j = 0; j < Coordinates.length; j++) {

                if (Coordinates[j].startsWith("0") && Coordinates[j].length() == 2) {
                    Coordinates[j] = Coordinates[j].substring(1);
                }

                if (Coordinates[j].startsWith("0") && Coordinates[j].length() > 2) {
                    Coordinates[j] = Coordinates[j].substring(1, 3) + "Grad";
                }

                if (!Coordinates[j].startsWith("0") && Coordinates[j].length() == 3) {
                    Coordinates[j] = Coordinates[j].substring(0, 3) + "Grad";
                }
            }

            String line = "";

            for (int b = 0; b < Coordinates.length; b++) {

                if (Coordinates[b].length() >= 7) {
                    line = line + "    " + Coordinates[b];
                } else if (Coordinates[b].length() < 7) {
                    line = line + "     " + Coordinates[b];
                }
            }
            CleanProtokoll.add("\n" + TimeStamp + " " + line);
        }
        System.out.println(CleanProtokoll);
    }
}

package Version_1.GUI;

import Version_1.GetSet;

import javax.swing.*;
import java.awt.*;

import static Version_1.GUI.Panels.p_Output;
import java.io.*;
public class Client {

    static JFrame f;
    static JLabel serverStream;
    static JLabel headline;
    static JLabel start;
    static JPanel p_Output;
    static JPanel p_Headline;
    static JPanel p_Start;

    GetSet getHeadline = new GetSet();

    //String Headline = "lel";


    public void client(){

        String Headline = getHeadline.getHeadline();
        serverStream = new JLabel("panel label");
        headline = new JLabel(Headline);
        start = new JLabel("Start");

        p_Headline = new JPanel(new BorderLayout());
        p_Headline.setBackground(Color.yellow);
        p_Headline.setPreferredSize(new Dimension(600,100));
        p_Headline.add(headline);

        p_Output = new JPanel(new BorderLayout());
        p_Output.setBackground(Color.GRAY);
        p_Output.setPreferredSize(new Dimension(600, 333));
        p_Output.add(serverStream);

        p_Start = new JPanel(new BorderLayout());
        p_Start.setBackground(Color.green);
        p_Start.setPreferredSize(new Dimension(300,100));
        p_Start.add(start);

        f = new JFrame("panel");

        f.setLayout(new BorderLayout(10,10));
        f.add(p_Output, BorderLayout.CENTER);
        f.add(p_Headline, BorderLayout.NORTH);
        f.add(p_Start, BorderLayout.SOUTH);
        f.setSize(600,1000);
        f.setVisible(true);
    }
}

package Version_1;

public class GetSet {

    private String headline;

    public String getHeadline(){
        return headline;
    }

    public void setHeadline(String N) {
        this.headline = N;
    }
}

Ich hoffe, Ihr könnt mir weiterhelfen, ich weiß nämlich grad nichtmal, wonach ich googeln könnte...
LG

 

[KonradN]

Du hast ja unterschiedliche GetSet Instanzen. In Client erzeugst Du eine Instanz (Zeile 20) und in ReaderPerList ebenso (Zeile 8).

Wenn Du in ReaderPerList da einen Wert rein schreibst erscheint der ja nicht in der anderen Instanz. Wenn Du darüber daten austauschen willst, dann erzeuge die Instanz z.B. in der main Methode und übergib diese dann in den Konstruktoren der beiden Klassen.

 

[Call_He_Tempi]

Du hast ja unterschiedliche GetSet Instanzen. In Client erzeugst Du eine Instanz (Zeile 20) und in ReaderPerList ebenso (Zeile 8).

Wenn Du in ReaderPerList da einen Wert rein schreibst erscheint der ja nicht in der anderen Instanz. Wenn Du darüber daten austauschen willst, dann erzeuge die Instanz z.B. in der main Methode und übergib diese dann in den Konstruktoren der beiden Klassen.

Vielen Dank! Mir ist doch tatsächlich aufgefallen, das ich genau die selbe Problemstellung hier schon einmal gestellt habe und du mir sogar genau da selbe mit den Instanzen und dem Konstruktor gesagt hast. Ist nur zu lange her und das Problem nicht 1:1 in mein jetziges Projekt zu übertragen. Ich mache mich dann mal erneut an des Rätsels Lösung und poste sie dann hier, wenn ich soweit bin. Verstehe die ganze Konstruktor und übergeben Sache noch nicht so ganz.

 

[Call_He_Tempi]

Also ich habs jetzt tatsächlich hinbekommen, ich habe aber nicht wirklich verstanden, was genau ich da gemacht habe. Könnte mir jemand mal mit den richtige Fachbegriffen erklären, wie ich jetzt den String "Headline" in die Class "Client" bekommen habe und diesen dort verwenden kann?

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

        ReaderPerList object = new ReaderPerList();
        object.readStream();
    }
}

public class ReaderPerList {
   
    public void readStream() {

        HTTPClientConnection connection = new HTTPClientConnection();
        String str = connection.sendRequest("http://localhost/Protokoll.csv");

        ArrayList<String> Protokoll = new ArrayList<>(Arrays.asList(str.split("\n")));
        String headline = Protokoll.get(0).replace(";", "  ");
        Client sendHeadline = new Client(headline);
        Protokoll.remove(0);
        cleanUp(Protokoll);
    }

public class Client {

    static JFrame f;
    static JLabel serverStream;
    static JLabel headline;
    static JLabel start;
    static JPanel p_Output;
    static JPanel p_Headline;
    static JPanel p_Start;
   
    String Headline;

    public Client(String headline){
        this.Headline = headline;
        client();
    }

    public void client(){

        serverStream = new JLabel("panel label");
        headline = new JLabel(Headline);
        start = new JLabel("Start");

Und warum, sagt mir "IntelliJ", das das Objekt "sendHeadLine" nie verwendet wird, obwohl ich hier doch den String "Headline" übergebe?

Client sendHeadline = new Client(headline);

 

[KonradN]

Hmm, was Du da gebaut hast mag erst einmal funktionieren, aber das ist absolut das, was man eben nicht machen sollte!

Du musst immer bedenken: static ist ein nicht unwichtiges Mittel. Das ist sozusagen ein Teil der Macht. Aber wenn Du es so viel einsetzt, dann wechselst Du zur "dunklen Seite der Macht". Das willst Du nicht! Du willst doch weiter hier bei uns Jedis sein!

ok, Spass bei Seite.

Was macht static: Damit existieren die Variablen nur einmal in der Klasse. Das ist entgegengesetzt zu dem, was man in Java eigentlich möchte: Objektorientiert entwickeln. Man will also einzelne Objekte erstellen.

Wenn Du mehrere Client Instanzen erzeugst, dann gibt es dennoch nur jeweils eine Variable f. Du kannst also nicht mehrere Fenster erstellen. Dein Code ist somit extrem limitiert und Du nimmst Dir die Chance, in die Welt der objektorientierten Entwicklung richtig einzutauchen.

 

[Call_He_Tempi]

Ne, ich bleib doch lieber bei den Jedis
Es geht tatsächlich erstmal mehr um die Übergabe des Strings "Headline" an public class Client{..} als um den Aufbau des Clienten

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

        ReaderPerList object = new ReaderPerList();
        object.readStream();
    }
}

public class ReaderPerList {
  
    public void readStream() {

        HTTPClientConnection connection = new HTTPClientConnection();
        String str = connection.sendRequest("http://localhost/Protokoll.csv");

        ArrayList<String> Protokoll = new ArrayList<>(Arrays.asList(str.split("\n")));
        String headline = Protokoll.get(0).replace(";", "  ");
        Client sendHeadline = new Client(headline);
        Protokoll.remove(0);
        cleanUp(Protokoll);
    }

public class Client {

    String Headline;

    public Client(String headline){
        this.Headline = headline;
        client();
    }

    public void client(){

        serverStream = new JLabel("panel label");
        headline = new JLabel(Headline);
        start = new JLabel("Start");

 

[KonradN]

Also der normale Aufbau bei so GUI Anwendungen ist immer etwas anders.

- Es startet immer der Client. Das ist der eigentliche Start. Du hast also etwas wie:

public class Main {
    public static void main(String[] args) {
        ClientApp app = new ClientApp();
        app.start(); // Das kann auch ein .view() sein oder der Client macht sich schon im Konstruktor sichtbar.
    }
}

ClientApp hatt dann einen JFrame und so - also das, was Du in Client bereits hast. Da kann dann aber der JLabel Eintrag noch etwas sein wie "Loading ..." oder so.

Die Anwendung macht dann irgendwas. Du hast da ein ReaderPerList gebaut. Das kann auch gerne so sein. Das packen wir aber dann in einen eigenen Thread, damit wir den Thread, der die Events der Anwendung verarbeitet, nicht blockieren. Das kann ein eigener Thread sein aber auch ein SwingWorker um das mal als Option zu nennen.

Jetzt ist die Frage: Wie soll denn das Ergebnis zurück kommen? Du hast eine Methode readStream und das Ergebnis ist hier einfach ein String. Das passt doch, denn einen einzelnen Wert kannst Du ja zurück geben:

    public String readStream() {
        HTTPClientConnection connection = new HTTPClientConnection();
        String str = connection.sendRequest("http://localhost/Protokoll.csv");

        ArrayList<String> Protokoll = new ArrayList<>(Arrays.asList(str.split("\n")));
        String headline = Protokoll.get(0).replace(";", "  ");
        Protokoll.remove(0);
        cleanUp(Protokoll);
        return headline;
    }

Damit hast Du sozusagen etwas Logik geschrieben, die unabhängig ist von der UI. Und das ist gut! Das kannst Du immer und überall einsetzen. Also auch in einer anderen Server Anwendung oder in einer Anwendung, die nur auf der Konsole läuft oder ....

Es gibt also eine andere Klasse, die das aufruft und das Ergebnis dann in der GUI präsentiert. Das kann dann die GUI kennen. Man kann z.B. ein JLabel übergeben. Aber morgen willst Du da kein JLabel nutzen sondern Du findest ein neues Control: SuperCoolAussehendesLabel. Dann könnte man das abstrahieren und z.B. ein Funktionales Interface nutzen. Da wäre dann ein Consumer<String> denkbar und die UI übergibt dann einfach eine Lambda Expression a.la. v -> myLabel.setText(v) oder so.
Aber das ist Zukunftsmusik. Fang an, dass Du da erst einmal einfach nur das JLabel übergibst und fertig.

 

[Call_He_Tempi]

Also der normale Aufbau bei so GUI Anwendungen ist immer etwas anders.

- Es startet immer der Client. Das ist der eigentliche Start. Du hast also etwas wie:

public class Main {
    public static void main(String[] args) {
        ClientApp app = new ClientApp();
        app.start(); // Das kann auch ein .view() sein oder der Client macht sich schon im Konstruktor sichtbar.
    }
}

ClientApp hatt dann einen JFrame und so - also das, was Du in Client bereits hast. Da kann dann aber der JLabel Eintrag noch etwas sein wie "Loading ..." oder so.

Die Anwendung macht dann irgendwas. Du hast da ein ReaderPerList gebaut. Das kann auch gerne so sein. Das packen wir aber dann in einen eigenen Thread, damit wir den Thread, der die Events der Anwendung verarbeitet, nicht blockieren. Das kann ein eigener Thread sein aber auch ein SwingWorker um das mal als Option zu nennen.

Jetzt ist die Frage: Wie soll denn das Ergebnis zurück kommen? Du hast eine Methode readStream und das Ergebnis ist hier einfach ein String. Das passt doch, denn einen einzelnen Wert kannst Du ja zurück geben:

    public String readStream() {
        HTTPClientConnection connection = new HTTPClientConnection();
        String str = connection.sendRequest("http://localhost/Protokoll.csv");

        ArrayList<String> Protokoll = new ArrayList<>(Arrays.asList(str.split("\n")));
        String headline = Protokoll.get(0).replace(";", "  ");
        Protokoll.remove(0);
        cleanUp(Protokoll);
        return headline;
    }

Damit hast Du sozusagen etwas Logik geschrieben, die unabhängig ist von der UI. Und das ist gut! Das kannst Du immer und überall einsetzen. Also auch in einer anderen Server Anwendung oder in einer Anwendung, die nur auf der Konsole läuft oder ....

Es gibt also eine andere Klasse, die das aufruft und das Ergebnis dann in der GUI präsentiert. Das kann dann die GUI kennen. Man kann z.B. ein JLabel übergeben. Aber morgen willst Du da kein JLabel nutzen sondern Du findest ein neues Control: SuperCoolAussehendesLabel. Dann könnte man das abstrahieren und z.B. ein Funktionales Interface nutzen. Da wäre dann ein Consumer<String> denkbar und die UI übergibt dann einfach eine Lambda Expression a.la. v -> myLabel.setText(v) oder so.
Aber das ist Zukunftsmusik. Fang an, dass Du da erst einmal einfach nur das JLabel übergibst und fertig.

Oh vielen dank, damit kann ich erstmal arbeiten bzw. hab ich ein sehr guten Anhaltspunkt. Vielen Dank!!

 

[Call_He_Tempi]

Konniiiiii

 

[Call_He_Tempi]

Ehrenmann!