ReadFile

[nazavrik]

Доброе время суток!

Считываю данные из файла:

size_t MyClass::RecvFrom(_TCHAR *buf, DWORD bufLen)
{
	HANDLE hFile = CreateFile(pBufData, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL);

	if(hFile == INVALID_HANDLE_VALUE)
	{
		_tprintf(_TEXT("File can not open\n"));
		return(-1);
	}
	
	DWORD nBytesRead;
	
	bool bFlag = ReadFile(hFile, buf, bufLen, &nBytesRead, NULL);

	if(bFlag)
	{
		_tprintf(_TEXT("File not read\n"));
		return(-1);
	}

	_tprintf(_TEXT("Buf - %s\n"), buf); //!!

	CloseHandle(hFile);
	return(0);
}

На экран выводятся квадратики непонятные, что не соответствует содержимому файла. В чем может быть дело?

[ROD]

В кодировке файла не верной причина быть может.

[pu4koff]

Меня смущает обилие макросов (_TCHAR, _tprintf, _TEXT) для переносимости кода между юникод/не юникод проектами. В свойствах проекта юникод включен? Файл юникодный или в какой-нибудь 1251 кодировке?

[lennon]

size_t MyClass::RecvFrom(_TCHAR *buf, DWORD bufLen)
{
	HANDLE hFile = CreateFile(pBufData, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL);

	if(hFile == INVALID_HANDLE_VALUE)
	{
		_tprintf(_TEXT("File can not open\n"));
		return(-1);
	}
	
	DWORD nBytesRead;
	
	bool bFlag = ReadFile(hFile, buf, bufLen, &nBytesRead, NULL);

	if(bFlag)
	{
		_tprintf(_TEXT("File not read\n"));
		return(-1);
	}

	printf("Buf - %s\n"), buf); //!!

	CloseHandle(hFile);
	return(0);
}

[nazavrik]

Спасибо! Действительно все дело было в выводе.

Сейчас встала другая проблема

Оказывается оператор new выделяет памяти больше, чем запрашивается (лишняя память, как я понял, необходима для хранения размера выделенной памяти, в которой нуждается оператор delete []). В связи с этим процесс выделения памяти для буфера определенного размера, записи в этот буфер такого же размера информации и последующий вывод буфера на печать приводит к выводу в конце лишних каракуль

[lennon]

=================================
предыдущая ошибка заключалась в следующем:
_tprintf выводит строки unicode, тоесть wchar_t строки, а у вас в файле данные находились в обычной кодировке ansi
=================================

Следующий ваш вопрос:

под буфером я так понял вы подразумеваете строку. Выделите на один байт больше и в этот последний байт поместите NULL - ноль байт.

 char [] test = "test";
 int len = strlen(test);
 char * str = new char[len+1];
 str[len+1] = 0; //или memset(str,0,len+1);
 strcpy(str,test);

 printf("%s",str);

неоднократно повторяю.... проблема в том, что строки в с++ должны заканчивать нулевым байтом, для того чтобы можно было определить какова их длина (пример strlen) и где они заканчиваються. Вот другой пример (собственно я так делаю всегда)

  char [] test = "test";
  char * str = (char*)calloc(strlen(test)+1,sizeof(char)); //здесь +1 - место для нулевого байта который свидетельствует о конце строки, так же эта функция сама заполняет весь массив NULL'ами
  strcpy(str,test);

 printf("%s",str);

пример функции strlen

int strlen(const char * str)
{
 int i = 0;
 while (str[i] != 0 /*NULL*/) i++;
 return i;
}

[nazavrik]

Проблема в следующем. Я использую оператор new:

	char *buf = new char[a];
	int i = strlen(buf);

количество выделяемой памяти определяется параметром a, так?

Если а = 1,2,3,4, то i принимает значение 16.
Если а = (5-12), то i принимает значение 24.
Если а = (13-20), то i принимает значение 32.
Если а = (21-28), то i принимает значение 40.

и т.д. Т.е. размер буфера всегда на 12 больше фактически запрашиваемого размера памяти. VS2008

[pu4koff]

Просто с мусором тебе так везёт. После выделения памяти, нужно обнулять данные, прежде, чем использовать их. К размеру выделяемого блока приведённый код вообще не имеет никакого отношения. Напиши так:

	char *buf = new char[a];
	buf[0] = '\0';
	int i = strlen(buf);

Теперь i всегда будет равна нулю. Значит компилятор вообще память выделять не стал чтоль?

[nazavrik]

Ок. Задача стоит так. Из файла прочесть заданное количество байтов- bufLen. Я уже приводил код ранее, повторюсь:

HANDLE hFile = CreateFile(pBufData, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_READONLY, NULL);

	if(hFile == INVALID_HANDLE_VALUE)
	{
		_tprintf(_TEXT("File can not open"));
		return(-1);
	}

	DWORD bRead;

	char *buff = new char[bufLen];

	bool bResult = ReadFile(hFile, buff, bufLen, &bRead, NULL);

	printf("buf - %s", buff); 

	CloseHandle(hFile);

Результат - всегда один - на экран выводится 40 символов, не зависимо от значения bufLen. Что конкретно не так7

Если бы задача ограничивалась выводом считанных байтов, то все было бы просто:

	int i=0;

	while(bRead!= 0)
	{
		std::cout<<buff[i];
		bRead--;
                i++;
	}

Но мне нужно оперировать ими. А для этого нужно их корректное представление в буфере.

[pu4koff]

Файл текстовый или с бинарными данными? В bRead 40 записано и на экран 40 символов выводится?