What is the problem? Please help

Question about the task

Under discussion

package com.codegym.task.task08.task0829;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;

/*
Software update

*/

public class Solution {
    public static void main(String[] args) throws IOException {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

        // List of addresses
        //List<String> addresses = new ArrayList<>();
        HashMap<String,String> addresses = new HashMap<String,String>();
        while (true) {
            String family = reader.readLine();
            String city = reader.readLine();
            if (family.isEmpty() || city.isEmpty()){
                break;
            }

            addresses.put(city,family);
        }

        // Read the house number
        //int houseNumber = Integer.parseInt(reader.readLine());
        String cityName = reader.readLine().toUpperCase();
        String searched = "";
        for (Map.Entry<String,String> entry : addresses.entrySet()){
            if(entry.getKey().equals(cityName)){
                searched = entry.getValue();
            }
        }
        System.out.println(searched);
    }
}

package com.codegym.task.task08.task0829; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import java.util.ArrayList; import java.util.HashMap; import java.util.List; import java.util.Map; /* Software update */ public class Solution { public static void main(String[] args) throws IOException { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); // List of addresses //List<String> addresses = new ArrayList<>(); HashMap<String,String> addresses = new HashMap<String,String>(); while (true) { String family = reader.readLine(); String city = reader.readLine(); if (family.isEmpty() || city.isEmpty()){ break; } addresses.put(city,family); } // Read the house number //int houseNumber = Integer.parseInt(reader.readLine()); String cityName = reader.readLine().toUpperCase(); String searched = ""; for (Map.Entry<String,String> entry : addresses.entrySet()){ if(entry.getKey().equals(cityName)){ searched = entry.getValue(); } } System.out.println(searched); } }

Comments (6)

Popular
New
Old

You must be signed in to leave a comment

KARAN BHOITE

Level 15 , Nagpur, India

21 August 2019, 12:20

why we shld make it complicated , if we can solve it by simple way public class Solution { public static void main(String[] args) throws IOException { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); // List of addresses List<String> addresses = new ArrayList<>(); while (true) { String family = reader.readLine(); if (family.isEmpty()) break; addresses.add(family); } String a = reader.readLine(); int n=0; for(int i=0;i<addresses.size();i++) { if(a.equals(addresses.get(i))) { n = i+1; } } System.out.println(addresses.get(n)); } } :);););););)

Vishnu

Level 25 , Richardson, United States

29 July 2019, 03:35

Try below, it should work! package com.codegym.task.task08.task0829; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import java.util.*; /* Software update */ public class Solution { public static void main(String[] args) throws IOException { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); // List of addresses HashMap <String, String> map = new HashMap<String, String>(); while (true) { String city = reader.readLine(); if (city.isEmpty()) { break; } String family = reader.readLine(); map.put(city, family); } // Read the house number String city = reader.readLine(); String familyName = map.get(city); System.out.println(familyName); } }

Clark

Level 17 , Berlin, Germany

27 July 2019, 08:33

Use no "UpperCase", maybe it is not necessary.

KIN SOCHEAT

Level 34 , Phnom Penh, Cambogia

25 July 2019, 10:23

try { family = reader.readLine().toUpperCase(); if (family.isEmpty() ) break; String city = reader.readLine().toUpperCase(); // addresses.put(city, family); addresses.put(family, city); } catch (IOException e) { e.printStackTrace(); }

KIN SOCHEAT

Level 34 , Phnom Penh, Cambogia

23 July 2019, 11:43

delete all this for (Map.Entry<String,String> entry : addresses.entrySet()){ if(entry.getKey().equals(cityName)){ searched = entry.getValue(); } } System.out.println(searched); try if(addresses.containKey(cityName){ System.out.println(addresses.get.(cityName)); }

Letenyei Ádám

Level 16 , Bakonyszombathely, Magyarország

23 July 2019, 20:46

I tried with this solution, it isn't working :(

package com.codegym.task.task08.task0829;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;

/* New task: The program should work with cities rather than house numbers:*/

public class Solution {
    public static void main(String[] args) throws IOException {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

        HashMap<String, String> addresses = new HashMap<String, String>();
        while (true) {
            String family = null;
            try {

                family = reader.readLine().toUpperCase();

                String city = reader.readLine().toUpperCase();
                if (family.isEmpty() || city.isEmpty()) break;
                addresses.put(city, family);
            } catch (IOException e) {
                e.printStackTrace();
            }
        }

        try {
            String cityName = reader.readLine().toUpperCase();
            if (addresses.containsKey(cityName)) System.out.println(addresses.get(cityName).toString());
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}