Ускорить алгоритм

@leva · Регистрация: 12.04.2012

Студворк — интернет-сервис помощи студентам

Есть код который сохраняет строку из StringList в 2-ой StringList, если этой строки нет в 3-ий StringList ...

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
unsigned __stdcall OST( void* Param )
{
std::auto_ptr<TStringList> SaveOst(new TStringList);
AnsiString ls=lst2->Text;
AnsiString a1="";
for (int i = 0; i < lst->Count; i++) {
count++;
 a1=lst->Strings[i];
   if(ls.Pos(a1)==0) SaveOst->Add(lst->Strings[i]);
 
}
 
SaveOst->SaveToFile("Ostatok.txt");
_endthreadex(0);
 return 0;
}

Работает не достаточно быстро, как можно эффективно изменить алгоритм для ускорения программы...

@BRcr · 01.07.2014, 01:39

Сообщение от leva

и я понимаю что это не самый лучший вариант...

На несколько порядков не лучший, но хозяин - барин... Заменить пять буковок кода - это, конечно же, излишние заморочки. ))
Кстати, о сортировке изначально не было сказано ни слова.

@kzru_hunter · 02.07.2014, 17:00

Сообщение от BRcr

vector < int > v1( size ), v2( size ), v3( size ), v4( size );

Заметил ошибку.

Нужно так:

Сообщение от BRcr

vector < int > v1( size ), v2( size ), v3, v4;

У меня так std::inserter на 0.1-0.5 мс быстрее отрабатывает

@Retyrn0 · 03.07.2014, 01:34

Что серьёзно? Никого кроме меня не смущает

Сообщение от leva

SaveOst->SaveToFile("Ostatok.txt");

?

@BRcr · 03.07.2014, 15:00

Сообщение от kzru_hunter

Заметил ошибку.

Все верно, моя ошибка.

Погонял на разных диапазонах - back_inserter у меня отрабатывает в среднем на 5 - 10 % быстрее.

Добавлено через 3 минуты
Retyrn0, сохранение происходит один раз по завершении выборки. Сохраняется стандартным методом класса TStringList. Так что нет - никого кроме тебя это не смущает.

Если есть желание сравнить скорости записи в файл различными способами - милости просим.

@Avazart · 03.07.2014, 15:10

Сообщение от BRcr

Погонял на разных диапазонах - back_inserter у меня отрабатывает в среднем на 5 - 10 % быстрее.

Ну оценивать в % не корректно, ибо разница будет тем больше чем больше элементов вставляться.
Кроме того логично было бы делать reserve() что бы исключить перераспределение памяти.

@Retyrn0 · 03.07.2014, 15:11

Сообщение от BRcr

сохранение происходит один раз по завершении выборки

Ну я не видел весь код, а ты? Если предположить что функция вызывается внутри другого цикла...мне трудно представить такой длинный список, чтобы ТС не хватало скорости...

@Avazart · 03.07.2014, 15:23

Ну логичнее было сохранять с помощью fstream,
тогда можно было бы с экономить время на излишнем преобразование std::vector<String> -> TStringList

@BRcr · 03.07.2014, 15:51

Сообщение от Retyrn0

Если предположить что функция вызывается внутри другого цикла...мне трудно представить такой длинный список, чтобы ТС не хватало скорости...

Зачем такое предполагать? На это нет оснований. Спроси у ТС - и всех делов.
А насчет длины списка и скорости - у ТС может быть малопроизводительная система либо очень большие объемы текста. В обоих случаях даже самый оптимизированный алгоритм может работать относительно долго.

Добавлено через 3 минуты
Avazart, провел тесты. Опять смешные результаты.

При увеличении количества элементов в векторах разница скорости уменьшается. При этом использование reserve никаких заметных изменений в картину не вносит. Парадокс какой-то...

@Avazart · 03.07.2014, 19:37

http://microfork.com/stdvector... -analisys/

Пример для понимания разницы

Код

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
#include <iostream>
#include <vector>
#include <string>
 
#include <iostream>
#include <vector>
#include <string>
 
class test
{
    public:
        test(const int n):n_(n)
        {
            std::cout<<"constructor "<<n_<<std::endl;
        };
 
        test(const test& some)
        {
           std::cout<<"copy constructor "<<some.n_<<std::endl;
           n_= some.n_;
        }
 
        test& operator=(const test& some)
        {
            std::cout<<"assign "<<n_<<"="<<some.n_<<std::endl;
            n_= some.n_;
            return *this;
        }
 
        ~test()
        {
            std::cout<<"destructor "<<n_<<std::endl;
        };
 
    private:
        int n_;
};
 
int main()
{
    std::vector<test> vtest1;
    std::vector<test> vtest2;
 
    const size_t size= 4;
    vtest1.reserve(size);
    vtest2.reserve(size);
 
    std::cout<<std::endl<<"insert():"<<std::endl;
    for(size_t i=0; i<size; ++i)
    {
        std::cout<<i<<")"<<std::endl;
        vtest1.insert(vtest1.begin(),test(i));
    }
 
    std::cout<<std::endl<<"push_back():"<<std::endl;
    for(size_t i=0; i<size; ++i)
    {
        std::cout<<i<<")"<<std::endl;
        vtest2.push_back(test(i));
    }
 
    getchar();
    return 0;
}

Вывод для МSVС2010

insert():
0)
constructor 0
copy constructor 0
destructor 0
1)
constructor 1
copy constructor 1
copy constructor 1
assign 1=0
assign 0=1
destructor 1
destructor 1
2)
constructor 2
copy constructor 2
copy constructor 0
assign 0=1
assign 1=0
destructor 0
copy constructor 0
assign 0=2
assign 2=0
destructor 0
destructor 2
3)
constructor 3
copy constructor 3
copy constructor 1
assign 1=2
assign 2=1
destructor 1
copy constructor 1
assign 1=3
assign 3=1
destructor 1
copy constructor 1
assign 1=0
assign 0=1
destructor 1
destructor 3

push_back():
0)
constructor 0
copy constructor 0
destructor 0
1)
constructor 1
copy constructor 1
destructor 1
2)
constructor 2
copy constructor 2
destructor 2
3)
constructor 3
copy constructor 3
destructor 3

Вывод для С++BuilderXE3 (x86)

insert():
0)
constructor 0
copy constructor 0
copy constructor 0
destructor 0
destructor 0
1)
constructor 1
copy constructor 1
copy constructor 0
assign 0=1
destructor 1
destructor 1
2)
constructor 2
copy constructor 2
copy constructor 0
assign 0=1
assign 1=2
destructor 2
destructor 2
3)
constructor 3
copy constructor 3
copy constructor 0
assign 0=1
assign 1=2
assign 2=3
destructor 3
destructor 3

push_back():
0)
constructor 0
copy constructor 0
destructor 0
1)
constructor 1
copy constructor 1
destructor 1
2)
constructor 2
copy constructor 2
destructor 2
3)
constructor 3
copy constructor 3
destructor 3

Онлайн компилятор http://codepad.org/qp9FP2b1

Чет с XE3 непонятки, как обычно...

@Retyrn0 · 03.07.2014, 21:09

Не по теме:

Сообщение от BRcr

На это нет оснований

Нет оснований утверждать обратное.

Сообщение от BRcr

Спроси у ТС - и всех делов

Вот тут согласен.

Новые блоги и статьи Все статьи Все блоги /
Вступаем в новую фазу генерации favicons, векторную. russiannick 02.11.2026 Теперь нет нужды скрывать что используется простой svg. Исходный код изображения помещаем в поле и жмем кнопку. Исходник преобразуется в вид корректный для помещения в favicon или в url(). Нажатие. . .	Генератор паролей (вариант №4) Maks 01.11.2026 После публикации 3-го варианта генератора паролей, разослал его своим коллегам (с целью оценки), в итоге были получены следующие рекомендации: 1. Вернуть возможность использования знаков \| 1 l I;. . .	Генератор паролей (вариант №3) Maks 30.10.2026 Финальная версия генератора паролей 1С. Доработка решения в этот раз коснулась только визуальной части, а именно, добавлен реквизит ДопПараметры (тип Булево), который скрывает/ отображает реквизиты:. . .	Магическое мышление и магия, современная наука не отличима от магии (говорит ИИ Джемени) zorxor 30.10.2026 Джемени, расскажи, пожалуйста, что такое магическое мышление, что такое магия, почему современная наука чем-то напоминает магию? Фраза «Любая достаточно развитая технология неотличима от магии». . .
За какое время ИИ может подобрать пароль по хешу (рассказывает Джемени ИИ) zorxor 30.10.2026 Расскажи, Джемени, если известен хэш МД5, например, за сколько ИИ сможет подобрать соответствующий пароль? Искусственный интеллект (ИИ) может подобрать простой или средний по сложности пароль по. . .	Кто убил Suno? Почему наше солнышко погасло? (диалог с ИИ Джемени) zorxor 30.10.2026 Как убивали Suno. Хронология. «Убийство» музыкальной нейросети Suno — это затяжной процесс, в котором технологический хайп стартапа разбился о жесткую реальность судебных исков от гигантов. . .	Генератор паролей (вариант №2) Maks 30.10.2026 Обновленная версия ранее созданной обработки. Добавлены следующие реквизиты типа Булево: - Буквы; - Цифры; - Символы. Изменена логика команды "Сгенерировать" и процедуры "ПриОткрытии":. . .	Генератор паролей (вариант №1) Maks 29.10.2026 Реализация из решения ниже выполнена исключительно по моей инициативе, дабы облегчить себе работу в создании паролей для администрируемых мною сервисов и пользователей. Самым главным критерием для. . .

@Avazart 8489 / 6156 / 615 Регистрация: 10.12.2010 Сообщений: 28,683 Записей в блоге: 30
	03.07.2014, 15:23
	Ну логичнее было сохранять с помощью fstream, тогда можно было бы с экономить время на излишнем преобразование std::vector<String> -> TStringList 0