C++ Function Pointer Syntax

Question

I am trying to create a function in VC++ that takes a function pointer but I keep getting syntax errors.

The declaration in my header file looks like this:

 void ApplyFuncToCellsInSelection(void(*func)(CPoint, *CSpreadWnd));

Here is the definition:

void CSpreadWnd::ApplyFuncToCellsInSelection(void(*func)(CPoint, *CSpreadWnd)) { ... }

And here are the error messages I'm getting:

c:\...\spreadwnd.h(274) : error C2059: syntax error : 'function-style cast'
c:\...\spreadwnd.h(274) : error C2059: syntax error : ')'
c:\...\spreadwnd.h(274) : error C2143: syntax error : missing ')' before ';'

I know its probably something really simple that I'm missing but I can't seem to figure it out.

You should probably consider using boost::function or std::function, if either is available to you. They're much easier to use and read. — Collin Dauphinee
– Collin Dauphinee, Commented Aug 3, 2010 at 15:11

Didier Trosset · Accepted Answer · 2010-08-03 14:59:20Z

9

It's usually a good idea to define a typedef for your function pointer type. It helps using it in further declarations, and avoids having to change it twice when you write an error. Here, you put the asterisk on the wrong side of CSpreadWnd.

typedef void (*MyFuncPtr)(CPoint, CSpreadWnd*);
void ApplyFuncToCellsInSelection(MyFuncPtr func);

And definition:

void CSpreadWnd::ApplyFuncToCellsInSelection(MyFuncPtr func) { ... }

answered Aug 3, 2010 at 14:59

Didier Trosset

37.6k14 gold badges89 silver badges126 bronze badges

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

Comments

James McNellis · Accepted Answer · 2010-08-03 14:54:36Z

6

You have the asterisk on the wrong side of CSpreadWnd:

void ApplyFuncToCellsInSelection(void(*func)(CPoint, CSpreadWnd*));
                                 the asterisk needs to go here ^

answered Aug 3, 2010 at 14:54

James McNellis

357k76 gold badges934 silver badges983 bronze badges

Comments

cHao · Accepted Answer · 2010-08-03 14:55:06Z

4

The CSpreadWnd pointer looks funny. It looks like it should be CSpreadWnd* rather than *CSpreadWnd.

answered Aug 3, 2010 at 14:55

cHao

87.1k21 gold badges147 silver badges178 bronze badges

Collectives™ on Stack Overflow

C++ Function Pointer Syntax

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related