如何从C中的线程返回一个值

Question

我是C的新手,想稍微玩一下线程.我想从一个线程中返回一些值pthread_exit()

我的代码如下:

#include <pthread.h>
#include <stdio.h>

void *myThread()
{
   int ret = 42;
   pthread_exit(&ret);
}

int main()
{
   pthread_t tid;
   void *status;

   pthread_create(&tid, NULL, myThread, NULL);
   pthread_join(tid, &status);

   printf("%d\n",*(int*)status);   

   return 0;
}

我希望程序输出"42 \n",但它输出一个随机数.如何打印返回的值？

编辑: 根据第一个答案,问题是我返回指向局部变量的指针.返回/存储多个线程的变量的最佳做法是什么？全局哈希表？

提前致谢

Answer 1

您正在返回局部变量的地址,当线程函数退出时,该地址变量不再存在.无论如何,为什么要调用pthread_exit？为什么不简单地从线程函数返回一个值？

void *myThread()
{
   return (void *) 42;
}

然后在主要:

printf("%d\n",(int)status);   

如果你需要返回一个复杂的值这样的结构,最简单的方法是通过malloc()动态分配它并返回一个指针.当然,启动该线程的代码将负责释放内存.

Answer 2

您已经返回了一个指向局部变量的指针.即使没有涉及线程,这也很糟糕.

执行此操作的常用方法是,当启动的线程是连接的同一个线程时,将在调用者管理的位置传递指向int的指针,作为pthread_create的第4个参数.然后它成为线程入口点的(唯一)参数.您可以(如果您愿意)使用线程退出值来表示成功:

#include <pthread.h>
#include <stdio.h>

int something_worked(void) {
    /* thread operation might fail, so here's a silly example */
    void *p = malloc(10);
    free(p);
    return p ? 1 : 0;
}

void *myThread(void *result)
{
   if (something_worked()) {
       *((int*)result) = 42;
       pthread_exit(result);
   } else {
       pthread_exit(0);
   }
}

int main()
{
   pthread_t tid;
   void *status = 0;
   int result;

   pthread_create(&tid, NULL, myThread, &result);
   pthread_join(tid, &status);

   if (status != 0) {
       printf("%d\n",result);
   } else {
       printf("thread failed\n");
   }

   return 0;
}

如果您必须使用结构的线程退出值,那么您将必须动态分配它(并确保加入该线程的任何人释放它).但这并不理想.

Answer 3

这是一个正确的解决方案.在这种情况下,tdata在主线程中分配,并且线程有一个空间来放置其结果.

#include <pthread.h>
#include <stdio.h>

typedef struct thread_data {
   int a;
   int b;
   int result;

} thread_data;

void *myThread(void *arg)
{
   thread_data *tdata=(thread_data *)arg;

   int a=tdata->a;
   int b=tdata->b;
   int result=a+b;

   tdata->result=result;
   pthread_exit(NULL);
}

int main()
{
   pthread_t tid;
   thread_data tdata;

   tdata.a=10;
   tdata.b=32;

   pthread_create(&tid, NULL, myThread, (void *)&tdata);
   pthread_join(tid, NULL);

   printf("%d + %d = %d\n", tdata.a, tdata.b, tdata.result);   

   return 0;
}

Answer 4

我认为您必须将数字存储在堆上。该int ret变量在堆栈上，并在函数执行结束时被销毁myThread。

void *myThread()
{
       int *ret = malloc(sizeof(int));
       if (ret == NULL) {
           // ...
       }
       *ret = 42;
       pthread_exit(ret);
}

free当您不需要它时，请不要忘记它：)

另一个解决方案是像Neil Butterworth所建议的那样，将数字作为指针的值返回。