Синхронизация потоков

@1Вирт1 · Регистрация: 25.08.2011

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

в общем есть код:

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
        DWORD WINAPI IndexSystemFile::GetFileFromDrive(void *DriveName){
    long hFile = 0;
    int fRes = 0;
    EnterCriticalSection(&cs);
    char *TmpPath = new char [_MAX_PATH];
    strcpy(TmpPath,(char*)DriveName);
    strcat(TmpPath,"\\*");
    hFile=_findfirst(TmpPath, &FindFileData);
    if(hFile!=-1)
        do{
            if((FindFileData.attrib&_A_SUBDIR)&&strcmp(FindFileData.name, ".") != 0 
                && strcmp(FindFileData.name, "..") != 0){
                strcpy(TmpPath,(char*)DriveName);
                strcat(TmpPath,"\\");
                strcat(TmpPath,FindFileData.name);
                GetFileFromDrive((void*)TmpPath);
            }
            else
                if(strcmp(FindFileData.name, ".") != 0 && strcmp(FindFileData.name, "..") != 0)
                    outf<<FindFileData.name<<" "<<FindFileData.size/1024<<" "
                    <<ctime(&(FindFileData.time_create))<<" "<<(char*)DriveName<<endl;
        }while(!(fRes = _findnext(hFile, &FindFileData)));
    _findclose(hFile);
    delete[] TmpPath;
    LeaveCriticalSection(&cs);
    return 0;
}
 
void IndexSystemFile::GetComputerDrive(){
    int n = 0;
    char dd[3] = " :";
    dr = GetLogicalDrives();
    for( int i = 0; i < 26; i++ ){
        n = ((dr>>i)&0x00000001);
        if( n == 1 ){
            dd[0] =  char(65 + i);
            hThreadMas[iThreadsCount] = CreateThread(NULL, 0, 
                (LPTHREAD_START_ROUTINE)GetFileFromDrive((void*)dd),
                (void*)dd, 0, &dwThreadsIdMas[iThreadsCount]);
            iThreadsCount++;
        }
    }
}

Пытаюсь синхронизовать потоки. При выполнении программа рушится и в редких случаях работает нормально, подскажите в чём может быть проблема, или хотя бы подкиньте полезную инфу по теме. Заранее спасибо.

@ssXXss · 12.04.2013, 23:46

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

@1Вирт1 · 13.04.2013, 00:26 **[ТС]**

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

@ssXXss · 13.04.2013, 00:36

http://msdn.microsoft.com/ru-r... 04870.aspx загляни сюда , возможно там найдешь решение , но нужен компилятор с поддержкой C ++11 .

@ISergey · 13.04.2013, 17:36

C++
1
DWORD WINAPI IndexSystemFile::GetFileFromDrive(void *DriveName)

Эта функция должна быть static.
CreateThread ожидает функцию с одним параметром http://msdn.microsoft.com/en-u... s.85).aspx

а у тебя неявно два. Както так

C++
1
DWORD WINAPI IndexSystemFile::GetFileFromDrive(void *this, void *DriveName)

@1Вирт1 · 13.04.2013, 19:08 **[ТС]**

ISergey, спасибо за подсказку! Исправил но прога всё равно не пашет(, продолжаю танци с бубном

@dmitry94 · 16.04.2013, 18:07

1Вирт1 Вы передаете в поток указатель на строку, а затем продолжаете цикл, изменяя значение этой строки. В общем суть в следующем, вы не должны изменять значение dd, пока не выполнено strcpy(TmpPath,(char*)DriveName); в потоке.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&d=1772460536 Одним из. . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .
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 секунды (а то и больше),. . .

@ssXXss 267 / 189 / 33 Регистрация: 15.01.2011 Сообщений: 681
	12.04.2013, 23:46
	может проблема в GetFileFromDrive() ,как то странно ты ее используешь, если это потоковая функция то зачем ее вызывать в этом же потоке в цикле да еще и войдя в критическую секцию , может я чего то не до понял ? )) 0

@1Вирт1 184 / 192 / 48 Регистрация: 25.08.2011 Сообщений: 792
	13.04.2013, 00:26 [ТС]
	задумка была создать несколько потоков использующих эту функцию, а цикл для того чтобы передать разные параметры для функции 0

@ssXXss 267 / 189 / 33 Регистрация: 15.01.2011 Сообщений: 681
	13.04.2013, 00:36
	http://msdn.microsoft.com/ru-r... 04870.aspx загляни сюда , возможно там найдешь решение , но нужен компилятор с поддержкой C ++11 . 0

@1Вирт1 184 / 192 / 48 Регистрация: 25.08.2011 Сообщений: 792
	13.04.2013, 19:08 [ТС]
	ISergey, спасибо за подсказку! Исправил но прога всё равно не пашет(, продолжаю танци с бубном 0

@dmitry94 567 / 198 / 70 Регистрация: 25.05.2012 Сообщений: 816
	16.04.2013, 18:07
	1Вирт1 Вы передаете в поток указатель на строку, а затем продолжаете цикл, изменяя значение этой строки. В общем суть в следующем, вы не должны изменять значение dd, пока не выполнено strcpy(TmpPath,(char*)DriveName); в потоке. 0