Inny sposób

Pytania dotyczące zadania

Powtórzenia nie są potrzebne

Składnia Java, Poziom 8, Lekcja 8

Rozwiązane

Ktoś pomoże, bo w pierwszej chwili wydawał mi się dobry sposób, ale nie działa.

Utwórz Map<String, String> i dodaj dziesięć wpisów, które odpowiadają parom (nazwisko, imie).
Usuń ludzi o takich samych imionach.

Wymagania:

Program nie powinien wyświetlać tekstu na ekranie.
Program nie powinien czytać wartości z klawiatury.
Metoda utworzMap() musi tworzyć i zwracać HashMap, który ma elementy (String, String) i zawiera 10 wpisów.
Metoda usunPowtorzoneImiona() musi usuwać z mapy wszystkich ludzi o takich samych imionach.
Metoda usunPowtorzoneImiona() musi wywoływać metodę usunElementZMapPoWartosci().

package pl.codegym.task.task08.task0817; import java.util.HashMap; import java.util.Iterator; import java.util.Map; /* Powtórzenia nie są potrzebne */ public class Solution { public static HashMap<String, String> utworzMap() { //tutaj wpisz swój kod HashMap<String, String> nazwiskoImie = new HashMap<>(); nazwiskoImie.put("Szyba", "Michał"); nazwiskoImie.put("szsasasa", "Piotr"); nazwiskoImie.put("sasasssaaa", "Beza"); nazwiskoImie.put("Kowalska", "Anna"); nazwiskoImie.put("Lis", "Ble"); nazwiskoImie.put("fdfd", "Michał"); nazwiskoImie.put("dsds", "Ble"); nazwiskoImie.put("sa", "Damian"); nazwiskoImie.put("qw", "Skywalker"); nazwiskoImie.put("sasassssss", "Nick"); return nazwiskoImie; } public static void usunPowtorzoneImiona(Map<String, String> mapa) { //tutaj wpisz swój kod Iterator<Map.Entry<String, String>> iterator = mapa.entrySet().iterator(); Iterator<Map.Entry<String, String>> iterator2 = mapa.entrySet().iterator(); while (iterator.hasNext()) { Map.Entry<String, String> entry = iterator.next(); String imie = entry.getValue(); while (iterator2.hasNext()) { Map.Entry<String, String> entry2 = iterator2.next(); String imie2 = entry2.getValue(); if (imie2.equals(imie)) { usunElementZMapPoWartosci(mapa, imie2); } } } } public static void usunElementZMapPoWartosci(Map<String, String> mapa, String wartosc) { HashMap<String, String> kopia = new HashMap<String, String>(mapa); for (Map.Entry<String, String> para : kopia.entrySet()) { if (para.getValue().equals(wartosc)) mapa.remove(para.getKey()); } } public static void main(String[] args) { } }

Komentarze (8)

Popularne
Najnowsze
Najstarsze

Musisz się zalogować, aby dodać komentarz

Molo

Poziom 41 , Lublin, Poland

11 września 2022, 12:21

Porównać dwie kopie ze sobą operatorem == , odrzucić pierwszy przebieg operatorem && porównując klucze .

HashMap<String , String> mapa2 = new HashMap<String , String>(mapa) ;
HashMap<String , String> mapa3 = new HashMap<String , String>(mapa) ;

  for(Map.Entry<String, String> para2 : mapa2.entrySet()) {

     for(Map.Entry<String, String> para3 : mapa3.entrySet()) {

       if(para2.getValue() == para3.getValue() && para2.getKey() != para3.getKey())
            usunElementZMapPoWartosci(mapa,para3.getValue()) ;     }   }
}

Do gotowych odp od codegym lepiej nie zaglądać bo szkoda oczu na ich twory . Po premi dostają pewnie jeżeli nie zaciemnią kodu włącznie z opisem zadań :]

Michał

Poziom 24 , Wrocław, Poland

25 sierpnia 2020, 08:01

Zaciąłem się na tym zadaniu. Wygląda ok, ale go nei zalicza. Jeśli próbuje debugować w IDE to też widzę, że nie idzie tak jak powinno. Napisanie wprost co jest nie tak będzie pomocne.

public static void usunPowtorzoneImiona(Map<String, String> mapa) {
    //tutaj wpisz swój kod
    HashMap<String, String> kopia1 = new HashMap<>(mapa);
    HashMap<String, String> kopia2 = new HashMap<>(mapa);

    Iterator<Map.Entry<String, String>> iterator1 = kopia1.entrySet().iterator();
    Iterator<Map.Entry<String, String>> iterator2 = kopia2.entrySet().iterator();

    while (iterator1.hasNext()) {
        Map.Entry<String, String> entry1 = iterator1.next();
        String imie1 = entry1.getValue();
        String nazwisko1 = entry1.getKey();

        //System.out.println(imie1 + " , " + nazwisko1);

        while (iterator2.hasNext()) {
            Map.Entry<String, String> entry2 = iterator2.next();
            String imie2 = entry2.getValue();
            String nazwisko2 = entry2.getKey();

            if (imie2.equals(imie1)) {
                if (!nazwisko2.equals(nazwisko1)) {
                    usunElementZMapPoWartosci(kopia2, imie1);

                }
            }
        }
    }
    //System.out.println(mapa);
}

public static void usunElementZMapPoWartosci(Map<String, String> mapa, String wartosc) {
    HashMap<String, String> kopia = new HashMap<String, String>(mapa);
    for (Map.Entry<String, String> para : kopia.entrySet()) {
        if (para.getValue().equals(wartosc))
            mapa.remove(para.getKey());
    }
}

Misiu

Poziom 41 , Gdansk, Poland

25 sierpnia 2020, 09:35

przydatny

Toć już napisałem. Po iteracji dla pierwszego elementu zewnętrznej pętli, pętla wewnętrzna ma iterator2 ustawiony na końcu mapy kopia2. Dla kolejnych iteracji "iterator2.hasNext()" jest FALSE. Wypadałoby po linii 27 dać instrukcję ustawiającą iterator2 na początku mapy kopia2. I don't know jak to zrobić. Albo zastosować pętlę(e) for-each: for (Map.Entry<String, String> para2 :kopia2.entrySet()) {...}

Michał

Poziom 24 , Wrocław, Poland

25 sierpnia 2020, 11:48

rozwiązanie

Zrezygnowałem z pętli while. Zrobiłem to od początku na pętli for-each i działa. Dzięki za pomoc!

public static void usunPowtorzoneImiona(Map<String, String> mapa) {

        HashMap<String, String> kopiaPierwsza = new HashMap<String, String>(mapa);
        HashMap<String, String> kopiaDruga = new HashMap<String, String>(mapa);

        for (Map.Entry<String, String> para1 : kopiaPierwsza.entrySet()) {
            String imie1 = para1.getValue();
            String nazwisko1 = para1.getKey();

            for (Map.Entry<String, String> para2 : kopiaDruga.entrySet()) {
                String imie2 = para2.getValue();
                String nazwisko2 = para2.getKey();

                if (imie2.equals(imie1) && !nazwisko2.equals(nazwisko1)) {
                    usunElementZMapPoWartosci(mapa, imie2);
                }
            }
        }
        //System.out.println(mapa);
    }

Kubańczyk92 QA Automation Engineer

10 kwietnia 2022, 09:59

dla referencji iterator2, możesz wywołać metodę iteratora ponownie aby zwróciła jego instację wskazującą na początek listy:

public static void usunPowtorzoneImiona(Map<String, String> mapa) {

     HashMap<String, String> kopia1 = new HashMap<>(mapa);
     HashMap<String, String> kopia2 = new HashMap<>(mapa);

     Iterator<Map.Entry<String, String>> iterator1 = kopia1.entrySet().iterator();
     Iterator<Map.Entry<String, String>> iterator2 = kopia2.entrySet().iterator();

     while (iterator1.hasNext()) {
         Map.Entry<String, String> entry1 = iterator1.next();
         String imie1 = entry1.getValue();
         String nazwisko1 = entry1.getKey();

         //System.out.println(imie1 + " , " + nazwisko1);

         while (iterator2.hasNext()) {
             Map.Entry<String, String> entry2 = iterator2.next();
             String imie2 = entry2.getValue();
             String nazwisko2 = entry2.getKey();

             if (imie2.equals(imie1)) {
                 if (!nazwisko2.equals(nazwisko1)) {
                     usunElementZMapPoWartosci(mapa, imie2);

                 }
             }
         }iterator2 = kopia2.entrySet().iterator();
     }

Michał

Poziom 24 , Wrocław, Poland

24 sierpnia 2020, 19:39

Nie wiem, czy za bardzo nie brnę w ten kierunek:

public static void usunPowtorzoneImiona(Map<String, String> mapa) {
        //tutaj wpisz swój kod
        HashMap<String, String> kopia1 = new HashMap<>(mapa);
        HashMap<String, String> kopia2 = new HashMap<>(mapa);

        Iterator<Map.Entry<String, String>> iterator = kopia1.entrySet().iterator();
        Iterator<Map.Entry<String, String>> iterator2 = kopia2.entrySet().iterator();

        while (iterator.hasNext()) {
            Map.Entry<String, String> entry = iterator.next();
            String imie = entry.getValue();
            String nazwisko = entry.getKey();

            while (iterator2.hasNext()) {
                Map.Entry<String, String> entry2 = iterator2.next();
                String imie2 = entry2.getValue();
                String nazwisko2 = entry2.getKey();

                if (imie2.equals(imie) && !nazwisko2.equals(nazwisko)) {
                    usunElementZMapPoWartosci(mapa, imie2);
                }
            }
        }
    }

    public static void usunElementZMapPoWartosci(Map<String, String> mapa, String wartosc) {
        HashMap<String, String> kopia = new HashMap<String, String>(mapa);
        for (Map.Entry<String, String> para : kopia.entrySet()) {
            if (para.getValue().equals(wartosc))
                mapa.remove(para.getKey());
        }
    }

Misiu

Poziom 41 , Gdansk, Poland

24 sierpnia 2020, 23:39

O to szło. Ja robiłem to pętlami for-each. U Ciebie kod działa poprawnie dla pierwszej pary z kopia1. Dla kolejnych nie działa, bo iterator2 ustawiony jest wciąż na końcu mapy. Czytelność kodu: OK, kolory też ładne ;) , ale skoro: kopia2, iterator2, imie2, nazwisko2 i skoro: kopia1 to też: iterator1, imie1, nazwisko1

Misiu

Poziom 41 , Gdansk, Poland

22 sierpnia 2020, 16:59

imie2.equals(imie) - to prawda dla np. pierwszego elementu, który nie ma powtórzeń. A potem dla drugiego, itd. Twój kod opróżnia mapę całkowicie. Potrzebny jest dodatkowy warunek. Iteracja po mapie i jednoczesne zmiany jej rozmiaru to droga ku błędom. Zrób dwie kopie mapy i je iteruj. Elementy usuwaj z mapy.