возникла необходимость создать несколько потоков

[Farrel]

Походу вопрос в эту ветку.
В общем возникла необходимость создать несколько потокв приблизительно таким образом:

for(i=0;i<20;i++)
  {
    f[0]=fopen(files[i],"r");	//open sourse
    i++;
    f[1]=fopen(files[i],"w+");	//open distanation file
	h[(i-1)/2]=CreateThread(0, 0,(LPTHREAD_START_ROUTINE) moveth, f, 0, 0);
	//moveth((LPVOID*)f);
  }
  WaitForMultipleObjects(10,h,true,INFINITE);

Весь код приводить в принципе бесмыссленно. В вышеописанном кусочке пытаюсь запустить 10 потоков, которые будут выпролнять функцию moveth (она занимается копированием файлов). Собственно вопрос в следующем: если при создании потоков использовать одну и ту же функцию, то будут ли потоки независимы? или же они могут помешать выполниться друг другу?(то что я видел при дебаге доказывало последнее). И если это так, то каким образом можно реализовать выполнение нескольких потоков созданных с исполльзованием одной и той же функции.

[rpy3uH]

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

код функции moveth в студию!

[Farrel]

DWORD moveth(LPVOID files)		//moving file function
{
  FILE** f=(FILE**)files;
  int filelen;
  int i;
  char c;
  
  fseek(f[0], 0, SEEK_END);		//fet file size
  filelen=ftell(f[0]);
  fseek(f[0], 0, SEEK_SET);
  for(i=0;i<filelen;i++)		//copy
  {
    fread(&c, 1, 1, f[0]);
    fwrite(&c, 1, 1, f[1]);
  }
  fclose(f[1]);
  fclose(f[0]);
	return NULL;
}

Будучи запущеной в цикле работает 100%, а вот при использовании потоков трабл.

[netrino]

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

DWORD moveth(LPVOID files)		//moving file function
{
  FILE** f=(FILE**)files;
  int filelen;
  int i;
  char c;
  
  fseek(f[0], 0, SEEK_END);		//fet file size
  filelen=ftell(f[0]);
  fseek(f[0], 0, SEEK_SET);
  for(i=0;i<filelen;i++)		//copy
  {
    fread(&c, 1, 1, f[0]);
    fwrite(&c, 1, 1, f[1]);
  }
  fclose(f[1]);
  fclose(f[0]);
	return NULL;
}

Будучи запущеной в цикле работает 100%, а вот при использовании потоков трабл.

Что, в общем-то, не странно, так как налицо несколько ошибок. Во-первых, необходимость принудительного преобразования к LPTHREAD_START_ROUTINE должна была насторожить вас и намекнуть, что moveth определена неправильно. Необходимо указать, что moveth должна использовать соглашение по вызову stdcall, а не ccall, это можно сделать директивой __stdcall или макросом WINAPI

DWORD WINAPI moveth(LPVOID files)

Во-вторых, Вы используете разделяемую переменную f, которая используется всеми созданными потоками, что не есть хорошо, так как новосозданный поток может не успеть прочесть имя файла до того, как оно будет затёрто на очередной итерации цикла основного потока.
Итак, чтобы избавиться от ошибок, необходимо:
1. Исправить определение функции moveth
2. Избежать использования разделяемой переменной f (сие сделать можно несколькими путями, один из них: на каждой итерации цикла выделять память для f в куче и передавать указатель на неё в moveth, а после открытия файлов, не забыть освободить память, уже внутри moveth)

[Farrel]

О, сеньк народ! Действительно с stdcall прокололся. Да и испоьзование разделяемой переменной тоже просмотрел. Во 2-й ошибке суда по всему и заключается проблема процентов на 90. Покопаюсь как раз в способах синхронизации.