C / C++ / MFC

1-Nov-08 13:31

Hi,
I'm hacking at this now for several hours, it will not compile.
I've the following declaration

<br />
class CCmdLineParserApp;// Advance Declaration<br />
typedef BOOL (CCmdLineParserApp::*PFN_CMDLINE_ACTION)(LPCSTR pAction=NULL);<br />
<br />
typedef struct {<br />
	char Switch[16];<br />
	PFN_CMDLINE_ACTION pCmdAction;<br />
	BOOL bIsTerminal;<br />
}CMD_LINE_ENTRY,*LP_CMD_LINE_ENTRY;<br />

Later on in the code I've found a LP_CMD_LINE_ENTRY pE. I try to execute
the Relevant pCmdAction function as follows:-

<br />
  (pE ->*pCmdAction)(ResultString);<br />

I get:

C:\Projects\CmdLineParser\CmdLineParser.cpp(143) : error C2065: 'pCmdAction' : undeclared identifier
C:\Projects\CmdLineParser\CmdLineParser.cpp(143) : error C2297: '->*' : bad right operand

I do not use this syntax very often, and I'm obviously making a very silly syntactic mistake. Any ideas??

regards

Confused | :confused:

Bram van Kampen

Mark Salsbery1-Nov-08 15:46

1-Nov-08 15:46

Bram van Kampen wrote:
'pCmdAction' : undeclared identifier

Where is pCmdAction declared? I don't see relevant code in your post
that would cause the error.

Mark

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

Bram van Kampen2-Nov-08 4:30

2-Nov-08 4:30

Mark Salsbery wrote:
Where is pCmdAction declared? I don't see relevant code in your post
that would cause the error.

<br />
class CCmdLineParserApp;// Advance Declaration<br />
typedef BOOL (CCmdLineParserApp::*PFN_CMDLINE_ACTION)(LPCSTR pAction=NULL);<br />
<br />
typedef struct {<br />
     char Switch[16];<br />
     PFN_CMDLINE_ACTION pCmdAction; <----HERE!<br />
     BOOL bIsTerminal;<br />
}CMD_LINE_ENTRY,*LP_CMD_LINE_ENTRY;<br />

pCmdAction is a member of structure CMD_LINE_ENTRY. CCmdLineParserApp, which is derrived from CWinApp, contains a static array of these. pE is a pointer into that array, and hence points to a structure of type CMD_LINE_ENTRY.

Bram van Kampen

David Crow1-Nov-08 16:13

David Crow

1-Nov-08 16:13

Bram van Kampen wrote:
(pE ->*pCmdAction)(ResultString);

You need to precede this with an instance of CCmdLineParserApp, like:

CCmdLineParserApp c;
(c.*(pE->pCmdAction))(ResultString);

"Love people and use things, not love things and use people." - Unknown

"The brick walls are there for a reason...to stop the people who don't want it badly enough." - Randy Pausch

Bram van Kampen2-Nov-08 4:38

2-Nov-08 4:38

DavidCrow wrote:
You need to precede this with an instance of CCmdLineParserApp, like:

No, I forgot to tell, the code snippet is called from within a memberfunction of CCmdLineParserApp.
however I tried :(this->*(pE ->pCmdAction))(ResultString) and it worked. Have no Idea why.

Bram van Kampen

Mark Salsbery2-Nov-08 8:03

2-Nov-08 8:03

Bram van Kampen wrote:
I tried :(this->*(pE ->pCmdAction))(ResultString) and it worked. Have no Idea why.

Because you were trying to call a CCmdLineParserApp method using a pointer
to a CMD_LINE_ENTRY as the CCmdLineParserApp object.

In your fixed version, "this" is the CCmdLineParserApp object the method is being
called in the context of, and "pE->pCmdAction" is the pointer to the method.

Much better Smile | :)

Mark

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

Bram van Kampen2-Nov-08 12:14

2-Nov-08 12:14

Thanks Mark.

Rose | [Rose]

Bram van Kampen

Michael Dunn2-Nov-08 8:17

Michael Dunn

2-Nov-08 8:17

Bram van Kampen wrote:
I tried :(this->*(pE ->pCmdAction))(ResultString) and it worked. Have no Idea why.

When you call through a pointer-to-member, you have to say which object the function should get as its this pointer. If you want the function to get the current this pointer as its own this pointer, you have to say so. That's why the version with this->* works.

--Mike--
Visual C++ MVP Cool | :cool:

LINKS~! CP SearchBar v3.0 | C++ Forum FAQ

I work for Keyser Söze

Bram van Kampen2-Nov-08 12:15

how to show/hide windows properly?

2-Nov-08 12:15

Thanks Mike.

Rose | [Rose]

Bram van Kampen

includeh101-Nov-08 10:56

includeh10

1-Nov-08 10:56

In my app, main frame has 5 child windows (base class is CWnd), the 5 children fully occupy client area of main frame and overlapped each others - similar to 5 property pages.

every time,only one of them is visible.
the 5 children are heavily painted in OnPaint().

problem is that when one is shown by ShowWindow(SW_SHOW) and others are hidden by ShowWindow(SW_HIDE), there is always some annoying visible flash at page changing time.

the reason I guess is that the paint is too heavy, so I use memory DC and disable OnEraseBkgnd() - howerver, the urgly flash still can be seen.

Do you have idea to kill the annoying flash?
Do I have to use ShowWindow() function to make one visible and others invisible?
Any suggestions are highly appreciated.

Thanks

Re: how to show/hide windows properly?

Mark Salsbery1-Nov-08 13:23

Re: how to show/hide windows properly?

1-Nov-08 13:23

What order are you making the ShowWindow(SW_SHOW) and ShowWindow(SW_HIDE)
calls in?

Try showing a child window before hiding the previous child window.

Also, if the main frame is always covered by one of the children, then
disable the main frame's OnEraseBkgnd.

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

Bram van Kampen1-Nov-08 13:58

Re: how to show/hide windows properly?

1-Nov-08 13:58

This apppears to be a problem with single threaded programs doing a lot of work! I consider it a design flaw in WINDOWS, and have been fighting with it for several years now. When the window you are working with gets a heavy task, and you send prior to that a SW_HIDE message, it hides your window. XP in particular (I Don't know about vista) seems will paint the most previous window immediately below it that it can find a thread for. This just could be the spreadsheet in which you were working out Staff Redundancies, or an email to the mistress. The only reason I can see behind this is that it avoids 'Blackness', and people thinking the machine crashed. A solution would be if the OS would show some non-descript blank screen (Nice Picture etc) instead of the last window of the previous app.

The only way I found out of this is by doing anything timeconsuming on a different thread, and show a 'wait box' This will even cure the short flashes you complain about.

Hope this is helpfull
Smile | :)

Bram van Kampen

Alan Balkany3-Nov-08 8:48

Alan Balkany

3-Nov-08 8:48

You might try LockWindowUpdate () http://msdn.microsoft.com/en-us/library/kc4ez6dk(VS.71).aspx[^] just before the ShowWindow calls, then UnlockWindowUpdate () when they're done.

This has helped me hide messy intermediate states when changing the display extensively.

sockaddr* - Casting from DWORD, get Ip address.

urbanyoung1-Nov-08 2:29

urbanyoung

1-Nov-08 2:29

Hey,

What I'm trying to do is get the ip address that a program is attempting to connect to. I can't use microsoft detours because if there is a similar application running the ip address will get modified before my program receives the data. At the moment I am codecaving on the connect statement below, from there I store the second argument (pSockAddr, EDI + 4) into a DWORD variable. I'm told that it's possible to get the address that the program is trying to connect to from pSockAddr, but I'm not too sure how. I've tried a reinterpret_cast statement, but of course that would be too simple :P

PUSH 10                                  ; /AddrLen = 10 (16.)
PUSH EDI                                 ; |pSockAddr
PUSH ECX                                 ; |Socket
CALL DWORD PTR DS:[<&WS2_32.#4>]         ; \connect

Could anybody guide me in the right direction? I've searched around this site and googled for about half an hour, without any luck. I'd really appreciate some help with this.

Thank you for your time,
urbanyoung.

Re: sockaddr* - Casting from DWORD, get Ip address.

Mark Salsbery1-Nov-08 9:04