illegal call of non-static member function -help-

[BlueOrca]

Im still new to c++, and class's are some thing im still trying to work out but i carnt figure out what im doing rong here.


: error C2352: 'ThreadClass::StopGoFunc_Go' : illegal call of non-static member function

: error C2597: illegal reference to data member 'ThreadClass::StopGo' in a static member function

class ThreadClass{
public:
 
	ThreadClass()
	{
		m_dwThreadID = 0;
		m_hThread    = 0;
	}

    ~ThreadClass() { 
		CloseHandle(m_hThread); 
	}
	 
	bool Creat_A_Thread()	 
	{
		m_hThread = CreateThread(0,0,StopGoThread,this,0,&m_dwThreadID);    
		
		if(!m_hThread)   
		{
            printf("Thread Failed");
			return false;
		}
		return true;              
	}
   
	void StopGoFunc_Go() {  StopGo = true;  }
       void StopGoFunc_Stop() {  StopGo = false;  }

private:
	DWORD  m_dwThreadID;
	HANDLE m_hThread; 
        static DWORD WINAPI StopGoThread(LPVOID pvParam);
	bool StopGo;
		
};

DWORD WINAPI ThreadClass::StopGoThread(LPVOID pvParam)
{
	StopGoFunc_Go();

    while(StopGo = true)
    {
       printf("Im Going"); 
	   Sleep(2500);
    }

	return 0;
}

int main()
{
   ThreadClass MakeThread;
   MakeThread.Creat_A_Thread();

   Sleep(10000); 

   MakeThread.StopGoFunc_Stop();

	return 0;
}

[Vedam Shashank]

What language have u programmed in before???

Consider the following class...

class CMyClass
{
public:
       int iNum;
       void Increament() {std::cout << ++iNum << endl;}
};

For the class, if u want to access the data member (iNum), or if u want to call the member function - Increament()... U MUST CREATE AN OBJECT of the class.

this is one way how u can create and call the members

CMyClass obj;    //Creates an object

obj.iNum = 5;     //Now u can access the objects members

obj.Increament();    //or call the function

iNum and Increament are non-static member and method respectively.

[BlueOrca]

I made the object in the main function to call the thread function :
Or did i do some thing rong ?

int main()
{
   ThreadClass MakeThread;           <-----------
   MakeThread.Creat_A_Thread();
...

I dont think i need to make an object in the thread function
DWORD WINAPI ThreadClass::StopGoThread(LPVOID pvParam)
because that is a member function !

PS: C++ is my first language.

[Kheun]

The problem is that you have to use a static member function for spawning a thread. However, in static member funciton, the "this" pointer isn't available. Therefore, to access other non-static member in a static member function, you have to pass the address of your object in the LPVOID parameter. When in the function, you have to use this pointer.

DWORD WINAPI ThreadClass::StopGoThread(LPVOID pvParam)
{
    ThreadClass *p = (ThreadClass *)pvParam;
    p->StopGoFunc_Go();

    while(StopGo == true)
    {
       printf("Im Going"); 
	   Sleep(2500);
    }

	return 0;
}

[BlueOrca]

Thank you very much

[atlas2050]

Hi, i'm usig POSIX Threads and i have a problem like that but, i have done all that you say and at the time of compile my program all is ok but when the ejecution arrives at the point where i call the function he quit the program whit this code: First-chance exception in OpenGL.exe: 0xC0000005: Access Violation. Here is my code.


SRRT::SRRT(Problem *p):RRT(p) {
p->Tipo=1;
}

bool SRRT::Plan()
{
int i;
Planner *Pl;



MSLVector InitialState(3);

pthread_t Threads[10]; //Numero de hilos maximos a crear es 10
READ_PARAMETER_OR_DEFAULT(NumBodies,1);



for (i=0; i<NumBodies; i++) {

pthread_create(&Threads[i],NULL,SRRT::Exp_princ,&Pl);
}


for (i=0; i<NumBodies; i++) {
pthread_join(Threads[i],NULL); // wait for threads
}

pthread_mutex_destroy(&Pl->CountMutex);
//pthread_mutex_destroy(&Pl->QueueMutex);


return true;
}


void *SRRT::Exp_princ(void *X)
{

Planner *Temp;
SRRT *S;
int i,id_hilo,m;
char cadena[1024];
ZeroMemory(cadena, 1024);

// (ThreadClass *)

Temp = static_cast<Planner *>(X);


if ((Temp->hilo < 0) || (Temp->hilo > 10))
{
Temp->CountMutex=PTHREAD_MUTEX_INITIALIZER;
pthread_mutex_lock(&Temp->CountMutex);
Temp->hilo=0;
pthread_mutex_unlock(&Temp->CountMutex);

}
else{
pthread_mutex_lock(&Temp->CountMutex);
Temp->hilo++;
pthread_mutex_unlock(&Temp->CountMutex);
}


id_hilo=Temp->hilo;

Temp->Tm1[id_hilo]=NULL;

wsprintf(cadena, "Hilo No. %d",id_hilo);

MessageBoxA( NULL,cadena, "Test", MB_OK );


m=Temp->prueba(3); //---------HERE IS MY PROBLEM

return (0);

}



That part is of a class named SRRT, there is other class named Planner



int Planner:rueba(int f)
{
int i=f;

for (int v=0; v<f;v++)
MessageBoxA( NULL,"La prueba", "Test", MB_OK );


return i;

}



I hope you can help me

[Kheun]

Please see the line highlighted in red. I believe you made a mistake by declaring a pointer and ended up passing an uninitialized pointer to the function, SRRT::Exp_princ().

bool SRRT::Plan()
{
int i;
//Planner *Pl;
Planner Pl;  //Should be declaring a real instance instead of a pointer.



MSLVector InitialState(3);

pthread_t Threads[10]; //Numero de hilos maximos a crear es 10
READ_PARAMETER_OR_DEFAULT(NumBodies,1);



for (i=0; i<NumBodies; i++) {

pthread_create(&Threads[i],NULL,SRRT::Exp_princ,&Pl);
}


for (i=0; i<NumBodies; i++) {
pthread_join(Threads[i],NULL); // wait for threads
}

pthread_mutex_destroy(&Pl->CountMutex);
//pthread_mutex_destroy(&Pl->QueueMutex);


return true;
}