Can not call function pointer of a struct to a class method

Question

use C++98. I have a struct t_fd which is used inside a class MS. In the struct there are two pointers to function: fct_read, fct_write. I designed that the function pointers are pointing to the two methods of the class. But then I have this error when trying to call them.

expression preceding parentheses of apparent call must have (pointer-to-) function type.

Please advice on the error, also on the design. I need the two functions are methods of that class because I need to use class's attributes (even though it isn't showed here for the shake of simplicity). Thank you for your time, I appreciate your help!

#include <vector>
#include <iostream>

typedef struct  s_fd {
    void(MS::*fct_read) (int);
    void(MS::*fct_write) (int);
}   t_fd;

class MS
{
    private:
        std::vector< t_fd >            _fdSet;

        void        server_accept(int s)
        {
            if (s % 2 == 0)
                _fdSet[cs].fct_read = MS::client_read;
            else
                _fdSet[cs].fct_write = MS::client_write;
        }

        void        client_read(int fd)
        {
            std::cout << "I'm reading\n";
        }

        void        client_write(int fd)
        {
            std::cout << "I'm writing\n";
        }

        void        check_fd()
        {
            int i = 0;
            int size = 10;

            while (i < size)
            {
                if (i < 5)
                    _fdSet[i].fct_read(i); //Error here!
                if (i >= 5)
                    _fdSet[i].fct_write(i); //Error here!
                i++;
            }
        }
};

At the beginning of server_accept, int cs; leaves cs with an indeterminate value. The code that follows that line use that value, so the code won't do anything meaningful. — Pete Becker
– Pete Becker, Commented May 16, 2022 at 20:30
The syntax for calling with a pointer-to-member-function uses .* or ->*; the calls as written simply can't work. Off the top of my head, (this->*_fdSet[I].fct_read)(i); is the way to do it. — Pete Becker
– Pete Becker, Commented May 16, 2022 at 20:35
The correct syntax for creating a pointer-to-member-function is &MS::client_read. Do you have a good book to learn from? — Pete Becker
– Pete Becker, Commented May 16, 2022 at 21:18

Ivan_a_bit_Ukrainivan · Accepted Answer · 2022-05-16 21:30:08Z

4

The intent of your code is difficult to understand (in its current form). But I would be happy to solve few issues in your code.

MS class needs to be declared before you reference it the type from s_fd structure definition :

class MS; // forward declaration
    
typedef struct  s_fd {
void(MS::* fct_read) (int);
void(MS::* fct_write) (int);
}   t_fd;
    
class MS
{  ... }

the syntax to assign function pointer is incorrect. You forgot &:

_fdSet[cs].fct_read = &MS::client_read;

fct_read and fct_write are member function pointers. They should be applied on instance of MS class. In case you want to apply them on this object:

if (i < 5) {
  auto fptr = _fdSet[i].fct_read;
  (this->*fptr)(i);
}

edited May 16, 2022 at 21:30

answered May 16, 2022 at 21:18

Ivan_a_bit_Ukrainivan

1,3872 gold badges15 silver badges27 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Can not call function pointer of a struct to a class method

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related