Копирование данных из файла в массив

[nickolay08]

Добрый день!
Имеется файл, в котором данные располагаются построчно в таком виде:
(0.5,0.5) 0.98
(0.6,0.5) 0.94
и так далее.
Нужно сформировать три массива, в которых будут храниться эти числа по столбикам.
Для этого написал код:

int APIENTRY WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow){
	HANDLE hFile = CreateFile ("c:\\text.txt",
		GENERIC_READ, 
		FILE_SHARE_READ, 
		NULL, 
		OPEN_EXISTING, 
		FILE_FLAG_SEQUENTIAL_SCAN, 
		NULL);

	
	unsigned int n_st = 0;
	if (hFile == INVALID_HANDLE_VALUE){
		DWORD dwError = GetLastError ();
		MessageBox(NULL,"Ошибка!","Error", MB_OK | MB_DEFBUTTON1 | MB_ICONEXCLAMATION | MB_DEFAULT_DESKTOP_ONLY);
		return 0; 
	}

	char buf[1];
	memset (buf, 0, 1);
	DWORD dwBytesRead = 0;
	DWORD dwOffset = 0;

        //Нахожу количество строк n_st
	while (dwOffset < GetFileSize (hFile, NULL)) {
		SetFilePointer (hFile, dwOffset, NULL, FILE_BEGIN);
		if (ReadFile (hFile, buf, 1, &dwBytesRead, NULL)){	
			if (!memchr (&buf[0], '(', 1)){
				dwOffset++;
				continue;
			} 
			else n_st++;
		}
		dwOffset++;
	}
	
	double 
		*X = new double [n_st],
		*Y = new double [n_st],
		*U = new double [n_st];

	dwOffset = 0;

	DWORD nBD = dwOffset; 
	for(int i=0; i<n_st; i++){
		dwOffset = nBD;
		while (nBD < GetFileSize (hFile, NULL)) {
			SetFilePointer (hFile, nBD, NULL, FILE_BEGIN);
			if (ReadFile (hFile, buf, 1, &dwBytesRead, NULL)){	
				if (!memchr (&buf[0], ',', 1)){
					nBD++;
					break;
				} 
			}
			nBD++;
		}	
		char *DigitX = new char[nBD-dwOffset];
		ReadFile (hFile, DigitX, nBD-dwOffset, &dwBytesRead, NULL);
		X[i] = atof(DigitX);

		delete [] DigitX;
	}

	delete [] buf;
	CloseHandle (hFile);
	return 0;
}

Ошибок не выдается, но и массив не заполняется. Подскажите, пожалуйста, в чем ошибка.

[Granus]

А winapi вам обязательно использовать? Чем обычные методы работы с файлами не нравятся?

[nickolay08]

А winapi вам обязательно использовать? Чем обычные методы работы с файлами не нравятся?

Вторая часть программы (не представленная здесь) использует winapi.

[Rififi]

nickolay08

Ошибок не выдается, но и массив не заполняется. Подскажите, пожалуйста, в чем ошибка.

Не пиши код в нотепаде - он не предназначен для поиска ошибок. Скачай нормальную среду разработки, в которой присутствует отладчик.

[f.hump]

мрак, конечно.

например

		if (!memchr (&buf[0], '(', 1)){
				dwOffset++;
				continue;
			} 
			else n_st++;

хотя ничто не мешает сделать так

if (buf[0] == '(')) n_st++; dwOffset++;

потом вот это FILE_FLAG_SEQUENTIAL_SCAN ставится если файл читается в один проход, без перечитывания и повторов.

дальше круче. ну, нашли вы запятую, и что читать собрались? где текущий указатель файла? и сколько вы байт собрались читать?

char buf[1];
...
delete [] buf;

внушает.

2 Rififi пользуюсь только нотепадом.

[nickolay08]

Код:

if (!memchr (&buf[0], '(', 1)){
dwOffset++;
continue;
}
else n_st++;

хотя ничто не мешает сделать так
Код:

if (buf[0] == '(')) n_st++; dwOffset++;

Согласен, усложняю - заменил.

потом вот это FILE_FLAG_SEQUENTIAL_SCAN ставится если файл читается в один проход, без перечитывания и повторов.

Заменил на FILE_ATTRIBUTE_NORMAL.

Код:

char buf[1];
...
delete [] buf;

внушает.

Ну да... убрал delete.

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

Аааа! Сделал так:

while (nBD < GetFileSize (hFile, NULL)) {
			SetFilePointer (hFile, nBD, NULL, FILE_BEGIN);
			if (ReadFile (hFile, buf, 1, &dwBytesRead, NULL)){	
				if (buf[0] == ',') break; 
			}
			nBD++;
		}
		char *DigitX = new char[nBD-dwOffset];
		SetFilePointer (hFile, dwOffset, NULL, FILE_BEGIN);
		ReadFile (hFile, DigitX, nBD-dwOffset, &dwBytesRead, NULL);
		X[i] = atof(DigitX);

и все заработало.
Спасибо большое!

[Rififi]

f.hump

2 Rififi пользуюсь только нотепадом.

ну так ты, вроде, и не путаешься с отладкой, в отличие от протагониста...

А вообще код действительно мракуха. Навернута телага их winapi, ручного выделения памяти. удаления статических массивов..

хотя сам парсинг занимает всего несколько строчек (вот у меня например - две :D )

includes...

struct Data
{
	float a_, b_, c_;
};

BOOST_FUSION_ADAPT_STRUCT(
	Data,
	(float, a_)
	(float, b_)
	(float, c_)
	);

template <typename InputIterator, typename Skipper>
struct grammar : public qi::grammar<InputIterator, std::vector<Data>(), Skipper>
{
	grammar() : grammar::base_type(start)
	{
		data = '(' >> qi::float_ >> ',' >> qi::float_ >> ')' >> qi::float_ >> qi::eol;
		start = +data;
	}

	qi::rule<InputIterator, Data(), Skipper> data;
	qi::rule<InputIterator, std::vector<Data>(), Skipper> start;
};

template <typename InputIterator>
void parse(InputIterator begin, const InputIterator end, std::vector<Data>& data)
{
	const qi::rule<InputIterator> skip = qi::char_(" \t");
	const grammar<InputIterator, qi::rule<InputIterator> > g;
	qi::phrase_parse(begin, end, g, skip, data);
}

int main()
{
	try
	{
		setlocale(LC_ALL, "");
		
		const ipc::file_mapping fp("data.txt", ipc::read_only);
		const ipc::mapped_region r(fp, ipc::read_only);

		const char* const addr = (const char*) r.get_address();
		const size_t size = r.get_size();

		std::vector<Data> data;
		parse(addr, addr + size, data);
	}
	catch (const std::exception& e)
	{
		std::cout << e.what() << std::endl;
	}

	return 0;
}