Question

我有一个我目前坚持的作业问题。问题的参数如下。 1.）它必须接受来自用户的总共5个整数。

2。）它必须有三个线程，每个线程执行不同的功能（平均值，最小值和最大值）。

我遇到的问题是声明一个包含5个元素的全局数组空数组，然后修改这些元素。每次我最终得到一个段错误，告诉我我做错了。顺便说一句，语言是C，重点不是C ++（我不允许使用它）。如果有人能帮我理解什么是错的，我将不胜感激。此外，如果它是重复的（我看了，我看到的没有解决这些问题），请指出我所处理的问题或文章谢谢。

代码：

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

void *avgWorker(int in[]);
void *minWorker(int in[]);
void *maxWorker(int in[]);

int main(void)
{
  int it, *input;

  int in[5];

  pthread_t tid1,tid2,tid3;
  pthread_attr_t attr1, attr2,attr3;

  for (it = 0; it < 5; ++it)
    {
      printf("Please enter number %d of 5\n", (it + 1));

      input[it] = scanf("%d");
    }

  pthread_attr_init(&attr1);
  pthread_attr_init(&attr2);
  pthread_attr_init(&attr3);

  pthread_create(&tid1, &attr1, avgWorker(in), NULL);
  pthread_create(&tid2, &attr2, minWorker(in), NULL);
  pthread_create(&tid3, &attr3, maxWorker(in), NULL);

  pthread_join(tid1,NULL);
  pthread_join(tid2,NULL);
  pthread_join(tid3,NULL);

  return 0;
}

void *avgWorker(int in[])
{
  int total, avg, it;

  total = 0;

  for (it = 0; it < 5; ++it)
    {
      total += in[it];
    }

  avg = 0;

  avg = total / 5;

  printf("\n The average value is: %d. \n", avg);
}

void *minWorker(int in[])
{
  int min, it;
  min = 99999;

  for (it = 0; it < 5; ++it)
    {
      if (in[it] < min)
    min = in[it];
    }

  printf("The minimum value is: %d\n", min);
}

void *maxWorker(int in[])
{
  int max, it;

  max = -99999;

  for (it = 0; it < 5; ++it)
    {
      if (in[it] > max)
    max = in[it];
    }

   printf("The maximum value is: %d \n", max);
}

最后，我正在使用gcc编译此代码，并使用-lpthread标志。再次感谢您提供的任何帮助。

Answer 1

这是应该崩溃的地方：

input[it] = scanf("%d");

scanf返回读取的字段数，而不是读取的数据。存储数据的变量的地址应作为参数传递。像这样：

scanf("%d", &input[it]);

同样input只是一个未初始化的指针。它没有指向任何地方（有意义）。我想你想要in，而不是input。你根本不需要input变量。

Answer 2

你的问题在

行

input[it] = scanf("%d");

查找scanf的语法...您需要提供存储数据的地址，所以

scanf("%d", input+it);

应该是一种改进。或者 - 正如@dreamlax所指出的，更易读的代码

scanf("%d", &input[it]);

多个POSIX线程

2 个答案: