Question

class Bot{
private:
       char *serveradr;
public:
       char *get_serveradr(); 
       char set_serveradr(char *server);
       void servcon();                
       };

char Bot::set_serveradr(char *server)
 {
 serveradr = server;
 }

 char *Bot::get_serveradr()
 {
 return serveradr;


 void servcon()
 {
  sin.sin_addr.s_addr           = inet_addr(this.get_serveradr());

  }

invalid use of `this’ in non-member function

Sorry for such a question but I’ve searched and know this is how it should be done but I’m not sure what I’m doing wrong.

Answer 1

You need to fully qualify the name of the member function defintion:

void Bot::servcon()
{
    sin.sin_addr.s_addr = inet_addr(this->get_serveradr()); // Note use of ->

}

Otherwise, it is treated as a free standing function which do not have a this pointer.

As this is C++, use a std::string instead of a char*. The argument passed to the Bot::set_serveradr() method must outlive Bot which is error prone (and who is responsible for delete[]ing it?). You can access a char* using std::string::c_str() method.