4

If I compile the code below, I get an

"Error C2064: term does not evaluate to a function taking 2 arguments"

in the doOpp() method. If I change q = f(p, i); for q = add(p, i); everything compiles fine (with add() being the second argument in the doOpp() method). I've tried a couple of solutions but nothing seems to work out.

template <class T = double>
class Points {
public: 
    Points(void);
    Points(Point3d<T> *, int);
    Points(T *, int);
    Points(Points &);
    ~Points(void);

    void clear();
    int size();
    void push_back(Point3d<T> &);
    typename vector<Point3d<T>>::iterator begin(void){return pnts.begin();};
    typename vector<Point3d<T>>::iterator end(void){return pnts.end();};

    void assign(Point3d<T> *, int);
    void assign(T *, int);
    void assign(Points &);

    Points valid(void);
    Points valid(int *);
    bool setValid(int, bool);
    int validSize();

// Pointer to operator function prototype
    typedef Point3d<T> (__thiscall Points<T>::*oppFunc) (Points<T> &, int);

    Points operator=(Points &);

// + Operator method
    Points operator+(Points &p){return doOpp(p, &Points<T>::add);};
    Points operator-(Points &){return doOpp(p, &Points<T>::sub);};
    Points operator*(Points &){return doOpp(p, &Points<T>::mlt);};;

private:

    Points(int);
    void assign(int);
    int assign(Points &, Points &);
    vector<Point3d<T>> pnts;
    bool isSameSize(Points<T> &, char *);

// General purpose operator function
    Points doOpp(Points<T> &, oppFunc);

// Addition method called by doOpp
    Point3d<T> add(Points &p, int i) {return pnts[i] + p.pnts[i];}
    Point3d<T> sub(Points &p, int i) {return pnts[i] - p.pnts[i];}
    Point3d<T> mlt(Points &p, int i) {return pnts[i] * p.pnts[i];}
};

template <class T> int
Points<T>::assign(Points &p1, Points &p2) {
    int s = min(p1.size(), p2.size());
    assign(s);

    return s;
}

template <class T> bool
Points<T>::isSameSize(Points &p, char *s) {
    bool    same = (pnts.size() == p.pnts.size());

    #ifndef POINTS_NO_THROW
    if (!same) {
        PointsException e;
        e.setMessage((string) s + " source sizes not equal");
        throw e;
    }
    #endif

    return same;
}

template <class T> Points<T>
Points<T>::doOpp(Points &p, oppFunc f) {
    isSameSize(p, __FUNCTION__);
    Points<T>   r;
    int         s = r.assign(*this, p);
    Point3d<T>  q;

    for (int i = 0; i < s; i++) {
        q = f(p, i); // Causes error C2064 but q = add(p, i); works fine
    }

    return r;
}


int
main () {
    Points<>    p, q;

    q.assign(a, 4);
    p = q + q;

    return 1;
}
0

2 Answers 2

8

Since oppFunc f is a member function pointer you should use the special syntax to make a call through a member function pointer:

q = (this->*f)(p, i);
Sign up to request clarification or add additional context in comments.

1 Comment

fyi, the brackets are needed to bind the ->* operator first.
0

Should this:

Points operator-(Points &){return doOpp(p, &Points<T>::sub);};

not be:

 Points operator-(Points &p){return doOpp(p, &Points<T>::sub);}; // note the p argument

Comments

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.