Как исправить ошибку «Кадр не находится в модуле» при создании потока?

Question

[Nulltiton]

Создал процесс, который должен выполнять функцию вычисления факториала, однако при попытке запуска появляется ошибка "Кадр не находится в модуле". С чем это может быть связано и как это исправить?

int gn = 0;
scanf_s("%d", &gn);
HANDLE h = CreateThread(
    NULL,
    0,
    factorial(gn),
    NULL,
    NULL,
    NULL
);
CloseHandle(h);
return 0;

Answer 1

[jcmvbkbc]

"Кадр не находится в модуле". Подскажите с чем это может быть связано

HANDLE h = CreateThread(
    NULL,
    0,
    factorial(gn),
    NULL,
    NULL,
    NULL
);

Третий параметр CreateThread -- указатель на функцию, которая будет запущена в создаваемом потоке. А у тебя записано скорее всего что-то другое. Если ты хотел запустить функцию factorial в потоке, она должна 1) иметь определённый прототип (вот такой), и 2) в функцию CreateThread надо передать её адрес, а не результат её вызова (например, так: CreateThread(NULL, 0, factorial, NULL, NULL, NULL)). 3) если ты сделаешь эти два изменения, тебе прийдётся также переделать передачу параметра в функцию factorial и получение результата её работы.

Comments

[Дмитрий]

jcmvbkbc, не подскажите литературу новичку для изучения C? Спасибо.

[jcmvbkbc]

Дмитрий, оффтопить -- плохо. Пользоваться поиском -- хорошо.

[Дмитрий]

jcmvbkbc, то что я нашёл, там совсем азы. Знаю что плохо. Но вдруг у вас где заметка есть.

[Nulltiton]

Большое спасибо за Ваш ответ, переделал, ошибки теперь нет, но как теперь передать параметр?

int (*fact)(int) = factorial;
    int gn = 0;
    HANDLE h = CreateThread(
        NULL,
        0,
        fact,
        NULL,
        NULL,
        NULL
    );
    CloseHandle(h);
    return 0;

[Nulltiton]

Дмитрий, на данный момент лично я пользуюсь учебниками: Керниган Б., Ритчи Д. Язык программирования С (2-е издание, 2009) и Харт. Системное программирование в среде Windows

[res2001]

Дмитрий, Не так давно появилась книжка "Экстремальный Си". Ничего особо экстремального нет, но материал там уровнем по выше, чем в обычных учебниках. Хоть в книге есть и ошибки/опечатки, но в целом рекомендую.
На счет учебников - сгодится люой для начала. Си язык простой и не большой. Дополнительную информацию ищите тут и в гугле то же всего полно.
Есть несколько тем, обычно плохо понимаемых начинающими: указатели и адресная арифметика, работа с динамической памятью. На них надо обратить особое внимание.

[res2001]

Nulltiton,

как теперь передать параметр?

Четвертый параметр CreateThread предназначен для этого. И почитайте наконец MSDN: https://docs.microsoft.com/en-us/windows/win32/api...
Обычно все необходимые параметры для потока собирают в структуру и передают указатель на структуру. Если у вас только целое число в качестве параметра, то можете конвертировать это целое в указатель и таким образом передать. В потоковой функции надо будет сделать обратное преобразование параметра.
И надо подумать о том, как будете возвращать значение. Можно возвращать в той же структуре, которую передаете в праметре. Но сначала надо дождаться завершения потока!
Вообще рекомендовал бы для потоков использовать функции стандартной библиотеки - потоки завезли в С11. Там синтаксис приближен к POSIX, он более прост и понятен, чем стандартные функции WinAPI. Кроме того, этим вы делаете код более платформонезависимым.

[Nulltiton]

res2001, Большое спасибо, от ошибки удалось избавиться, дожидаюсь завершения потока при помощи WaitForSingleObject(hThread, INFINITE);.
Но теперь никак не могу понять, каким образом возвращать значение, каким образом должно реализовываться возвращение из потока в той же структуры, если я передал ссылку на параметр? (int *)gn

[res2001]

Nulltiton, (int *)gn
Это не ссылка на параметр. Если gn целое, то вы просто меняете тип данных из целого в указатель, при этом само значение не меняется.
Соответственно в потоковой функции вы должны сделать (int)arg для получения из указателя снова целого числа.
В таком случае вы никак не можете вернуть из потока значение, разве что используя глобальные переменные, но это плохой вариант (для средних и крупных проектов).

Если у вас возврат из потоковой функции имеет тот же тип, что и параметр, то можете передавать именно указатель на gn: &gn и через этот же указатель возвращать значение.
Если же тип возвращаемого значения другой, то лучше всего объявить структуру, содержащую и параметр и возвращаемое значение и передавать указатель на структуру. А кроме того это самый наглядный и простоя для понимания вариант.
Кстати, в реализации потоков стандартной библиотеки предусмотрен вариант возврата значения из потока в виде целого числа: https://en.cppreference.com/w/c/thread/thrd_join

[res2001]

Nulltiton, Я тут нахваливаю потоки стандартной библиотеки, но вы должны понимать, что все более высокоуровневые реализации используют реализацию потоков предоставляемую операционной системой, поэтому внутри того же thrd_create() из стандартной библиотеки спрятан вызов CreateThread в винде или pthread_create() в POSIX совместимых ОС. Никаких других потоков (так же как и процессов), кроме предоставляемых ОС быть не может.

[Nulltiton]

res2001, Возврат из потоковой функции имеет тот же тип, что и параметр, я стал передавать указатель, как написано в Вашем сообщении, но никак не могу понять, как будет выглядеть конструкция для, вот допустим, вывода на экран возвращаемого значения функции? Я бы с радостью обратился к стандартной библиотеке, но, к сожалению, передо мной стоит задача изучить именно CreateThread, возможно есть еще какие-либо источники, где я мог бы увидеть правильный вариант написания. И, еще раз, большое спасибо, что откликнулись и помогли дойти хотя бы до этого этапа.

[res2001]

Nulltiton,

передо мной стоит задача изучить именно CreateThread

Изучай конечно. Просто работа с потоками она +-одинакова везде, т.к. поток - совершенно определенная сущность ОС, одинаковая для всех ОС. POSIX вариант потоков считаю наиболее продуманным и простым, а вариант стандартной библиотеки по сути копирует POSIX. Внутренняя реализация потоков в ядре ОС, конечно, разная для разных ОС, но это не должно волновать прикладного программиста.

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

Выкладывай код, который у тебя получился и вопросы по нему. Дальше в слепую уже трудно что-то советовать.

Answer 2

[Nulltiton]

Решил проблему при помощи глобальной, воспользовался примером из книги Побегайло - "Системное программирование в Windows".
Код:

#include <stdio.h>
#include <Windows.h>

int n;

DWORD WINAPI Add(int Num) {
	printf("Thread is started\n");
	n += Num;
	printf("Thread is finished\n");
}

int main() {

	int inc = 10;
	HANDLE hThread;
	DWORD IDThread;

	printf("n = %d\n", n);

	hThread = CreateThread(
		NULL,
		0,
		Add,
		(void*)inc,
		0,
		&IDThread);

	if (hThread == NULL)
		return GetLastError();

	WaitForSingleObject(hThread, INFINITE);
	CloseHandle(hThread);

	printf("n = %d\n", n);

	return 0;
}

Comments

[res2001]

Кода я и не заметил :-)
Я писал выше про использование глобальных переменных. Вариант имеет право на существование, но когда у тебя будет проект побольше и с несколькими потоками глобальные переменные вообще не вариант, как правило.
Вообще исопльзоавние глобальных переменных в принципе не очень хорошая идея, а не только для потоков.
В некоторых случаях это допустимо, но таких случаев не много.
И если уж ты хочешь научиться работать с потоками, то важно научиться и правильно передавать в поток параметры и принимать возвращаемое значение. И параметров и возвращаемых значений может быть не одно.
И это я еще не начинал говорить о совместном использовании данных разными потоками - это вообще отдельная огромная тема.

Обрати внимание, что твоя потоковая функция по сигнатуре не соответствует документации. Она должна быть такой: https://docs.microsoft.com/en-us/previous-versions...
Твой код в принципе рабочий, но стоит добавить опций компилятору, как он начнет ругаться на эту ошибку, по формальным признакам.
И еще - потоковая функция возвращает значение типа DWORD, в твоем случае можешь передавать возвращаемое значение потока через него, не используя глобальную переменную.
В POSIX потоковые функции возвращают указатель void*

[Nulltiton]

Извините за долгое отсутствие, все верно, кодом это вряд ли назовешь, но этот пример нужен был только для понимания работы. Я отдаю себе отчет, что данное решение, вероятно, наихудшее, но, еще раз, я только изучаю потоки, а значит мне нужно видеть, что это хоть как-то, но работает, дабы далее можно было погрузиться глубже. Большое спасибо за Вашу ссылку, приму это к сведению. Хотел бы задать достаточно глупый вопрос, потому как за терминами серьезной литературы иногда и правда тяжело уловить суть. Правильно ли я понимаю, что процессы - это (на примитивном уровне) возможность запускать какое-либо приложение из своего проекта, а потоки в свою очередь служат для того, чтобы распараллелить этот проект или добавить еще что-то?