在文件中写入char数组时访问冲突

时间:2017-02-23 09:40:51

标签: c++ multithreading pointers ofstream

我正在编写一个小程序,它为主程序执行所有“写入文件”工作。数据是结构,大小不一致。它们中有很多,将来还会添加新的。 因此,我决定使用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类的对象。

2 个答案:

答案 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;功能