C / C++ / MFC

Re: creating simple dilog box without using resource script

10-Jan-10 23:02

This is the third time you have asked this question (or a variation of it). Try reading the documentation on MSDN[^], and using one of the examples.

Adam Roderick J10-Jan-10 23:04

Adam Roderick J

10-Jan-10 23:04

Just check this link. [^]

Величие не Бога может быть недооценена.

Re: creating simple dilog box without using resource script

Nelek10-Jan-10 23:54

Nelek

10-Jan-10 23:54

In addition... To check this[^] won't hurt you.

Regards.
--------
M.D.V. Wink | ;)

If something has a solution... Why do we have to worry about?. If it has no solution... For what reason do we have to worry about?
Help me to understand what I'm saying, and I'll explain it better to you
Rating helpfull answers is nice, but saying thanks can be even nicer.

Re: creating simple dilog box without using resource script

KingsGambit11-Jan-10 0:55

KingsGambit

11-Jan-10 0:55

Following link would be helpful:http://msdn.microsoft.com/en-us/library/ms644996(VS.85).aspx#template_in_memory[^]

creating modless dialog box dynamically

kir_MFC10-Jan-10 20:25

kir_MFC

10-Jan-10 20:25

please let me know the procedure of creating modelessdilog box dynamically

kir_MFC

Re: creating modless dialog box dynamically

«_Superman_»10-Jan-10 20:27

«_Superman_»

10-Jan-10 20:27

The procedure in your previous post is the correct way.
You should try the suggestions given there and follow up on that instead of posting the question again.

«_Superman_»
I love work. It gives me something to do between weekends.

Microsoft MVP (Visual C++)

InternetOpenUrl - urgent help needed

YaronNir10-Jan-10 20:13

10-Jan-10 20:13

i use InternetOpenUrl (as async) to receive a file from a url.

this is the code i use :

BOOL CMyCode::RequestFile( LPCTSTR lpszUrl )<br />
{<br />
	m_hInternet = InternetOpen(_T("MyCode"), INTERNET_OPEN_TYPE_DIRECT, NULL, NULL, INTERNET_FLAG_ASYNC);<br />
	if (!m_hInternet)<br />
	{<br />
		return FALSE;<br />
	}<br />
<br />
	InternetSetStatusCallback(m_hInternet, CMyCode::InternetStatusCallback);<br />
<br />
	InternetOpenUrl(m_hInternet, <br />
			lpszUrl, <br />
			NULL,<br />
			0, <br />
			INTERNET_FLAG_RELOAD | INTERNET_FLAG_PRAGMA_NOCACHE | INTERNET_FLAG_NO_CACHE_WRITE, <br />
			(DWORD) this);<br />
	return TRUE;<br />
}<br />
<br />
void __stdcall CMyCode::InternetStatusCallback (HINTERNET hSession, DWORD Context, DWORD Status, LPVOID pInformation, DWORD InfoLength)<br />
{<br />
	CMyCode* pThis = (CMyCode*) Context;<br />
	if (!pThis)<br />
	{<br />
		return;<br />
	}<br />
<br />
        switch (Status) <br />
	{<br />
	case INTERNET_STATUS_RESPONSE_RECEIVED:<br />
		m_dwNumytes2Read = (*(DWORD *)pInformation);<br />
		break;<br />
	case INTERNET_STATUS_HANDLE_CREATED:<br />
		m_hHttpSession = * (HINTERNET *) pInformation;	// Copy<br />
		break;<br />
	case INTERNET_STATUS_REQUEST_COMPLETE:<br />
		CompleteRequest();<br />
		break;<br />
	case INTERNET_STATUS_HANDLE_CLOSING:<br />
		break;<br />
	default:<br />
		ATLTRACE(_T("Status is %ld\n"), Status);<br />
		break;<br />
	}	<br />
}<br />
<br />
void CMyCode::CompleteRequest()<br />
{<br />
	LPBYTE pBuff = new BYTE[m_dwNumytes2Read];<br />
	ZeroMemory(pBuff, m_dwNumytes2Read);<br />
<br />
	DWORD dwBytesRead(0);<br />
	BOOL bOK(TRUE);<br />
	while (bOK)<br />
	{<br />
		bOK = InternetReadFile (m_hHttpSession, pBuff, m_dwNumytes2Read, &dwBytesRead);<br />
		if (!dwBytesRead)<br />
			bOK = FALSE;<br />
		// .. progress handling<br />
	}<br />
<br />
	if (pBuff)<br />
		delete []pBuff;<br />
	<br />
	Abort();<br />
<br />
        //... done<br />
}<br />
<br />

the code works ok only i have 2 major problems:

1. the file is received in a chunks of 267/268 Bytes each time i call InternetReadFile????
2. the CPU reaches 80% until the file is download completely

can any1 help me here?
thanks in advanced

Interface basics click here :
http://www.codeproject.com/com/COMBasics.asp

don't forget to vote Smile | :)

«_Superman_»10-Jan-10 20:16

«_Superman_»

10-Jan-10 20:16

You can try URLDownloadToFile function.

«_Superman_»
I love work. It gives me something to do between weekends.

Microsoft MVP (Visual C++)

YaronNir10-Jan-10 20:20

10-Jan-10 20:20

thanks for the reply...

i can not use other methods other than what i've posted in my code (for stupid reasons go figure)
so i would still like any help regarding my issues

thanks again

Interface basics click here :
http://www.codeproject.com/com/COMBasics.asp

don't forget to vote Smile | :)

Bram van Kampen11-Jan-10 16:28

Bram van Kampen

11-Jan-10 16:28

YaronNir wrote:
i can not use other methods other than what i've posted in my code (for stupid reasons go figure)
so i would still like any help regarding my issues

Then You're at a dead end! We cannot help you any further!(for stupid reasons go figure).

Smile | :)

Bram van Kampen

YaronNir11-Jan-10 20:49

11-Jan-10 20:49

this sort of reply is in appropriate,

if you can see below i got a really good reply who took my question seriously

shame on you....

Interface basics click here :
http://www.codeproject.com/com/COMBasics.asp

don't forget to vote Smile | :)

Richard MacCutchan11-Jan-10 1:53

11-Jan-10 1:53

Have you checked the value of m_dwNumytes2Read bfore your call to InternetReadFile()?

YaronNir11-Jan-10 1:55

11-Jan-10 1:55

what do you mean? can you give an example

Interface basics click here :
http://www.codeproject.com/com/COMBasics.asp

don't forget to vote Smile | :)

Richard MacCutchan11-Jan-10 3:15

11-Jan-10 3:15

You have the following code (btw could you edit your original message and put the code between <pre></pre> tags):

case INTERNET_STATUS_RESPONSE_RECEIVED:
    m_dwNumytes2Read = (*(DWORD *)pInformation);
    break;
    // intermediate lines omitted

void CMyCode::CompleteRequest()
{
    LPBYTE pBuff = new BYTE[m_dwNumytes2Read];
    ZeroMemory(pBuff, m_dwNumytes2Read);

    DWORD dwBytesRead(0);
    BOOL bOK(TRUE);
    while (bOK)
    {
    bOK = InternetReadFile (m_hHttpSession, pBuff, m_dwNumytes2Read, &dwBytesRead);
    // intermediate lines omitted
}

The value of m_dwNumytes2Read is set when you process the INTERNET_STATUS_RESPONSE_RECEIVED message and is relevant to the status information, so it is probably less than the amount of data that may be available. Try resetting it to some bigger value before calling InternetReadFile() for example:

dwNumytes2Read = 2048;
LPBYTE pBuff = new BYTE[m_dwNumytes2Read];
DWORD dwBytesRead;
BOOL bOK(TRUE);
while (bOK)
{
bOK = InternetReadFile (m_hHttpSession, pBuff, m_dwNumytes2Read, &dwBytesRead);
// ...

YaronNir11-Jan-10 3:19

11-Jan-10 3:19

thanks for the reply - it did solve the problem now the chunck are 2048 bytes each time - good!

the second problem still remains - high CPU about 80% - how can i solve this?

thanks again for your kind reply

Interface basics click here :
http://www.codeproject.com/com/COMBasics.asp

don't forget to vote Smile | :)

Richard MacCutchan11-Jan-10 4:09

11-Jan-10 4:09

First thing is to run some tests with different buffer sizes to see how that affects performance, 2048 was just a number plucked from the air.

Looking at your code I notice that you use ZeroMemory on the input buffer just before reading data into it; this is wasteful and serves no purpose, just allocate it with the new call.

Alternatively you could allocate your buffer at the beginning of the program, either via new or as a static data area. This would save repeated new delete[] pairs, and the consequent garbage collection code.

There may be other issues in the rest of your code, but I leave that for you to investigate.

YaronNir11-Jan-10 4:32