Go Premium for a chance to win a PS4. Enter to Win

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 339
  • Last Modified:

Dynamic Menu Options

In the following snippet of code I have created a new popup menu and added a couple menu items with AppendMenu().  The popup is then appended to the menu associated with the CFrame window of the program.  The function is called from the OnCreate() message handler of the CView class of the
program:
      
int CSigmaView::OnCreate( LPCREATESTRUCT lpCreateStruct)
{
      ...

      cCWReport.SetRptMenu(this->GetParent());
            
      ...

      return 0;
}

  When the program runs, the top-level menu option "Reports" is apparently created OK because it appears at the end of the menu bar for the frame window. When the new menu option is selected it becomes highlighted and the program appears to hang for a few seconds as though it is processing something but the popup menu never appears and no error is generated.  I have used standard command ID's for the added menu items just for the sake of creating and testing the function.

void CSigmaReport::SetRptMenu(CWnd *pWnd)
{
      CMenu *pMenu;
      CMenu cPopup;

      cPopup.CreatePopupMenu();

      cPopup.AppendMenu(MF_ENABLED|MF_STRING,                           ID_FILE_OPEN, "&Bay Details");
      cPopup.AppendMenu(MF_ENABLED|MF_STRING,                           ID_FILE_OPEN, "&Vac Details");
      pMenu = pWnd->GetMenu();

      pMenu->AppendMenu(MF_POPUP|MF_STRING,                           (UINT)cPopup.GetSafeHmenu(),
                  "&Reports");
      pWnd->DrawMenuBar();
};
0
armg
Asked:
armg
1 Solution
 
chensuCommented:
In your program, cPopup is created on the stack. As soon as it goes out of the function (SetRptMenu) scope, it is destroyed automatically (the destructor CMenu::~CMenu() is called).

You should either put cPopup as a member varible of CSigmaView or allocate it dynamically by using new and delete.

Method 1:
CMenu cPopup; // member varible of CSigmaView

Method 2:
CMenu *pcPopup; // member varible of CSigmaView

pcPopup = new CMenu;
pcPopup->CreatePopupMenu();

void CSigmaView::OnDestroy()
{
    delete pcPopup;
    ...
}

0

Featured Post

What does it mean to be "Always On"?

Is your cloud always on? With an Always On cloud you won't have to worry about downtime for maintenance or software application code updates, ensuring that your bottom line isn't affected.

Tackle projects and never again get stuck behind a technical roadblock.
Join Now