Форум программистов, компьютерный форум, киберфорум
C# .NET
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.76/21: Рейтинг темы: голосов - 21, средняя оценка - 4.76
28 / 28 / 11
Регистрация: 08.08.2011
Сообщений: 1,173

Вычитание списков, как ускорить код

19.06.2012, 17:45. Показов 4264. Ответов 14
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Есть два списка: GarList и KonList. В каждом списке хранятся объекты "Document", у каждого документа есть поле "ID" формата string. Необходимо сравнить два списка по их параметру ID. Таким образом нужно построить список NoKon - документы, которые есть в GarList, но нет в KonList.

Следующий код работает очень медленно (порядка 20 секунд при сравнении двух списков по 5 тыс. документов):


C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
            foreach (Document g in GarList)
            {
                string ID = g.ID;
                bool err = true;
 
                foreach (Document k in KonList)
                {
                    if (k.ID == ID)
                        err = false;
                }
                if (err)
                {
                    NoKon.Add(g);
                    cNoKon++;
                }
            }
Как его ускорить?

Добавлено через 4 минуты
Один важный момент:
в каждом из списков есть только один документ с данным ID.

Добавлено через 13 минут
Проект на 2.0, то есть LINQ нельзя использовать.
0
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
19.06.2012, 17:45
Ответы с готовыми решениями:

Как ускорить код?
var subs = Database.Subscription; var ss = !subType ? subs.Where(s => !s.SubscriptionType) :...

Как ускорить код ?
тут были уже вопросы на эту тему но я все же хотел узнать как ускорить мой код в задаче Ириска весит X грамм, мандарин – Y...

Как ускорить код?
Есть код : #include <cstdio> #include <cstring> #include <memory.h> #include <iostream> using namespace std; const int N =...

14
Эксперт .NET
 Аватар для kolorotur
17823 / 12973 / 3382
Регистрация: 17.09.2011
Сообщений: 21,261
19.06.2012, 17:45
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
class DocumentComparer : IEqualityComparer<Document>
{
   public bool Equals(Document x, Document y)
   {
      return x.ID == y.ID;
   }
 
   public int GetHashCode(Document obj)
   {
      return obj.ID;
   }
}
 
List<Document> NoKon = GarList.Except(KonList, new DocumentComparer()).ToList();
1
28 / 28 / 11
Регистрация: 08.08.2011
Сообщений: 1,173
19.06.2012, 18:01  [ТС]
kolorotur, строку

return obj.ID;

подчеркивает и пишет ошибку: "Неявное преобразование типа "string" в "int" невозможно"

И еще, проект под 2.0. Ругается на GarList.Except. Не знает такого метода.

Добавлено через 12 минут
Попробовал вот так сделать (получение третьего списка путем удаления части GarList)

C#
1
2
3
4
5
6
7
8
9
            foreach (Document k in KonList)
            {
                string ID = k.ID;
 
                GarList.RemoveAll(w => (w.ID == ID));
 
            }
 
            NoKon = GarList;
Но тоже медленно работает. Скорость не изменилась.
0
Эксперт .NET
 Аватар для kolorotur
17823 / 12973 / 3382
Регистрация: 17.09.2011
Сообщений: 21,261
19.06.2012, 18:06
Цитата Сообщение от Suppir Посмотреть сообщение
kolorotur, строку
return obj.ID;
подчеркивает и пишет ошибку: "Неявное преобразование типа "string" в "int" невозможно"
Ага, там ошибка - должно быть obj.ID.GetHashCode();

Цитата Сообщение от Suppir Посмотреть сообщение
проект под 2.0. Ругается на GarList.Except.
Ваше дополнение о .NET 2.0 появилось уже после того, как я ответ отправил.

Не помню, есть ли в двойке генерики, потому вот пример, предполагающий, что их там нет:
C#
1
2
3
4
5
6
7
8
9
10
11
12
class DocumentComparer : IEqualityComparer
{
   public bool Equals(object x, object y)
   {
      return ((Document)x).ID == ((Document)y).ID;
   }
 
   public int GetHashCode(object obj)
   {
      return ((Temp)obj).ID.GetHashCode();
   }
}
Создание списка:
C#
1
2
3
4
5
6
Hashtable hash = new Hashtable(KonList.Count, new DocumentComparer());
foreach (Document doc in KonList) hash.Add(doc, doc);
 
ArrayList NoKon = new ArrayList();
foreach (Document doc in GarList)
   if (!hash.Contains(doc)) NoKon.Add(doc);
1
19.06.2012, 19:44

Не по теме:

Цитата Сообщение от kolorotur Посмотреть сообщение
Не помню, есть ли в двойке генерики
есть.

0
28 / 28 / 11
Регистрация: 08.08.2011
Сообщений: 1,173
20.06.2012, 12:13  [ТС]
kolorotur, на строке
C#
1
return ((Temp)obj).ID.GetHashCode();
выдает ошибку:

Code
1
Не удалось найти имя типа или пространства имен "Temp" (пропущена директива using или ссылка на сборку?)
0
Эксперт .NET
 Аватар для kolorotur
17823 / 12973 / 3382
Регистрация: 17.09.2011
Сообщений: 21,261
20.06.2012, 12:23
Ну как обычно: думал об одном, а писал другое.
Замените Temp на Document.
0
28 / 28 / 11
Регистрация: 08.08.2011
Сообщений: 1,173
20.06.2012, 12:27  [ТС]
Сделал. Теперь на строке

C#
1
Hashtable hash = new Hashtable(KonList.Count, new DocumentComparer());
пишет:

Code
1
2
Ошибка    1   Наиболее подходящий перегруженный метод для "System.Collections.Hashtable.Hashtable(int, System.Collections.IEqualityComparer)" имеет несколько недопустимых аргументов
Ошибка    2   Аргумент "2": преобразование типа из "Comparator_2011.DocumentComparer" в "System.Collections.IEqualityComparer" невозможно
0
Эксперт .NET
 Аватар для kolorotur
17823 / 12973 / 3382
Регистрация: 17.09.2011
Сообщений: 21,261
20.06.2012, 12:47
Цитата Сообщение от Suppir Посмотреть сообщение
Аргумент "2": преобразование типа из "Comparator_2011.DocumentComparer" в "System.Collections.IEqualityCompare r" невозможно
Покажите код класса DocumentComparer как он у вас в программе выглядит.
0
28 / 28 / 11
Регистрация: 08.08.2011
Сообщений: 1,173
20.06.2012, 13:01  [ТС]
Цитата Сообщение от kolorotur Посмотреть сообщение
Покажите код класса DocumentComparer как он у вас в программе выглядит.
Добавлено через 2 минуты
Я понял. Нужно написать не "class DocumentComparer", а "class DocumentComparer : IEqualityComparer".

--------------


Отлично! Теперь сравнение списков проходит в 10 раз быстрее! (вместо 20 сек. - 2 сек.)

Добавлено через 8 минут
Только теперь на строке
C#
1
        public bool Equals(object x, object y)
выдает предупреждение

Code
1
Предупреждение    1   "Comparator_2011.DocumentComparer.Equals(object, object)" скрывает наследуемый член "object.Equals(object, object)". Если предполагается скрытие, используйте ключевое слово new.
0
Эксперт .NET
 Аватар для kolorotur
17823 / 12973 / 3382
Регистрация: 17.09.2011
Сообщений: 21,261
20.06.2012, 13:26
Цитата Сообщение от Suppir Посмотреть сообщение
выдает предупреждение
Это ничего страшного, если вы класс DocumentComparer не собираетесь использовать в другом контексте.
Чтобы "заткнуть" компилятор, можно метод Equals реализовать явно:
C#
1
2
3
4
bool IEqualityComparer.Equals(object x, object y)
{
   // ...
}
Цитата Сообщение от Suppir Посмотреть сообщение
вместо 20 сек. - 2 сек.
Многовато для 10000 элементов - должно быть порядка пары милисекунд.
Чем замеряется время и с какого момента начинается замер?
1
28 / 28 / 11
Регистрация: 08.08.2011
Сообщений: 1,173
20.06.2012, 14:25  [ТС]
Спасибо! Теперь не ругается.
0
Эксперт .NET
 Аватар для kolorotur
17823 / 12973 / 3382
Регистрация: 17.09.2011
Сообщений: 21,261
20.06.2012, 16:02
Чем замеряется время и с какого момента начинается замер?
0
28 / 28 / 11
Регистрация: 08.08.2011
Сообщений: 1,173
20.06.2012, 16:18  [ТС]
Время замеряю на глазок. Но в этой функции еще несколько операций проходит. Возможно, они тормозят процесс. Потестирую код с таймером.
0
Эксперт .NET
 Аватар для kolorotur
17823 / 12973 / 3382
Регистрация: 17.09.2011
Сообщений: 21,261
20.06.2012, 18:39
C#
1
2
3
4
5
6
7
8
9
10
11
12
System.Diagnostics.Stopwatch stopwatch = new System.Diagnostics.Stopwatch();
 
stopwatch.Start();
Hashtable hash = new Hashtable(KonList.Count, new DocumentComparer());
foreach (Document doc in KonList) hash.Add(doc, doc);
 
ArrayList NoKon = new ArrayList();
foreach (Document doc in GarList)
   if (!hash.Contains(doc)) NoKon.Add(doc);
 
stopwatch.Stop();
Console.WriteLine(stopwatch.Elapsed);
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
20.06.2012, 18:39
Помогаю со студенческими работами здесь

Как ускорить код?
string ttt = &quot; &quot;; for (int i = 0; i &lt; text.Length; i++) { for(int j=0; j&lt;nt.Count; j++) ...

Вычитание списков в десятичной системе счисления
Вычитание списков в десятичной системе счисления - Lisp

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

Вычитание списков в десятичной системе счисления столбиком
Добрый вечер. Требуется реализовать вычитание списков в десятичной системе счисления столбиком. Общий принцип вычитания понятен: на вход...

Как ускорить код (VBA Excel)?
Привет. Я тут написал код к простой програмке. Суть такая: выскакивает форма 4 опшен-кнопок (верх, низ, право, лево) и кнопка...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
15
Ответ Создать тему
Новые блоги и статьи
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров
8Observer8 12.02.2026
Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 11.02.2026
Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru