Pthread_join 返回变量值为 0

Question

我想在以下代码中插入 pthread_join 函数来终止我的线程并更新变量值。之后我的想法是创建一个变量来添加我从线程中获得的新值并将其打印出来。我有以下代码，似乎工作正常，但我想直接更新变量 a 和 b 而无需任何额外变量（我不想使用 aux_soma 和 aux_multiplicacao）。如果我尝试直接更新它们，它们的值会设置为 0（主要是变量 a）。有什么办法可以按照我想要的方式进行吗？代码是用葡萄牙语写的，很抱歉，我希望你无论如何都能理解它。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/types.h>
#include <unistd.h>
#include <pthread.h>

void *soma(void *valor)
{
  int a = (intptr_t) valor;
  a = 10 + a;
  //printf("A thread soma terminou.\n");
  pthread_exit((void*)(intptr_t)a);
}

void *multiplicacao(void *valor)
{
  int a = (intptr_t) valor;
  a = 10 * a;
  //printf("A thread multiplicacao terminou.\n");
  pthread_exit((void *)(intptr_t)a);
}

int main()
{
  pthread_t p, t;

  int a = 5, b = 5;
  int aux_soma, aux_multiplicacao; // Variáveis auxiliares para evitar erros

  printf("\n\n"); // Duas linhas em branco; Para ficar separado e mais apresentável

  // Estava a dar erro no valor das variáveis e da soma, desta maneira não há erros
  int ra = pthread_create(&p, NULL, soma, (void *)(intptr_t)a);
  int rb = pthread_create(&t, NULL, multiplicacao, (void *)(intptr_t)b);
  pthread_join(t, (void **) &aux_multiplicacao);
  pthread_join(p, (void **) &aux_soma);

  a = aux_soma;
  b = aux_multiplicacao;

  int soma_ab = a + b;

  printf("\nIDthread soma = %d\n", (int) p);
  printf("IDthread multiplicacao = %d\n", (int) t);
  printf("a = %d\n", a);
  printf("b = %d\n", b);
  printf("Soma: a + b = %d\n", soma_ab);

  exit(0);
}

提前致谢。

Answer 1

题中的代码是将a和b的值传递给线程函数，然后通过pthread_join取回结果。更好的解决方案是将 a 和 b 的地址传递给线程函数。然后函数可以得到a和b的初始值。并且函数可以直接更新 a 和 b 的值。

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

void *soma(void *valor)
{
    int *aptr = valor;
    int a = *aptr;    // get the initial value of 'a'

    a = 10 + a;       // do something with 'a'

    *aptr = a;        // set the final value of 'a'
    return NULL;
}

void *multiplicacao(void *valor)
{
    int *aptr = valor;
    int a = *aptr;    // get the initial value of 'a'

    a = 10 * a;       // do something with 'a'

    *aptr = a;        // set the final value of 'a'
    return NULL;
}

int main()
{
    int a = 5, b = 5;

    pthread_t p, t;
    int ra = pthread_create(&p, NULL, soma, &a);
    int rb = pthread_create(&t, NULL, multiplicacao, &b);
    if (ra != 0 || rb != 0)
        exit(1);
    pthread_join(t, NULL);
    pthread_join(p, NULL);

    int soma_ab = a + b;

    printf("a = %d\n", a);
    printf("b = %d\n", b);
    printf("Soma: a + b = %d\n", soma_ab);
}

Answer 2

您所做的大部分工作在实践中都很好，并且可以说是最佳的，这依赖于通过指针类型来回整数值的能力。这避免了管理指向对象的生命周期的需要，这可能是危险错误的来源。但是你的程序有严重的 UB，编译器显然通过警告告诉你，然后你试图用强制转换来掩盖：

pthread_join(t, (void **) &aux_multiplicacao);
pthread_join(p, (void **) &aux_soma);

转换为 void ** 几乎总是错误的。

在这里，您告诉 pthread_join 将类型 void * 的对象存储在 &aux_multiplicacao（然后在 &aux_soma），但是该地址的对象没有正确的类型（或者甚至大小，一般来说）来存储这样的对象。相反，您需要：

void *tmp;
pthread_join(t, &tmp);
aux_multiplicacao = (intptr_t)tmp;
pthread_join(p, &tmp);
aux_soma = (intptr_t)tmp;

Pthread_join 返回变量值为 0

Pthread_join returning variable value as 0

c

pthreads

pthread-join