我正在编写一个小程序,它为主程序执行所有“写入文件”工作。数据是结构,大小不一致。它们中有很多,将来还会添加新的。 因此,我决定使用char数组,并从方法到方法将指针移交给这些数组。
70%的时间代码按预期运行,但是在写入数组时经常会出现“访问冲突”错误,有时这个错误会连续发生。我只是找不到一种模式。
要注意的是Execute在不同的线程上运行。
struct BufferElement
{
char* ptrData;
int TotalBytes;
int StartPosition;
std::string FileName;
};
class FileThread
{
private:
std::vector<BufferElement> Queue;
bool BoolThread;
std::ofstream writestream;
//This Method calls the "WriteInFile" Method for the first element in the
//Queue and erases it from the vector
void Execute( void )
{
while( BoolThread )
{
if( Queue.size() > 0 )
{
if( WriteInFile() )
{
delete[] Queue.at( 0 ).ptrData;
Queue.erase( Queue.begin() );
}
}
}
}
//This Method writes the first Element of the Queue in the file
bool WriteInFile( void )
{
if( Queue.at(0).ptrData == NULL )
{
return true;
}
writestream.open( Queue.at(0).FileName.c_str(), std::ios::in |
std::ios::out | std::ios::binary );
if( !writestream.is_open() )
{
writestream.close();
writestream.clear();
return false;
}
writestream.seekp( Queue.at( 0 ).StartPosition );
writestream.write( Queue.at( 0 ).ptrData, Queue.at( 0 ).TotalBytes );
writestream.close();
writestream.clear();
return true;
}
public:
void EndThread(void)
{
BoolThread = false;
for( int i = 0; i < Queue.size(); i++ )
{
delete[] Queue.at( i ).ptrData;
}
}
template< typename T >
void WriteOrder( std::string _FileName, T _Data, int _StartPosition )
{
BufferElement Temp_BufferElement;
Temp_BufferElement.TotalBytes = sizeof( _Data );
Temp_BufferElement.StartPosition = _StartPosition;
Temp_BufferElement.FileName = _FileName;
Temp_BufferElement.ptrData = new char[ Temp_BufferElement.TotalBytes ];
memcpy( Temp_BufferElement.DataPtr, _Data, Temp_BufferElement.TotalBytes );
Queue.push_back( Temp_BufferElement );
}
};
int main(void)
{
std::string Path = "..\\Data\\Test.dat";
FileThread Writer;
for( int i = 0; i < 1000; i++ )
{
char array[] = {'H','e','l','l','o',' ','W','o','r','l','d','!','\0'};
Writer.WriteOrder( Path, array, i * sizeof( array );
}
system("pause");
Writer.EndThread();
return 0;
}
如果有人可以查看代码,我会很高兴。也许我只是忽视了一些事情 我正在使用Borland Turbo C ++ Builder,而Thread是来自vcl类的对象。
答案 0 :(得分:2)
您需要同步对队列的访问权限。如果主线程在工作线程修改它时写入队列,则可能会崩溃。此外,如果执行push_back
,则阵列内存可能无效,而工作者仍在使用该无效内存。
我也看不到BoolThread
初始化或更改的位置。所以要么这个例子不完整,要么就可能表现得很奇怪。
答案 1 :(得分:1)
void WriteOrder( std::string _FileName, T _Data, int _StartPosition )
{
BufferElement Temp_BufferElement;
Temp_BufferElement.TotalBytes = sizeof( _Data );
Temp_BufferElement.StartPosition = _StartPosition;
Temp_BufferElement.FileName = _FileName;
Temp_BufferElement.ptrData = new char[ Temp_BufferElement.TotalBytes ];
memcpy( Temp_BufferElement.DataPtr, _Data, Temp_BufferElement.TotalBytes );
Queue.push_back( Temp_BufferElement );
}
如果这样称呼:
Writer.WriteOrder( Path, array, i * sizeof( array ));
Temp_BufferElement.TotalBytes将保持4(或8)(sizeof(_Data))似乎这不是你想要的。这一行:
Temp_BufferElement.ptrData = new char[ Temp_BufferElement.TotalBytes ];
memcpy( Temp_BufferElement.DataPtr, _Data, Temp_BufferElement.TotalBytes );
将在堆上分配并复制4个字节(取决于指针的大小,32/64位程序)
此代码:
Queue.push_back( Temp_BufferElement );
在Queue.size()== Queue.capacity()执行此代码之前,向量将重新分配自身,导致执行线程的同步问题&#34;执行&#34;功能