Почему не получается создать Hash Map c нужной информацией?

Question

[Maks Burkov]

Как правильно добавить данные в Hash Map ?
Почему добавляется только один и тот же ключ ?

@Before
    public void createCoupons() {
        System.out.println("Setting coupons data! ");

        Coupon coupon1 = companyServiceTesting.createCompanyCoupon(setOfCompanies.get("company1"), callForCoupon(), "Travelling low prices", "FamilyTravelling",
                23, 765475475, "C://ProgramFiles", 35.8, CouponType.TRAVELING, CouponSystemDate.dateFormat(2017, 12, 12)
                , CouponSystemDate.dateFormat(2019, 12, 12));

        Coupon coupon2 = companyServiceTesting.createCompanyCoupon(setOfCompanies.get("company2"), callForCoupon(), "Camping low prices", "FamilyCamping",
                13, 9999887, "C://ProgramFiles", 55.8, CouponType.CAMPING, CouponSystemDate.dateFormat(2016, 10, 10)
                , CouponSystemDate.dateFormat(2018, 9, 1));

        Coupon coupon3 = companyServiceTesting.createCompanyCoupon(setOfCompanies.get("company1"), callForCoupon(), "Electricity low prices", "Electricity",
                12, 5424352, "C://ProgramFiles", 25.8, CouponType.ELECTRICITY, CouponSystemDate.dateFormat(2017, 7, 3)
                , CouponSystemDate.dateFormat(2018, 4, 5));

        Coupon coupon4 = companyServiceTesting.createCompanyCoupon(setOfCompanies.get("company2"), callForCoupon(), "Food low prices", "FamilyDinner",
                53, 989873, "C://ProgramFiles", 15.8, CouponType.FOOD, CouponSystemDate.dateFormat(2017, 4, 11)
                , CouponSystemDate.dateFormat(2018, 4, 11));

        Coupon coupon5 = companyServiceTesting.createCompanyCoupon(setOfCompanies.get("company1"), callForCoupon(), "Health insurance low prices", "FamilyInsurance",
                83, 33344455, "C://ProgramFiles", 88.8, CouponType.HEALTH, CouponSystemDate.dateFormat(2017, 8, 9)
                , CouponSystemDate.dateFormat(2018, 8, 9));


        Coupon coupon6 = companyServiceTesting.createCompanyCoupon(setOfCompanies.get("company2"), callForCoupon(), "Sport village low prices", "SportVacation",
                73, 6665432, "C://ProgramFiles", 95.8, CouponType.SPORTS, CouponSystemDate.dateFormat(2017, 7, 11)
                , CouponSystemDate.dateFormat(2019, 7, 11));

        setInCouponCollection("coupon1", coupon1);
        setInCouponCollection("coupon2", coupon2);
        setInCouponCollection("coupon3", coupon3);
        setInCouponCollection("coupon4", coupon4);
        setInCouponCollection("coupon5", coupon5);
        setInCouponCollection("coupon6", coupon6);

    }

public Map<String, Coupon> setInCouponCollection(String key, Coupon coupon) {
        if (setOfCoupons == null) {
            setOfCoupons = new HashMap<>();
        }
        setOfCoupons.put(key, coupon);
        System.out.println("Added object key: " + setOfCoupons.entrySet().iterator().next().getKey() + " added object value: " + setOfCoupons.get(key));
        return setOfCoupons;
    }

Отладка:

Setting coupons data! 
Added object key: coupon1 added object value: Coupon [id=6665432, title=SportVacation, startDate=2017-07-11, endDate=2019-07-11, amount=73, message=Sport village low prices, price=95.8, image=C://ProgramFiles, type=SPORTS]
Added object key: coupon1 added object value: Coupon [id=6665432, title=SportVacation, startDate=2017-07-11, endDate=2019-07-11, amount=73, message=Sport village low prices, price=95.8, image=C://ProgramFiles, type=SPORTS]
Added object key: coupon1 added object value: Coupon [id=6665432, title=SportVacation, startDate=2017-07-11, endDate=2019-07-11, amount=73, message=Sport village low prices, price=95.8, image=C://ProgramFiles, type=SPORTS]
Added object key: coupon1 added object value: Coupon [id=6665432, title=SportVacation, startDate=2017-07-11, endDate=2019-07-11, amount=73, message=Sport village low prices, price=95.8, image=C://ProgramFiles, type=SPORTS]
Added object key: coupon1 added object value: Coupon [id=6665432, title=SportVacation, startDate=2017-07-11, endDate=2019-07-11, amount=73, message=Sport village low prices, price=95.8, image=C://ProgramFiles, type=SPORTS]
Added object key: coupon1 added object value: Coupon [id=6665432, title=SportVacation, startDate=2017-07-11, endDate=2019-07-11, amount=73, message=Sport village low prices, price=95.8, image=C://ProgramFiles, type=SPORTS]

Answer 1

[Денис Загаевский]

Потому что неправильно пишешь в консоль.
setOfCoupons.entrySet().iterator().next().getKey()
entrySet - это множество ключей, и ты берешь от него итератор - перечислитель, берешь всегда первый элемент и берешь его ключ. Нужно пройти по множеству полностью, например так:

for (Map.Entry<String, Coupon> entry: setOfCoupons.entrySet()){
    print(entry.getKey() + " ->" + entry.getValue());
}

Кстати, ходить по хешмапе итератором - это так себе занятие, потому что порядок не гарантирован. Если хочется это делать, лучше использовать LinkedHashMap.

Comments

[Дмитрий Александров]

Уже в предыдущей теме разжевывали ему.

[Денис Загаевский]

Дмитрий Александров: печально, потратил время

[Maks Burkov]

Дмитрий Александров: Вы нечего не разжевали мне ..
Для вас то как вы объяснили может понятно и то что вы называете разжеванием.
К сожалению дима потратил время согласен.

[Maks Burkov]

Результат от проверки для купонов . После добавки всех .

Checking the setted data! 
Key: coupon1 Value: Coupon [id=6665432, title=SportVacation, startDate=2017-07-11, endDate=2019-07-11, amount=73, message=Sport village low prices, price=95.8, image=C://ProgramFiles, type=SPORTS]
Key: coupon3 Value: Coupon [id=6665432, title=SportVacation, startDate=2017-07-11, endDate=2019-07-11, amount=73, message=Sport village low prices, price=95.8, image=C://ProgramFiles, type=SPORTS]
Key: coupon5 Value: Coupon [id=6665432, title=SportVacation, startDate=2017-07-11, endDate=2019-07-11, amount=73, message=Sport village low prices, price=95.8, image=C://ProgramFiles, type=SPORTS]

@AfterClass
    public static void checkTheSetedObjects() {
        System.out.println("Checking the setted data! ");
        Iterator<Map.Entry<String, Coupon>> iterator3 = setOfCoupons.entrySet().iterator();
        while(iterator3.hasNext()){
           String key =  iterator3.next().getKey();
           Coupon value = iterator3.next().getValue();
           System.out.println("Key: "+key+" Value: "+value);
        }
}

[Денис Загаевский]

Maks Burkov: Т_Т
я же дал тебе рабочий код. По итератору так не ходят. Положи туда нечетное количество записей - оно вообще упадёт. Потому что next() каждый раз сдвигает итератор вперед. Правильный код(разворот for-each по факту):

Iterator<Map.Entry<String, Coupon>> iterator3 = setOfCoupons.entrySet().iterator();
        while(iterator3.hasNext()){
           Map.Entry<String, Coupon> entry = iterator3.next();
           String key =  entry.getKey();
           Coupon value = entry.getValue();
           System.out.println("Key: "+key+" Value: "+value);
        }

и по сути это полный аналог кода, который есть в моём ответе.

[Maks Burkov]

Денис Загаевский: Ну так ..
По итератору так не ходят или это полный аналог того что ты дал в ответе и что называеш "правильным кодом" ? В любом случаи СПС за потраченное время!

[Денис Загаевский]

Maks Burkov: Чего?
"так не ходят" - это значит, что каждый вызов next() должен предваряться вызовом hasNext(). Нельзя делать next() два раза подряд, потому что исключение будет выброшено.
"полный аналог" - то, что я написал в предыдущем комментарии, это полный аналог того, что в ответе. for - это синтаксический сахар для такого цикла.
"Правильный код" - это код, который делает то, что задумано и не кидает исключений, когда не надо. Это тот код, который я привёл.
Ещё раз подчеркну - код, который привёл я, совсем не аналогичен коду, который у тебя.
Для СПС есть "отметить решением" и "нравится".

Answer 2

[Дмитрий Александров]

вам той темы мало было.
1) setOfCoupons.entrySet().iterator().next().getKey() не вернет то что вы думаете в данной ситуации, будет всегда возвращать только первый элемент.

Comments

[Maks Burkov]

После той темы я изменил код .. На ваш и работало также всё с проблемами . Незнаю где вы там что разжевали . Тут я его изменил.

[Дмитрий Александров]

Maks Burkov: сделай вывод через prinln() каждого купона после его создания но перед запихиванием в хэшмап. По типу
Coupon coupon5 = "создал";
system.err.prinln("coupon: "+coupon5.toString());
для каждого купона.

[Maks Burkov]

Дмитрий Александров:

@Before
    public void createCoupons() {
        System.out.println("Setting coupons data! ");

        Coupon coupon1 = companyServiceTesting.createCompanyCoupon(setOfCompanies.get("company1"), callForCoupon(), "Travelling low prices", "FamilyTravelling",
                23, 765475475, "C://ProgramFiles", 35.8, CouponType.TRAVELING, CouponSystemDate.dateFormat(2017, 12, 12)
                , CouponSystemDate.dateFormat(2019, 12, 12));

        Coupon coupon2 = companyServiceTesting.createCompanyCoupon(setOfCompanies.get("company2"), callForCoupon(), "Camping low prices", "FamilyCamping",
                13, 9999887, "C://ProgramFiles", 55.8, CouponType.CAMPING, CouponSystemDate.dateFormat(2016, 10, 10)
                , CouponSystemDate.dateFormat(2018, 9, 1));

        Coupon coupon3 = companyServiceTesting.createCompanyCoupon(setOfCompanies.get("company1"), callForCoupon(), "Electricity low prices", "Electricity",
                12, 5424352, "C://ProgramFiles", 25.8, CouponType.ELECTRICITY, CouponSystemDate.dateFormat(2017, 7, 3)
                , CouponSystemDate.dateFormat(2018, 4, 5));

        Coupon coupon4 = companyServiceTesting.createCompanyCoupon(setOfCompanies.get("company2"), callForCoupon(), "Food low prices", "FamilyDinner",
                53, 989873, "C://ProgramFiles", 15.8, CouponType.FOOD, CouponSystemDate.dateFormat(2017, 4, 11)
                , CouponSystemDate.dateFormat(2018, 4, 11));

        Coupon coupon5 = companyServiceTesting.createCompanyCoupon(setOfCompanies.get("company1"), callForCoupon(), "Health insurance low prices", "FamilyInsurance",
                83, 33344455, "C://ProgramFiles", 88.8, CouponType.HEALTH, CouponSystemDate.dateFormat(2017, 8, 9)
                , CouponSystemDate.dateFormat(2018, 8, 9));


        Coupon coupon6 = companyServiceTesting.createCompanyCoupon(setOfCompanies.get("company2"), callForCoupon(), "Sport village low prices", "SportVacation",
                73, 6665432, "C://ProgramFiles", 95.8, CouponType.SPORTS, CouponSystemDate.dateFormat(2017, 7, 11)
                , CouponSystemDate.dateFormat(2019, 7, 11));

        System.out.println("Coupon1: "+coupon1);
        System.out.println("Coupon2: "+coupon2);

        setInCouponCollection("coupon1", coupon1);
        setInCouponCollection("coupon2", coupon2);
        setInCouponCollection("coupon3", coupon3);
        setInCouponCollection("coupon4", coupon4);
        setInCouponCollection("coupon5", coupon5);
        setInCouponCollection("coupon6", coupon6);

    }

Output:

Coupon1: Coupon [id=6665432, title=SportVacation, startDate=2017-07-11, endDate=2019-07-11, amount=73, message=Sport village low prices, price=95.8, image=C://ProgramFiles, type=SPORTS]
Coupon2: Coupon [id=6665432, title=SportVacation, startDate=2017-07-11, endDate=2019-07-11, amount=73, message=Sport village low prices, price=95.8, image=C://ProgramFiles, type=SPORTS]

[Дмитрий Александров]

Maks Burkov: Проблема точно внутри companyServiceTesting.createCompanyCoupon().
Показывай внутренности метода этого =).

[Дмитрий Александров]

В createCompanyCoupon() ты возвращаешь ссылку на один и тотже объект.
В couponCreator() аналогичная проблема, возвращаете всегда одну и туже ссылку.
Вот и ваши проблемы все.

[Maks Burkov]

Дмитрий Александров:

private static Coupon couponCreator(Company company , Coupon coupon , String message , String title, int amount,
                                        long id , String image , double price , CouponType type  , Date start_date , Date end_date){
        coupon.setCompany(company);
        coupon.setMessage(message);
        coupon.setAmount(amount);
        coupon.setId(id);
        coupon.setType(type);
        coupon.setTitle(title);
        coupon.setImage(image);
        coupon.setPrice(price);
        coupon.setEndDate(end_date);
        coupon.setStartDate(start_date);
        return coupon;
    }

public Coupon createCompanyCoupon(Company company , Coupon coupon , String message , String title, int amount,
                                      long id , String image , double price , CouponType type  , Date start_date , Date end_date) {
        coupon = couponCreator(company, coupon, message, title, amount, id, image, price, type, start_date, end_date);
        return coupon;
    }

[Maks Burkov]

Дмитрий Александров: Отлично спасибо! Теперь разжевали!!

[Дмитрий Александров]

Maks Burkov: Отметь и в той теме вопрос решенным со ссылкой сюда. А то карму и столько терпения сливать бывает сложно.