Получите ключ HashMap, введя значение HashMap, которое не работает должным образом - PullRequest
Новая
       41

Получите ключ HashMap, введя значение HashMap, которое не работает должным образом

0 голосов
/ 28 февраля 2019

Проблема в методе getName()

public class Phonebook implements PhonebookInterface {

    private Map<String, Set<String>> phonebook;
    private Map<String, Address> address;

    public Phonebook() {
        this.phonebook = new HashMap<String, Set<String>>();
        this.address = new HashMap<String, Address>();
    }

    @Override
    public void addNumber(String person, String number) {
        if (!this.phonebook.containsKey(person)) {
            this.phonebook.put(person, new HashSet<String>());
        }

        this.phonebook.get(person).add(number);
    }

    @Override
    public void getName(String number) {
        for (String person : this.phonebook.keySet()) {
            if (this.phonebook.get(person).contains(number)) {
                System.out.println(" " + person);
            } else {
                System.out.println("  not found");
            }
        }
    }

    @Override
    public void getNumber(String person) {
        try {
            for (String n : this.phonebook.get(person)) {
                if (this.phonebook.get(person).size() > 1) {
                    System.out.println(" " + n);
                } else {
                    System.out.println("number: " + n);
                }
            }
        } catch (Exception e) {
            System.out.println("  not found");
        }
    }

    public void addAddress(String person, String street, String city) {
        this.address.put(person, new Address(street, city));
    }

    public void getInfo(String person) {
        if (this.address.containsKey(person)) {
            System.out.println("  address: " + this.address.get(person));

            if (this.phonebook.containsKey(person)) {
                System.out.println("  phone numbers:");
                for (String n : this.phonebook.get(person)) {
                    System.out.println("  " + n);
                }
            } else {
                System.out.println("  phone number not found");
            }
        } else {
            System.out.println("  not found");
        }
    }

    public void removeInfo(String person) {
        if (this.address.containsKey(person)) {
            this.address.remove(person);
        } else {
            System.out.println(" not found");
        }
    }
}

public class Address {

    private String street;
    private String city;

    public Address() {
    }

    Address(String street, String city) {
        this.street = street;
        this.city = city;
    }

    public String getStreet() {
        return this.street;
    }

    public String getCity() {
        return this.city;
    }

    public String toString() {
        return this.street + " " + this.city;
    }

}

class Main {

    public static void main(String[] args) {
        Phonebook p = new Phonebook();

        System.out.println("Search for number: ");

        System.out.println("Pekka :");
        p.addNumber("Pekka", "014-1234");
        p.addNumber("Pekka", "015-5344");
        p.getNumber("Pekka");

        System.out.println("Matti :");
        p.addNumber("Matti", "013-4321");
        p.getNumber("Matti");

        System.out.println("\nsearch for a person by phone number: ");

        p.getName("013-4321");
        p.getName("1234567");
    }
}

Ожидаемый результат:

Поиск числа:
Пекка:
014-1234
015-5344
Матти:
номер: 013-4321

поиск человека по номеру телефона:
Матти
не найдено

Но вывод был:

Поиск по номеру:
Пекка:
014-1234
015-5344
Матти:
номер: 013-4321

Поиск человека по номеру телефона:
не найдено
Matti
не найдено
не найдено

Почему выводится печать "не найден"3 раза, а только один раз?

1 Ответ

0 голосов
/ 28 февраля 2019

Вы перебираете все ключи Map и сравниваете соответствующие значения со значением, которое вы ищете.Если совпадения нет, вы печатаете «not found».

Ваш Map имеет два ключа, поэтому, если вы ищете значение, присутствующее в Map, вы напечатаете соответствующий ключ, новы также напечатаете «not found» для другой клавиши.

Если вы ищете значение, отсутствующее в Map, вы напечатаете «not found» дважды (один раз для каждой клавиши).

Вы должны вывести «not found» только после перебора всех ключей: 

public void getName(String number) {   
    for (String person : this.phonebook.keySet()) {   
        if (this.phonebook.get(person).contains(number)) {   
            System.out.println(" " + person);
            return;
        }    
    }
    System.out.println("  not found"); 
}
...