Ошибка в алгоритме

@Nightwalker · Регистрация: 21.11.2014

Всем доброго времени суток. Есть кусок программы, в общем он проверяет одинаковые элементы в масиве если есть такие до установить в их поля значения null
вот код :

Java

       for (int i = 0; i < toStorageList.size(); i++) {
                    ToStorage toStorage = toStorageList.get(i);
                    for (int m = i + 1; m < toStorageList.size(); m++) {
                        ToStorage toStorage2 = toStorageList.get(m);
                            if (toStorage.getMatchId().equals(toStorage2.getMatchId())) {
                                toStorage.setRuleId(null);
                                toStorage.setMatchId(null);
                                toStorage.setMatchType(null);
                                toStorage2.setRuleId(null);
                                toStorage2.setMatchId(null);
                                toStorage2.setMatchType(null);
                               // тут будет типа saveToDb(toStorage) и saveToDb(toStorage2)
                        }
                    }
                }

в массиве у меня вот такой список
123
111
123
111
123
111
123
я пишу это все в базу данных,и в ней по идее в после выполнения алгоритма везде должно быть Null но в базе у мнея вот так
null
null
null
null
null
111
null
предпоследний элемент не стал Null, подскажите плиз что я не так делаю в алгоритме ???

@Goongala · 21.10.2019, 15:41

пушто ты обнуляешь переменную, с которой сравниваешь все остальные, сразу же, как находишь первое совпадение. Надо сначала проверить массив до конца, а потом обнулять toStorage

@Nightwalker · 21.10.2019, 15:45 **[ТС]**

Я думал в эту сторону, не подскажите как правильно проверить а потом обнулять ??

@Goongala · 21.10.2019, 15:49

Можно так

Java

        boolean found;
        for (int i = 0; i < toStorageList.size(); i++) {
            ToStorage toStorage = toStorageList.get(i);
            found = false;
            for (int m = i + 1; m < toStorageList.size(); m++) {
                ToStorage toStorage2 = toStorageList.get(m);
                if (toStorage.getMatchId().equals(toStorage2.getMatchId())) {
                    found = true;
                    toStorage2.setRuleId(null);
                    toStorage2.setMatchId(null);
                    toStorage2.setMatchType(null);
                    // тут будет типа saveToDb(toStorage) и saveToDb(toStorage2)
                }
            }
            if (found) {
                toStorage.setВсё(null);
            }
        }

@Nightwalker · 21.10.2019, 18:07 **[ТС]**

Спасибо большое насчет флага совсем не подумал!!! Тема закрыта!!!

Добавлено через 1 час 51 минуту
А еще вопрос это хорошо когда все надо обнулять, а как быть когда не все совпали и надо только конкретные поля обнулять??

Добавлено через 5 минут
хотя можно собрать в лист только те обьекты которые совпали и потом их обнулять используя функционал БД)))

Добавлено через 1 минуту
сделал так но теперь последний обьект не обнулился((((

Java

 for (int i = 0; i < toStorageList.size(); i++) {
                    List<String> toStorageId = new ArrayList<>();
 
                    ToStorage toStorage = toStorageList.get(i);
                    for (int m = i + 1; m < toStorageList.size(); m++) {
                        ToStorage toStorage2 = toStorageList.get(m);
                        if (Objects.equals(ofNullable(toStorage.getMatchId()).orElse(null), ofNullable(toStorage2.getMatchId()).orElse(null))) {
                                found = true;
                                toStorageId.add(toStorageList.get(i).getId());
                                toStorage2.setRuleId(null);
                                toStorage2.setMatchId(null);
                                toStorage2.setMatchType(null);
                            }
                    }
                    if (found) {
                        Query queryFields = new Query();
                        queryFields.addCriteria(Criteria.where("_id").in(toStorageId));
                        Update update = new Update();
                        update.set("ruleId", null);
                        update.set("matchId", null);
                        update.set("matchType", null);
                        mongoTemplate.updateMulti(queryFields, update, ToStorage.class);
 
                    }
                }

Добавлено через 13 минут
Я задал не похожее на все значения в предпоследний элемент а последний оставил 123 и он не обнулился

Добавлено через 4 минуты
что то типа
123
111
123
111
123
3213
123

@Nightwalker · 22.10.2019, 09:14 **[ТС]**

В общем решил все ок не надо было обнулять в if ничего и норм там все собрать в лист и обнулять!!

@Goongala 918 / 521 / 174 Регистрация: 18.08.2013 Сообщений: 1,963 Записей в блоге: 2
	21.10.2019, 15:41	2
	пушто ты обнуляешь переменную, с которой сравниваешь все остальные, сразу же, как находишь первое совпадение. Надо сначала проверить массив до конца, а потом обнулять toStorage 1

@Nightwalker 16 / 14 / 4 Регистрация: 21.11.2014 Сообщений: 296
	21.10.2019, 15:45 [ТС]	3
	Я думал в эту сторону, не подскажите как правильно проверить а потом обнулять ?? 0

@Nightwalker 16 / 14 / 4 Регистрация: 21.11.2014 Сообщений: 296
	22.10.2019, 09:14 [ТС]	6
	В общем решил все ок не надо было обнулять в if ничего и норм там все собрать в лист и обнулять!! 0

Опции темы

Ошибка в алгоритме

Решение