我正在尝试在单独的线程中处理向量的元素,并将结果放在不同的向量中。我已尝试使用互斥体和代码周围的关键部分,我检查并从输入向量中取出元素,但是当我运行代码时,我遇到了访问冲突。
编辑:我已经更新了代码,将结果放回到不同关键部分的向量中,并在线程开始之前初始化向量。
#include "stdafx.h"
#include <windows.h>
#include <stdlib.h>
#include <string.h>
#include <stdio.h>
#include <conio.h>
#include <process.h>
#include <iostream>
#include <vector>
#define MAX_THREADS 4
void InvertProc( void * MyID ); // Threads 2 to n: display
void ShutDown( void ); // Program shutdown
int ThreadNr; // Number of threads started
CRITICAL_SECTION cs, cs2;
std::vector<int> _oTempVector;
std::vector<int> _oOutVector;
int OutCounter;
int _tmain(int argc, _TCHAR* argv[])
{
ThreadNr = 0;
OutCounter = 0;
for ( int i = 0; i < 50000; i++ ) {
_oTempVector.push_back( i );
_oOutVector.push_back( 0 );
}
InitializeCriticalSection( &cs );
InitializeCriticalSection( &cs2 );
std::vector<HANDLE> events;
for ( ThreadNr = 0; ThreadNr < MAX_THREADS; ThreadNr++ ) {
HANDLE handle = (HANDLE)_beginthread( InvertProc, 0, &ThreadNr );
events.push_back( handle );
}
WaitForMultipleObjects( events.size(), &events[0], true, INFINITE );
std::cout << "outvector contains:" << _oOutVector.size() << "elements";
std::cout << '\n';
}
void InvertProc( void *pMyID )
{
do {
EnterCriticalSection( &cs );
if ( _oTempVector.size() > 0 ) {
int iElement = _oTempVector.back();
_oTempVector.pop_back();
LeaveCriticalSection( &cs );
iElement *= -1;
EnterCriticalSection( &cs2 );
_oOutVector[OutCounter] = iElement;
OutCounter++;
LeaveCriticalSection( &cs2 );
}
} while ( _oTempVector.size() > 0 );
}
答案 0 :(得分:0)
你的输出向量不在临界区内作为它应该的共享对象.... 因为如果有几个线程同时尝试push_back,那么你将面临Write Over Write Data竞赛!!!!
EnterCriticalSection( &cs );
if ( _oTempVector.size() > 0 ) {
int iElement = _oTempVector.back();
_oTempVector.pop_back();
iElement *= -1;
_oOutVector.push_back( iElement );
LeaveCriticalSection( &cs );
}
} while ( _oTempVector.size() > 0 );
生成的运行线程将比并发运行更顺序
要解决这个问题,你必须采取不同的方式: 1)你可以拆分问题,每个线程应该对输入向量的连续元素起作用 2)如果输出向量在输入的大小开始时初始化,则删除由于推送引起的所有问题!
每个正在运行的线程将* -1一个给定的数字范围输入,将结果放在输出向量的特定位置(没有交互)与另一个线程,这样你就可以删除锁
void InvertProc( void *pMyID )
{
int threadnum = *((int*)pMyID);
int chunk = input.size()/THREAD_NUM;
int start = threadnum*chunk;
int end = start+chunk;
for (I = start ; I < end ; ++I)
{
output[i] = input[i]*-1;
}
}