nameAndAmoun никъде не го зареждаш със стойност, преди да го пъхнеш в orders на 16^ти ред.

2-та ключа bananas и apples сочат към един и същи обект - nameAndAmoun. Прави TreeMap-а вътре в цикъла - така ще създаваш (и слагаш в orders) нов TreeMap всеки път... което реално искаш да направиш. А дали е най-добрият начин - това е друг въпрос :)

Ето нещо примерно което написах набързо, но трябва да се подобри, най-малкото да се сортират.

GitHub

Здравей. ЕТО едно решение на задачата от мен надявам се да ти помогне. Успех!

Здравей,

Първо това е моето решение. (без да го мисля много)

При теб първо трябва вътре във for да направиш обект TreeMap, Тъй като при всяка интерация nameAndAmoun сочи към един и същ обект ( тъй е към един и същ блок от Heap-а).

С прости думи и за bananas и apples ти добавяш един и същ обект nameAndAmoun(референцията е една и съща). Затова вътре във for трябва да създадеш TreeMap и него да го предадеш към HasMap. Когато напишеш TreeMap<String, Integer> nameAndAmoun = new TreeMap<String, Integer>();, това означава нека nameAndAmoun да сочи към обект с определен адрес в памета. Той винаги ще сочи към него докато не му присвоиш друг адрес (например nameAndAmoun = new TreeMap<String, Integer>();). В този случай се заделя нава памет в Heap-а и nameAndAmoun сочи към нея. От тук приизлиза и втория вариант.

2 Вариант: Вътре в for-а да промениш референцита на nameAndAmoun . По следния начин nameAndAmoun = new TreeMap<String, Integer>(); и след това да му добавяш данни.

Също така трябва да проверяваш данните за да се избегне дублирането на информация.

Не съм обяснил как се решава проблема, тъй като въпросът е друг а и нямам много време, но се надявам да съм бил полезен и ясен в обясненията.