Thread & Classes

[kargo]

Hey i have this class:

class ClientHandler
{
      private:
          //    
      public:
         ClientHandler();
         ~ClientHandler(){}
         DWORD WINAPI ServeClient(LPVOID lpParam);
         DWORD WINAPI WaitClients(PVOID pvParam);
};

and in function WaitClients Im trying to create thread :

CreateThread(NULL, 0, ServeClient, (LPVOID)NextFreeSpace, 0, &thread_id);

Units/ClientHandler.cpp: In member function `DWORD ClientHandler::WaitClients(void*)':
Units/ClientHandler.cpp:115: error: argument of type `DWORD (ClientHandler:(void*)' does not match `DWORD (*)(void*)'

Units/ClientHandler.cpp: In member function `DWORD ClientHandler::ServeClient(void*)':

[Elkvis]

you cannot create a thread from a class member function using CreateThread() unless it is static. you can do it with boost::thread and boost::bind, so I would suggest looking at those to see if they will work for you, if this is something that you need to do with non-static functions.

[kargo]

This Boost C++ Libraries - Browse /boost/1.46.1 at SourceForge.net ?

There is no other way?

[grumpy]

Sure. Create a function (either a global function, or a static member function of a class) that, in turn, invokes the member function on an object.

[kargo]

Ok, i will try. But i have question about crypt my connection, i would like to use openssl lib. So should it be done?

1. Client is sending to server his public key
2. Server random big number and crypt it with client's public key
3. Client use this big number to create secret key and send to server
4. Server use this same big number to create his secret key and send to client

And then server is using client's secret key to crypt and client is using server's secret key to crypt?

[Elysia]

Why do you not want to use boost? You are aware that CreateThread is dangerous if you use C/C++ library functions, yes?

[kargo]

Why do you not want to use boost? You are aware that CreateThread is dangerous if you use C/C++ library functions, yes?

I will try, i just don't like to complicate life by writing 10 lines instead of 1

[Elysia]

o_O
Boost makes life easier. Here is what boost would look like:

boost::thread thread(&ServeClient, NextFreeSpace);

And we're done!
If you want to wait on the thread, add

thread.join();

[kargo]

o_O
Boost makes life easier. Here is what boost would look like:

boost::thread thread(&ServeClient, NextFreeSpace);

And we're done!
If you want to wait on the thread, add

thread.join();

Then i was wrong :P

But any clues about ssl?

[Elysia]

I am not well versed in networking stuff, sorry. I have never used ssl from C++.

[kargo]

Ok

Btw, im trying to create thread this way u wrote but :

Units/ClientHandler.cpp:122: error: ISO C++ forbids taking the address of an unqualified or parenthesized non-static member function to form a pointer to member function. Say `&ClientHandler::ServeClient'

And when i try :

boost::thread thread(&ClientHandler::ServeClient, NextFreeSpace);

Then it shows me errors in bind_template.hpp

[Elysia]

Not quite how it works. Member functions are coupled to instances, so you need to provide an instance in your parameter list. Eg:

ClientHandler handler;
boost::thread thread(&ClientHandler::ServeClient, handler, NextFreeSpace);

Where basically handler would be your instance of the class whose member function ServeClient you would like to call.
Btw, this would never work with CreateThread
Savor the power of boost!

[kargo]

Not quite how it works. Member functions are coupled to instances, so you need to provide an instance in your parameter list. Eg:

ClientHandler handler;
boost::thread thread(&ClientHandler::ServeClient, handler, NextFreeSpace);

Where basically handler would be your instance of the class whose member function ServeClient you would like to call.
Btw, this would never work with CreateThread
Savor the power of boost!

Doing like this again shows me errors in bind_template.hpp

And this :

boost::thread thread(&ClientHandler::ServeClient, handler, (LPVOID)NextFreeSpace);

Units/ClientHandler.o(.text+0x2891):ClientHandler.cpp: undefined reference to `boost::thread::~thread()'
Units/ClientHandler.o(.text$_ZN5boost6threadC1IM13Client HandlerFmPvES2_S3_EET_T0_T1_[boost::thread::thread<unsigned long (ClientHandler::*)(void*), ClientHandler, void*>(unsigned long (ClientHandler::*)(void*), ClientHandler, void*)]+0xfa):ClientHandler.cpp: undefined reference to `boost::thread::start_thread()'

I'm using win7 and dev-c++.

[Elysia]

This is right, actually. You need to compile boost and link to it (which the library should fix by itself, I believe). There are instructions at the boost homepage, or alternatively there might be binaries ready. I know there are for Visual Studio at least.
Oh, and don't cast NextFreeSpace to LPVOID.

EDIT: You might get binaries from here:
http://ascendwiki.cheme.cmu.edu/Bina...Boost_on_MinGW

[kargo]

Weird.. ihave installed binary and if i don't cast NextFreeSpace to LPVOID then i got errors in bind_template.hpp, but now if i cast to LPVOID it works, atleast compile