Java

[frager2345]

Hi, also ich melde mich mal wieder , da ich noch etwas Probleme mit Ausnahmen und Streams habe. Ich habe jetzt hier mal ein Programm geschrieben:
Die Klasse ist Task mit den Attributen :

private String description;

private int priority;

Als seter habe ich:

public void setDescription(String description) throws  IllegalArgumentException{
        this.description = description;
        if(!checkDescription(description)){
            throw new IllegalArgumentException("Fehler bei description");
        }
    }

Bitte nicht die Fehlermeldung beachten, wird noch ausgebessert
Die gleiche seter für priority.
Nun habe ich noch eine Klasse geschrieben. Die erstmal eine arraylist mit task Objekten in der Darstellung : "Testaufgabe n" (n soll nur die Nummerierung angeben) befüllt. Nun will ich aus dieser Arraylist erstmal ein Stream machen und dann ein IntStream Objekt, der nur die Prioritäten enthält. Hier hänge ich gerade noch etwas.
Mein derzeitiger Code:

public static void main(String[] args) {
        ArrayList<Task> mylist = new ArrayList<>();
        for(int i = 1; i <= 10; ++i){
            Random n = new Random();
            try{
                Task a = new Task("Testaufgabe " + (char) i, n.nextInt(10) + 1);
                mylist.add(a);
            }catch(IllegalArgumentException e) {
                throw new IllegalArgumentException("Falsches Format!"); //hier vielleicht sogar e.printstacktrace();
            }
        }
        Stream<Task> stream = mylist.stream();

Ich hab die Behandlung der expetions in der main gemacht, da ich die Expetions hier gebündelt haben wollte .
Zu dem Intstream hab ich irgendwie noch mapping im Kopf aber versteh nicht ganz wie dies funktioniert.
Hoffe es findet sich Hilfe.
Grüße

 

[httpdigest]

Also: Du hast einen Stream<Task> und möchtest daraus einen IntStream erzeugen, der nur aus den jeweiligen Prioritäten der Tasks aus dem ersten Stream besteht?
"Mapping" ist hier schon ganz richtig und du brauchst hier vermutlich die Methode Stream.mapToInt().
Diese kannst du auf deinem "stream" Stream<Task> aufrufen mit einer Lambda oder Methodenreferenz als Argument, um aus einem Task seine Priorität zu erhalten.

 

[mihe7]

mylist.stream().mapToInt(task -> task.getPriority())

mylist.stream().mapToInt(Task::getPriority)

 

[frager2345]

Also: Du hast einen Stream<Task> und möchtest daraus einen IntStream erzeugen, der nur aus den jeweiligen Prioritäten der Tasks aus dem ersten Stream besteht?
"Mapping" ist hier schon ganz richtig und du brauchst hier vermutlich die Methode Stream.mapToInt().
Diese kannst du auf deinem "stream" Stream<Task> aufrufen mit einer Lambda oder Methodenreferenz als Argument, um aus einem Task seine Priorität zu erhalten.

Ja genau. also grob so :

mylist.stream().mapToInt('lamda expression').forEach(System.out::println);

bloß das Problem war bei mir, dass ich nicht ganz versteh wie ich die Prioritäten aus den "Task" Objekten bekomme. Bzw wie der dazugehörige Lamdaausdruck aussieht.

 

[mihe7]

Bzw wie der dazugehörige Lamdaausdruck aussieht.

s. beispielhaft #3 erstes Beispiel. Das zweite Beispiel nutzt die Methodenreferenz.

 

[frager2345]

mylist.stream().mapToInt(task -> task.getPriority())

mylist.stream().mapToInt(Task::getPriority)[/code
[/QUOTE]
ahcsooo. Oh man, hab ja geter Funktionen , ganz vergessen :/

 

[frager2345]

s. Beispielhaft #3 erstes Beispiel. Das zweite Beispiel nutzt die Methodenreferenz.

Ja, hab ewig rum überlegt, hatte die geter ganz vergessen , danke dir!

 

[Robert Zenz]

Bei deiner setDescription Methode solltest du zuerst die Pruefung machen und dann zuweisen. So koennte es sein dass dein Objekt dann in einem inkonsistenten Zustand verbleibt. Also wenn jemand einen "falschen" Wert setzt, die Ausnahme faengt und wegignoriert ist trotzdem der Wert geaendert auf einen ungueltigen.

 

[frager2345]

Bei deiner setDescription Methode solltest du zuerst die Pruefung machen und dann zuweisen. So koennte es sein dass dein Objekt dann in einem inkonsistenten Zustand verbleibt. Also wenn jemand einen "falschen" Wert setzt, die Ausnahme faengt und wegignoriert ist trotzdem der Wert geaendert auf einen ungueltigen.

Ja, aber ich wollte eigentlich die Exeptions bis in die main reichen, um so ein Bündel an Exeptions zu haben , hoffe man versteh so ein bisschen was ich mein .

 

[Oneixee5]

Ich verstehe noch nicht so richtig was du willst. Aber hier sind schon mehrere Probleme:

public void setDescription(String description) throws  IllegalArgumentException{
        this.description = description;
        if(!checkDescription(description)){
            throw new IllegalArgumentException("Fehler bei description");
        }
    }

• IllegalArgumentException ist eine RuntimeException - die deklariert man nicht.
• this.description = description; wird ausgeführt bevor die Validierung durchgeführt wird. Also die Klasse besitzt die 'description' obwohl sie illegal ist.
• checkDescription sollte nicht boolean zurückgeben sondern direkt die Exception werfen.

• public void setDescription(String description) {
      this.description = checkDescription(description, "Fehler bei description");
  }

 

[frager2345]

checkDescription sollte nicht boolean zurückgeben sondern direkt die Exception werfen.

Wie genau meinst du, also wenn Description illegal ist wirft checkDescription die Exeption , also dann so:

private static void checkDescription(String description, "Fehler bei description") { //paramter sollen natürlich anders sein, aber der Übersicht hab ich jetzt mal "Fehler bei description" s
geschrieben

        if (description.equals("")){
            throw new IllegalArgumentException("Fehler bei description");
        }
    }

 

[mihe7]

private static String checkDescription(String description, String errorMsg) {
    if ("".equals(description)) {
        throw new IllegalArgumentException(errorMsg);
    }
    return description;
}

 

[frager2345]

Ah, danke dir.

 

[Neumi5694]

Wenn man sicher ist, dass description garantiert nicht null ist, geht auch

description.isEmpty()
//oder
description.trim().isEmpty() //nur Leerzeichen oder leer
//oder in neueren Javaversionen
description.isBlank() //modernere Variante des obigen Befehls

 

[Fragerjava]

Wenn man sicher ist, dass description garantiert nicht null ist, geht auch

description.isEmpty()
//oder
description.trim().isEmpty() //nur Leerzeichen oder leer
//oder in neueren Javaversionen
description.isBlank() //modernere Variante des obigen Befehls

Vielen Dank, habs ausgetauscht 👍🏽