_beginthreadex线程安全问题

[merry_wanwan]

class testClass
{
   public:
         testClass();
         testClass(testClass & test);
   string param1;
   string param2;
}

//默认构造
testClass::testClass():param1("hi"), param("hello")
{}

//复制构造
testClass::testClass(testClass & test)
{
   param1 = test.param1;
   param2 = test.param2;
}

//这个函数开启线程
void CallThread( testClass param )
{
     HANDLE hThread;
     unsigned threadId;
     hThread = _beginthread(NULL, 0, ThreadWork, ¶m, 0, &threadId);
}

//线程执行函数
unsigned __stdcall ThreadWork(void * param)
{
   //在这里取出传进来的值
   testClass test = *((testClass*)param);
   //这里再做许多的事情，也许会很久
}

当CallThread创建进程之后，函数会释放param ，导致线程中取值为野指针。
好像“线程传参”只能传递指针，不能复制传递。
请教各位如何解决这个问题，先谢谢。（代码是直接在这里手打的，也许有点问题）

[bdmh]

传testClass * 或testClass &，否则 会创建一个，然后释放

[merry_wanwan]

[Quote=引用 1 楼 bdmh 的回复:]
传testClass * 或testClass &，否则 会创建一个，然后释放
[/Quote]

bdmh的意思是这样吗?
void CallThread( testClass * param )
void CallThread( testClass & param )

//例如这样
void CallThread( testClass & param );
main()
{
  while(1)
  {
     testClass param = GetTestClass();
     CallThread(param );
  }
}

这样也会由于上层调用的释放而变成野指针吧？

[高性能架构探索]

testClass *test = (testClass*)param;
用指针吧，我最开始也犯过你这样的错误

[merry_wanwan]

[Quote=引用 3 楼 namelij 的回复:]
testClass *test = (testClass*)param;
用指针吧，我最开始也犯过你这样的错误
[/Quote]


是这样吗？

//线程执行函数
unsigned __stdcall ThreadWork(void * param)
{
   //在这里取出传进来的值
   testClass * test = (testClass*)param;
   //这里再做许多的事情，也许会很久
}

测试了一下，还是野指针。
个人觉得，如果传的是指针，当原始指针释放的时候，还是会野吧？
或者是如果有指针在使用这个地址就不会释放？（好像没在书上看到过。。）

[PG]

顶~~

[bdmh]

看看你具体怎么用的，是全局指针还是局部指针

[merry_wanwan]

都是局部的，没有全局变量
有没有不用全局可以解决的方法呢？

[PG]

在hThread = _beginthread(NULL, 0, ThreadWork, ¶m, 0, &threadId);
后面sleep(1000)吧；

[merry_wanwan]

[Quote=引用 8 楼 peige2008 的回复:]
在hThread = _beginthread(NULL, 0, ThreadWork, &param, 0, &threadId);
后面sleep(1000)吧；
[/Quote]
100也许做不完，如果waitFOrSingleOBject也不行，因为要多线程循环接受任务

[PG]

testClass test = *((testClass*)param);
你这个东西能取出来就可以了^ ^!!!

[迷途的书童]

如果你是个全局变变量或成员函数，只要程序不挂就不会有问题！

[merry_wanwan]

[Quote=引用 10 楼 peige2008 的回复:]
testClass test = *((testClass*)param);
你这个东西能取出来就可以了^ ^!!!
[/Quote]
如果waitFOrSingleOBject，会导致上层没有释放，可以取的，但是程序流程不允许使用waitFOrSingleOBject....

[Quote=引用 11 楼 guanyijun123 的回复:]
如果你是个全局变变量或成员函数，只要程序不挂就不会有问题！
[/Quote]
全局不会释放。成员函数？这个没有看懂，可以解释下吗？

[PG]

你原来的代码加个sleep就可以运行了。
至于你信不信，由你，我反正是信了~

[merry_wanwan]

[Quote=引用 13 楼 peige2008 的回复:]

你原来的代码加个sleep就可以运行了。
至于你信不信，由你，我反正是信了~
[/Quote]
这个应该是有隐患的
例如sleep(1000);
如果线程中在这1000ms之后，再去读取这个变量，则出错

但是如果进入线程马上取出来保存，是可以的。（隐患在于:如果在1000ms内，该线程还没有启动呢？）

[captainlee007]

[Quote=引用 4 楼 merry_wanwan 的回复:]

引用 3 楼 namelij 的回复:
testClass *test = (testClass*)param;
用指针吧，我最开始也犯过你这样的错误



是这样吗？
C/C++ code

//线程执行函数
unsigned __stdcall ThreadWork(void * param)
{
//在这里取出传进来的值
testClass * test = (……
[/Quote]

void CallThread( testClass *param )

[PG]

[Quote=引用 14 楼 merry_wanwan 的回复:]

引用 13 楼 peige2008 的回复:

你原来的代码加个sleep就可以运行了。
至于你信不信，由你，我反正是信了~

这个应该是有隐患的
例如sleep(1000);
如果线程中在这1000ms之后，再去读取这个变量，则出错

但是如果进入线程马上取出来保存，是可以的。（隐患在于:如果在1000ms内，该线程还没有启动呢？）
[/Quote]
那就5秒~

[merry_wanwan]

[Quote=引用 15 楼 captainlee007 的回复:]
void CallThread( testClass *param )
[/Quote]

//例如这样
void CallThread( testClass * param );
main()
{
  while(1)
  {
     testClass * param = GetTestClass();
     CallThread(param );
  }
}

也是会释放啊

[merry_wanwan]

[Quote=引用 16 楼 peige2008 的回复:]

引用 14 楼 merry_wanwan 的回复:

引用 13 楼 peige2008 的回复:

你原来的代码加个sleep就可以运行了。
至于你信不信，由你，我反正是信了~

这个应该是有隐患的
例如sleep(1000);
如果线程中在这1000ms之后，再去读取这个变量，则出错

但是如果进入线程马上取出来保存，是可以的。（隐患在于:如果在1000ms内，该线程还……
[/Quote]
这个方法和全局其实是一样的，如果在取值之前，变量被释放，或者全局变量被再次调用覆盖，还是会出问题。。难道架构设计得不合理吗

[merry_wanwan]

好吧，用一个比较挫的方法
在线程外面new,线程里面delete。但是如果忘记释放了就泄露了。