Re: URLDownloadToFile, Cancel Button - C / C++ / MFC Discussion Boards

I don't understand the question. At a minimus perhaps it has two parts.

Part 1: If you want to write code to create a dll (not use an existing one) then you need to deal with doing just that. HOW the dll is used after that is irrelevant to this part of the code. And if I was doing it I would create source code and then plan on providing a compiler/linker which I would then call from the code so that those created the dll.

Part 2: There is code X that uses a dll Y. If Y does not exist when X is created then X must call the dll dynamically. Y of course must be a dll (or another type of binary that can be dynamically loaded.) X cannot randomly use any Y. X must know/assume some behavior of Y. So X loads Y, then looks for exactly the functionality that it demands. Then it calls that.

The two parts have nothing to do with each other. Implement them and test them independently from each other.

straight win32 app

I used the URLDownloadToFile to download a program file, probably a mistake, but a start at least. I wrote my IBindStatusCallback, and attached it to the URL program.

So I have a Window, not a dialog box, probably another mistake, I call the function in my window, and the function runs, but the wndproc in the window stops running, so my cancel button won't fire.

I tried to think of a way, that allows the OnProcess loop to give control back to the window, so I can capture the cancel button, but no luck.

Any points in the right direction would be helpful. I did see the data exchange thing, but thought it was for dialog boxes. This is my first experience in this.

Well, you could start a separate thread to do the URLDownloadToFile call and let the OnProcess (which I assume is your button) to return processing windows messages.

You would have to implement a "completion" message back from the thread to the main window to tell it when the download was complete and return the status of the download (success, fail, cancelled).

You cancel button should then work but, of course, you now need to tell the thread that the cancel button was hit or test a "cancelled" flag in the callback.

I was just reading up on creating threads.

So I guess I would create a thread in the window code, and run the class code in the thread.

Creating a thread looks like it takes alot of code.

Maybe the cancel button is not worth it.

Nah, creating thread is pretty straightforward, there are lots of examples here on code project. A quick search should find several like this CThread - a Worker Thread wrapper class[^]
Or you can use CWinThread (if you're derived from CWinApp). Or use _beginthreadex()[^] native Windows API.

Threads is a good thing to learn early. In this case, you can create the simplest of thread routines and not worry about complicated events and synchronization later. Just get it to do your download and send a completion message to your main windows process.

Well since I used the simplest way to download, the simplest thread sounds like a good idea. I'll start that tomorrow.

Right now I'm experimenting with creating a cancel button in the download class, and seeing if I can capture the message when the button is clicked. I got the button to appear and position itself with the right font in the Callback under BINDSTATUS_CONNECTING

I wrapped the original call to the class in a do loop, to keep it from progressing. Got the idea from the socket code I wrote.

I'm just messing around right now, experimenting till morning.

C#

do {
        bResult = caDownload._download_SQLServerExpress_GUI(cWnd, pb_SQL_Server_Download_Status, lbl_SQL_Server_Download_Status, bAreWeX64 );
    } while (bResult == FALSE);

Which one was the easy one?, they both look intimidating.

They're both a lot easier having done them than before. I had no idea about threads or sockets 2 1/2 weeks ago, but have now learnt enough to get by and have cobbled together a rss reader - one which can pre-load all subscribed feeds in parallel, each in it's own thread.

Notifying the main window of a thread's completion is the easier of the two. Without MFC, it's as (1) defining a user message, and deciding on the format (if any) of the passed information (2) sending that message to the main window upon completion (3) handling that message in the main window.

E.g
Part 1 - Main window completion notification mechanism

C++

#define WM_GALLERY_HTML_AVAILABLE WM_USER+1
#define WM_FILE_DOWNLOAD_COMPLETE WM_USER+2
...
...
...
__stdcall void galleryHtmlDownloadedCallbackFunc(PVOID data)
{
    downloadHeader_t *hdr;
    hdr = (downloadHeader_t*)data;

    galHtmlStr = hdr->result;
    galUrlStr = hdr->szUrl;

    free(hdr->result);
    hdr->result = NULL;
    hdr->contentLen = 0;

    PostMessage(mainDlgWnd, WM_GALLERY_HTML_AVAILABLE, 0, 0);
}

...
...
...
...

    case WM_INITDIALOG:
        mainDlgWnd = hwndDlg;
        editWnd = GetDlgItem(hwndDlg, IDC_RICHEDIT1);
        SetFocus(GetDlgItem(hwndDlg, IDC_RICHEDIT1));
        return FALSE;

    case WM_FILE_DOWNLOAD_COMPLETE:
        return true;

    case WM_GALLERY_HTML_AVAILABLE:
        findLinks((char*)galHtmlStr.c_str(), links, validExts);
        strLinks.clear();
        printf("Number of links: %d\n", links.size());
        for (i=0; i<links.size(); i++)
            strLinks += links[i] + "\n";
        SetDlgItemText(mainDlgWnd, IDC_RICHEDIT1, strLinks.c_str());
        return true;

    case WM_CLOSE:
        EndDialog(hwndDlg, 0);
        return TRUE;

Part 2 - Threading mechanism

C++

///////// usage example single (extra) thread //////////

    static downloadHeader_t curDownload;
    GetWindowText(urlHwnd, szUrl, urlLen+1);
    savePath = getFolderName(mainDlgWnd, "C:\\001");

    if (strlen(szUrl) && strlen(savePath))
    {
        curDownload.szUrl = szUrl;
        curDownload.callback = (voidFuncPtr)galleryHtmlDownloadedCallbackFunc;
        curDownload.saveFilePath = savePath;
        _beginthread(DownloadThread, NULL, (void*)&(curDownload));
    }

    delete savePath;
    delete szUrl;

////////////////////////////////////////////////////////////////////////


typedef void (*voidFuncPtr)(void*);
struct downloadHeader_t
{
    string szUrl;             // in
    voidFuncPtr callback;     // in
    string saveFilePath;     // in
    PVOID lParam;             // in

    SOCKET conn;
    char *readBuffer, *sendBuffer, *tmpBuffer, *result;
    string server, filepath, filename;
    long thisReadSize, headerLen, contentLen;
};


// seperate a url into it's components
void mParseUrl(char *mUrl, string &serverName, string &filepath, string &filename)
{
    string::size_type n;
    string url = mUrl;

    if (url.substr(0,7) == "http://")
        url.erase(0,7);

    n = url.find('/');
    if (n != string::npos)
    {
        serverName = url.substr(0,n);
        filepath = url.substr(n);
        n = filepath.rfind('/');
        filename = filepath.substr(n+1);
    }

    else
    {
        serverName = url;
        filepath = "/";
        filename = "";
    }
}


// return the socket number for a TCP connection
SOCKET connectToServer(char *szServerName, WORD portNum)
{
    struct hostent *hp;
    unsigned int addr;
    struct sockaddr_in server;
    SOCKET conn;
//SOCKHANDLE_USE_OVERLAPPED
    conn = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
    if (conn == INVALID_SOCKET)
        return NULL;

    if(inet_addr(szServerName)==INADDR_NONE)
    {
        hp=gethostbyname(szServerName);
    }
    else
    {
        addr=inet_addr(szServerName);
        hp=gethostbyaddr((char*)&addr,sizeof(addr),AF_INET);
    }

    if(hp==NULL)
    {
        closesocket(conn);
        return NULL;
    }

    server.sin_addr.s_addr=*((unsigned long*)hp->h_addr);
    server.sin_family=AF_INET;
    server.sin_port=htons(portNum);
    if(connect(conn,(struct sockaddr*)&server,sizeof(server)))
    {
        closesocket(conn);
        return NULL;
    }
    return conn;
}

// searches the content returned from a server for 2 newlines in a row - this signifies the end of the header.
// the RFC (I forget the number) states that "\r\n\r\n" to be the valid terminator, however not all sites conform
// to the standard.
int getHeaderLength(char *content)
{
    const char *srchStr1 = "\r\n\r\n", *srchStr2 = "\n\r\n\r";
    char *findPos;
    int ofset = -1;

    findPos = strstr(content, srchStr1);
    if (findPos != NULL)
    {
        ofset = findPos - content;
        ofset += strlen(srchStr1);
    }

    else
    {
        findPos = strstr(content, srchStr2);
        if (findPos != NULL)
        {
            ofset = findPos - content;
            ofset += strlen(srchStr2);
        }
    }
    return ofset;
}






/*
struct downloadHeader_t
{
    string szUrl;            // in
    voidFuncPtr callback;     // in
    channelDef_t *channel;  // in

    SOCKET conn;
    char *readBuffer, *sendBuffer, *tmpBuffer, *result;
    string server, filepath, filename;
    long thisReadSize, headerLen, contentLen;
};
*/
void __stdcall DownloadThread(void *dat)
{
    downloadHeader_t *hdr;
    hdr = (downloadHeader_t *)dat;
    const int bufSize = 512;

    printf("download thread - \n");

    hdr->result = NULL;
    hdr->readBuffer = (char*)malloc(bufSize);
    hdr->sendBuffer = (char*)malloc(bufSize);
    hdr->tmpBuffer = (char*)malloc(bufSize);

    printf("download thread - \n");

    mParseUrl((char*)hdr->szUrl.c_str(), hdr->server, hdr->filepath, hdr->filename);

    printf("download thread - \n");
//http://forums.devshed.com/c-programming-42/any-ideas-863939.html
    ///////////// step 1, connect //////////////////////
//    printf("connecting to %s\n", hdr->server.c_str());
    hdr->conn = connectToServer((char*)hdr->server.c_str(), 80);
    if (hdr->conn)
        printf("download thread - connected?\n");

    ///////////// step 2, send GET request /////////////
    sprintf(hdr->tmpBuffer, "GET %s HTTP/1.0", hdr->filepath.c_str());
    strcpy(hdr->sendBuffer, hdr->tmpBuffer);
    strcat(hdr->sendBuffer, "\r\n");
    sprintf(hdr->tmpBuffer, "Host: %s", hdr->server.c_str());
    strcat(hdr->sendBuffer, hdr->tmpBuffer);
    strcat(hdr->sendBuffer, "\r\n");
    strcat(hdr->sendBuffer, "\r\n");
//    printf("sending:\n'%s'", hdr->sendBuffer);
    send(hdr->conn, hdr->sendBuffer, strlen(hdr->sendBuffer), 0);

    ///////////// step 3 - get received bytes ////////////////
    // Receive until the peer closes the connection
    hdr->contentLen = 0;
    while(1)
    {
        memset(hdr->readBuffer, 0, bufSize);
        hdr->thisReadSize = recv (hdr->conn, hdr->readBuffer, bufSize, 0);
        if ( hdr->thisReadSize <= 0 )
            break;

        // nasty - but seemingly necessary. Headers don't always include the content-length
        // without a known content length, we cannot accurately allocate the memory required to hold
        // the full (header+message) response. Therefore, we re-allocate our memory each time a successful read is performed.
        // the alternative would be to write directly to file, or allocate (hopefully) more memory than we
        // need (Too bad if we allocate 10 Mb & a resource is 10.1 Mb !)
        hdr->result = (char*)realloc(hdr->result, hdr->thisReadSize+hdr->contentLen);

        memcpy(hdr->result+hdr->contentLen, hdr->readBuffer, hdr->thisReadSize);
        hdr->contentLen += hdr->thisReadSize;
    }

    hdr->headerLen = getHeaderLength(hdr->result);
    hdr->contentLen -= hdr->headerLen;
    memmove(hdr->result, hdr->result+hdr->headerLen, hdr->contentLen);
    realloc(hdr->result, hdr->contentLen+1);
    hdr->result[hdr->contentLen] = 0x0;
    closesocket(hdr->conn);
    free(hdr->readBuffer);
    free(hdr->sendBuffer);
    free(hdr->tmpBuffer);

    if (hdr->callback)
        hdr->callback(hdr);
}

Nice job on the code modules. I can see that creating a thread to run the program in is not that difficult. I have an idea of the components needed now after examining your sample.

Let's see what I can some up with today. I'll be back.

Not a problem.
I just noticed that I declared the DownloadThread function as being __stdcall. It's a mistake, and should read __cdecl. Errors with calling conventions are, I suspect, begin the majority of my program crashes.

FWIW, it's _beginthreadex that wants a thread function called using __stdcall.

I didn't run your sample. I just examined it for reverse engineering to suit my needs.

I spend half a day yesterday cleaning up my code, but was not able to successfully test my thread. I got confused with how the process worked. In other words, I show the cancel button, then I run a function, and the cancel button won't send a message back to wndproc until it completes. So I messsed around with the way I made the original class, and fixed my mistakes.

Today I will try to get my thread working.

He, he. I'm familiar with that feeling. It took me quite a while to wrap my head around when and where was the appropriate place to do things. I've since re-done the function such that it can send progress callbacks too.

Haven't bothered with a cancel functionality yet - though I'd imagine it to be a simple matter of setting a flag from the main program thread, checking this flag each time through the receive loop in the download thread. (may have to use a mutex to access the flag in a safe manner)

I have the create thread working now. just went simple on it.

I already had the progress bar and status text working, now the new thread launches the download, and the cancel button responds to my click. I know how to stop the Download Callback with E_ABORT, I just need to figure out how to send that message to the progress loop.

After that, I need to figure out how to destroy the thread on cancel, or when the download is complete, _endthread,
but I'm not sure if I need to close the handle, and the WaitForObject.

Getting there. Still experimenting with it, so it's not finalized.

C#

///////////////////////////////////////////////////////////////////////////////////////////
    // Ready for Take Off
    hThread = (HANDLE)_beginthreadex(NULL, 0, SQL_SoftwareDownload, NULL, 0, (unsigned *)&thread_id);

    return resultCode;
}

unsigned int __stdcall SQL_SoftwareDownload(void *) {

    int iReturn = 0;
    HRESULT hr = E_FAIL;

    callback = new SQL_Servers_BindCallback;
    callback->m_MDIChild = hSQL_Servers_Download;
    callback->m_Progress_Text = lbl_SQL_Server_Download_Status;
    callback->m_Progress_Bar = pb_SQL_Server_Download_Status;
    callback->b_Downloading = TRUE;

    hr = URLDownloadToFile(NULL, szUrl, szFileName, 0, callback);
    if (hr == S_OK)
        iReturn = 1;

    return iReturn;

}

Congrats.

The HANDLE that you get back from _beginthreadex() should eventually be cleaned up with CloseHandle(). There is a resource that is backed by the HANDLE and you should clean that up when you are done with it.

The HANDLE remains a "waitable object" even after the thread exits. That way, you can do a WaitForSingleObject() on the HANDLE to either wait for the thread to finish or to test for completion (by adding a timeout value to the wait). Once the thread exits, the HANDLE becomes "signaled" and that will satisfy any wait, even one that comes along after the thread completes. That's why you have to eventually close it.

That sounds tricky.

I'm not sure what to do yet, but I'm moving forward. Here is my process

Button Click Next ->

_download_SQLServer_Begin - GUI stuff - Call _download_SQLServer_Package
_download_SQLServer_Package - Download Prep - Start Thread
SQL_SoftwareDownload - load callback, call urldownload function
_download_SQLServer_End - End Thread stuff, GUI unwind

cancel button click!
_cancel_SQLServer_Package - stop download process- send message to bind callback to E_ABORT
call _download_SQLServerEnd for thread and GUI unwind

I had to split the gui stuff in half - sorry to bombard you with code, I'm sure theirs much to delete

BOOL _download_SQLServer_Begin( HWND cWnd ) 
{
	BOOL			bResult								= FALSE;
	BOOL			bPreFlightCheck						= FALSE;
	BOOL			bAreWeX64							= FALSE;

	CA_Windows caWin;
		
	// Hide the Instructions
	ShowWindow(lbl_SQL_Server_Download_Instructions, SW_HIDE);
	// Hide the Navigation Buttons
	ShowWindow(bt_SQL_Server_Download_Next, SW_HIDE);
	ShowWindow(bt_SQL_Server_Download_Exit, SW_HIDE);
	ShowWindow(bt_SQL_Server_Download_Cancel, SW_SHOW);
	
	// Hide the Listbox and Intialize it
	ShowWindow(lbl_SQL_Server_Download_Label, SW_HIDE);		
	// Show the Status Bar
	ShowWindow(lbl_SQL_Server_Download_Status, SW_SHOW);
	
	// Are we on a x64 OS 
	bAreWeX64 = caWin._getProcessorArchitecture();
	if (bAreWeX64 == TRUE) {
		SetWindowText(lbl_SQL_Server_Download_Status, L"Requesting X64 Version from microsoft.com");		
	}
	else {
		SetWindowText(lbl_SQL_Server_Download_Status, L"Requesting X86 Version from microsoft.com");		
	}
	SendMessage(pb_SQL_Server_Download_Status, PBM_SETPOS, (WPARAM)0, LPARAM(100));
	SendMessage(pb_SQL_Server_Download_Status, PBM_SETSTEP, (WPARAM)1, NULL);
	ShowWindow(pb_SQL_Server_Download_Status, SW_SHOW);
	UpdateWindow(pb_SQL_Server_Download_Status);
	// Update the Stage 1 more time
	UpdateWindow(cWnd);
	// Call the Program Download Function
	bResult = _download_SQLServer_Package( bAreWeX64 );
	return bResult;
}

BOOL _download_SQLServer_End( HWND cWnd ) {
		
	BOOL			bResult								= FALSE;

	//WaitForSingleObject( hThread, INFINITE );
	//_endthread();
	//CloseHandle(hThread);
	
	ShowWindow(bt_SQL_Server_Download_Cancel, SW_HIDE);
	ShowWindow(bt_SQL_Server_Download_Exit, SW_SHOW);	
	
	if ( bResult == TRUE ) {
		SetWindowText(lbl_SQL_Server_Download_Status, L"Download has completed successfully");		
		// Show the Listbox and Intialize it
		ShowWindow(lbl_SQL_Server_Download_Label, SW_SHOW);
		BOOL preFlightCheck = FALSE;
		Sleep(2000);

		int msgboxID = MessageBoxEx(hSQL_Servers_Download,
			L"Would you like to install SQL Server now?",
			L"SQL Server has been downloaded successfully",
			MB_OKCANCEL | MB_DEFBUTTON1 | MB_ICONINFORMATION,
			LANG_NEUTRAL
		);

		switch ( msgboxID ) 
		{			
			case IDOK:
				preFlightCheck = TRUE;					
				break;

			case IDCANCEL:
				preFlightCheck = FALSE;
				break;
		} 
	}
	return bResult;
}

BOOL _cancel_SQLServer_Package( HWND cWnd )
{
	BOOL bResult  = FALSE;
	int msgboxID;
		
	msgboxID = MessageBoxEx(hSQL_Servers_Download,
			L"Canceling the download will prevent you from installing SQL Server?",
			L"SQL Server is downloading now",
			MB_OKCANCEL | MB_DEFBUTTON1 | MB_ICONINFORMATION,
			LANG_NEUTRAL
	);

	switch ( msgboxID ) 
	{			
		case IDOK:
			// Kill the Callback Loop
			_download_SQLServer_End( cWnd );				
			bResult = FALSE;
			break;
		case IDCANCEL:			
			bResult = FALSE;
			break;
	}
	return bResult;
}

BOOL _download_SQLServer_Package( BOOL X64 )
{
	BOOL					resultCode				= FALSE;
	BOOL					bThread					= FALSE;
	HRESULT					hr						= S_OK;
	
	char					*fs_X86					= "762761216";
	char					*fs_X64					= "834555904";	
	char					*ckSum_X86				= "10ef23fe7bd9b6a64dcaf1927dd19df7";
	char					*ckSum_X64				= "5122f358a63e12a3095aaabba8203292";	
	WCHAR					*szUrl_X86				= L"http://download.microsoft.com/download/9/7/6/9761DF25-CBD8-4D48-9415-065F6BD4E63D/SQLEXPRADV_x86_ENU.exe";
	WCHAR					*szUrl_X64				= L"http://download.microsoft.com/download/9/7/6/9761DF25-CBD8-4D48-9415-065F6BD4E63D/SQLEXPRADV_x64_ENU.exe";
	
	WCHAR					*szFileName_X86			= L"SQLEXPRADV_x86_ENU.EXE";
	WCHAR					*szFileName_X64			= L"SQLEXPRADV_x64_ENU.EXE";
	WCHAR					*szTargetFolder			= NULL;
	WCHAR					*szSystemDrive			= NULL;
	WCHAR					*szFileName_X			= NULL;
		
	DWORD					dwReserved				= 0;
	LPCWSTR					szError					= NULL;
	
	/////////////////////////////////////////////////////////////////////////////////////////
	// Pre Flight Check - See if the file is already downloaded, make sure it's legit	
	// Select the URL target for download based on cpu type	
	if (X64 == FALSE) {
		// Download the X86 Version of SQL Server
		int iSize = wcslen(szUrl_X86)*sizeof(szUrl_X86);
		szUrl = szUrl_X86;
	}
	else {
		// Download the X64 Version of SQL Server
		int iSize = wcslen(szUrl_X64)*sizeof(szUrl_X64);
		szUrl = szUrl_X64;
	}

	// Get the System Drive Letter and Windows Folder
	CA_Windows *caWin = new CA_Windows;	
	szSystemDrive = caWin->_getWindowsFolder();
	caWin = NULL;

	// Figure out where to store the file - prefer appdata folder	
	wcsncpy_s(szFileName, szSystemDrive, wcslen(szSystemDrive) );
	wcsncat_s(szFileName, L"\\Temp\\", wcslen(L"\\Temp\\") );
	
	if (X64 == FALSE) {
		wcsncat_s(szFileName, szFileName_X86, wcslen(szFileName_X86) );		
	} 
	else {
		wcsncat_s(szFileName, szFileName_X64, wcslen(szFileName_X64) );		
	}
		
	// Combine the system drive and app data folder.
	// End of Pre Flight Check
	///////////////////////////////////////////////////////////////////////////////////////////
	// Ready for Take Off
	hThread = (HANDLE)_beginthreadex(NULL, 0, SQL_SoftwareDownload, NULL, 0, (unsigned *)&thread_id);		
	return resultCode;
}

unsigned int __stdcall SQL_SoftwareDownload(void *) {
	int iReturn = 0;
	HRESULT hr = E_FAIL;

	SQL_Servers_BindCallback *callback;
	callback = new SQL_Servers_BindCallback;
	callback->m_MDIChild = hSQL_Servers_Download;
	callback->m_Progress_Text = lbl_SQL_Server_Download_Status;
	callback->m_Progress_Bar = pb_SQL_Server_Download_Status;
	callback->Release();
		
	hr = URLDownloadToFile(NULL, szUrl, szFileName, 0, callback);
	if (hr == S_OK)
		iReturn = 1;	

	return iReturn;
}

From a look over your code, I think that you're seeing the way to cancel the download as being to kill the thread. Given that you're using URLDownloadToFile (a blocking function), there's no apparent way to ask it to stop. With this in mind the clear choice would be to kill the thread - PROVIDING it's safe to nuke a thread in the middle of this function.

Providing it's safe, I guess you could just kill the thread. From what I can tell, you can only terminate a thread started with _beginthreadex from within it, using _endthreadex (you can't pass a thread ID to the function). With this in mind - it seems that _beginthreadex and _endthreadex are insufficient for your needs. If you were looping in the download function (like I was with recv in my download function), you could check a flag to see if the download should be discontinued - if so, simply exit the receive loop gracefully, cleaning up any/all memory resources.

Since you're not running a loop that you can check for the flag, it seems that your only option is to cancel the download be exiting/terminating the thread. In this case, it may perhaps be that you need to use the OpenThread, CloseThread, SuspendThread, ResumeThread, TerminateThread family of functions, since they allow you to pass a thread identifier to Suspend, Resume and Terminate any given thread.

There's some history associated with beginthread, _beginthreadex and OpenThread. I forget which supersedes which and for quite what reason, though I recall that there were inherent problems that caused the 're'-implementation. I'm inclide to think it had something to do with memory leaks or issues with handles.

I tried killing the thread, but it destroyed my window. but that was before I split the functions.

Here's the other half of the code - Callback Statu. I think somehow, there is a way for the callback to receive messages or values during the OnProcess loop, or it's just optimism on my behalf. I should be able to get the loop to ask the window if cancel has been invoked.

There is a code sample on CP that uses a dialog box, in which a handle is created back to the dialog box, I'm not sure if the handle is bi-directional.

FYI:
I'm not not looking for code, just ideas or pointers.
This is the callback, the h file is seperate. I'm fuzzy about this line in the constructor
: m_Progress_Text(NULL), m_Progress_Bar(NULL) {
in which I' not really sure what it means, or what it does.

#include "stdafx.h"
#include <stdlib.h>

#include "atlstr.h"
#include <string>
#include <tchar.h>

#include <math.h>

#include "resource.h"
#include "stdio.h"
#include <commctrl.h>
#include <Windows.h>
#include <Winuser.h>
#include <Winbase.h>
#include "SQL_Servers_BindCallback.h"

#pragma comment(lib, "urlmon.lib")
#pragma comment(lib,"wininet.lib")
#define RETURN_ON_FAILURE(hr) if (FAILED(hr)) return (hr);	
		
	SQL_Servers_BindCallback::SQL_Servers_BindCallback ( ) : m_Progress_Text(NULL), m_Progress_Bar(NULL) {
		ShowWindow(m_Progress_Bar, SW_SHOW);
		UpdateWindow(m_Progress_Bar);
		UpdateWindow(m_MDIChild);

		SendMessage(m_Progress_Bar, PBM_SETRANGE, 0, MAKELPARAM(0, 5000) );
		SendMessage(m_Progress_Bar, PBM_SETPOS, (WPARAM)10, 0);		
	}

	SQL_Servers_BindCallback::~SQL_Servers_BindCallback( ) {	
	
	}
			
	HRESULT SQL_Servers_BindCallback::OnProgress(
		/* [in] */ ULONG ulProgress, 
		/* [in] */ ULONG ulProgressMax, 
		/* [in] */ ULONG ulStatusCode, 
		/* [in] */ LPCWSTR wszStatusText)
	{				
		static int iMsgLen;
		static double dBytes;
		static double dTotal;
		static double dPercent;
		static double dPB;	
		static TCHAR szStatusMessage[80];			
   		
		wcsncpy_s(szStatusMessage, L"please wait...", wcslen(L"please wait...") );

		switch (ulStatusCode) {		
			case BINDSTATUS_CONNECTING:
				wcsncpy_s(szStatusMessage, L"Connecting to download.microsoft.com", wcslen(L"Connecting to download.microsoft.com") );
				break;

			case BINDSTATUS_DOWNLOADINGDATA:
				// Calculate the MegaBytes Downloaded		
				dBytes = ((float)ulProgress / (1024*1024));
				dTotal = ((float)ulProgressMax / (1024*1024));
				dPercent = dBytes/dTotal*100;

				if ((dBytes > 0.01) && (dTotal > 0.01)) {
					swprintf_s(szStatusMessage,
						80,
						L"Downloading %.2f MB of %.2f MB [%.1f%%]",
						dBytes, dTotal, dPercent
					);					
				}
				break;

			case BINDSTATUS_ENDDOWNLOADDATA:				
				wcsncpy_s(szStatusMessage, L"Disconnecting from download.microsoft.com", wcslen(L"Disconnecting from download.microsoft.com") );
				Sleep(1000);
				return TRUE;
				break;
		}		
		
		// Publish Status Message
		SetWindowText(m_Progress_Text, szStatusMessage);
				
		// Format the Progress Bar Value
		dPB = (float)ulProgress/ulProgressMax*5000.0;
		SendMessage(m_Progress_Bar, PBM_SETPOS, (WPARAM)dPB, 0);
		SendMessage(m_Progress_Bar, PBM_SETRANGE, 0, MAKELPARAM(0, 5000) );		
		iLoopCount++;
		return S_OK;	
	}

	HRESULT SQL_Servers_BindCallback::OnStartBinding(
		/* [in] */ DWORD dwReserved, 
		/* [in] */ IBinding __RPC_FAR *pib)
	{ return S_OK; }
	
	HRESULT SQL_Servers_BindCallback::GetPriority(
		/* [out] */ LONG __RPC_FAR *pnPriority)
	{ return E_NOTIMPL;	}

	HRESULT SQL_Servers_BindCallback::OnLowResource(
		/* [in] */ DWORD reserved) 
	{ return E_NOTIMPL;	}

	HRESULT SQL_Servers_BindCallback::OnStopBinding(
		/* [in] */ HRESULT hresult, 
		/* [unique][in] */ LPCWSTR szError)
	{ return S_OK; }

	HRESULT SQL_Servers_BindCallback::GetBindInfo(
		/* [out] */ DWORD __RPC_FAR *grfBINDF, 
		/* [unique][out][in] */ BINDINFO __RPC_FAR *pbindinfo)
	{ return E_NOTIMPL; }

	HRESULT SQL_Servers_BindCallback::OnDataAvailable(
		/* [in] */ DWORD grfBSCF, 
		/* [in] */ DWORD dwSize, 
		/* [in] */ FORMATETC __RPC_FAR *pformatetc, 
		/* [in] */ STGMEDIUM __RPC_FAR *pstgmed)
	{ return S_OK; }

	HRESULT SQL_Servers_BindCallback::OnObjectAvailable(
		/* [in] */ REFIID riid, 
		/* [iid_is][in] */ IUnknown __RPC_FAR *punk)
	{ return E_NOTIMPL;	}

	ULONG SQL_Servers_BindCallback::AddRef()
	{ return 0;	}

	ULONG SQL_Servers_BindCallback::Release()	
	{ return 0; }

	HRESULT SQL_Servers_BindCallback::QueryInterface(
		/* [in] */ REFIID riid, 
		/* [iid_is][out] */ void __RPC_FAR *__RPC_FAR *ppvObject)
	{ return E_NOTIMPL;	}

jkirkerx wrote:
I should be able to get the loop to ask the window if cancel has been invoked.

I'm not too sure about this hope. My understanding had been that the communication should be the other way around - i.e The main window will tell the thread to cancel the download, rather than the thread asking the window if it should continue/cancel

jkirkerx wrote:
FYI:

I'm not not looking for code, just ideas or pointers.

This is the callback, the h file is seperate. I'm fuzzy about this line in the constructor

: m_Progress_Text(NULL), m_Progress_Bar(NULL) {

in which I' not really sure what it means, or what it does.

Sure thing, sorry if I've innundated you with useless code, confusing the matter.
I think you'l find that the above snippet will call the constructors for the progress text and the progress bar. I reckon you'd find that if you changed the NULLs to "myProgTest" and 50, respectively that you'd have a progress bar at 50% with the text myProgTest - why not give it a try?

After thinking over this some more last night, I decided to have a go at implementing a cancel button for my downloads. In this case, I added an extra field to the struct that I pass to the thread function - a simple boolean flag, bCancelled. Cancelling the download is a simple matter of setting this flag in the struct from the main thread in response to the cancel button.
In the download function, during the receive loop I simply check the status of this flag. If true, I just clean-up the memory I used and exit. It seems to work flawlessly. Also, since it's a graceful exit, I can be certain of the number of bytes downloaded before it was cancelled. This has a two-fold benefit -
(1) I can close the handle on the file I'm saving it too, avoiding total loss of the data and
(2) I know where the download is up to, so I can resume it at a later time of my choosing

Initially, my aim had been to get the size of a resource if given a url. Since wininet didn't seem to provide the functionality, I had to get access to the raw HTTP headers. From what I could determine, this required low-level use of sockets. However, the rewards have been, well worth-while.

C++

case WM_COMMAND:
    switch(LOWORD(wParam))
    {
        case IDC_BTN_CANCEL:
            singleDownload.bCancelled = true;
            return true;

C++

///////////// step 3 - get received bytes ////////////////
// Receive until the peer closes the connection
hdr->contentLen = 0;
while(1)
{
    if (hdr->bCancelled)
        break;

    memset(hdr->readBuffer, 0, bufSize);
    hdr->thisReadSize = recv (hdr->conn, hdr->readBuffer, bufSize, 0);
    if ( hdr->thisReadSize <= 0 )
        break;

I just didn't want to sound like I wanted someone to fix my code because I posted it. No insult intended.

I got it to work this afternoon. I sort of did the same thing, but not as elaborate. I examined the code sample in CP, and put the if statement back in, and made it a long this time instead of a boolean.

I just needed to get the mechanics working, so I didn't go around in circles trying complex things. I cleaned up this program alot, and got rid of some the erratic side effects of experimenting. It's not perfect in the purest sense, but it's a start. Now I can clean it up some more based on feedback, and write a check to see if the file is already downloaded, and integrate the installation program I already wrote.

Edit:
After reading your post several times, I realize that all of the benefits you listed are well worth it. And you right about pushing the cancel to the thread. But having the byte count, and being able to resume is pretty sweet, when you need to pause to do something else, and pick up again. I wanted to write a socket based program, but I've only been doing this since oct 5, so I had to keep it simple for now. Are you checksuming your download?

I put the variable in the h file for the callback

SQL_Servers_BindCallback::SQL_Servers_BindCallback ( ) : m_Progress_Text(NULL), m_Progress_Bar(NULL) {
				
	}

	SQL_Servers_BindCallback::~SQL_Servers_BindCallback( ) {	
	
	}
			
	HRESULT SQL_Servers_BindCallback::OnProgress(
		/* [in] */ ULONG ulProgress, 
		/* [in] */ ULONG ulProgressMax, 
		/* [in] */ ULONG ulStatusCode, 
		/* [in] */ LPCWSTR wszStatusText)
	{				
		static int iMsgLen;
		static double dBytes;
		static double dTotal;
		static double dPercent;
		static double dPB;	
		static TCHAR szStatusMessage[80];

		if ( g_fAbortDownload == 0 ) <- put in h file of this cpp file
			return E_ABORT;

Then I made the callback global to the window page, and changed the way I close the thread.

C#

BOOL _download_SQLServer_Begin( BOOL bCancel ) {

    // lots of code ........

    int errorNumber;
    hThread = (HANDLE)_beginthreadex(NULL, 0, SQL_SoftwareDownload, NULL, 0, (unsigned *)&thread_id);
    if (hThread == 0) {
        errorNumber = errno;
    }
    Sleep(2000);
    int rv = WaitForSingleObjectEx(hThread, 3000, FALSE);
    if (rv == WAIT_OBJECT_0)
        resultCode = 1;

    return resultCode;
}

unsigned int __stdcall SQL_SoftwareDownload(void *) {
    int iReturn = 0;
    HRESULT hr = E_FAIL;

    // Globally declare the callback
    callback = new SQL_Servers_BindCallback;
    callback->m_MDIChild = hSQL_Servers_Download;
    callback->m_Progress_Text = lbl_SQL_Server_Download_Status;
    callback->m_Progress_Bar = pb_SQL_Server_Download_Status;

    hr = URLDownloadToFile(NULL, szUrl, szFileName, 0, callback);
    if (hr == S_OK)
        iReturn = 1;

    return iReturn;
}

BOOL _download_SQLServer_End( BOOL bCancel ) {
		
	BOOL			bResult								= FALSE;
	BOOL			bHandle								= FALSE;
	unsigned 		iThread;
		
	// Signal the Callback Loop
	callback->g_fAbortDownload = 0; < - made the call here, callback picked it up and aborted
	callback->Release();

	SetWindowText(lbl_SQL_Server_Download_Status, L"Disconnecting from download.microsoft.com");
	UpdateWindow(hSQL_Servers_Download);
	
	iThread = WaitForSingleObjectEx(hThread, 3000, FALSE); <- I thought this was needed
	bHandle = CloseHandle(hThread); < - I know _endthread is called automatically
	if (bHandle == 0) <- Not sure, but it shuts down real smooth like.
		Sleep(2000);			
		
	ShowWindow(pb_SQL_Server_Download_Status, SW_HIDE);
	ShowWindow(bt_SQL_Server_Download_Cancel, SW_HIDE);
	ShowWindow(bt_SQL_Server_Download_Exit, SW_SHOW);

jkirkerx wrote:
I just didn't want to sound like I wanted someone to fix my code because I posted it. No insult intended.

Oh, no no no no no. None taken - I'd feared having been an irritation. Laugh | :laugh:

Fantastic! Glad to hear you've got it working. 'twas quite the source of satisfaction for me when I had a running stable program, rather than just crash after crash[after crash, after crash]

Indeed, I used URLDownloadToFile or something similar in a VBA crud app in the last job I was at, though found it to be a pain for all sorts of reasons - not the least of which being that it's a blocking call - not so special in MS office apps when the whole UI freezes.. :grin:

I also hate the idea of having to continually allocate/reallocate memory to receive the download, hence the idea of being able to retrieve the size of content from the HTTP headers. Delving into the headers, I discovered the "Byte-Range:" or something similarly named, as a tag in the headers. This allows you to specify which _part_ of a particular resource you want. This is the thing that gives you the download resume functionality, it's also a way to (a) download large files with a number of parallel connections or (b)download only part of a file - e.g first part of an image file to determine dimensions (c) thinking about one of your questions, it would also mean you could checksum smaller portions of the file, minimizing the number of bytes that need to be re-downloaded in the event of a check-sum failure - think along the lines of torrent programs, where the download is split up into pieces, each of which may be check-summed and repeated in the event of failure. A 700mb file may be split up into 700 pieces of 1 meg, for instance. Much nicer to re-download 1mb rather than the whole 700

Sony PlayStation network, I hate you for this oversight. I am ever so tired of using 190mb of my monthly 4000mb plan to download a system-update, necessary to continue to play online - only to find that it fails 3, 4, 5 times in a row. Brilliant! 20% of my internet gone because nobody thought to checksum pieces rather than the whole payload Mad | :mad:

Oh, and in answer to your question, nope - not check-summing the download at this stage, thanks for causing me to start thing about it. Thumbs Up | :thumbsup:

Given that it would appear that you'd only ever have one instance of the above class at any given time, looks like the global flag in the .h file works okay. Of course, if it were a member of the class itself, you could have multiple instances of the class co-existing - though I can see no reason for that for this particular implementation. A more generic threaded file download class whether it be inherited or not, would be the place. In my implementation I've tested with about 15 simultaneous threads all downloading rss feeds, also with a test-bed app that grabs 10 files simultaneously and updates the download progress of each in a listview as each progress callback is received. - I can cancel any of them by choice. Next comes resume! Big Grin | :-D

Looks like the checksuming is a lot of work at this point.

I testing my code now, 742 meg download, and starting and stopping (Cancel). Strange, it picks up where it left off. I added a filesize check, to see if I have a partial or complete download before starting. If complete just install the file, that what the checksum was for, make sure it's legit.

I put that global in the callback h file, and cleaned up the h and cpp files for a class, so I suspect I can call it several times for concurrent use. I made the callback class global in the window code, so I call reference it from several different functions, and then release it at the end. All I need are more threads. I think I'll keep the downloads one at a time to get all the programs needed.

So far 554 megs, no stalls or freezes, progress bar looks accurate, percentage is correct.

No you were not a pain, I just had to much written already, and in theory, my code should work whether it was a dialog box or mdi child window, I didn't see the difference. I understood the last sample, just had to cherry pick what I needed.

What's your program for?. Commercial use?

I make web applications, and I've been waiting 8 years for a program that can install web applications with ease. Kind of like buying a bar-b-Que already build, having it delivered - with a bag of coals and matches. I tried very good instructions as a pdf, but the 1 click window app seems to be the way to go. I have a 99% failure rate on installation right now, so the goal is a single app packaged in installshield, in which it installs the window program, and sets up everything you need, and launches the browser window with the web code ready to go. Then add data import capability, like yahoo store, and an interface that sets your company info and preferences.

Download almost over, I need to stop and save a copy of a partial download so I don't have to keep downloading it.

Window Code:
SQL_Servers_BindCallback *callback;
HANDLE hThread;
long thread_id;

Function in Window Code:
callback = new SQL_Servers_BindCallback;
callback->m_MDIChild = hSQL_Servers_Download;
callback->m_Progress_Text = lbl_SQL_Server_Download_Status;
callback->m_Progress_Bar = pb_SQL_Server_Download_Status;

Class H:
#if !defined SQL_Servers_BindCallback_H
#define SQL_Servers_BindCallback_H

class SQL_Servers_BindCallback : public IBindStatusCallback
{

public:
SQL_Servers_BindCallback();
~SQL_Servers_BindCallback();

// Pointer to the download progress dialog.
LONG g_fAbortDownload;
HWND m_MDIChild;
HWND m_Progress_Text;
HWND m_Progress_Bar;

int iLoopCount;

// The time when the download should timeout.
BOOL m_bUseTimeout;
CTime m_timeToStop;

STDMETHOD(OnProgress)(
/* [in] */ ULONG ulProgress,
/* [in] */ ULONG ulProgressMax,
/* [in] */ ULONG ulStatusCode,
/* [in] */ LPCWSTR wszStatusText);

Hi everyone, I am trying to successfully run the Recipe Preview Sample from MSDN and cannot get it to work. I have downloaded the sample and the SDK, tried to change target platforms, changed the AppID, and followed all the instructions. I can get the handler to register successfully, but the .recipe file they provide as an example does not show in the preview pane.

Here is the link to the MSDN site. The code builds fine....

http://msdn.microsoft.com/en-us/library/windows/desktop/dd940374(v=vs.85).aspx

Any help would be appreciated, the source code they provide for download is very small if someone gets a chance to try it.

Thanks for the help,

AG

Sorry for non VC / MFC post.
Here is a base of my code to open COM port. Works as advertized.
The problem is that the DTR/RTS lines are used to key up a ham radio transmitter and are enabled when the file is first created.
Even this code still have considerable "flash" on these lines. I am just monitoring the lines with test LED's – I did not measure the flash lenght.
I could use a hardware delay circuit to stop this but would prefer to do this in software.

Any suggestions ?

Thanks for your time,
Vaclav

DCB dcb;
HANDLE hFile = ::CreateFile( strCOM ,
0, // querry only
0,
0,
OPEN_EXISTING, // communication file
0,
0);
dcb.fDtrControl = 0;
dcb.fRtsControl = RTS_CONTROL_DISABLE;
if(!::SetCommState(hFile,&dcb))
{
TRACE("\nObtain the error code");
}

Vaclav_Sal wrote:
Any suggestions ?

It would help if you formatted your code within <pre> tags so it is easier to read. You have also not really explained what your problem is, e.g. what is meant by the expression "flash" on these lines?

Unrequited desire is character building. OriginalGriff
I'm sitting here giving you a standing ovation - Len Goodman

"The problem is that the DTR/RTS lines are used to key up a ham radio transmitter and <b>are enabled when the file is first created</b>."

Disabling the DTR/RTS - setting to 0 - in DCB clears these lines but it takes a time to do that ( my test LED "flashes") , thus generating unwanted "key up " of the transmitter.