-1

我有5个作家，20个读者。我想用二进制信号量解决读写器问题。

但是我的代码有问题。存在分段错误（核心转储）。我认为创建线程时存在问题。我该如何解决这个问题？这是解决读写问题的正确代码吗？我使用了教科书的伪代码。

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

sem_t mutex, rw_mutex;

int data = 0;
int readcount = 0;

void *reader(void* i)
{
    int num = *((int*)i);
    sem_wait(&mutex);
    readcount += 1; 
    if(readcount == 1)
            sem_wait(&rw_mutex);
    sem_post(&mutex);

    printf("I'm reader%d, data is %d \n", num, data);
    sem_wait(&mutex);

    readcount -= 1;
    if( readcount == 0)
            sem_post(&rw_mutex);
    sem_post(&mutex);
}

void *writer(void *i)
{
      int num = *((int*)i);
      sem_wait(&rw_mutex);
      data++;
      printf("I'm writer%d, data is %d\n", num, data);
      sem_post(&rw_mutex);
}

void main()
{
      int i;
      pthread_t writer[5], reader[20];
      sem_init(&rw_mutex, 0, 1);
      sem_init(&mutex, 0, 1);

      for(i=0; i<5; i++)
              pthread_create(&writer[i], NULL, writer, &i);
      for(i=0; i<20; i++)
              pthread_create(&reader[i], NULL, reader, &i);
      for(i=0; i<5; i++)
            pthread_join(writer[i], NULL);
      for(i=0; i<20; i++)
              pthread_join(reader[i], NULL);
      printf("End \n");
}
4

1 回答 1

2

您是否检查过编译器的警告？我收到几个警告。一个例子是：

警告：从不兼容的指针类型传递“pthread_create”的参数 3 [默认启用] pthread_create(&reader[i], NULL, reader, &i);

问题是在main你有一个名称为数组reader但在程序中你也有一个名为reader. 所以编译器（即至少我的编译器）在您实际需要该函数时使用该数组。和程序崩溃。

修复警告！通过重命名函数reader或writer重命名数组。

之后，我再也看不到程序崩溃了。

于 2019-05-28T12:52:52.883 回答