Контейнер unordered_map, содержащий потоки

@Artishok · Регистрация: 27.12.2009

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

постановка задачи:
есть некий класс Item

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
class Item
{
private:
    int id;
    string text;
public:
    Item() 
    {
        text = "default";
        id = -1;
    }
    void SetText(string text) { this->text = text; }
    void SetId(int id) { this->id = id; }
    string GetText() { return text; }
    int GetId() { return id; }
    ~Item() {}
};

есть класс ItemSubSystem который имеет vector объектов Item* и unordered_map, где ключ int а значение Item*. Класс имеет два метода - один выводит список элементов вектора, другой - модифицирует элементы map. Так как что вектор что map содержат указатели на одни и те же объекты - то есть изменение значений полей элемента Item* в map приводит к изменению значений в vector.

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
class ItemSubSystem
{
private:
    vector<Item*> toDraw;
    boost::unordered_map<int, Item*> controllerMap;
    int size;
public:
    ItemSubSystem() {};
    void InitAll()
    {
        size = 10;
        for(int i = 0; i < size; i++)
        {
            Item *it = new Item();
            it->SetId(i);
            string text = "item-" + boost::lexical_cast<string>(i);
            it->SetText(text);
            toDraw.push_back(it);
            controllerMap.insert(std::make_pair(i, it));
        }
    }
    void DrawAll() {
        BOOST_FOREACH(auto i, toDraw)
        {
            cout<<";"<<i->GetText();
        }
        cout<<endl;
    }
    void ModifyAll()
    {
        for(int i = 0; i < size; i++)
        {
            Item *it = controllerMap[i];
            int t = it->GetId();
            it->SetId(++t);
            string text = "item-" + boost::lexical_cast<string>(t);
            it->SetText(text);
        }
    }
    ~ItemSubSystem() 
    {
        BOOST_FOREACH(auto i, toDraw) delete i;
        toDraw.clear();
 
        BOOST_FOREACH(auto& i, controllerMap) delete i.second;
        controllerMap.clear();
    }
};

Необходимо чтобы в одном потоке вызывался метод drawAll, который бы выводил элементы вектора, а в другом - изменялись значения в map.

изначательно я создавал потоки в функции main. я создал две функции потоков

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
void DrawThread()
{
    for(;;)
    {
        try 
        {
            boost::this_thread::sleep(boost::posix_time::milliseconds(2000));
            items->DrawAll();
        }
        catch(boost::thread_interrupted&)
        {
            cout<<"Draw thread is stopped"<<endl;
            return;
        }
    }   
};
 
void ModifyThread()
{
    for(;;)
    {
        try 
        {
            boost::this_thread::sleep(boost::posix_time::milliseconds(15000));
            items->ModifyAll();
        }
        catch(boost::thread_interrupted&)
        {
            cout<<"Modify thread is stopped"<<endl;
            return;
        }
    }
};

где items

C++
1
ItemSubSystem *items;

Все работало если создавал потоки в main и там же останавливал.

Но потом я решил создать класс, который содержал бы map, где ключ - идентификатор потока, а значение - собственно сам поток. класс должен был позволить добавлять и останавливать/удалять потоки.

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
class ItemThreadingController
{
private:
    boost::unordered_map<long int, boost::thread> controllerMap;
    long int id;
public:
    ItemThreadingController() 
    {
        id = -1;
    }
    int AddThread(void (*func)())
    {
        boost::thread t(func);
        id++;
        controllerMap.insert(std::make_pair(id, t));
        return id;
    }
    void StopThread(long int id)
    {
        controllerMap[id].interrupt();
        controllerMap[id].join();
        controllerMap.erase(id);
        cout<<"Thread "<<id<<"has stopped and erased!"<<endl;
    }
    ~ItemThreadingController() {};
};

проблема в том, что если я оставляю код без изменений - то выдаётся ошибка при сборке

Code
1
2
13  IntelliSense: #error directive: "Incompatible build options"    111
Error   1   error C2248: 'boost::thread::thread' : cannot access private member declared in class 'boost::thread'   247

но если я закомментирую строку

C++
1
2
3
4
5
6
7
int AddThread(void (*func)())
    {
        boost::thread t(func);
        id++;
        //controllerMap.insert(std::make_pair(id, t));
        return id;
    }

то собирается без ошибок.

функция main выглядит так

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
int main(int argc, char** argv)
{
    items = new ItemSubSystem();
    items->InitAll();
 
    ItemThreadingController *threads = new ItemThreadingController();
    int t1 = threads->AddThread(DrawThread);
    int t2 = threads->AddThread(ModifyThread);
 
    boost::this_thread::sleep(boost::posix_time::milliseconds(50000));
    threads->StopThread(t1);
    threads->StopThread(t2);
 
    return 0;
}

почему возникает ошибка?

@gray_fox · 26.01.2014, 01:47

Artishok, boost::thread не поддерживает копирование. Если есть поддержка C++11, то можно использовать move

C++
1
controllerMap.insert(std::make_pair(id, std::move(t)));

Если нет, то можно хранить указатель на thread, например boost::shared_ptr<boost::thread>

Jupiter · 26.01.2014, 01:48

del

@Artishok · 26.01.2014, 01:56 **[ТС]**

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

Artishok, boost::thread не поддерживает копирование. Если есть поддержка C++11, то можно использовать move

C++
1
controllerMap.insert(std::make_pair(id, std::move(t)));

Если нет, то можно хранить указатель на thread, например boost::shared_ptr<boost::thread>

это получается нужно сделать метод таким

C++
1
2
3
4
5
6
7
int AddThread(void (*func)())
    {
        boost::shared_ptr<boost::thread> p (new boost::thread(func));
        id++;
        controllerMap.insert(std::make_pair(id, p));
        return id;
    }

а контейнер таким

C++
1
boost::unordered_map<long int, boost::shared_ptr<boost::thread>> controllerMap;

?

@gray_fox · 26.01.2014, 02:02

Artishok, именно. Либо, что бы не возиться с умными указателями, можно использовать boost::ptr_unordered_map как вариант.

@Artishok · 26.01.2014, 02:12 **[ТС]**

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

Artishok, именно. Либо, что бы не возиться с умными указателями, можно использовать boost:

tr_unordered_map как вариант.

Ну так у меня по сути не будет обращений к элементам использующих один и тот же элемент shared_ptr так что можно и так оставить.

разве что я так понимаю нужно в деструктор добавить

C++
1
2
3
4
5
~ItemThreadingController() 
    {
        BOOST_FOREACH(auto& i, controllerMap) 
            i.second.reset();
    };

?

@gray_fox · 26.01.2014, 02:18

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

разве что я так понимаю нужно в деструктор добавить

не надо, shared_ptr в своём деструкторе вызовет delete, умный же)

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

Ну так у меня по сути не будет обращений к элементам использующих один и тот же элемент shared_ptr так что можно и так оставить.

Ну ptr_unordered_map поэффективней должен быть, как минимум он не занимается подсчётом ссылок, в отличие от shared_ptr. Я правда не знаю, в какой версии boost этот контейнер добавили.

@Artishok · 26.01.2014, 02:18 **[ТС]**

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

не надо, shared_ptr в своём деструкторе вызовет delete, умный же)

Ну ptr_unordered_map поэффективней должен быть, как минимум он не занимается подсчётом ссылок, в отличие от shared_ptr. Я правда не знаю, в какой версии boost этот контейнер добавили.

ну у меня 1_55_0 так что он должен быть точно

@gray_fox · 26.01.2014, 02:26

Так то можно просто хранить голые указатели в unordered_map, в деструкторе вызывать delete )

Добавлено через 7 минут
В принципе, если учесть, что ItemThreadingController нельзя копировать, то наверное можно использовать и auto_ptr. Если копирировать можно, то проще с shared_ptr.

@Artishok · 26.01.2014, 02:31 **[ТС]**

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

Так то можно просто хранить голые указатели в unordered_map, в деструкторе вызывать delete )

Добавлено через 7 минут
В принципе, если учесть, что ItemThreadingController нельзя копировать, то наверное можно использовать и auto_ptr. Если копирировать можно, то проще с shared_ptr.

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

как добавить тогда элемент в ptr_unordered_map?

C++
1
boost::ptr_unordered_map<int, boost::thread> controllerMap;

я пытаюсь и так

C++
1
2
3
4
5
boost::thread t(func);
        std::auto_ptr<boost::thread> tp(&t);
        id++;
        controllerMap.insert(std::make_pair(id, tp));
        return id;

и по-другому...ругается.

@gray_fox · 26.01.2014, 02:36

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

всмысле нельзя копировать?

В прямом, если не подразумевается такое

C++
1
ItemThreadingController copy(original);

либо

C++
1
copy = original;

@Artishok · 26.01.2014, 02:40 **[ТС]**

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

В прямом, если не подразумевается такое

C++
1
ItemThreadingController copy(original);

либо

C++
1
copy = original;

а понятно...

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера 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. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

Jupiter Каратель 6610 / 4029 / 401 Регистрация: 26.03.2010 Сообщений: 9,273 Записей в блоге: 1
	26.01.2014, 01:48
	del 0

@gray_fox What a waste! 1610 / 1302 / 180 Регистрация: 21.04.2012 Сообщений: 2,733
	26.01.2014, 02:02
	Artishok, именно. Либо, что бы не возиться с умными указателями, можно использовать boost::ptr_unordered_map как вариант. 1

@gray_fox What a waste! 1610 / 1302 / 180 Регистрация: 21.04.2012 Сообщений: 2,733
	26.01.2014, 02:26
	Так то можно просто хранить голые указатели в unordered_map, в деструкторе вызывать delete ) Добавлено через 7 минут В принципе, если учесть, что ItemThreadingController нельзя копировать, то наверное можно использовать и auto_ptr. Если копирировать можно, то проще с shared_ptr. 1