В данной процедуре поиск дублей происходит по всем колонкам строк. Тоесть дублем считается повтор значений в двух строках по всем колонкам.
Если повторяющих строк больше одной, то они все будут показаны в сообщении, с порядковым номером исходной строки и повторяющейся строки.
Данную процедура лучше использовать в общем модуле и вызывать её из процедур "ПриЗаписи" или "ОбработкаПроведения", и т.п. То есть в тех, где можно сделать отказ от выполнения дальнейших дествий
В качестве Таблицы — можно указать "ТаблицуЗначений", либо "ТабличнуюЧасть" документа, Справочника и т.д.
// !!! Павел С.С. 2 июня 2011 г. 17:49:14
// Проверка любой таблицы на повторяющиеся строки
// Данную процедура лучше использовать в процедурах ПриЗаписи, ОбработкаПроведения, и т.п. Тоесть в тех, где можно сделать отказ от выполениядальнейших дествий
// Параметры:
// ТЧ — можно указать «ТаблицуЗначений», либо «ТабличнуюЧасть» документа
// Отказ — параметр «Отказ» или «Истина», «Ложь»
Процедура ПоискДублейСтрокВТабЧасти(ТЧ, Отказ) Экспорт
Если Тип(ТЧ) = Тип(«ТаблицаЗначений») Тогда
ТаблЗнач = ТЧ;
Иначе
ТаблЗнач = ТЧ.Выгрузить();
КонецЕсли;
Отбор = Новый Структура();
Для каждого Стр Из ТаблЗнач Цикл
Отбор.Очистить();
Для каждого Колонки Из ТаблЗнач.Колонки Цикл
Если Колонки.Имя <> «НомерСтроки» Тогда
Отбор.Вставить(Колонки.Имя, Стр[Колонки.Имя]);
КонецЕсли;
КонецЦикла;
Строки = ТаблЗнач.НайтиСтроки(Отбор);
Если Строки.Количество() > 1 Тогда
Для каждого НайденныеСтроки Из Строки Цикл
Если Строки.Найти(НайденныеСтроки) > 0 Тогда
СтрокаСообщения = «Строка № » + Строки[0].НомерСтроки + » совпадает со строкой № » + НайденныеСтроки.НомерСтроки;
СообщитьОбОшибке(СтрокаСообщения, Отказ, «Уберите задвоения строк!»);
КонецЕсли;
КонецЦикла;
КонецЕсли;
КонецЦикла;
КонецПроцедуры
Проверка любой таблицы на повторяющиеся строки, с выводом сообщения какие строки задублированны, и режимом «Отказ» для проведения документов.
В данной процедуре поиск дублей происходит по всем колонкам строк. Тоесть дублем считается повтор значений в двух строках по всем колонкам.
Если повторяющих строк больше одной, то они все будут показаны в сообщении, с порядковым номером исходной строки и повторяющейся строки.
Данную процедура лучше использовать в общем модуле и вызывать её из процедур «ПриЗаписи» или «ОбработкаПроведения», и т.п. То есть в тех, где можно сделать отказ от выполнения дальнейших дествий
В качестве Таблицы — можно указать «ТаблицуЗначений», либо «ТабличнуюЧасть» документа, Справочника и т.д.
Перейти к публикации
Думаю, небрежно сделано.
1. В приведенном куске кода последняя команда «Найти» лишняя:
Если Строки.Количество() > 1 Тогда
Для каждого НайденныеСтроки Из Строки Цикл
Если Строки.Найти(НайденныеСтроки) > 0 Тогда
— так как все строки для данного отбора уже найдены, то незачем делать последнее «Найти». Нужно просто вывести номера найденных строк.
2. Данная процедура будет делать отбор для каждой строки, в том числе и для тех, которые дублируют строки, обработанные ранее. То есть, в частности, будет выдаваться несколько сообщений по поводу одних и тех же дублей.
3. Для каждой строки ТЧ идет цикл по колонкам Отбор.Вставить… Вставить все колонки в отбор правильнее перед циклом по строкам ТЧ. А в цикле по строкам — заполнять значения отбора.
4. Отбор делается по строковому представлению, что неправильно. Есть куча примеров, когда объекты разные, но их строковое представление совпадает. Например, наименования номенклатуры совпадают, но разные артикулы.
5. Более того, зачем вообще эти многократные отборы и поиски? Лучше отсортировать таблицу значений по всем колонкам. Тогда все дублирующиеся строки окажутся рядом друг с другом. Останется только один раз пройтись циклом сверху вниз, сравнивая текущую строку со следующей.
(Если п.5 еще можно снять по той причине, что сортировка меняет исходную ТЧ — что нежелательно — то с п.п.1-4 вряд ли можно спорить)
Так что, наверно, «минус».
на работоспособность не проверял, особо сильно не вникал, но за интересную мысль плюс, сложил в «судук», авось пригодится.
К тому же обработка отлавливает только АБСОЛЮТНЫХ двойников! Если например, ТЧ Товары содержит две строки с одинаковыми товарами, характеристиками, сериями и т.п., но с разным количеством по строкам, то такие строки не будут дублями….?
А если ТЧ документа устанавливающего свойства, и в одной строке устанавливается Истина, а вдругой Ложь? Это ошибочное состояние документа не будет распознано!!!
Сыровато, недодлано, крайне ограниченное применение идеи как таковой!
Минус пока..
Для дальнейшего развития, так сказать, если твоя табличная часть будет довольно большая, не плохо было использовать индексы и исключить из проверки колонки с числовым типом.
Ну и вообщем — метод типовой. Есть куда более интересные и более продуктивные решения.
Если кому не нравиться, можете не пользоваться, или пишите свои, но для начинающих программеров как пример вполне может сойти. А вы типа продвинутые программеры, если такие умные возьмите и выложите свою реализацию данного примера. А то умничать все крутые, а взять написать и выложить слабо. Так что …
(6)
Не кипи. Ты не в детском саду и здесь не будут радостно хлопать в ладоши только потому, что ты ЧТО-ТО сделал. Скопируют твой код люди не глядючи и кое-кто получит проблему на свою голову. Поэтому и указывают на недостатки в коде. А минус как предупреждение, чтобы хоть в комментарии заглянули отчего так.
Не кипи. Ты не в детском саду и здесь не будут радостно хлопать в ладоши только потому, что ты ЧТО-ТО сделал. Скопируют твой код люди не глядючи и кое-кто получит проблему на свою голову. Поэтому и указывают на недостатки в коде. А минус как предупреждение, чтобы хоть в комментарии заглянули отчего так.
глуп, тот программист, что просто тупо копирует чужой код, не разобравшись в нем. 🙂
Думаю что колонка Номерстроки есть не во всех таблиц значений.
Проще всего создать временную таблицу, поместить туда таблицу значений + 1 колонка с цифрой 1 Свернуть всё это просуммировав колонку с цифрой 1 и отбор на итог >1. В результате получим таблицу дублей. При желании можно настроить не на все колонки, а только определенные.
Проще всего создать временную таблицу, поместить туда таблицу значений + 1 колонка с цифрой 1 Свернуть всё это просуммировав колонку с цифрой 1 и отбор на итог >1. В результате получим таблицу дублей. При желании можно настроить не на все колонки, а только определенные.
да согласен, тоже как вариант., единственное, что так ты не сможешь сообщить какие строки одинаковые, и пользователю придется искать по наименованию, а если в табл. 500 строк то это увеличит время поиска.
Думаю что колонка Номерстроки есть не во всех таблиц значений.
если я не ошибаюсь, то ее может не быть только в том случе, если таб. ты создаешь сам кодом. А если это табл. часть док. то там эта колонка есть всегда.
Согласен с (1) — небрежно.
1. Проверка в самом начале:
Если Тип(ТЧ) = Тип(«ТаблицаЗначений») Тогда
ТаблЗнач = ТЧ;
Иначе
ТаблЗнач = ТЧ.Выгрузить();
КонецЕсли;
В комментарии к функции описано, что передаваться может или таблица значений или табличная часть документа.
Проверка на таблицу значений есть, а проверки на табличную часть нет. Непорядок.
Т.е. если мы передадим в параметре ТЧ значение типа СписокЗначений (в нем же тоже можно дубли поискать), то завал обеспечен.
2. Не помню где, но сами 1С-овцы говорили, что при обработке таблиц значений вместо метода НайтиСтроки лучше использовать построитель запросов с соответствующим источником — и быстрее, и надежнее, и возможностей больше.
3. В дополнение к (1) о формировании отбора перед основным циклом добавлю, что заполнение такого отбора в основном цикле лучше производить методом ЗаполнитьЗначенияСвойств(…) — просто, быстро, надежно.
(6) «Слабо/не слабо» — не аргумент и не ответ на критику. А критика реализации, на мой взгдяд, вполне конструктивная. И если учесть все то, что здесь говорилось, то можно написать очень даже неплохой и эффективный обработчик.
(11) Ваш вариант сработает только в том случае, если все колонки в ТЗ типизированы.
А данная реализация претендует на универсальность.
(12) Можно пойти дальше и отобранные строки дубль соединить с исходной таблицей по полной — тогда можно получить номера строк.
Признаяюсь, код не был доведен до ума, по причине загруженности, ну и в моем случае просто не нужна была универсальность. Поэтому благодарю всех за обсуждение, ну а если кто то хочет предложить что то еще, то пожалуйста. Ну и может либо я сам, либо кто нить другой учтет все высказывания и выложит реально универсальный алгоритм, может кому и пригодится.
Знания бесценны и ими надо делиться, чтобы они совершенствовались — это так от себя 🙂
(0) Будем каждый программный «чих» сюда выкладывать? 🙁
Присоединяюсь к предудыщим словам — слабо, не все проверено, медленно, без ключевых полей дубли искать не смысла и т.п.
Минусую!
(12)
единственное, что так ты не сможешь сообщить какие строки одинаковые, и пользователю придется искать по наименованию, а если в табл. 500 строк то это увеличит время поиска.
А кто мешает сделать еще один запрос к полученной таблице дублей и присоединить таблицу наименований?
и все это на сервере.
+(18) вот простой и не универсальный, но рабочий код
Жесткий вариант конечно, юзать цикл.
Не проще так?)
Для каждого Строка Из Тз Цикл
НайденныеСтроки = Тз.НайтиСтроки(Новый Структура(«Сотрудник», Строка.Сотрудник));
Если НайденныеСтроки.Количество()>1 Тогда
ОбщегоНазначения.СообщитьОбОшибке(«По сотруднику «+Строка.Сотрудник+» имеются дубли!»);
КонецЕсли;
КонецЦикла;
Здесь идет поиск дублей по 1 полю, но нет проблем в структуру добавить все поля.
Спасибо за обработку, молодец! У меня вопрос появился, быть может я не сталкивался с такой потребностью, но для чего такая обработка, если можно всё содержимое в ТаблицеЗначений (или её клоне) при сворачивании определить точное количество дублей, всего «пара» строк, м?
(21) freebsdd, нужно было обрабатывать в нескольких видов документов табличные части и сообщить пользователю какая позиция в таблице является дублем, т.к. табличная часть могла быть от 10 до 300 позиций. А время работы запроса не значительно бы уменьшило время ожидания, а то и вообще не был бы эффективен.
(18) — верно только в случае для проверки документов «записанных», а если проверять из текущего документа — не будут проверены только что добавленные строки.
минус поставил не из вредности, а для того чтобы обратили внимание.
и не тратили свое драгоценное время.
Аналогичная функция: Удаление дублей из Таблицы значений:
Показать
Товары.Свернуть(«Номенклатура»,»Количество»); и ВСЕ 🙂
Вот тоже придумал,
Показать
Гораздо быстрее удалить дубли в таблице, по моему мнению, при помощи запроса |ВЫБРАТЬ РАЗЛИЧНЫЕ
Показать
Так ведь у меня несколько колонок, не одна. И «Выбрать Различные» не будет работать на строках неограниченной длины.
По крайней мере, сравнивая ТЗ со справочником по реквизиту «Строка неограниченной длины» выдавало ошибку «нельзя сравнивать поля бла-бла несовместимые типы»
(30) Bublik2011, разумеется. Вот между прочим, в посте (28) вероятно вместо
надо наверное
У меня синтакс контроль браузера ошибку нашел. Не знаю, может, неправильно.
(29) SkyLink2012,
{Форма.Форма.Форма(26)}: Ошибка при вызове метода контекста (Выполнить)
Результат = Запрос.Выполнить().Выгрузить();
по причине:
{(6, 5)}: Тип не может быть выбран в запросе
<<?>>&ТЗ1 КАК ТЗ1
Взяла за основу код из статьи и убрала дублирование сообщений:
Показать
Как пишет автор, вдруг пригодится новичкам 🙂
А так например можно просто проверить — есть дубли или нет:
Подпишусь, интересно знать все способы, но если бы кто-то оформил все способы в статью — цены б этому автору не было)
Из моих старых закромов, оформил.
Проверка таблицы на дублирование строк
(34) Еще доработал:
Показать
Находит дубли по определенным полям.
(38) Или по всем полям (кроме служебных), если конкретные поля не указаны:
Показать
(39) Если нет поля «НомерСтроки», то можно в начале функции продублировать индекс ТЗ в отдельное (дополнительное) поле и вернуть массив индексов с дублями. При этом ТЗ передавать по значению, а неизмененный источник уже потом кроить. Например, чтобы удалить дубли в источнике по полученным индексам, можно добавить поле ПризнакУдаления, заполнить его по индексам, а потом удалить помеченные строки с последующим удалением добавленного поля. Можно просто построчно перенести источник в новую ТЗ, исключая дублированные строки. В моем случае колхозить надо было с ТЧ и просто информировать юзера о проблемах.
(26) Я так понял что это удаление дублей из одного документа с таб доком? А как , вот есть несколько доков и в каждом есть таб док, и чтобы удаляло дубли во всех где они встречаются или выдавало ошибку. При сохранении документа или проведении
(41) , Да мое решение тебе не подойдет. Поэтому решение не дам, нужно пробовать, но самым лучшим вариантом в данном случае будет (как здесь писали) через запрос. К тому же не до конца понятно, что ты хочешь сделать. Что за документы в которых ты хочешь удалить дубли, уже проведенные или это новый документ в отношении старых?
(35) Или так
Отбор = Новый Структура(«Номенклатура»);
Отбор.Вставить(«Номенклатура»,СтрокаТаблицы.Номенклатура);
НайденныеСтроки = Параметры.ТаблицаТовары.НайтиСтроки(Отбор);
Если НайденныеСтроки.Количество()>1 Тогда
КонецЕсли;
В моем случае, при обходе табличной части, мне нужно было выполнять определенные условия, если уже был или будет дальше дубль этой строки.