Функция потока в классе

@klynxe · Регистрация: 07.09.2012

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

Подскажите, как запихать в класс функцию потока (небольшой пример, если есть)?
когда пишу так:

C++

class abc{
DWORD WINAPI MyThread(LPVOID);
void StartTr();
};
void abc::StartTr()
{
CreateThread(NULL, 0, MyThread, NULL, 0, NULL);
}

Соответственно получаю ошибку
E2034 Cannot convert 'unsigned long (__stdcall * (_closure )(void *))(void *)' to 'unsigned long (__stdcall *)(void *)'
E2342 Type mismatch in parameter 'lpStartAddress' (wanted 'unsigned long (__stdcall *)(void *)', got 'void')

@DrOffset · 10.11.2015, 12:53

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

Подскажите, как запихать в класс функцию потока (небольшой пример, если есть)?

Принцип можно срисовать отсюда: https://www.cyberforum.ru/post8225359.html. Для всего низкоуровневого поточного API он одинаковый.

Добавлено через 3 минуты
PS. Параметр, приходящий в поточную функцию, задается четвертым по счету (lpParameter).

@klynxe · 12.11.2015, 16:05 **[ТС]**

Хм, а можно без SDL_Thread? С CreateThread

@pproger · 12.11.2015, 16:08

klynxe, тебеж принцип говорят посмотри

@klynxe · 12.11.2015, 16:15 **[ТС]**

C++

class abc{
DWORD WINAPI MyThread(LPVOID);
void StartTr();
static int StaticThread(void* self);
};
void abc::StartTr()
{
CreateThread(NULL, 0, &abc::StaticThread, NULL, 0, NULL);
}
static int StaticThread(void* self)
{
    return static_cast<abc*>(self)->MyThread();
}

не работает
E2034 Cannot convert 'int (*)(void *)' to 'unsigned long (__stdcall *)(void *)'
E2342 Type mismatch in parameter 'lpStartAddress' (wanted 'unsigned long (__stdcall *)(void *)', got 'int (*)(void *)')
E2247 '__stdcall abc::ReadThread(void *)' is not accessible

Croessmah · 12.11.2015, 16:19

Прототип функции:

C++

1
2
3

DWORD WINAPI ThreadProc(
  _In_ LPVOID lpParameter
);

https://msdn.microsoft.com/ru-... 85%29.aspx

@klynxe · 12.11.2015, 16:26 **[ТС]**

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

Прототип функции:

Не понял, к чему это?

Croessmah · 12.11.2015, 16:27

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

Не понял к чему это?

к этому:

C++

static DWORD WINAPI StaticThread(LPVOID self)
{
//...
}

@klynxe · 12.11.2015, 16:32 **[ТС]**

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

к этому:

спасибо, помогло, а еще
E2247 '__stdcall abc::ReadThread(void *)' is not accessible

Croessmah · 12.11.2015, 16:35

Ну и

C++

void abc::StartTr()
{
CreateThread(NULL, 0, &abc::StaticThread, NULL, 0, NULL);
}

исправить, иначе получите параметр self равным 0.

C++

1	CreateThread(NULL, 0, &abc::StaticThread, (void*)this, 0, NULL);

Добавлено через 2 минуты

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

а еще
E2247 '__stdcall abc::ReadThread(void *)' is not accessible

в приведенном куске кода нет упоминания abc::ReadThread, так что только гадать что там.

@klynxe · 12.11.2015, 16:41 **[ТС]**

как-то так

C++

class abc{
DWORD WINAPI MyThread(LPVOID);
void StartTr();
static DWORD WINAPI StaticThread(LPVOID self);
};
void abc::StartTr()
{
CreateThread(NULL, 0, &abc::StaticThread, (void*)this, 0, NULL);
}
DWORD WINAPI abc::StaticThread(LPVOID self)
{
    return static_cast<abc*>(self)->MyThread(NULL);
}

Добавлено через 42 секунды

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

в приведенном куске кода нет упоминания abc::ReadThread, так что только гадать что там.

Да, сторонняя ошибка(

@Kastaneda · 12.11.2015, 16:45

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

CreateThread(NULL, 0, &abc::StaticThread, (void*)this, 0, NULL);

Очень не безопасно, объект может быть уже уничтожен, а поток может еще работать. В таких случаях класс наследует std::enable_shared_from_this<> и в поток передается shared_from_this(). Ну и конструкторы прячутся в private.

Добавлено через 1 минуту
да и потоки можно использовать из C++11, либо enable_shared_from_this взять из boost'а, либо реализовать все это руками.

Croessmah · 12.11.2015, 16:53

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

Очень не безопасно, объект может быть уже уничтожен

ну тогда

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

В таких случаях класс наследует std::enable_shared_from_this<> и в поток передается shared_from_this(). Ну и конструкторы прячутся в private.

компилятор может не поддерживать c++11.

Нам не ведано, что там и как у автора будет устроено, как будет синхронизироваться и т.д.

@klynxe · 13.11.2015, 12:37 **[ТС]**

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

компилятор может не поддерживать c++11.
Нам не ведано, что там и как у автора будет устроено, как будет синхронизироваться и т.д.

Пишу код, который должен работать в VS13 и RadStudio XE8, думаю C++11 должны поддерживать.

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

Очень не безопасно, объект может быть уже уничтожен, а поток может еще работать. В таких случаях класс наследует std::enable_shared_from_this<> и в поток передается shared_from_this(). Ну и конструкторы прячутся в private.
Добавлено через 1 минуту
да и потоки можно использовать из C++11, либо enable_shared_from_this взять из boost'а, либо реализовать все это руками.

А можно чуть подробнее (без boots)? Пример какой, а то так я не очень знаю что такое enable_shared_from_this<>

@Kastaneda · 13.11.2015, 14:33

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

А можно чуть подробнее (без boots)? Пример какой, а то так я не очень знаю что такое enable_shared_from_this<>

Хотел пример написать, но потом подумал - WinAPI это сишный стиль, в поток передается void*, т.е. максимум, что ты можешь туда передать это указатель на shared_ptr, но shared_ptr уже может быть уничтожен к моменту запуска потока. Поэтому тут так не получится, нужна другая реализация создания потоков.
В общем следи за тем, чтобы время жизни объекта было >= времени жизни потока.

@pproger · 13.11.2015, 14:49

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

Хотел пример написать, но потом подумал - WinAPI это сишный стиль, в поток передается void*, т.е. максимум, что ты можешь туда передать это указатель на shared_ptr, но shared_ptr уже может быть уничтожен к моменту запуска потока. Поэтому тут так не получится, нужна другая реализация создания потоков

все получится. в деструкторе объекта сигнализируем потоку, что пора на выход и делаем join.

@Kastaneda · 13.11.2015, 15:45

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

все получится. в деструкторе объекта сигнализируем потоку, что пора на выход и делаем join.

с таким подходом shared_ptr получается не нужен.

@DrOffset · 13.11.2015, 15:51

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

Поэтому тут так не получится, нужна другая реализация создания потоков.

Справедливости ради, std::thread через те же сишные функции реализован. Если ты ему join забудешь позвать, то будет та же самая картина: объект потока уничтожился, а сам поток еще работает.

@pproger · 13.11.2015, 15:53

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

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

таки нефиг пихать его везде подряд

@Kastaneda · 13.11.2015, 15:55

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

Если ты ему join забудешь позвать, то будет та же самая картина: объект потока уничтожился, а сам поток еще работает.

Я имел ввиду другое.

C++

//псевдокод
ThreadPtr tp;
{
    MyClass ob;
    tp = createThread(ob);
}
tp->join();

join вызван, но поток работает с несуществующим объектом.

Добавлено через 51 секунду

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

Справедливости ради, std::thread через те же сишные функции реализован.

я внуть не смотрел, но подозреваю, что объект thread хранит в себе shared_ptr (или что ему передали) и он будет жить пока поток не завершится.

@Kastaneda 5231 / 3204 / 362 Регистрация: 12.12.2009 Сообщений: 8,116 Записей в блоге: 2
	13.11.2015, 15:45	17
	Сообщение от pproger все получится. в деструкторе объекта сигнализируем потоку, что пора на выход и делаем join. с таким подходом shared_ptr получается не нужен. 1

@DrOffset 18842 / 9841 / 2408 Регистрация: 30.01.2014 Сообщений: 17,284
	13.11.2015, 15:51	18
	Сообщение от Kastaneda Поэтому тут так не получится, нужна другая реализация создания потоков. Справедливости ради, std::thread через те же сишные функции реализован. Если ты ему join забудешь позвать, то будет та же самая картина: объект потока уничтожился, а сам поток еще работает. 1

@DrOffset 18842 / 9841 / 2408 Регистрация: 30.01.2014 Сообщений: 17,284
	10.11.2015, 12:53	2
	Сообщение от klynxe Подскажите, как запихать в класс функцию потока (небольшой пример, если есть)? Принцип можно срисовать отсюда: https://www.cyberforum.ru/post8225359.html. Для всего низкоуровневого поточного API он одинаковый. Добавлено через 3 минуты PS. Параметр, приходящий в поточную функцию, задается четвертым по счету (lpParameter). 1

@klynxe 12 / 12 / 4 Регистрация: 07.09.2012 Сообщений: 542
	12.11.2015, 16:05 [ТС]	3
	Хм, а можно без SDL_Thread? С CreateThread 0

@pproger 166 / 69 / 17 Регистрация: 22.03.2011 Сообщений: 196
	12.11.2015, 16:08	4
	klynxe, тебеж принцип говорят посмотри 1

@klynxe 12 / 12 / 4 Регистрация: 07.09.2012 Сообщений: 542
	12.11.2015, 16:26 [ТС]	7
	Сообщение от Croessmah Прототип функции: Не понял, к чему это? 0

@klynxe 12 / 12 / 4 Регистрация: 07.09.2012 Сообщений: 542
	12.11.2015, 16:32 [ТС]	9
	Сообщение от Croessmah к этому: спасибо, помогло, а еще E2247 '__stdcall abc::ReadThread(void *)' is not accessible 0

@Kastaneda 5231 / 3204 / 362 Регистрация: 12.12.2009 Сообщений: 8,116 Записей в блоге: 2
	12.11.2015, 16:45	12
	Сообщение от Croessmah CreateThread(NULL, 0, &abc::StaticThread, (void)this, 0, NULL); Очень не безопасно, объект может быть уже уничтожен, а поток может еще работать. В таких случаях класс наследует std::enable_shared_from_this<> и в поток передается shared_from_this(). Ну и конструкторы прячутся в private. Добавлено через 1 минуту* да и потоки можно использовать из C++11, либо enable_shared_from_this взять из boost'а, либо реализовать все это руками. 2

Croessmah Неэпический 17870 / 10635 / 2054 Регистрация: 27.09.2012 Сообщений: 26,737 Записей в блоге: 1
	12.11.2015, 16:53	13
	Сообщение от Kastaneda Очень не безопасно, объект может быть уже уничтожен ну тогда Сообщение от Kastaneda В таких случаях класс наследует std::enable_shared_from_this<> и в поток передается shared_from_this(). Ну и конструкторы прячутся в private. компилятор может не поддерживать c++11. Нам не ведано, что там и как у автора будет устроено, как будет синхронизироваться и т.д. 1

@klynxe 12 / 12 / 4 Регистрация: 07.09.2012 Сообщений: 542
	13.11.2015, 12:37 [ТС]	14
	Сообщение от Croessmah компилятор может не поддерживать c++11. Нам не ведано, что там и как у автора будет устроено, как будет синхронизироваться и т.д. Пишу код, который должен работать в VS13 и RadStudio XE8, думаю C++11 должны поддерживать. Сообщение от Kastaneda Очень не безопасно, объект может быть уже уничтожен, а поток может еще работать. В таких случаях класс наследует std::enable_shared_from_this<> и в поток передается shared_from_this(). Ну и конструкторы прячутся в private. Добавлено через 1 минуту да и потоки можно использовать из C++11, либо enable_shared_from_this взять из boost'а, либо реализовать все это руками. А можно чуть подробнее (без boots)? Пример какой, а то так я не очень знаю что такое enable_shared_from_this<> 0

@Kastaneda 5231 / 3204 / 362 Регистрация: 12.12.2009 Сообщений: 8,116 Записей в блоге: 2
	13.11.2015, 14:33	15
	Сообщение от klynxe А можно чуть подробнее (без boots)? Пример какой, а то так я не очень знаю что такое enable_shared_from_this<> Хотел пример написать, но потом подумал - WinAPI это сишный стиль, в поток передается void*, т.е. максимум, что ты можешь туда передать это указатель на shared_ptr, но shared_ptr уже может быть уничтожен к моменту запуска потока. Поэтому тут так не получится, нужна другая реализация создания потоков. В общем следи за тем, чтобы время жизни объекта было >= времени жизни потока. 1

@pproger 166 / 69 / 17 Регистрация: 22.03.2011 Сообщений: 196
	13.11.2015, 14:49	16
	Сообщение от Kastaneda Хотел пример написать, но потом подумал - WinAPI это сишный стиль, в поток передается void*, т.е. максимум, что ты можешь туда передать это указатель на shared_ptr, но shared_ptr уже может быть уничтожен к моменту запуска потока. Поэтому тут так не получится, нужна другая реализация создания потоков все получится. в деструкторе объекта сигнализируем потоку, что пора на выход и делаем join. 1

@pproger 166 / 69 / 17 Регистрация: 22.03.2011 Сообщений: 196
	13.11.2015, 15:53	19
	Сообщение от Kastaneda с таким подходом shared_ptr получается не нужен таки нефиг пихать его везде подряд 1