использование мьютексов

[Sparky]

Добрый вечер! вот задача: Дан файл содержащий текст произвольной длины. Найти контрольную сумму текста, как сумму кодов символов по модулю 256. Реализовать два потока: первый для чтения содержимого файла буфер длиной m символов, второй – для расчета контрольной суммы.
Вот что я написала:

/*
	подсчет контрольной суммы по тексту
*/

#include "stdafx.h"
#include "iostream"
#include "windows.h"
#include "time.h"
/*==================================================================================
					описание  данных и переменных
===================================================================================*/

	HANDLE hMutexRead;	  // Мьютекс "информация считана"
    HANDLE hMutexSum;// Мьютекс "сумма подсчитана"


	int sum=0;	//переменная накапливающая сумму
	FILE *file;	//файловая переменная
	char* file_name = "input.txt";	//имя файла
	std::string str;	//буфер
	int m=4;
	int control_sum=0;	//контрольная сумма


DWORD WINAPI reader(LPVOID param) // Функция потока чтения из файла
{
	int i;
	int scale=0;	//масштаб для массива
	WaitForSingleObject(hMutexSum,INFINITE);	//захват мьтекса "сумма подсчитана"
	while(! feof(file))	//до тех пор не конец файла читает и записываем
	{
		for(i=0;i<m;i++)
			fscanf(file,"%c",&str[i+scale]);
		scale+=m;	//увеличить масштаб
	}
	ReleaseMutex(hMutexRead);	//освободить мьютекс "информация считана"
	return 0;
}

DWORD WINAPI summator(LPVOID param)	//функция вычисления контрольной суммы
{
	int i;
	int count=str.length();
	WaitForSingleObject(hMutexRead,INFINITE);	//захват мьтекса "информация считана"	
	for (i=0;i<=count;i++)
	{
		sum+=int(str[i]);
	}
	ReleaseMutex(hMutexSum);	//освободить мьютекс "сумма подсчитана"	
	control_sum=sum%256;
	std::cout<<control_sum;
	return 0;
}

/*==================================================================================
			главная программа
==================================================================================*/
void main()
{
	file = fopen(file_name,"r");	//открытие файла для чтения 
/*==================================================================================
			создание мьютексов
==================================================================================*/
	hMutexRead = CreateMutex(NULL,false,NULL);
	hMutexSum = CreateMutex(NULL,false,NULL);

/*==================================================================================
			создание потоков
==================================================================================*/
	HANDLE HRead = CreateThread(NULL,0,reader,NULL,0,NULL);
	HANDLE HSum = CreateThread(NULL,0,summator,NULL,0,NULL);
	HANDLE hThread[2];
	hThread[0] = HRead;
	hThread[1] = HSum;

	//завершение потоков
	WaitForMultipleObjects(2,hThread,true,INFINITE);
	CloseHandle(HRead);
	CloseHandle(HSum);
}

Но программа не работает, помогите разобраться

[|{ot]

Изначально 2 мьютекса свободны и потоки "одновременно" начинают выполнение....
Попробуй так:

hMutexRead = CreateMutex(NULL,true,NULL);

[Sparky]

виснет можете еще посмотреть что не так?

[Sparky]

вот исправила:

/*
	подсчет контрольной суммы по тексту
*/

#include "stdafx.h"
#include "iostream"
#include "windows.h"
#include "time.h"
/*==================================================================================
					описание  данных и переменных
===================================================================================*/

	HANDLE hMutexRead;	  // Мьютекс "информация считана"
    HANDLE hMutexSum;// Мьютекс "сумма подсчитана"


	int sum=0;	//переменная накапливающая сумму
	FILE *file;	//файловая переменная
	char* file_name = "input.txt";	//имя файла
	std::string str;	//буфер
	int m=4;
	int control_sum=0;	//контрольная сумма


DWORD WINAPI reader(LPVOID param) // Функция потока чтения из файла
{
	int i;
	int scale=0;	//масштаб для массива
	WaitForSingleObject(hMutexRead,INFINITE);	//захват мьтекса "сумма подсчитана"
	while(! feof(file))	//до тех пор не конец файла читает и записываем
	{
		for(i=0;i<m;i++)
		//	fscanf(file,"%c",&str[i+scale]);
		str[i+scale]=getc(file);
		scale+=m;	//увеличить масштаб
	}
	//str="qwew";
	ReleaseMutex(hMutexRead);	//освободить мьютекс "информация считана"
	return 0;
}

DWORD WINAPI summator(LPVOID param)	//функция вычисления контрольной суммы
{
	int i;
	int count=str.length();
	WaitForSingleObject(hMutexRead,INFINITE);	//захват мьтекса "информация считана"	
	for (i=0;i<=count;i++)
	{
		sum+=int(str[i]);
	}
	ReleaseMutex(hMutexSum);	//освободить мьютекс "сумма подсчитана"	
	control_sum=sum%256;
	std::cout<<control_sum;
	return 0;
}

/*==================================================================================
			главная программа
==================================================================================*/
void main()
{
	file = fopen(file_name,"r");	//открытие файла для чтения 

/*==================================================================================
			создание мьютексов
==================================================================================*/
	hMutexRead = CreateMutex(NULL,false,NULL);
	hMutexSum = CreateMutex(NULL,true,NULL);

/*==================================================================================
			создание потоков
==================================================================================*/
	HANDLE HRead = CreateThread(NULL,0,reader,NULL,0,NULL);
	HANDLE HSum = CreateThread(NULL,0,summator,NULL,0,NULL);
	HANDLE hThread[2];
	hThread[0] = HRead;
	hThread[1] = HSum;

	//завершение потоков
	WaitForMultipleObjects(2,hThread,true,INFINITE);
	CloseHandle(HRead);
	CloseHandle(HSum);
	getchar();
}

Но почему-то не хочет считывать из файла когда программно записываю строку все номально что исправить?

[ds.Dante]

У меня программа вылетает на втором проходе строки
str[i+scale]=getc(file);
потому что по умолчанию строка содержит один нулевой символ, и, соответственно, буфер размером 1 байт. Проще сделать так:
str += getc(file);
буфер будет автоматически расширяться.


Кстати, функции reader() и summator() у тебя выполняются в случайном порядке, потому что новый поток запускается не сразу при его создании, а когда ось решит, что время пришло. На самом деле, формулировка самой задачи дурацкая - потоки тут вообще никаким боком. Можно сделать так:

	WaitForSingleObject(hMutexRead,INFINITE);
	HANDLE HRead = CreateThread(NULL,0,reader,NULL,0,NULL);
	WaitForSingleObject(hMutexRead,INFINITE);
	HANDLE HSum = CreateThread(NULL,0,summator,NULL,0,NULL);

а в самих функциях убрать захват мьютекса.

[Sparky]

спасибо за совет, приду домой буду пробовать