Klassen Berechnen mehrerer Map-Werte

[PaulGustav]

Hallo zusammen,

ich habe mal wieder eine Frage an euch und würde mich freuen, wieder eine kompakte Lösung zu erhalten.

Ich versuche derzeit Werte von immer wieder auftauchenden Schlüssel-IDs aufzurechnen. Dafür habe ich eine HashMap in der Form
Map <Long, KlasseX> map = new HashMap<>()
erstellt. Ich weiß, dass man mittels der neuen Java8-Methode map.computeIfPresent() den Wert eines Schlüssels berechnen kann und würde das gerne auch auf eine ganze Klasse und deren Werte übertragen. Die KlasseX, so wie ich sie habe, enthält ca 4 Integer- bzw Floar-Objekte , welche auch schon mittels Getter und Setter versehen sind.

Folgendes theoretisches Beispiel:
Schlüssel, (Eigenschaften)
1000000001, (10, 150.50, 600.00)
1000000002, (0, 100.00, 200.50)
1000000001, (20, 110.50, 730.00

ich möchte quasi, dass der Schlüssel 1000000001 die Werte (30, 261.00, 1300.00) erhält.

Ist mein Sachverhalt so mit der Funktion möglich oder gibt es da bereits bessere Methoden? Braucht ihr vllt ein Minimalbeispiel samt Klasse und Objekte?

Viele Grüße und vielen Dank,
Paul

 

[Thallius]

Ich finde das benutzen einer Map hier nicht sehr sinnvoll. Damit schränkst du dich von Anfang an zu sehr ein.

Mach doch eine Klasse mit zwei Attributen. Einem Long für den Schlüssel und einer List (oder wenn es wirklich immer nur 4 sein sollen einem Array) für die Elemente.

Gruß

Claus

 

[PaulGustav]

Okay, den Gedanken hatte ich auch mal gefast, jedoch wieder verworfen. Für die Liste muss ja dann sicherlich eine Schleife geschrieben werden, die die Attribute des jeweligen Schlüssels aktualisiert.

Falls ich auf die Lösung gekommen bin, werde ich sie mal hier posten.

Ich danke dir.

 

[PaulGustav]

Muss dann für jeden Schlüssel eine eigene Liste her?

Ich überlege gerade wie die Liste auszusehen hat..? etwa so:

Inhalt des SourceFiles "appleFile.csv":
appleID, announcedAppels, announcedWeight,
1000000001, 10, 150.50
1000000002, 0, 100.00
1000000001, 20, 110.50

-----------------------------------------------

String appleID, announcedAppels, announcedWeight;
Long id;
int numberOfApples;
float weigth;

List<String[]> appleBoxes = new ArrayList<>;

//reading line of appleFile.csv...
String[] content = line.split(",");
appleID = content[0]);
announcedAppels = (content[1];
announcedWeight = (content[2];

for (int i=0; i < appleBoxes.size(); i++){
    if (appleBoxes.get(i).contains(String.valueOf(id)){
        numberOfApples += Int.parseInt(announcedAppels);
        weigth += float.ParseFloat(announcedWeight);
    }
    appleBoxes.set(i, Array.asList(appleID, announcedAppels, announcedWeight));
}

 

[Thallius]

Warum in Gottes Namen benutzt du keine Klassen? Dafür sind die doch erfunden worden. Wer soll diesen Chaoscode denn später noch verstehen geschweige denn warten? Wie fragst du dann zum Beispiel die Anzahl Äpfel ab? Mit box[1]? Wer soll das kapieren?

 

[mrBrown]

Was willst du denn überhaupt erreichen?
Generell ist ID auf Objekt Mappen nicht das schlechteste...

 

[PaulGustav]

Ich habe schon eine Klasse verwendet. So ist das nicht. Da die ganzen Zusammenhänge meines Erachtens jedoch zu komplex wären, habe ich nur ein minimales Beispiel gebracht.

Hier nun nochmal folgendes Skript:

Inhalt des SourceFiles "appleFile.csv":
timestamp, appleID, announcedAppels, announcedWeight,
2015-01-01Z12:12:00T,1000000001, 10, 150.50
,1000000002, 0, 100.00
,1000000001, 20, 110.50
2015-01-02Z12:12:00T,1000000001, 15, 100.00
,1000000002, 10, 200.00
,1000000001, 25, 140.00

-----------------------------------------------

public class AppleBoxes{
   
    private static final int COLUMN_WEIGHT = 2;
    private static final int COLUMN_NUMBEROFAPPLES = 1;
   
    private Long id;
    private float weigth;
    private int numberOfApples;
   
    public AppleBoxes (Long id){
        this.id = id;
    }
   
    public void setDataLine(String[] content) {
        if (isNullOrBlank((content[COLUMN_WEIGHT])))
            weight += 0;
        else
            weigth += Float.parseFloat(content[COLUMN_WEIGHT]);
        if (isNullOrBlank(content[COLUMN_NUMBEROFAPPLES]))
            numberOfApples += 0;
        else
            numberOfApples += Float.parseFloat(content[COLUMN_NUMBEROFAPPLES]);
    }
   
    public float getWeight() {
        return weight;
    }

    public int getnumberOfApples() {
        return numberOfApples;
    }
   
}

public Class BoxRanking {
    public static void main(String[] args) throws Exception {
        AppleBoxes id = null;
        List<AppleBoxes> aggregatedAppleBoxes = new ArrayList<>;
        try {......
        //reading lines of appleFile.csv...
            while (line = br.readLine() != null){
                String[] content = line.split(",");
                if (StringUtils.isEmpty(content[COLUMN_TIMESTAMP]) == false){
                    AppleBoxes id = new AppleBoxes(content);
                    id.setDataLine(content);
                    aggregatedAppleBoxes.add(id)
                }
                else if (content[COLUMN_TIMESTAMP].isEmpty()){
                    id.setDataLine(content);
                    aggregatedAppleBoxes.add(id)
                }
       
       
            }
        }
        catch(IOException ex) {
            ex.printStackTrace();
        }
    }
   
}

Es soll eine Liste / Map generiert werden, in der die Schlüssel nur einmal vorkommen und und die entsprechenden Werte (Gewicht, Anzahl) aggregiert werden. Die Sache mit dem Zeitstempel kommt dann aufgrund des Filterns noch hinzu, aber wenn ich die Aggregation auf die Schlüssel hinbekomme, wäre das schon sehr gut.

Es handelt sich nach wie vor nur um ein Beispiel.

Vielen Dank für eure Hilfe.

 

[mrBrown]

Ich finde den Ansatz mit der Map prinzipiell sinnvoll.

Ich würd nur AppleBox anders aufbauen und immutable machen, dann könnte man das Recht leicht mit Map::merge machen

 

[PaulGustav]

Hallo mal wieder.

ich habe die Problematik nun gelöst und es hinbekommen über die Map-API Werte meiner Schlüsselobjekte zu berechnen. Es klappt wie folgt:
Nehmen wir noch einmal das Beispiel der AppleBox

Inhalt des SourceFiles "appleFile.csv":
timestamp, appleID, announcedAppels, announcedWeight,
2015-01-01Z12:12:00T,1000000001, 10, 150.50
,1000000002, 0, 100.00
,1000000001, 20, 110.50
2015-01-02Z12:12:00T,1000000001, 15, 100.00
,1000000002, 10, 200.00
,1000000001, 25, 140.00

-----------------------------------------------

public class AppleBoxes{
 
    private static final int COLUMN_WEIGHT = 3;
    private static final int COLUMN_NUMBEROFAPPLES = 2;
 
    private float weigth;
    private int numberOfApples;
 
    public AppleBoxes(String[] content) {
        if (isNullOrBlank((content[COLUMN_WEIGHT])))
            weight = 0;
        else
            weigth = Float.parseFloat(content[COLUMN_WEIGHT]);
        if (isNullOrBlank(content[COLUMN_NUMBEROFAPPLES]))
            numberOfApples = 0;
        else
            numberOfApples = Float.parseFloat(content[COLUMN_NUMBEROFAPPLES]);
    }
  
    public void updateAppleBoxes(String[] content) {
         if (isNullOrBlank((content[COLUMN_WEIGHT])))
            weight += 0;
        else
            weigth = Float.parseFloat(content[COLUMN_WEIGHT]);
        if (isNullOrBlank(content[COLUMN_NUMBEROFAPPLES]))
            numberOfApples += 0;
        else
            numberOfApples = Float.parseFloat(content[COLUMN_NUMBEROFAPPLES]);
    }
  
 
    public float getWeight() {
        return weight;
    }

    public int getnumberOfApples() {
        return numberOfApples;
    }
 
}

public Class BoxRanking {
  
    private static final int COLUMN_BOXID = 1;
  
    public static void main(String[] args) throws Exception {
        AppleBoxes id = null;
        Map<Long ,AppleBoxes> aggregatedAppleBoxes = new hashMap<>();
        // we need to assume, that the listOfBoxIDs is filled with all IDs
        List<Long> listOfBoxIDs = new List<>();
        try {......
        //reading lines of appleFile.csv...
            while (line = br.readLine() != null){
                String[] content = line.split(",");
                if (StringUtils.isEmpty(content[COLUMN_TIMESTAMP]) == false){
                    Long boxID = content[COLUMN_BOXID]);
listOfBoxIDs.add(boxID);
                    for (Long boxID : listOfBoxIDs){
                    AppleBoxes appleBoxes = aggregatedAppleBoxes.get(boxID);
                    if (appleBoxes == null){
                        appleBoxes = new AppleBoxes (content);
                        aggregatedAppleBoxes.add(boxID,appleBoxes)
                    }
                    else if (appleBoxes != null){
                        appleBoxes.updateAppleBoxes(content);
                        aggregatedAppleBoxes.add(boxID, appleBoxes);
                    }
                }
                else if (content[COLUMN_TIMESTAMP].isEmpty()){
                    id.setDataLine(content);
                    aggregatedAppleBoxes.add(id)
                }
            }
        }
        catch(IOException ex) {
            ex.printStackTrace();
        }
    }
 
}

Es genügt also innerhalb der Klasse Apples eine neue Methode zu schreiben und diese Daten wie im Code anzuwenden.

Ich danke euch trotzdem für die vielen Ratschläge.

Gruß Paul

 

[mrBrown]

Ein paar Anmerkungen:

public AppleBoxes(String[] content) {
        if (isNullOrBlank((content[COLUMN_WEIGHT])))
            weight = 0;
        else
            weigth = Float.parseFloat(content[COLUMN_WEIGHT]);
        if (isNullOrBlank(content[COLUMN_NUMBEROFAPPLES]))
            numberOfApples = 0;
        else
            numberOfApples = Float.parseFloat(content[COLUMN_NUMBEROFAPPLES]);
    }
 
    public void updateAppleBoxes(String[] content) {
         if (isNullOrBlank((content[COLUMN_WEIGHT])))
            weight += 0;
        else
            weigth = Float.parseFloat(content[COLUMN_WEIGHT]);
        if (isNullOrBlank(content[COLUMN_NUMBEROFAPPLES]))
            numberOfApples += 0;
        else
            numberOfApples = Float.parseFloat(content[COLUMN_NUMBEROFAPPLES]);
    }

Das erstellen und das updaten sollte besser unabhängig von der Repräsentation in der Datei sein. Der Konstuktor kann einfach die beiden entsprechenden Werte bekommen, und update entweder die entsprechenden Werte, oder eine andere AppleBox (was ein besserer Name als AppleBoxes wäre, Klassennamen sollten im Singular gehalten werden).

Deine BoxRanking ist doch überhaupt nicht kompilierbar.
So Dinge wie:

Long boxID = content[COLUMN_BOXID]);
for (Long boxID : listOfBoxIDs){

sind auch totaler Unsinn, genauso wie der Inhalt der for-Schleife, hast du das mal getestet?

 

[PaulGustav]

Das erstellen und das updaten sollte besser unabhängig von der Repräsentation in der Datei sein. Der Konstuktor kann einfach die beiden entsprechenden Werte bekommen, und update entweder die entsprechenden Werte, oder eine andere AppleBox (was ein besserer Name als AppleBoxes wäre, Klassennamen sollten im Singular gehalten werden).

Was genau meinst du Repräsentation in der Datei. In meinem vorliegenden Fall habe ich eine separate Klasse, wo die Eigenschaften gespeichert werden und dort habe ich den Konstruktor und die Methode genauso angewandt.

Deine BoxRanking ist doch überhaupt nicht kompilierbar.
So Dinge wie:

Long boxID = content[COLUMN_BOXID]);
for (Long boxID : listOfBoxIDs){

sind auch totaler Unsinn, genauso wie der Inhalt der for-Schleife, hast du das mal getestet?

Ja es kann gut sein, dass der Code nicht einwandfrei kompilierbar ist. Tut mir Leid, dass ich mir da nicht mehr Mühe gegeben habe. Mein eigentlicher Code ist länger und enthält mitunter noch einige Methoden, die ich hier nicht alle erklären wollten. Zudem handelt es sich bei mir nicht um Apfelkisten.

Hier mal nur meine Schleifen zum Auslesen der Daten. Die läuft zumindest:

if (StringUtils.isEmpty(content[COLUMN_TIMESTAMP]) == false && line.contains(",") == true) {
                    List<Long> dsegColumnList = new ArrayList<>(100);
                    String dsegColumn = content[COLUMN_DSEGID];
                    getDsegs(dsegColumn, dsegColumnList);
                    for (Long segmentID : dsegColumnList){
                        DsegProperties dsegProperties = mapping.get(segmentID);
                        if (dsegProperties == null){
                        dsegProperties = new DsegProperties(content,mapProperties,segmentID);
                        mapping.put(segmentID, dsegProperties);
                        }
                        else if(dsegProperties != null){
                        dsegProperties.updateDsegID(content, mapProperties, segmentID);
                        mapping.put(segmentID, dsegProperties);
                        }
                        }
                    }
                   
                else if (...){...
                     }

 

[mrBrown]

Was genau meinst du Repräsentation in der Datei. In meinem vorliegenden Fall habe ich eine separate Klasse, wo die Eigenschaften gespeichert werden und dort habe ich den Konstruktor und die Methode genauso angewandt.

AppleBoxes enthält die Daten und übernimmt das Umwandeln der Daten aus der Datei, das trennt man besser.

Hier mal nur meine Schleifen zum Auslesen der Daten. Die läuft zumindest:

Was soll denn dsegColumnList sein, Liste aller IDs?
Dann gibt dir mapping.get(segmentID); immer ein Objekt zurück und es greift immer der else-Fall. Das if kannst du dir im else-Fall übrigens sparen, da ist schon klar, das es ungleich null ist.

 

[PaulGustav]

ja in der dsegColumnList werden die IDs (aus der Spalte der gemeldeten IDs) gespeichert. Aber wenn die segmentID noch nicht mit Werten versehen ist, gibt mir Java doch zuerst null wieder.

 

[mrBrown]

segmentID kann nur null sein, wenn du dsegColumnList explizit null hinzufügst. Wenn die Liste leer ist, wird die for-Schleife überhaupt nicht ausgeführt.