0

私は winapi を少しだけ使って、所有者がメニューに描画するプロジェクトを作成しています。を呼び出すとGetMenuItemInfo、メニュー項目のテキストが設定されますが、fType UINT 変数フラグは設定されません。
現在、私は宣言しています：

MenuItem->fMask = MIIM_TYPE

そして、MSDN は次のように述べています。

MIIM_TYPE Retrieves or sets the fType and dwTypeData members.

MIIM_TYPE フラグと混同したかどうかはわかりません。

これが私のコードです：

void SetOwnerDrawMenu(HMENU * menu)
{
MENUIF * menu_item_information;
HMENU sub_menu_ocational;

UINT uId_menuitem;
int nMenuCountItems = GetMenuItemCount(*menu);

MENUITEMINFO * MenuItem = (MENUITEMINFO*)malloc(sizeof(MENUITEMINFO));

for(int i=0;i<nMenuCountItems;i++)
{

    menu_item_information = (MENUIF*)malloc(sizeof(MENUIF));
    menu_item_information->isSeparator=false;
    menu_item_information->max_width=0;
    sub_menu_ocational = 0;
    uId_menuitem = GetMenuItemID(*menu,i);
    memset(&MenuItem,0,sizeof(MenuItem));
    MenuItem = (MENUITEMINFO*)malloc(sizeof(MENUITEMINFO));
    MenuItem->cbSize = sizeof(MenuItem);
    MenuItem->fMask = MIIM_TYPE;
    MenuItem->cch = MAX_ODM_CCH;
    MenuItem->dwTypeData = menu_item_information->szItemText;

    GetMenuItemInfo(*menu,uId_menuitem,FALSE,MenuItem);

    UINT final_flags = MF_BYPOSITION | MF_OWNERDRAW;

    if( ( MFT_SEPARATOR & MenuItem->fType ) == MFT_SEPARATOR )
    {
        final_flags |= MF_SEPARATOR;
        menu_item_information->isSeparator = true;
    }
    else
    {
        // Not important stuff
    }

    sub_menu_ocational = GetSubMenu(*menu,i);

    if(sub_menu_ocational!=NULL)
    {
        ModifyMenu(*menu,i,final_flags,0,(LPCTSTR)menu_item_information);
        // We got a submenu, repeat this operation
        SetOwnerDrawMenu(&sub_menu_ocational);
    }
    else
    {
        ModifyMenu(*menu,i,final_flags,0,(LPCTSTR)menu_item_information);
    }
}
}

関数を使用してメニューを挿入していInsertMenuます：

InsertMenu(tid_cmenu,0,MF_BYPOSITION | MF_SEPARATOR,0,NULL);
InsertMenu(tid_cmenu,0, MF_BYPOSITION | MF_STRING, TID_EXIT, "Exit");

正確には、なぜGetMenuItemInfofType を取得しないのですか?

4

2 に答える 2

3

からの戻りコードをチェックしていた場合、GetMenuItemInfo失敗していることがわかります。エラーは次の行にあります。

MenuItem->cbSize = sizeof(MenuItem);

メンバーは構造体のサイズに設定されるはずですが、ポインターのサイズに設定していますMENUITEMINFO::cbSize(つまり、プラットフォームに応じて 4 または 8 バイト)。MENUITEMINFOMENUITEMINFO*

コードを次のように変更します。

MenuItem->cbSize = sizeof(MENUITEMINFO);

また、コードがMenuItemループの外側に割り当てられているだけでなく、ループ内で反復ごとに 1 回割り当てられているため、メモリ リークが発生しています。

于 2013-07-24T04:06:13.747 に答える