9. User Logs homework

MarinaShortyMarinova 12 Точки

9. User Logs homework

Здравейте колеги,

Правя 9та задача от домашното -> https://softuni.bg/trainings/resources/officedocument/12002/exercise-problem-descriptions-java-advanced-january-2017

https://gist.github.com/anonymous/838ce1ff1a14e28a48c7df43a2a0a23c

Използвам LinkedHashMap, защото доколкото знам, би трябвало да не съхранява еднакви ключове, ами да ги презаписва. Очевидно нещо не ми се получава, защото за еднакви IP-та ми прави различни записи. Отбелязала съм реда, в който има проблем, когато дебъгвам. Сравнява IP-тата и излиза, че такъв запис не съществува, а той всъщност съществува, което ме навежда на мисълта, че проблемът е в проверката, но не знам точно къде. Моля за малко помощ.

Тагове:

22/03/2017 15:02:19 Java Advanced

Teodor92 569 Точки

Проблема идва от това как update-ваш вътрешния LinkedHashMap на:

TreeMap<String,LinkedHashMap<String, Integer>> usersIPs=new TreeMap<>();

Или по-точно от следната част на кода:

                    if(usersIPs.get(username).containsKey(ip)){
                        int countUpdate=usersIPs.get(username).get(ip)+1;
                        newData.put(ip,countUpdate);
                        usersIPs.put(username,newData);

                    }
                    else {
                        newData.put(ip,1);
                        usersIPs.put(username,newData);

                    }

Какъв е проблема тук? Че всеки път заместваш вътрешния Map с нов (newData), което изтрива всички ip-та срещнати до сега. Това което искаш да направиш е, да вземеш същестуващия вътрешен Map и да добавш новите update/увеличиш coutner-a на същестуващо ip. Примерен фикс:

                    if (usersIPs.get(username).containsKey(ip)) {
                        int countUpdate = usersIPs.get(username).get(ip) + 1;
                        usersIPs.get(username).put(ip, countUpdate);
                    } else {
                        usersIPs.get(username).put(ip, 1);
                    }

Note: всички видове Map-ве гарантират уникалност на ключовете.

22/03/2017 19:50:49 22/03/2017 19:52:09

MarinaShortyMarinova 12 Точки

Ох, да, благодаря, видях си грешката, иначе първоначалната ми логика беше точно този фикс, който си ми дал, но нещо ми гърмеше и не успях да го догодя да заработи. Иначе, честно казано не бях съвсем убедена защо мапът ми изтрива всички запазени ip-та, всъщност си мислех, че просто ще ми добави ново entry в treemap-а, а не че ще ми ги изтрие всичките. Разбрах сега каква е логиката му. Благодаря

22/03/2017 22:58:18

IvoVencislavov 0 Точки

Прикачвам едно решение 100/100 за следващите, които тази задача ще ги мъчи. ;)

https://pastebin.com/6UbBSih4

16/09/2021 07:55:41