Solved

Dynamic Menu Options

Posted on 1997-02-21
1
329 Views
Last Modified: 2013-11-20
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
Comment
Question by:armg
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
1 Comment
 
LVL 23

Accepted Solution

by:
chensu earned 70 total points
ID: 1300304
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

Independent Software Vendors: We Want Your Opinion

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Introduction: Ownerdraw of the grid button.  A singleton class implentation and usage. Continuing from the fifth article about sudoku.   Open the project in visual studio. Go to the class view – CGridButton should be visible as a class.  R…
If you use Adobe Reader X it is possible you can't open OLE PDF documents in the standard. The reason is the 'save box mode' in adobe reader X. Many people think the protected Mode of adobe reader x is only to stop the write access. But this fe…
This video will show you how to get GIT to work in Eclipse.   It will walk you through how to install the EGit plugin in eclipse and how to checkout an existing repository.
This video shows how to use Hyena, from SystemTools Software, to update 100 user accounts from an external text file. View in 1080p for best video quality.

738 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question