Сохранение данных COM-порта в несколько файлов

[bedouin]

to de-em

В идеале файл должен создаваться в том же самом потоке.
Но, т.к. мне необходимо закрыть предыдущий файл, то я обрываю поток

 {
 if(reader)		   //если поток чтения работает, завершить его; проверка if(reader) обязательна, иначе возникают ошибки
  {TerminateThread(reader,0);
   CloseHandle(overlapped.hEvent);	//нужно закрыть объект-событие
   CloseHandle(reader);
  }

Конечно, это можно было сделать изящнее, пока делаю как есть.
После обрыва потока закрываю старый файл

myFile.Close();

И пытаюсь создать новый

newFile.Open("myfile.txt", CFile::modeCreate | CFile::modeWrite);

И после этого я должен возобновит поток (пока не сделал).
Затык в том, что файл не создается. А следовательно, писать поток некуда

[waleri]

Мне не совсем понятно - зачем обрывать поток при закрытии файла? Закройте да откройте все в одном потоке (который читает из COM порта). Далее, у вас все время есть то myFile то newFile. В принципе должен быть один файловый объект и надо открывать и закрывать его в одном потоке, либо надо синхронизировать доступ в объекту, в противном случае могут случаться Плохие Вещи. Далее, посмотрите на последний, третий аргумент у CFile::Open и воспользуйтесь им. Наверняка есть код ошибки - надо посмотреть что ломается. Вы уверены, что поток уже закончен и файл уже закрыт?

[bedouin]

Подправил код таким образом, чтобы можно было приостанавливать и стартовать поток при помощи Suspend/Resume, а не уничтожать его:
Создаю поток

reader = CreateThread(NULL, 0, ReadThread, NULL, CREATE_SUSPENDED, NULL);
ResumeThread(reader);

Процедура записи в файл:

void ReadPrinting()
{
 	if (GetFileSize(myFile,NULL) > 100000)
  {
    newFile.Open("newFile.txt", CFile::modeCreate | CFile::modeWrite);
   
    SuspendThread(reader);
     myFile.Close();
    ResumeThread(reader);

	}
	else
  {	
  myFile.Write(bufrd, strlen(bufrd));  //записать в файл данные из приёмного буфера
  memset(bufrd, 0, BUFSIZE);	        //очистить буфер (чтобы данные не накладывались друг на друга)
  }

}

Файл newFile.txt создается.
Но для меня осталось непонятным, почему если я открытие/создание файла ставлю после приостановки потока и закрытия файла

{
    SuspendThread(reader);
    myFile.Close();
    ResumeThread(reader);

newFile.Open("newFile.txt", CFile::modeCreate | CFile::modeWrite);

	}

то ничего не работает (newFile.txt не создается)?
Если наоборот, то все идет. Из-за чего так происходит?

[waleri]

Да зачем останавливать и запускат поток?
Откройте COM порт
Создайте поток и передайте параметром HANDLE порта
В потоке откройте файл, циклично читайте из порта и пишите в файл. Если файл стал большим закройте файл и откройте новый.

Когда надо завершить программу из главного потока закройте COM порт, тогда чтение прервется ошибкой.

[bedouin]

Когда я закрываю файл, не обрывая поток, вываливается ошибка, ссылающаяся на файл filecore.cpp (номер строки не скажу, надо уточнить).
Она связана с некорректным закрытием файла.
Из-за этого программу периодически "выбивает", а мне надо, чтобы она писала файлы в цикле.
Вот поэтому я и "камасутрюсь" с потоком (его запуском и остановом).

[waleri]

Я бы на вашем месте разобрался бы с вываливанием. Если это то вываливание, о котором шла речь пару страниц назад, то насколько я помню оно было связано с тем, что вы закрываете файл, который не был открыт, либо закрывается дважды.

[bedouin]

ОК, перепроверю.
Хотя в прошлый раз ошибка уходила, когда перед тем, как закрыть файл, предварительно останавливал поток данных с порта.
Но пересмотрю код еще раз. Возможно это было совпадение

[bedouin]

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

void ReadPrinting()
{
 	if (GetFileSize(myFile,NULL) > 100000)
  {
 	myFile.Close(); 

	 //ПРОЦЕДУРА ФОРМИРОВАНИЯ ИМЕНИ ФАЙЛА
    
	newTime = CTime::GetCurrentTime(); 
   
	SYSTEMTIME lt;
GetLocalTime(&lt);

CString stroke;
stroke = "";
char temp[5];
// Преобразуем целое в строку в десятичном формате
itoa(lt.wYear, temp, 10);
stroke += temp;
stroke += "-";
itoa(lt.wMonth, temp, 10);
stroke += temp;
stroke += "-";
itoa(lt.wDay, temp, 10);
stroke += temp;
stroke += "-";
itoa(lt.wHour, temp, 10);
stroke += temp;
stroke += "-";
itoa(lt.wMinute, temp, 10);
stroke += temp;
stroke += "-";
itoa(lt.wSecond, temp, 10);
stroke += temp;
stroke += ",";
itoa(lt.wMilliseconds, temp, 10);
stroke += temp;
stroke += ".dat";
	
	new_fileName = stroke;

   
    char new_data[30];
    strcpy(new_data,new_fileName);
    szNewFileName = new_data;
	
	myFile.Open(szNewFileName, CFile::modeCreate | CFile::modeWrite);
  
	}
	else
  {	
  myFile.Write(bufrd, strlen(bufrd));  //записать в файл данные из приёмного буфера
  memset(bufrd, 0, BUFSIZE);	        //очистить буфер (чтобы данные не накладывались друг на друга)
  }

}

Функция позволяет производить запись нового (2-го) и последующего файлов, присваивая ему имя аналогично предыдущему (1-му):

ГГГГ-ММ-ДД-ЧЧ-ММ-СС,МСС.dat

Согласен, что код далек от идеала. Поэтому будет дорабатываться. Готов выслушать замечания и предложения.
Отдельное спасибо waleri за помощь.
Очень помог!

[waleri]

Что мешает копировать напрямую из stroke в szNewFileName?

У CString должен быть метод Format() - наверно будет лучше его использовать.

[bedouin]

Что мешает копировать напрямую из stroke в szNewFileName?

Наибольшие сложности при формировании имени у меня возникли с командой strcpy. Возможно, что я еще "плаваю" в форматах данных. Поэтому и получилась такая сложная конструкция:

    new_fileName = stroke;
    char new_data[30];
    strcpy(new_data,new_fileName);
    szNewFileName = new_data;

При случае попробую сделать присваивание напрямую.

У CString должен быть метод Format()

Не до конца понял, что Вы имели в виду.
Можно поподробнее