C / C++ / MFC

Trupti Mehta wrote:

i have a practise of calling just dd.~DeptDetails(), but someone pointed out, "Calling destructor without freeing the memory", so where am I going wrong. And keeping that in mind, in many such places I have called free(&dd) before the destructor.

Trupti Mehta wrote:

t present I pass object via parent & update the parent back by CHANGED_DS message. Is their any better way to work out which might reduce sending msg to parent for updated object.

Jijo raj wrote:

I can see a chance of serious synchronization error. Your UpdateData() is called after sending notification to the parent? In that case, your child dialog will get deleted by parent dialog and after deletion you might be calling UpdateData() which will endup in assertion from MFC. Its a wild guess, since i don't have the full picture of your code.

void CPage1Dlg::OnGetData()
{
	AfxMessageBox(_T("Into OnGetData"));
	UpdateData(true);

	// Search operator no
	OperatorDetails od;
//	if (operatorsMap->Lookup(m_number, od) != 0) {	// Found operator number
//		m_operName = od.GetOperName();

	if (true) {
		m_operName = "Trial";
		UpdateData (false);
		AfxMessageBox(m_operName);
		EnableConfirm(true);
	} else {
		m_operName = "";
		EnableConfirm(false);
	}

	od.~OperatorDetails();
	UpdateData (true);

	return;	
}
// Enables /Disables the Confirm Y/N? & Yes,No buttons
void CPage1Dlg::EnableConfirm(bool enable)
{
	if (enable) {
		GetDlgItem(IDC_NAME_BTN)->ShowWindow(false);
		GetDlgItem(IDC_CONFIRM_LABEL)->ShowWindow(true);
		GetDlgItem(IDC_YES1_BTN)->ShowWindow(true);
		GetDlgItem(IDC_WRONG_BTN)->ShowWindow(true);
	} else {
		GetDlgItem(IDC_NAME_BTN)->ShowWindow(true);
		GetDlgItem(IDC_CONFIRM_LABEL)->ShowWindow(false);
		GetDlgItem(IDC_YES1_BTN)->ShowWindow(false);
		GetDlgItem(IDC_WRONG_BTN)->ShowWindow(false);
	}
}

void CPage1Dlg::OnYes()
{
	// Create object of OperatorDetails with provided info
	OperatorDetails* odet(m_number, m_operName);
	// Update the DeptStock
	deptStock.SetOperator(odet);
	// Free up the memory of OpertatorDetails
	free(&odet);
	odet.~OperatorDetails();

	// Communicate with the Parent
	TellParent();

	return;
}
void CPage1Dlg::TellParent()
{
//	AfxMessageBox(_T("Inside TellPArent"));

	// Update deptStock obj with the parent
	this->CRHPostMessageToParent(WM_CHANGED_DS, (int)&deptStock);
	this->ShowWindow(SW_HIDE);
	this->CRHPostMessageToParent(WM_SHOW_NEXT_VIEW, 2);
//	AfxMessageBox(_T("Finished TellParent"));

	return;
}

Jijo raj wrote:

Keep DeptStock as a member of parent and Let the child windows keep a reference to your DeptStock and update that. When the parent window create you child window, pass DeptStock as constructor parameter by reference. So the child dialog can keep a reference to it. So that you can directly modify the DeptStock, by being in the child dialog itself and can avoid that message updating mechanism. You can also use pointers.

Artem Kustikov wrote:

It seems to me direct call of destructor (od.~OperatorDetails() and so on) is very bad technique (((

/************************************************
*  Virtual Function : OnDblclk(....)
*
*  Purpose : Called by the Framework when the user
*            double-click on a the List Control area
*
*  Comment : if the clicked item is a folder, 
*            SelectThisItem(..) function in the 
*            CShellTreeView class is called to
*            expand the the Treeview after searching
*            for the passed character string
*
*************************************************/
void CShellListView::OnDblclk(NMHDR* pNMHDR, LRESULT* pResult) 
{ 
	CString text;
	char szBuff[MAX_PATH];
	CString ptrPath[5];
	int i = 0;

	// User has double-clicked, get the clicked Item
	// depending on the clicked point
	LVHITTESTINFO lvhInf;
	GetCursorPos(&lvhInf.pt);
	ScreenToClient(&lvhInf.pt);
	int item = ListView_HitTest(GetListCtrl().m_hWnd, &lvhInf);
	if((LVHT_ONITEMLABEL & lvhInf.flags ) || (LVHT_ONITEMICON & lvhInf.flags))
	{
		
	
       LPTVITEMDATA* lptvid = NULL;
       lptvid = (LPTVITEMDATA*) m_pMalloc->Alloc (sizeof (LPTVITEMDATA));

	   LVITEM lvi;
	   lvi.mask = LVIF_PARAM;
	   lvi.iItem = lvhInf.iItem;
       ListView_GetItem(GetListCtrl().m_hWnd, &lvi);
	   lptvid = (LPTVITEMDATA*)lvi.lParam;

       ULONG uAttr = SFGAO_FOLDER;
       lptvid->lpsfParent->GetAttributesOf(1, (LPCITEMIDLIST *) &lptvid->lpi, &uAttr);

	   // is the item a Folder
	   if(uAttr & SFGAO_FOLDER)
	   {
		   CShellClass csc;
		   csc.GetName(lptvid->lpsfParent , lptvid->lpi , SHGDN_NORMAL, szBuff);
		   this->m_pShellTreeView->SelectThisItem(szBuff);

		 
		   IShellFolder *psfProgFiles = NULL;
		   HRESULT hr = lptvid->lpsfParent->BindToObject(lptvid->lpi, NULL, IID_IShellFolder, (LPVOID *) &psfProgFiles);
		   if(FAILED(hr))
			   return;
		   lptvid->lpsfParent = psfProgFiles;
		   LVPopulateFiles(lptvid); 
		   
		  

		   text = szBuff;
		   ptrPath[i] = text;
		  //  AfxMessageBox(szBuff);
		   AfxMessageBox(ptrPath[i]);
		   i++;

	   }
       else
	   {

       // display a popup-menu
	// 		ShowStdMenu(FALSE, lptvid);AfxMessageBox(szBuff);

AfxMessageBox(ptrPath[0]+ptrPath[1]+ptrPath[2]+ptrPath[3]);//"It's a file not a folder"
	   }
	}

	*pResult = 0;
}

CrocodileBuck wrote:

I think a need a shell function or something like that?

LPITEMIDLIST    idl;
TCHAR           szPath[MAX_PATH];
SHGetPathFromIDList(idl, szPath);

CrocodileBuck wrote:

how to use this function

sumit.durg wrote:

but not able to get dynamic generated URL (URL contain ? mark ).

Somethings seem HARD to do, until we know how to do them.
_AnShUmAn_