Question

我有以下代码，其中两个数组被不同地修改（在一种情况下，每个元素递增1，在另一种情况下每个元素递减1）由两个线程修改。它给出了分段错误。你能指出错误吗？

#include <stdio.h>
#include <pthread.h>
#include <stdlib.h>
void *decrement(void *object){

  int *array = (int *)object;
  for(int i=0;i<5;i++){
    array[i]=array[i]-1;
  }

}

void *increment(void *object){

  int *array = (int *)object;
  for(int i=0;i<5;i++){
    array[i]=array[i]+1;
  }
}
int main(int argc, char const *argv[]){
  int segments = 2;
  pthread_t threads[segments];
  int i;

  int numbers[5] = {1,3,2,9,11}; 
  int numbers1[5] = {0,14,12,4,10};



  int *array;
  array=numbers;
  pthread_create(&threads[0], NULL, increment, (void *) &array);

  int *array1;
  array1=numbers1;
  pthread_create(&threads[1], NULL, decrement, (void *) &array1);
  for(i = 0; i < 2; i++){
      pthread_join(threads[i], NULL);
  }
  for(i = 0; i < 5; i++){
      printf("%d\n", array[i]);
      printf("%d\n", array1[i]);
  }
  pthread_exit(NULL);
}

Answer 1

让我们走这些路线：

int *array;
array=numbers;
pthread_create(&threads[0], NULL, increment, (void *) &array);

这里将变量array的地址传递给线程函数，但是array本身指向数组，变量的地址不是。

如果要使用临时变量，只需单独使用变量：

pthread_create(&threads[0], NULL, increment, array);

请注意缺少强制转换和缺少地址操作符。

但是你并不真正需要临时array变量，因为数组会衰减到指针，所以你可以这样做

pthread_create(&threads[0], NULL, increment, numbers);

为了更加图解地说明您的问题，您拥有的是：

+--------+      +-------+      +---------+
| &array | ---> | array | ---> | numbers |
+--------+      +-------+      +---------+

Pthread连接分段错误

1 个答案: