Map - remove() ?

[programmer77]

Hallo,
Wie kann man kontrollieren, ob die entsprechende Aufgabe gelöscht oder nicht vorhanden ist?
Also output --> " No Entry" statt null (wenn nicht vorhanden) .
" Gelöscht " statt null (wenn gelöscht.)
class Period {

private Map<String, Period> zeitPlan = new TreeMap<String, Period>();
public String rem(String aufgabe) {
return null;
}

 

[Xyz1]

Machen wirs mit etwas Professionalität:

    public static class Period {

        private final Map<String, Period> zeitPlan = new TreeMap<>();

        public Period() {
            zeitPlan.put("hallo", this);
            zeitPlan.put("du", this);
            zeitPlan.put("da!", this);
        }

        public String rem1(String aufgabe) {
            if (rem2(aufgabe)) {
                return "Geloescht";
            } else {
                return "No Entry";
            }
        }

        private final boolean rem2(String aufgabe) {
            if (zeitPlan.containsKey(aufgabe)) {
                zeitPlan.remove(aufgabe);
                return true;
            } else {
                return false;
            }
        }        
    }
    
    public static void main(String[] args) {
        Period period = new Period();
        System.out.println(period.rem1("hallo"));
        System.out.println(period.rem1("hallo"));
    }

 

[programmer77]

Danke für die Antwort, aber ich habe bissl falsch erklärt.
Als Output brauche ich 3 Optionen.
1- task removed, 2- no entry , 3 - null
System.out.println(period.rem1("hallo")); -- output : task removed
System.out.println(period.rem1("Wie gehts?")); -- output : no entry
System.out.println(period.rem1("hallo")); -- output : null

 

[mrBrown]

Nur mit Nutzung von Map gar nicht - du musst dir die Logik dafür selbst schreiben.

Wobei sich da erstmal die Frage stellt, was der Unterschied zwischen task removed, no entry und null sein soll

 

[programmer77]

Unterschied ist wie folgendes :
public Period() {
zeitPlan.put("hallo", this);
zeitPlan.put("du", this);
zeitPlan.put("da!", this);
}
System.out.println(period.rem1("hallo")); ---> gelöscht
System.out.println(period.rem1("wie gehts")); ---> no entry (weil es nicht vorhanden war)
System.out.println(period.rem1("hallo")); ---> null

 

[mrBrown]

Also ist gelöscht kein Zustand, sondern nur einmalige Ausgabe beim löschen?
Es gibt also im wesentlichen nur einen Unterscheid zwischen nie vorhanden und nicht mehr vorhanden?

 

[programmer77]

Ja , wie du geschrieben hast. z.B "wie gehts" ist nie vorhanden. wie soll man das impl ?

 

[mrBrown]

Man könnte Optional darür missbrauchen.

null = nie vorhandne
Optional.empty() = gelöscht
Optional.of(...) = vorhanden
Beim entfernen einfach mit contains prüfen, ob was drin lag, und entsprechende handeln

 

[programmer77]

mit containsKey(...) kann man überprüfen ob es vorhanden ist oder nicht. Aber wie ist mit der null ?
Z.B
if(!zeitplan.containsKey(aufgabe)){
return "nicht vorhanden";
}
zeitplan.remove(aufgabe);
return "gelöscht";
}

 

[Xyz1]

Nimm doch bitte Java -Tags, eine Erklärung steht in meiner Sig.

Dann muss sich die Map zusätzlich merken, ob ein String schon mal vorhanden war.

Dafür könnte man value benutzen, oder man nimmt 'ne zweite Map deleted.

 

[programmer77]

Machen wirs mit etwas Professionalität:

    public static class Period {

        private final Map<String, Period> zeitPlan = new TreeMap<>();

        public Period() {
            zeitPlan.put("hallo", this);
            zeitPlan.put("du", this);
            zeitPlan.put("da!", this);
        }

        public String rem1(String aufgabe) {
            if (rem2(aufgabe)) {
                return "Geloescht";
            } else {
                return "No Entry";
            }
        }

        private final boolean rem2(String aufgabe) {
            if (zeitPlan.containsKey(aufgabe)) {
                zeitPlan.remove(aufgabe);
                return true;
            } else {
                return false;
            }
        }       
    }
   
    public static void main(String[] args) {
        Period period = new Period();
        System.out.println(period.rem1("hallo"));
        System.out.println(period.rem1("how")); // da muss nicht vorhanden ausgeben.
        System.out.println(period.rem1("hallo")); // da muss null ausgeben. 
    }

 

[programmer77]

"No Entry" heisst nicht , dass Map leer ist ..
Als Ergebnis kommt "task removed" zurück wenn diese Aufgabe
erfolgreich entfernt wurde, bzw. "no entry" falls die entsprechende
Aufgabe nicht vorhanden war.

 

[Xyz1]

Verwirrt
Wenn das Elem. vorhanden ist, gelingt das removen IMMER...

Task Removed: immer dann, wenn?
No Entry: immer dann, wenn?
Null: immer dann, wenn?

Schüssel' das mal auf.

 

[programmer77]

zeitPlan.put("hallo", this);
zeitPlan.put("du", this);
zeitPlan.put("da!", this);
Da haben wir z.B 3 Elemente.
Wenn --> System.out.println(period.rem1("hallo")); ist dann "task removed"
Wenn --> System.out.println(period.rem1("Wie gehts")); ist dann " no entry" (weil wir es vorher nicht hinzugefügt haben. d.h wir haben es nicht. wir haben jetzt nur "du" , "da".)
Wenn --> System.out.println(period.rem1("hallo")); ist dann null, weil es sich gerade in Map nicht befindet.(weil wir es schon gelöscht haben).

 

[mrBrown]

Wie man das umsetzen könnte steht in #8^^

 

[programmer77]

Kannst du einen Beispiel mit Optional geben? Das werde ich erstes mal umsetzen..

 

[Meniskusschaden]

Wenn --> System.out.println(period.rem1("Hallo")); ist dann null, weil es sich gerade in Map nicht befindet.(weil wir es schon gelöscht haben).

Wozu soll das eigentlich gut sein? Warum genügt da nicht "no entry"? Vielleicht versteht man dann besser, was du eigentlich erreichen möchtest. Mir ist es nämlich noch immer nicht wirklich klar.

 

[mrBrown]

Alles was du in die Map legst, legst du als Optional.of(...) in die Map.
Beim entfernen prüfst du, ob es null ist, dann null.
Sonst ist es entweder Optional.empty oder Optional.of(...), im ersten fall "no entry", im zweiten legst du Optional.empty mit dem gleichen Key in die Map, und gibst "task removed" zurück

 

[programmer77]

@Meniskusschaden , weil es in der Aufgabe so vorgegeben ist ..

 

[Meniskusschaden]

@Meniskusschaden , weil es in der Aufgabe so vorgegeben ist ..

Dann poste doch mal die Aufgabenstellung.

 

[programmer77]

Unten ist die Aufgabestellung - Methode "rem()" ...

import java.util.Map;
import java.util.TreeMap;


    class Period {
        private String from;
        private String to;

        public Period(String from, String to) {
            this.from = from;
            this.to = to;
        }
        public String from(){
            return from;
        }
        public String to(){
            return to;
        }
    }

    public class Aufgabe1 {
            public static void main(String[] args) {
                Aufgabe1 plan1 = new Aufgabe1();
                Aufgabe1 plan2 = new Aufgabe1();
                Period period = new Period("0.0.2", "0.0.3");
                System.out.println(plan1.add("Sonnenaufgang betrachten", new Period("1.95.0", "2.5.0")));
                System.out.println(plan1.add("Frühstück", new Period("2.15.0", "2.30.0")));
                System.out.println(plan1.add("Muschelsammeln", new Period("3.0.0", "3.50.0")));
System.out.println(plan1.add("Sonnenhut tragen", new Period("2.0.0", "6.0.0")));
                System.out.println(plan1.rem("Sonnenhut tragen")); // removed
                System.out.println(plan1.rem("Abendessen")); // no entry
                System.out.println(plan1.rem("Sonnenhut tragen")); // null

                private Map<String, Period> zeitPlan = new TreeMap<String, Period>();
            }


    public boolean add(String aufgabe, Period period) {
       Period period1 = zeitPlan.put(aufgabe, period);
        if (period1== null) {
            return false;
        }
        return true;
    }

     public String from(String aufgabe) {
        Period period = zeitPlan.get(aufgabe);
        if (period != null) {
            return period.from();
        }
        return null;
    }

    public String to(String aufgabe) {
        Period period = zeitPlan.get(aufgabe);
        if (period != null) {
            return period.to();
        }
        return null;
    }
    // TODO
    /*wird für das Entfernen von Tätigkeiten aus dem Tagesablaufplan
    benötigt; hat einen Parameter vom Typ String und gibt eine Tätigkeit
    an; als Ergebnis kommt "task removed" zurück wenn diese Tätigkeit
    erfolgreich entfernt wurde, bzw. "no entry" falls die entsprechende
    Tätigkeit nicht vorhanden war.*/
    public String rem(String aufgabe) {
        return null // TODO;

    }
   
}

 

[mrBrown]

Hm, Optional ist eigentlich überflüssig.

removen kannst du als put(..., null) sehen, dann gibt es einen unterschied zwischen null und nicht enthalten.

 

[Meniskusschaden]

Wenn --> System.out.println(period.rem1("hallo")); ist dann null, weil es sich gerade in Map nicht befindet.(weil wir es schon gelöscht haben).

Ich lese aus der Aufgabenstellung, dass in dem Fall "no entry" zurück gegeben werden soll.

 

[mrBrown]

Ich lese aus der Aufgabenstellung, dass in dem Fall "no entry" zurück gegeben werden soll.

In der main steht für diesen Fall als Kommentar null dabei, das fehlt leider nur unten

 

[programmer77]

In der main kannst du als Kommentar die Outputs sehen.
System.out.println(plan1.rem("Sonnenhut tragen")); // removed
System.out.println(plan1.rem("Abendessen")); // no entry
System.out.println(plan1.rem("Sonnenhut tragen")); // null

 

[Meniskusschaden]

In der main steht für diesen Fall als Kommentar null dabei, das fehlt leider nur unten

Habe ich tatsächlich nicht gesehen. Für mich ist die Aufgabenstellung damit widersprüchlich. Ehrlich gesagt, würde ich da die TODO-Kommentierung für maßgeblich halten, zumal die im Gegensatz zum main-Kommentar auch praktisch sinnvoll ist. Außerdem gehört so eine Spezifikation doch eher an die Stelle der Methoden-Deklaration, als an die Stellen der Verwendung. Deshalb spekuliere ich mal, dass das ein Kommentierungsfehler in der main-Methode ist.

 

[mrBrown]

Habe ich tatsächlich nicht gesehen. Für mich ist die Aufgabenstellung damit widersprüchlich. Ehrlich gesagt, würde ich da die TODO-Kommentierung für maßgeblich halten, zumal die im Gegensatz zum main-Kommentar auch praktisch sinnvoll ist. Außerdem gehört so eine Spezifikation doch eher an die Stelle der Methoden-Deklaration, als an die Stellen der Verwendung. Deshalb spekuliere ich mal, dass das ein Kommentierungsfehler in der main-Methode ist.

Das würde ich auch denken, aber wer weiß, die Steller solcher Aufgaben sind manchmal etwas komisch drauf^^

 

[programmer77]

Es gibt keinen Fehler in der Aufgabenstellung.

 

[mrBrown]

Es gibt keinen Fehler in der Aufgabenstellung.

Naja, ich seh da zwei Aufgabenstellungen, die gegensätzlich sind, und die untere würde auch ich als richtiger ansehen...von daher...

 

[programmer77]

#21 ist die richtige Aufgabenstellung ...

 

[mrBrown]

#21 ist die richtige Aufgabenstellung ...

Das Todo in #21 enthält kein "null", sondern nur "task removed" und "no entry".

 

[programmer77]

Ja in TestCases steht aber null. wenn man versucht ein Element zu löschen, das schon vorher gelöscht wurde. (dann null..)
Es ist nicht so kompliziert. Z.b wir haben "a", "b", "c" .
wenn remove("a") --> dann "Removed"
wenn remove ("d") --> dann " No Entry" (weil wir d nicht haben)
wenn remove("a") --> null (weil es "a" nicht mehr haben)

Was findest du genau so unverständlich ?

 

[Xyz1]

Es gibt keinen Fehler in der Aufgabenstellung

Glaub mir, es gibt fehlerhafte Aufgabenstellungen....... Das wäre auch zum Vorteil des Bearbeitenden.

Also, vielleicht wurde auch an Zusicherungen gedacht, null zu erlauben... oder vielleicht soll der value auf null gesetzt werden.

Aber Mensch, das Thema kocht ja gerade hoch hier.^^

 

[programmer77]

Am besten schreibe ich hier nicht mehr , danke an allen die beantwortet haben. Aber ich bin mir sicher , dass es keinen Fehler gibt, die Lösung werde ich später posten.

 

[mrBrown]

Die Lösung mit null steht oben schon als Text, du musst es nur in Code überführen^^

 

[Meniskusschaden]

Was findest du genau so unverständlich ?

Ich finde daran unverständlich, dass das der Vorgabe des TODO-Kommentars widerspricht. Weiterhin finde ich unverständlich, warum man in diesem Fall überhaupt null als Antwort fordern sollte.

Aber ich bin mir sicher , dass es keinen Fehler gibt

Ich denke schon, dass es mindestens einen Fehler gibt, denn meines Erachtens ist zeitPlan in der main-Methode falsch deklariert, so dass der Code nicht compiliert. Deshalb sind Zweifel angebracht, dass auch bei dem Rest etwas faul ist.

 

[programmer77]

Ich finde daran unverständlich, dass das der Vorgabe des TODO-Kommentars widerspricht. Weiterhin finde ich unverständlich, warum man in diesem Fall überhaupt null als Antwort fordern sollte.

Ich denke schon, dass es mindestens einen Fehler gibt, denn meines Erachtens ist zeitPlan in der main-Methode falsch deklariert, so dass der Code nicht compiliert. Deshalb sind Zweifel angebracht, dass auch bei dem Rest etwas faul ist.

Sorry, ich habe einen Fehler bei Copy+paste . Das kannst du kompilieren ..

import java.util.Map;
import java.util.TreeMap;


    class Period {
        private String from;
        private String to;

        public Period(String from, String to) {
            this.from = from;
            this.to = to;
        }
        public String from(){
            return from;
        }
        public String to(){
            return to;
        }
    }

    public class Aufgabe1 {

            public static void main(String[] args) {


                Aufgabe1 plan1 = new Aufgabe1();
                Aufgabe1 plan2 = new Aufgabe1();
                Period period = new Period("0.0.2", "0.0.3");
                System.out.println(plan1.add("Sonnenaufgang betrachten", new Period("1.95.0", "2.5.0")));
                System.out.println(plan1.add("Frühstück", new Period("2.15.0", "2.30.0")));
                System.out.println(plan1.add("Muschelsammeln", new Period("3.0.0", "3.50.0")));
                System.out.println(plan1.add("Sonnenhut tragen", new Period("2.0.0", "6.0.0")));
                System.out.println(plan1.rem("Sonnenhut tragen")); // removed
                System.out.println(plan1.rem("Abendessen")); // no entry
                System.out.println(plan1.rem("Sonnenhut tragen")); // null

            }


    private Map<String, Period> zeitPlan = new TreeMap<String, Period>();



    public boolean add(String aufgabe, Period period) {
       Period period1 = zeitPlan.put(aufgabe, period);
        if (period1== null) {
            return false;
        }
        return true;
    }

     public String from(String aufgabe) {
        Period period = zeitPlan.get(aufgabe);
        if (period != null) {
            return period.from();
        }
        return null;
    }

    public String to(String aufgabe) {
        Period period = zeitPlan.get(aufgabe);
        if (period != null) {
            return period.to();
        }
        return null;
    }
    // TODO
    /*wird für das Entfernen von Tätigkeiten aus dem Tagesablaufplan
    benötigt; hat einen Parameter vom Typ String und gibt eine Tätigkeit
    an; als Ergebnis kommt "task removed" zurück wenn diese Tätigkeit
    erfolgreich entfernt wurde, bzw. "no entry" falls die entsprechende
    Tätigkeit nicht vorhanden war.*/
    public String rem(String aufgabe) {
        return null; // TODO

    }
   
}

 

[Meniskusschaden]

Ja, jetzt kompiliert es. Mit Post#22 von @mrBrown erhältst du auch die gewünschte Ausgabe. Von meiner Seite allerdings mit einigen Zweifeln, dass das vom Aufgabensteller wirklich so gedacht war.

 

[programmer77]

Wenn jemand rem() method implementieren kann, würde ich wirklich sehr freuen. Bei mir gehts leider nicht mehr

 

[mrBrown]

Kommt drauf an - was zahlst du?

 

[Meniskusschaden]

Wenn jemand rem() method implementieren kann, würde ich wirklich sehr freuen.

Ohne Eigenleistung nicht. Wie sieht deine rem-Methode denn jetzt aus?

 

[programmer77]

Wie soll ich kontrollieren, ob dieses Element in Map vorhanden ist? Also "no Entry" kann ich leider nicht.

public String rem(String aufgabe) {
    if (zeitPlan.containsKey(aufgabe)){
        zeitPlan.remove(aufgabe);
        return "removed";
    }
return null;

 

[mrBrown]

Wie soll ich kontrollieren, ob dieses Element in Map vorhanden ist?

Das tust du grad schon.

 

[programmer77]

Ja mit containsKey(aufgabe) schon . Aber wo soll man "no Entry " ausgeben?

 

[Meniskusschaden]

Die Idee von @mrBrown war ja, beim "Löschen" die Aufgabe nicht wirklich zu entfernen, sondern nur ihre Periode auf null zu setzen. Du hast also die Fälle "Kein Eintrag", "Eintrag mit null" und "Eintrag mit Periode".

 

[programmer77]

Meinst du so ?
if (zeitPlan.containsKey(aufgabe)) {
zeitPlan.put(aufgabe,null);
return "removed";
}

 

[mrBrown]

contains ist auch true, wenn null drin ist, es also eigentlich schon gelöscht ist