ArrayList in 2D-Array konvertieren.

[Joerg66]

Hallo,
ich lese mit:

               List<String> list = new ArrayList<String>();
               try {
                   while((buffer = ident_event.readLine()) != null){
                       list.add(buffer);
                   }
               } catch (IOException e) {
                   e.printStackTrace();
               }

Eine Datei ein, von der ich die Länge vorher nicht kenne. In der Datei stehen untereinander String-Paare:
Fahrrad Pedale
Moped Fußraste
usw.
Nun kann ich mit:

String[] ie_Array = list.toArray(new String[0]);

aus der Liste ein 1D Array machen.
Ich brauche aber ein 2D Array, also die Zeilen noch am Leerzeichen getrennt.
Mir fehl jede Idee, wie ich das anstelle, ohne jetzt das 2D Array zusätzlich zu erzeugen und das dann per Schleife zu befüllen.
Kann mir da wer helfen?
Gruß Jörg

 

[Robert Zenz]

Mir fehl jede Idee, wie ich das anstelle, ohne jetzt das 2D Array zusätzlich zu erzeugen und das dann per Schleife zu befüllen.

Wird nicht ohne gehen. Vermutlich gibt es irgendeine Stream-Magie dafuer, aber eine einfache Schleife dafuer ist schnell getippt und leicht verstaendlich. Unmittelbare Array-Operationen fuer einen solchen Fall gibt es nicht.

 

[Joerg66]

Ok, danke .... und schade

 

[KonradN]

Vermutlich gibt es irgendeine Stream-Magie dafuer,

Das ist sogar sehr einfach:

• Files.lines() gibt einem ein Stream von Strings
• per map dann das String.split aufrufen und schon hat man ein Stream von String Arrays
• Stream zu Array ist doof … dann einfach erst zu List und dann zum Array. Also erst Stream mit toList abschließen, Zielatray erstellen und dann toArray auf der List aufrufen mit neuem Array als Parameter.

Ein durchaus typisches Beispiel für Streams mit gut lesbarem Ergebnis (finde ich)

 

[httpdigest]

Ich würde mal einfach behaupten, dass das, wonach gefragt wurde "ArrayList in 2D-Array konvertieren." gar nicht das ist, was eigentlich getan werden sollte.

Die Frage ist vermutlich eher: "Wie deserialisiere ich eine zeilenweise Textdatei bestehend aus Zeilen 'string1 string2' in eine passende Datenstruktur?"

Und die Antwort wäre: Baue dir eine Klasse oder Record, die zwei Felder hat (z.B. Fahrzeug und Bauteil) und lese Zeile für Zeile ein, splitte am Leerzeichen und erstelle eine java.util.List gefüllt mit Instanzen dieser Klasse/Record.

Denn weder ist die ursprüngliche Form der Information eine ArrayList (sie ist eine Datei), noch ist vermutlich das Konvertieren in ein 2D Array hier sinnvoll.

 

[Joerg66]

Erstmal vielen Dank für eure Antworten, die aber vom Level etwas zu hoch sind. Ich hab sie nicht verstanden.
Allerdings habe ich das Problem gelöst und packe diese hier mal hin.
Ist das so eine saubere Sache?

           try {
               ident_event = new BufferedReader(new FileReader("ident_event.ini"));

               // ident_event.ini einlesen
               List<String> list = new ArrayList<String>();
               try {
                   while((buffer = ident_event.readLine()) != null){
                       list.add(buffer);
                   }
               } catch (IOException e) {
                   e.printStackTrace();
               }
               count = list.size();

               // ident_event-Array anlegen
               String[][] i_e_arr = new String[2][count];

               for (Zstart = 0; Zstart <= count - 1; Zstart++) {
                   Zend = list.get(Zstart).indexOf(" ");
                   i_e_arr[0][Zstart] = list.get(Zstart).substring(0, Zend);
                   i_e_arr[1][Zstart] = list.get(Zstart).substring(Zend + 1).trim();
                   System.out.println("Account: " + i_e_arr[0][Zstart] + "        *.evdata: " + i_e_arr[1][Zstart]);
               }
               System.out.println("ident_event.ini        OK, " + count + " Einträge");
               try {
                ident_event.close();
            } catch (IOException e) {
                e.printStackTrace();
            }
           } catch (FileNotFoundException e) {
               e.printStackTrace();
           }

 

[httpdigest]

Ist das so eine saubere Sache?

Ich würde sagen: Nein. Wenn es funktioniert, okay. Aber "sauber" im Sinne von "es erfüllt die Anforderungen mit der geringstmöglichen Komplexität" dann: Nein.
Es ist aber auch schwer zu bewerten, ob das jetzt eine "saubere" Lösung ist, weil wir deine genauen Anforderungen nicht kennen.
Also, allem Anschein nach (was dein zuletzt geposteter Code tut) willst du ja lediglich die Zeilen in der Datei formatiert ausgeben, also mit "Account: " und mit "*.evdata: ". Plus am Ende noch die Anzahl der Zeilen. Und danach vergisst du ja sowohl die ArrayList als auch das Array.
Wenn du lediglich nur diese Informationen auf der Konsole ausgeben willst, brauchst du weder eine ArrayList, noch ein Array.

Vielleicht sagst du jetzt aber auch: "Ja, das ist nur ein Beispiel und dieser Code soll sich woanders reinintegrieren, wo ich dann doch wieder das 2D-Array später brauche."

 

[Joerg66]

Ja genau, die Ausgabe dient nur der Kontrolle.
Die Daten werden später noch benötigt.
Wenn da also kein offensichtlicher Fehler drin ist, dann lass ich das jetzt so, denn weil ich von allein drauf gekommen bin, ist das die Art die ich verstehe.

 

[mihe7]

Ist das so eine saubere Sache?

Nein, zumindest der Versuch der Strukturierung des Codes mit Hilfe von Kommentaren ist eine ganz schlechte Idee (dafür verwendet man z. B. Methoden), ebenso wie der Verzicht auf try-with-resources.

Wenn da also kein offensichtlicher Fehler drin ist, dann lass ich das jetzt so, denn weil ich von allein drauf gekommen bin, ist das die Art die ich verstehe.

Wenn Du die Daten später noch einmal benötigst, ist der Code nicht unmittelbar verwendbar.


Was Du machen könntest wäre, dass Du Dir erstmal einen eigenen Datentyp (Klasse/Record) schreibst, der die Daten strukturiert speichert:

record AccountEvent(String account, String evdata) {}

Jetzt kannst Du sagen: hey, lies mir mal alle Zeilen aus der Datei, füge jede Zeile als AccountEvent zu einer Liste hinzu und gib mir die Liste zurück. Das kann dann so aussehen:

    List<AccountEvent> readAccountEventsFromFile(String filename) throws IOException {
        // try-with-resources -> schließt die Datei automatisch
        try(BufferedReader reader = new BufferedReader(new FileReader(filename)))) {
           List<AccountEvent> eventList = new ArrayList<String>();
           String line;
           while ((line = reader.readLine()) != null) { // lies alle Zeilen aus der Datei
               addAsAccountEvent(eventList, line);      // füge jede Zeile als AccountEvent zur Liste eventList hinzu
           }                                            // und
           return eventList;                            // gib mir die Liste zurück
        }
    }

Fehlt natürlich noch die Methode addAsAccountEvent:

    private void addAccountEvent(List<AccountEvent> events, String line) {
        AccountEvent event = accountEventFromLine(line); // mach mir aus der Zeile ein AccountEvent
        // ein bisschen Fehlerbehandlung
        if (event != null) { // und wenn das geklappt hat,
            events.add(event); // dann füge das Objekt zur Liste hinzu.
        }
    }

Fehlt wieder eine Methode... accountEventFromLine:

    private AccountEvent accountEventFromLine(String line) {
        int indexOfSpace = line.trim().indexOf(" "); // wir trimmen die Zeile, um keine bösen Überraschungen zu erhalten
        // ein bisschen Fehlerbehandlung
        if (indexOfSpace == -1) {  // wenn kein Leerzeichen enthalten,
            return null; // enthält die Zeile keine vernünftigen Daten.
                         // Ich gebe hier einfach mal null zurück
        }
       
        String account = line.substring(0, indexOfSpace);
        String evdata = line.substring(indexOfSpace + 1); // da line getrimmt wurde, MUSS es weitere
        return new AccountEvent(account, evdata);
    }

Das sieht jetzt erstmal viel umständlicher aus. Tatsächlich sind das jetzt aber nur drei Methoden, die eine klare Aufgabe haben. Außerdem ist der Code robust ggü. Zeilen ohne Leerzeichen bzw. Zeilen, die nur ein Leerzeichen am Zeilenende haben.

Man kann die Methode readAcountEventsFromFile nun allerdings auch kürzer schreiben:

    List<AccountEvent> readAccountEventsFromFile(String filename) throws IOException {
        return Files.lines(Paths.get(filename)) // gib mir einen Stream der Zeilen aus der Datei filename
                .map(this::accountEventFromLine) // bilde jedes Element (=Zeile) mit Hilfe der accountEventFromLine Methode auf ein AccountEvent ab.
                .filter(Objects::nonNull) // verwerfe alle Elemente, die null sind
                .toList(); // und gib mir den Spaß gesammelt in einer Liste zurück
    }

 

[Joerg66]

Hallo,
mih7, ich danke die, für die Zeit und Arbeit die du dir gemacht hast, echt klasse.
Aber am Ende hast du mir nur gezeigt, was ich alles nicht kann :-(
Habe mir deinen Code jetzt ausgiebig angesehen und glaube das so einigermaßen nachvollziehen zu können- auch wenn ich das selber jetzt so immer noch nicht hinbekommen würde.
Ein Array kann ich mir bildlich vorstellen, in meinem Fall eine Tabelle mit 2 Spalten und benötigt viele Zeilen und ich kann darauf zugreifen, indem ich Zeile und Spalte angebe.
Bei deinem Code habe ich kein Bild und kann mir auch nicht vorstellen, wie ich jetzt an die einzelnen Werte komme.
Bin grad etwas deprimiert .....

 

[mihe7]

Okay, ich versuche mal, das etwas zu erklären.

Was ist ein Array? Zunächst einmal ist in Java ein Array einfach ein Speicher für eine gewisse Anzahl von Elementen eines bestimmten Typs. Ein zweidimensionales Array ist also nichts anderes als ein Array dessen Elemente vom Typ Array (eines bestimmten Typs) sind. Der Zugriff auf eine bestimmte Stelle im Array erfolgt über den Index-Operator (eckige Klammern).

Du kannst zum Beispiel ein String-Array erzeugen, das aus zwei Elementen besteht. Und Du kannst ein Array erzeugen, das aus 100 solcher String-Arrays besteht. In dem Fall hättest Du bildlich eine Tabelle mit 100 Zeilen und 2 Spalten.

So weit, so gut. Der Ansatz hat allerdings entscheidende Nachteile.

Wir wollen möglichst Code erhalten, den man gut lesen und schnell verstehen kann. Wenn wir Code vor uns haben, möchten wir Dinge wie "Kunde", "Adresse", "Rechnung", "Rechnungspositionen" usw. sehen und nicht irgendwelche x-elementigen Arrays, womöglich noch mit bescheidenen Bezeichnern und int-Literalen als Indizes, zumal der Typ für alle Elemente eines Arrays ja immer der selbe ist.

Nehmen wir mal eine Rechnung als Beispiel, die enthält einen Kunden, mehrere Rechnungspositionen, ein Rechnungsdatum usw. Versuch das mal in ein Array zu packen. Da würdest Du eine Krücke nach der anderen einbauen - der Code wird immer schlimmer. Das macht Dein Hirn ein paar Stunden mit, bevor es aussteigt.

Leider hast Du kaum Infos gegeben, was genau Deine Datei enthält, daher habe ich mich auf account und evdata beschränkt. Statt eines zwei-elementigen String-Arrays habe ich nun einen eigenen Datentyp für einen solchen Eintrag angelegt. Das ist der Record AccountEvent. Wie üblich können mit dem new-Operator Instanzen dieses Typs (sprich: Objekte) erzeugt werden.

AccountEvent event = new AccountEvent("DE23528213423829", "Nix passiert");
System.out.println(event.account());
System.out.println(event.evdata());

Die Bezeichner für die Methoden leitet Java automatisch aus der Record-Definition ab:

record AccountEvent(String account, String evdata) {}

Hätte ich dort String x, String y geschrieben, würde der Zugriff auf die Daten über die Methoden x() und y() erfolgen. Das macht Java für uns automatisch, daher habe ich auch einen record verwendet -> weniger Tipparbeit.

Natürlich kannst Du statt eines records auch eine eigene Klasse schreiben - ist halt mehr Arbeit. Allerdings ist ein Record immutable. Du kannst account und evdata im Nachhinein also nicht auf einen anderen Wert setzen. Mit einer eigenen Klasse wäre das natürlich möglich.

Statt mit event[0] und event[1] hat man jetzt also schön event.account() und event.evdata() im Code stehen.

Mit dem Typ könnten wir nun statt new String[100][2] schon einmal new AccountEvent[100] schreiben, was schon einmal eine erhebliche Verbesserung darstellt: events[0].account() ist wesentlich aussagekräftiger als events[0][0].

Allerdings wollen wir kein AccountEvent-Array, weil ein Array nur eine vorab definierte Anzahl an Elementen aufnehmen kann. Wir wollen eine Liste von AccountEvent-Objekten, die dynamisch in ihrer Größe ist. Eine Liste ist ein "abstrakter Datentyp", d. h. es ist bekannt, welche Operationen auf einer Liste ausgeführt werden können, und was sie im Ergebnis bewirken, die konkrete Implementierung ist allerdings offen, d. h.

List<AccountEvent> events;

besagt nur, dass events eine Liste von AccountEvent-Objekten darstellt, jedoch nicht, welche Implementierung verwendet wird. Welche man wählt, hängt von den konkreten Umständen ab, meist ist ArrayList aber nicht die schlechteste Wahl - insbesondere, wenn man sie statt eines Arrays verwenden will.

List<AccountEvent> events = new ArrayList<>();

erzeugt nun ein neues ArrayList-Objekt, das Elemente vom Typ AccountEvent in einer Liste verwaltet. Welche Methoden List anbietet, kann man der Dokumentation entnehmen. Um ein Element in der Liste auszulesen, verwendet man die Methode get, die Methode set wird verwendet, um ein Element der Liste zu setzen. Hinzugefügt werden Elemente mit add, entfernt mit remove. Die Anzahl der in der Liste gespeicherten Elemente erhält man mit size.

Um z. B. jedes Element der Liste auszugeben, kann man beispielsweise schreiben:

for (int i = 0; i < events.size(); i++) {
     System.out.println(events.get(i));
}

Natürlich funktioniert hier auch die for-each-Schleife. Außerdem hat List eine forEach-Methode usw.

Konkret auf den Code oben bezogen bedeutet dies, dass Du mit

    List<AccountEvent> events = readAccountEventsFromFile("meineDatei.txt");

Eine Liste von AccountEvent-Objekten erhältst und in der Variablen events speicherst. Danach kannst Du wie im Array auf bestimmte Elemente zugreifen:

    AccountEvent event = events.get(5);
    String account = event.account();
    String evdata = event.evdata();

vs

    String[] event = events[5];
    String account = event[0];
    String evdata = event[1];

 

[M.L.]

Array

Im Zusammenhang mit Arrays als Datenstrukturen gibt es noch weitere Empfehlungen um (häufigen) Erweiterungen / Änderungen / ... gegenüber möglichst flexibel (und robust) (*) zu bleiben: https://www.javaguides.net/2018/06/java-collection-framework-best-practices.html

(*)gilt aber generell für SW-Architektur oder -Implementierung

 

[Joerg66]

Oh man, du hast dir so viel Arbeit gemacht ... vielen vielen Dank.
Ich bekomme aber schon bei

record AccountEvent(String account, String evdata) {}

den Fehler : record can not be resolved to a type
Gefunden habe ich dazu, das der Class-Path eingetragen werden muß.
"Add ClassFolder" und dann das Verzeichnis "bin" auswählen.
Nur ... das gibt es nicht.

 

[Jw456]

record geht glaube erst mit Java 14 .

 

[mihe7]

den Fehler : record can not be resolved to a type

Zu altes Java.

Wie aber schon mehrfach erwähnt, kannst Du auch eine Klasse schreiben, hier mal das Äquivalent zum record (aus Faulheitsgründen per ChatGPT generiert):

public class AccountEvent {
    private final String account;
    private final String evdata;

    public AccountEvent(String account, String evdata) {
        this.account = account;
        this.evdata = evdata;
    }

    public String getAccount() {
        return account;
    }

    public String getEvdata() {
        return evdata;
    }

    // Implementierung der hashCode-Methode
    @Override
    public int hashCode() {
        int result = 17;
        result = 31 * result + account.hashCode();
        result = 31 * result + evdata.hashCode();
        return result;
    }

    // Implementierung der equals-Methode
    @Override
    public boolean equals(Object obj) {
        if (this == obj) {
            return true;
        }
        if (obj == null || getClass() != obj.getClass()) {
            return false;
        }
        Record other = (Record) obj;
        return account.equals(other.account) && evdata.equals(other.evdata);
    }

    // Implementierung der toString-Methode
    @Override
    public String toString() {
        return "Record{" +
                "account='" + account + '\'' +
                ", evdata='" + evdata + '\'' +
                '}';
    }
}

 

[mihe7]

Jetzt sehe ich gerade, dass ChatGPT Mist gebaut hat: die Methoden müssten account() und evdata() und nicht getAccount() und getEvdata() heißen. Dann kommen wir dem record schon ziemlich nah. Die toString()-Methode liefert auch nicht exakt das, was ein record liefert, aber das ist mal Nebensache.

 

[Jw456]

Da sieht man das die KI doch nicht so schlau ist.
Solange in Netz viel Müll ist wie soll dann die KI Schlau werden.

 

[mihe7]

Da sieht man das die KI doch nicht so schlau ist.
Solange in Netz viel Müll ist wie soll dann die KI Schlau werden.

Das ist zwar völlig OT, aber ich muss sagen, dass ich zu doof für ChatGPT bin. Zumindest habe ich es noch nie (kann mich jedenfalls nicht daran entsinnen) fertiggebracht, dem Teil wirklich Code zu entlocken, den ich haben möchte bzw. unmittelbar nutzen kann.

 

[lonLat]

die Methoden müssten account() und evdata() und nicht getAccount() und getEvdata() heißen.

Dieser Punkt wird kontrovers diskutiert. Ich bin für die alte Schreibweise: getAccount() --- das neue records Namensschema ist falsch. Lässt sich jetzt aber nicht mehr revidieren, da es sonst Legacy-Code gäbe. 🤷‍♂️ nobody is perfect.

 

[M.L.]

Code zu entlocken

Und woher die KI den Code bezieht (am Bspl. GitHub Copilot): (YT, "thenativeweb", "Jetzt bewiesen: GitHub Copilot stiehlt Code")

 

[mihe7]

Dieser Punkt wird kontrovers diskutiert.

Mag sein, der Fehler aber ist, dass die Methoden in records nun einmal xyz() und nicht getXyz() heißen. Wenn ich also nach dem class-Äquivalent zum record frage, dann erwarte ich schon, dass ich auch eine äquivalente Klasse bekomme.

Und woher die KI den Code bezieht

Anscheinend aus der falschen Quelle 🤣

 

[Joerg66]

Und wieder habe ich in Zeile 36

Record other = (Record) obj;

den Fehler : Record can not be resolved to a type
Ich muß den Code nunmal für Java 8 machen. Darauf habe ich keinen Einfluß.

 

[Jw456]

Es wurde doch nun schon gesagt das du keinen record mit Java 8 machen kannst da es denn da noch nicht gibt .
Also Alternative die dir schon gegeben wurde.

 

[Jw456]

Natürlich kannst Du statt eines records auch eine eigene Klasse schreiben - ist halt mehr Arbeit. Allerdings ist ein Record immutable. Du kannst account und evdata im Nachhinein also nicht auf einen anderen Wert setzen. Mit einer eigenen Klasse wäre das natürlich möglich.

wurde ja schon gesagt.

 

[Joerg66]

Aber das steht doch in der Alternative

 

[lonLat]

@Joerg66 Nomm doch halbwegs aktuelles Java. Nicht das aus der Steinzeit.

 

[Joerg66]

Das kann ich nicht. Auf dem System, wo das Programm laufen soll, ist Java 8 installiert.

 

[lonLat]

ChatGPT sagt:

Gerne! Hier ist eine Datenklasse namens "DataRecord" mit zwei finalen String-Attributen, die auch die equals()- und hashCode()-Methoden überschreibt:

public class DataRecord {
    private final String attribute1;
    private final String attribute2;

    public DataRecord(String attribute1, String attribute2) {
        this.attribute1 = attribute1;
        this.attribute2 = attribute2;
    }

    public String getAttribute1() {
        return attribute1;
    }

    public String getAttribute2() {
        return attribute2;
    }

    @Override
    public boolean equals(Object o) {
        if (this == o) return true;
        if (o == null || getClass() != o.getClass()) return false;

        DataRecord that = (DataRecord) o;

        if (!attribute1.equals(that.attribute1)) return false;
        return attribute2.equals(that.attribute2);
    }

    @Override
    public int hashCode() {
        int result = attribute1.hashCode();
        result = 31 * result + attribute2.hashCode();
        return result;
    }
}

In dieser Klasse wurden die Attribute attribute1 und attribute2 als final deklariert, sodass sie nach der Initialisierung nicht mehr geändert werden können. Die equals()- und hashCode()-Methoden wurden überschrieben, um den Wertvergleich und die Hash-Funktion für Objekte der Klasse DataRecord zu implementieren.

Hier ist ein Beispiel, wie Sie diese Klasse verwenden können:

public class Main {
    public static void main(String[] args) {
        DataRecord record1 = new DataRecord("Wert 1", "Wert 2");
        DataRecord record2 = new DataRecord("Wert 1", "Wert 2");
        DataRecord record3 = new DataRecord("Anderer Wert", "Noch ein anderer Wert");

        // Überprüfen der Gleichheit mit equals()
        System.out.println("record1 und record2 sind gleich: " + record1.equals(record2)); // Ausgabe: true
        System.out.println("record1 und record3 sind gleich: " + record1.equals(record3)); // Ausgabe: false

        // Hash-Codes
        System.out.println("Hash-Code von record1: " + record1.hashCode());
        System.out.println("Hash-Code von record2: " + record2.hashCode());
        System.out.println("Hash-Code von record3: " + record3.hashCode());
    }
}

Beachten Sie, dass das Überschreiben von equals() und hashCode() wichtig ist, um die korrekte Verwendung von Datenklassen in Collections wie HashMap oder HashSet zu gewährleisten.

 

[mihe7]

den Fehler : Record can not be resolved to a type

Mein Fehler. ChatGPT hatte eine Klasse "Record" erstellt, ich habe die im Nachhinein zu AccountEvent umbenannt. Also: überall, wo "Record" (Großbuchstabe am Anfang) steht, durch AccountEvent ersetzen. Bzw die Klasse DataRecord von @lonLat verwenden

 

[lonLat]

ich habe die im Nachhinein zu AccountEvent umbenannt.

Das musst du ChatGPT auch deutlich sagen nicht einfach selber pfuschen

 

[lonLat]

result = 31 * result + attribute2.hashCode();

Die Zeile macht mich aber noch etwas unglücklich...

 

[lonLat]

Dabei weiß er/sie/es ja, wie es genauer ginge:

Um zwei Hash-Codes in Java zu kombinieren, können Sie die Methode Objects.hash() aus der Java Standardbibliothek verwenden. Diese Methode akzeptiert eine beliebige Anzahl von Argumenten und kombiniert die Hash-Codes dieser Argumente.

Hier ist ein Beispiel, wie Sie zwei Hash-Codes kombinieren können:

import java.util.Objects;

public class Main {
    public static void main(String[] args) {
        String string1 = "Hello";
        String string2 = "World";

        int hashCode1 = string1.hashCode();
        int hashCode2 = string2.hashCode();

        int combinedHashCode = Objects.hash(hashCode1, hashCode2);

        System.out.println("HashCode von string1: " + hashCode1);
        System.out.println("HashCode von string2: " + hashCode2);
        System.out.println("Kombinierter HashCode: " + combinedHashCode);
    }
}

In diesem Beispiel wurden die Hash-Codes der beiden Strings "Hello" und "World" durch Aufruf der hashCode()-Methode erhalten. Anschließend wurden diese beiden Hash-Codes durch den Aufruf von Objects.hash() kombiniert, um einen einzelnen kombinierten Hash-Code zu erhalten.

Bitte beachten Sie, dass das Kombinieren von Hash-Codes auf diese Weise eine einfache Möglichkeit ist, aber es ist nicht immer perfekt. In einigen Fällen kann es zu sogenannten Hash-Kollisionen kommen, bei denen unterschiedliche Kombinationen von Werten denselben Hash-Code ergeben. In komplexeren Szenarien, wie beim Erstellen von Hash-Codes für Datenobjekte, kann es sinnvoll sein, eine spezielle Hash-Funktion zu implementieren, um solche Kollisionen zu minimieren.

 

[mihe7]

Die Zeile macht mich aber noch etwas unglücklich...

Wer weiß, von wem das Teil den Code geklaut hat?

 

[lonLat]

Wer weiß, von wem das Teil den Code geklaut hat?

Nicht von mir Ich hatte nur selten etwas über Hash-Funktionen geschrieben ...

Btw. ich hab ChatGPT gerade etwas zu LineIntersection gefragt, und es gibt da effiziente Verfahren das betrifft jetzt aber das andere Thema

 

[Joerg66]

Vielen Dank euch allen, aber ich gebe auf.
Ich bin mir sicher, das dieser Betreff vielen anderen weiterhelfen wird.
Der Fehler kommt jetzt bei DataRecord.
Eigentlich habe ich seit Donnerstag Urlaub und sitze hier trotzdem.
Scheinbar ist das Thema zu viel gewollt.
Werde meinem Chef nach dem Urlaub meine Kapitulation bekanntgeben.
Ich entschuldige mich bei euch für die vertane Zeit.
Gruß Jörg

 

[lonLat]

Zu kündigen ist zwar nie schön, andererseits hat man dann mehr Freizeit. Machs gut

 

[Jw456]

Der Fehler kommt jetzt bei DataRecord.

Was kommt den da für ein Fehler?

Lasse das mit dem ChatGPT siehst ja was daraus kommt. mache es selber.


Und wenn du die Anleitung von mihe7 durchgehen willst . Dann heißt die Klasse auch nicht DataRecord.
Wie gesagt ist das eine Klasse die auch in eine eigne Datei gehört.

 

[mihe7]

Hier mal eine Version, die auch unter Java 8 laufen dürfte:

Spoiler: AccountEvent.java

import java.util.Objects;

public class AccountEvent {
    private final String account;
    private final String evdata;

    public AccountEvent(String account, String evdata) {
        this.account = account;
        this.evdata = evdata;
    }

    public String account() {
        return account;
    }

    public String evdata() {
        return evdata;
    }

    // Implementierung der hashCode-Methode
    @Override
    public int hashCode() {
        return Objects.hash(account, evdata);
    }

    // Implementierung der equals-Methode
    @Override
    public boolean equals(Object obj) { 
        if (this == obj) {
            return true;
        }
        if (obj == null || getClass() != obj.getClass()) {
            return false;
        }
        AccountEvent other = (AccountEvent) obj;
        return account.equals(other.account) && evdata.equals(other.evdata);
    }

    // Implementierung der toString-Methode
    @Override
    public String toString() {
        return "AccountEvent[" +
                "account='" + account + '\'' +
                ", evdata='" + evdata + '\'' +
                ']';
    }
}

Spoiler: Test.java

import java.io.*;
import java.util.*;

public class Test {
    public static void main(String[] args) throws Exception {
        Test test = new Test();
        List<AccountEvent> events = test.readAccountEventsFromFile("ident_event.ini");

        for (AccountEvent event : events) {
            System.out.println("Konto: " + event.account() + ", evdata: " + event.evdata());
        }
    }

    public List<AccountEvent> readAccountEventsFromFile(String filename) throws IOException {
        // try-with-resources -> schließt die Datei automatisch
        try(BufferedReader reader = new BufferedReader(new FileReader(filename))) {
           List<AccountEvent> eventList = new ArrayList<>();
           String line;
           while ((line = reader.readLine()) != null) { // lies alle Zeilen aus der Datei
               addAsAccountEvent(eventList, line);      // füge jede Zeile als AccountEvent zur Liste eventList hinzu
           }                                            // und
           return eventList;                            // gib mir die Liste zurück
        }
    }

    private void addAsAccountEvent(List<AccountEvent> events, String line) {
        AccountEvent event = accountEventFromLine(line); // mach mir aus der Zeile ein AccountEvent
        // ein bisschen Fehlerbehandlung
        if (event != null) { // und wenn das geklappt hat,
            events.add(event); // dann füge das Objekt zur Liste hinzu.
        }
    }

    private AccountEvent accountEventFromLine(String line) {
        int indexOfSpace = line.trim().indexOf(" "); // wir trimmen die Zeile, um keine bösen Überraschungen zu erhalten
        // ein bisschen Fehlerbehandlung
        if (indexOfSpace == -1) {  // wenn kein Leerzeichen enthalten,
            return null; // enthält die Zeile keine vernünftigen Daten.
                         // Ich gebe hier einfach mal null zurück
        }
       
        String account = line.substring(0, indexOfSpace);
        String evdata = line.substring(indexOfSpace + 1); // da line getrimmt wurde, MUSS es weitere
        return new AccountEvent(account, evdata);
    }
}

 

[lonLat]

Was kommt den da für ein Fehler?

Ich vermute eher, dass er es nicht geschafft hat, den Code (richtig) zu kopieren ... dann wird es mit dem Programmieren schwer. Er ist mit anderen Aufgaben sicherlich besser dran.

 

[KonradN]

Mag sein, der Fehler aber ist, dass die Methoden in records nun einmal xyz() und nicht getXyz() heißen. Wenn ich also nach dem class-Äquivalent zum record frage, dann erwarte ich schon, dass ich auch eine äquivalente Klasse bekomme.

Hier ist evtl. auch das Problem, dass er Code generieren will, der sich an die coding guidelines hält. Ich selbst finde es etwas unglücklich, dass bei Records auf das get verzichtet wurde.

Aber so Dinge hatten wir hier ja auch schon öfters im Forum angesprochen, denn so Dinge findet man durchaus öfters.

Aber man will ja nicht meckern sondern sich einfach nur freuen, dass es Records gibt

 

[lonLat]

Ah, ich hab mich schon gefragt, wann der hier aufschlägt ...

Aber man will ja nicht meckern sondern sich einfach nur freuen, dass es Records gibt

Ich nutze Records inzwischen auch gerne, einfach, weil es z B Tipparbeit (Boilerplate-Code) spart.

Dennoch sollte man ja das Verständnis dafür erlangen (vielleicht gerade am Anfang), wie Records intern umgesetzt wurden. Zumindest würde ich mir das wünschen.

 

[KonradN]

Ich nutze Records inzwischen auch gerne, einfach, weil es z B Tipparbeit (Boilerplate-Code) spart.

Ich habe leider immer noch viel Java 8 und 11 im beruflichen Umfeld. Da setze ich dann gerne auf Lombok um Boilerplate Code zu vermeiden.

 

[mihe7]

Aber man will ja nicht meckern sondern sich einfach nur freuen, dass es Records gibt

Rischdisch.

 

[Jw456]

Ich denke das wir die records erwas der Koltin Sprache zuverdanken haben.
Denn da gehört die data Klasse einfach dazu. und da wird auch nicht mit einen vorangestelten get auf Properties zugegriffen.

 

[mihe7]

Wie? Copy & Paste-Fehler?

 

[M.L.]

Der Fehler kommt jetzt bei DataRecord.

Welche Fehlermeldung(en) ? Der Code aus Posting #28 funktioniert (unter Eclipse und Java 17) aber grundsätzlich:

 

[M.L.]

Der Fehler

Teil II: die Codes von #38 und #28 funktionieren unter Eclipse mit Java 8 (passende JRE verlinkt, Compiler-Level auf 1.8 gesenkt) grundsätzlich

 

[lonLat]

funktionieren unter Eclipse mit Java 8

Natürlich tun sie dies.

Ich vermute eher, dass der TO mit dem gegebenen Code nicht umgehen kann ...

Btw. nicht Eclipse verwenden

 

[M.L.]

nicht Eclipse

Im Screenshot bei Posting #13 ist aber ein (älteres) Eclipse zu sehen. Könnte sein, dass es neben Java 8 eine Zwangsanforderung ist.