Problem Creating Threads

[Walta69]

My problem is simple... I am creating a TCP server and I need threads to process requests
and track time. However when i try create a thread it gives the error:

"error C2064: term does not evaluate to a function taking 1 arguments"

Here is some of the code:

sf::Thread * t = new sf::Thread(&Server::time, 0);
t->launch();

Where there is a method:

void Server::time(int clientIndex)

Any help would be welcomed

[G.]

you can use several types of entry points:

non-member functions with no argument
non-member functions with one argument of any type
functors with no argument (this one is particularly useful for compatibility with boost/std::bind)
functors with one argument of any type
member functions from any class with no argument

I don't see a "member function with an argument" in the doc.

[Walta69]

Alright :p so any advice on how to fix this issue?

[Walta69]

Note:

Even if I change it to std::thread th(&Server::time); (firstly changing the time function),
it still doesn't work and gives the error "term does not evaluate to a function taking 0 arguments"

Here are portions of the code:

#include "Server.h"
void Server::time()
{
   int clientIndex = 0;
   sf::Clock c;
   while (true)
   {
      sf::Time time = c.getElapsedTime();
      if (time.asSeconds() > 6)
      {
         sf::Packet p;
         std::string message = "Time Done";
         p << message;
         clients[clientIndex]->send(p);
         return;
      }
   }
}

Server::Server(int port)
{

...
...
...

sf::Thread th(&Server::time);
}

This is all done in the implementation of Server.h

[Walta69]

sf::Thread,

not std::thread

[G.]

sf::Thread needs to know what variable to use. Which in your case is "this".
sf::Thread th(&Server::time, this);

[Laurent]

Use std::thread (and lambdas and/or std::bind) if your compiler supports it. It's much better.

[Walta69]

Even with std::thread I still get problems:

std::thread th(&time);

I get the error: "error C2276: '&' : illegal operation on bound member function expression   "

Note I am doing all of this in Server.cpp which is the implementation of Server.h.

I'm really confused as to why its not working :/

[Laurent]

The syntax to take the address of a member functions is the following:

&Class::function

When you pass a non-static member function to something that will try to call it, you often need to pass the object on which it will be called as well ("this"), because you can't call a member function without an instance of the class.

So...

std::thread th(&Server::time, this);

I'm pretty sure any tutorial about std::thread explains this kind of stuff...

[Walta69]

Ok so how would I make a thread then from the constructor?
( I don't do well with tutorials because they all use big words )

Here is basically what I want to achieve:

Server::Server()
{
 ...
...
...
std::thread (&Server::time, this) // Note this still gives errors
}
void Server::time()
{
 ...
...
}


Also, if you could maybe explain, what if I wanted my time() function to take in arguments , e.g: time(int x)?

Thanks alot for help

[Walta69]

I followed your advice and did :

std::thread th(&(Server::time), this);
th.join();

and I get a different error now :

Error   4 error C2100: illegal indirection

[Nexus]

I followed your advice

No, you did not.  There's no parenthesis in Laurent's code.

You should really read a tutorial about threads. Multithreading is a complex topic, and if you're already getting problems at the syntax, this will become really difficult.

[Laurent]

Also, if you could maybe explain, what if I wanted my time() function to take in arguments , e.g: time(int x)?

You would use std::bind. Or maybe the constructor of std::thread can handle that directly. Just pass x as a third argument.