C / C++ / MFC

"Ideas are a dime a dozen. People who put them into action are priceless." - Unknown

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~<br />
Peter Weyzen<br />
Staff Engineer<br />
<A HREF="http://www.santacruznetworks.com">Santa Cruz Networks</A>

for (itvPhoneNumbers = vPersonNumbers.begin();<br />
     itvPhoneNumbers != vPersonNumbers.end();<br />
     itvPhoneNumbers++, iCpt++) {<br />
         PhonesMenu.AppendMenu(MF_STRING, iCpt, *itvPhoneNumbers);<br />
}

Run-Time Check Failure #2 - Stack around the variable 'dlg' was corrupted.

CMenu* popup_menu = PhonesMenu.GetSubMenu(0);
popup_menu->TrackPopupMenu(TPM_LEFTALIGN | TPM_RIGHTBUTTON, rect.right, rect.top, AfxGetMainWnd());

CMenu popup_menu;<br />
popup_menu.CreatePopupMenu();<br />
<br />
popup_menu.AppendMenu(MF_STRING, 2, _T("I know they're stolen"));<br />
popup_menu.AppendMenu(MF_STRING, 3, _T("But I don't feel bad"));<br />
<br />
CRect rect;<br />
// ... get a valid rect from somewhere<br />
<br />
popup_menu.TrackPopupMenu(TPM_LEFTALIGN | TPM_RIGHTBUTTON, rect.right, rect.top, AfxGetMainWnd());

FactEditor.h :
<font color=blue>class</font> CFactEditorDlg : <font color=blue>public</font> CDialog {
    <font color=green>//...</font>
 
<font color=blue>protected</font>:
    <font color=green>//I added these two members in the dialog class...</font>
    std::vector<CString> m_vPersonPhoneNumbers;
    <font color=blue>bool</font>                 m_bPhonesMenuCalled;
 
    <font color=green>//...</font>
};
 
 
FactEditor.cpp :
<font color=blue>void</font> CFactEditorDlg::OnSClickShowNumbers() {
    <font color=blue>using namespace</font> std;
    m_vPersonPhoneNumbers.clear();
    map<CString, TOperatorPersonPair>::const_iterator itmPersonNumbers;
 
    <font color=green>// Populates the Person's phone numbers list</font> 
    <font color=blue>for</font> (itmPersonNumbers = m_mapPhoneWithOpPersPair.begin();
         itmPersonNumbers != m_mapPhoneWithOpPersPair.end();
         itmPersonNumbers++) {
         <font color=blue>if</font> (itmPersonNumbers->second.m_strPerson == m_strCalledPerson) {
            m_vPersonPhoneNumbers.push_back(itmPersonNumbers->first);
        }
    }
    sort(m_vPersonPhoneNumbers.begin(), m_vPersonPhoneNumbers.end());
 
    <font color=green>// Displays the popup menu from the vector</font>
    <font color=blue>if</font> (m_vPersonPhoneNumbers.size() != 0) {
        CMenu PhonesMenu;
        <font color=blue>if</font> (PhonesMenu.CreatePopupMenu()) {
            vector<CString>::const_iterator itvPhoneNumbers;
            CString strRes, strMenuTitle;
            strRes.LoadString(IDS_FACTEDITORDLGTITLE);
            strMenuTitle.Format(strRes, m_strEditorVersion);
            PhonesMenu.AppendMenu(MF_STRING, 1, strMenuTitle);
            PhonesMenu.AppendMenu(MF_SEPARATOR);
            <font color=blue>int</font> iCpt = 3;
            <font color=blue>for</font> (itvPhoneNumbers = m_vPersonPhoneNumbers.begin();
                 itvPhoneNumbers != m_vPersonPhoneNumbers.end();
                 itvPhoneNumbers++, iCpt++) {
                PhonesMenu.AppendMenu(MF_STRING, iCpt, *itvPhoneNumbers);
            }
            CRect rect;
            m_pbShowNumbers->GetWindowRect(&rect);
            PhonesMenu.EnableMenuItem(1, MF_GRAYED);
            m_bPhonesMenuCalled = <font color=blue>true</font>;
            PhonesMenu.TrackPopupMenu(TPM_LEFTALIGN | TPM_RIGHTBUTTON, rect.right+1, rect.top-1, AfxGetMainWnd());
        }
    }
}
 
 
BOOL CFactEditorDlg::OnCommand(WPARAM wParam, LPARAM lParam) {
    <font color=blue>if</font> (HIWORD(wParam) == (UINT)0) {    <font color=green>// Called from a menu</font>
        <font color=blue>if</font> (m_bPhonesMenuCalled) {    <font color=green>// Called from Person Phone Numbers Popup Menu</font>
            m_peCalledNumber->SetWindowText(m_vPersonPhoneNumbers[LOWORD(wParam)-3]);
            m_bPhonesMenuCalled = <font color=blue>false</font>;
        }
    }
    <font color=blue>return</font> CDialog::OnCommand(wParam, lParam);
}