Блог

У меня возникают проблемы с вызовом моих функций из моей основной программы.
Эти функции ДОЛЖНЫ быть в моем классе.
Как мне получить к ним доступ из моего int main()?

 #include <iostream>
#include <stdio.h>
#include <pthread.h>
#include <stdlib.h>
#include <math.h>
#include <sys/types.h>
#include <semaphore.h>
#include <synch.h>
using namespace std;   


class myCountingSemaphoreUsingBinarySemaphore {  
public:  
    void waitSemaphore(pthread_mutex_t *thread)  
    {  
        pthread_mutex_lock(*thread);// Makes value 1 (Not Available)  
    }  
    void signalSemaphore(pthread_mutex_t *thread)  
    {  
        pthread_mutex_unlock(*thread); // Makes value 0 (Available)  
    }  
    void deleteSemaphore(pthread_mutex_t *thread)   
    {  
        pthread_mutex_destroy(*thread);// Deletes  
    }  
};  
int readerCount;   
int database = (rand() / 100); // Number less than 1000  
void reader_writer(void);   
int main(int argc, char *argv[])  
{     
    myCountingSemaphoreUsingBinarySemaphore obj;  
    pthread_mutex_t mutex1;  
    pthread_mutex_t wrt;  
    pthread_create( amp;mutex1, NULL, reader_writer, void);  
    pthread_create( amp;wrt, NULL, reader_writer, void);  
    //----------------------READER------------------------//  
    do{  
        cout << "Database Before Read = " << database << endl;  
        obj.waitSemaphore(mutex1);//lock  
        readerCount  ;  
        if (readerCount == 1)  
        {  
        obj.waitSemaphore(wrt);//lock  
        obj.signalSemaphore(mutex1);//unlock  
        //reading is preformed  
        obj.waitSemaphore(mutex1); // lock  
        readerCount--;  
        }  
        if(readerCount == 0)  
        {  
            obj.signalSemaphore(wrt);//unlock  
            obj.signalSemaphore(mutex1); // unlock  
        }  
        cout << "Database After Read = " << database << endl;  
    }while (true);  
    //-----------------------WRITER---------------------//  
    do{  
        cout << "Database Before Write = " << database << endl;  
        obj.waitSemaphore(wrt);//lock  
        //writing is preformed  
        database = database   10;  
        obj.signalSemaphore(mutex1);//unlock  
        cout << "Database After Write = " << database << endl;  
    }while(true);  
    pthread_join( mutex1, NULL);  
    pthread_join( wrt, NULL);   
    obj.deleteSemaphore(* mutex1);  
    obj.deleteSemaphore(* wrt);  
    return 0;   
}  
void reader_writer () {}  
  

Вот ошибка, которую я получаю:

какого типа они должны быть? pthread_mutex_t_create? или pthread_t_create?
каков правильный тип?

Функции внутри класса называются методами. Вам необходимо создать экземпляр объекта этого класса, чтобы иметь возможность использовать его методы:

 myCountingSemaphoreUsingBinarySemaphore obj; // obj is an instance of the class

obj.waitSemaphore(amp;mutex1);

obj.signalSemaphore(amp;mutex1);
  

Редактировать:

Кстати, pthread_create и pthread_join принимают pthread_t* , а не мьютекс!

 int pthread_create(pthread_t* thread, 
                   pthread_attr_t* attr, 
                   void* (*start_routine)(void*), 
                   void* arg);
  

Вы можете либо объявить эти методы как статические, либо использовать объект для выполнения вызовов:

 myCountingSemaphoreUsingBinarySemaphore s;
s.waitSemaphore(wrt);
  

Вы вызываете методы класса просто waitSemaphore без создания объекта myCountingSemaphoreUsingBinarySemaphore.

Сначала вы должны создать объект.

 myCountingSemaphoreUsingBinarySemaphore obj;
obj.waitSemaphore(mutex1);
  

Два потока, которые вы создаете (через reader_writer() ), ничего не делают. main() просто переходит в первый do цикл без возможности выхода.

Кроме того, вы, похоже, перепутали мьютекс, семафоры и переменные условия. Имена функций создают впечатление, что вы пытаетесь реализовать переменные условия в своем классе. Но вы создаете ее как просто оболочку для блокировок мьютексов.

И, наконец, вы вызываете pthread_mutex_lock() и др. на a pthread_t когда эти функции должны вызываться на a pthread_mutex_t .

Вероятно, есть и другие ошибки, но это те, которые действительно выскакивают наружу. В принципе, вам нужно пересмотреть многопоточное программирование, как с точки зрения того, как создаются потоки, так и с точки зрения того, как они синхронизируются.

Вам нужно создать экземпляр класса (объект), чтобы вызывать его функции-члены.

В этом конкретном коде функции-члены не имеют оснований быть экземплярами и могут быть статическими:

 class foo{
    public:

    static void bar(int val)
    {
    //do something
    }
    };

int main()
{
   foo::bar(10);
}
  

карлфиллип был прав, вам нужно передавать по указателю вместо ссылки

Кстати, следующая строка также является ошибкой, pthread_create принимает и pthread_t вместо pthread_mutex_t

pthread_create( amp;mutex1, NULL, reader_writer, void);

pthread_create( amp;wrt, NULL, reader_writer, void);