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

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

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

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

C++

        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); в потоке.

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

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

@ssXXss 267 / 189 / 33 Регистрация: 15.01.2011 Сообщений: 681
	13.04.2013, 00:36	4
	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 [ТС]	6
	ISergey, спасибо за подсказку! Исправил но прога всё равно не пашет(, продолжаю танци с бубном 0

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