ポップアップメニューをクリックして、exeを開けるようにしたい

Question

右クリックでポップアップメニューを表示する→ポップアップメニューの１つを選択→あるexeを開く
という機能を作りたいです。

開発環境はVC++, MFCを使ってます。
ポップアップメニューを出すところまでは出来たのですが、現在はメニューを選択しても、何も起こりません。
ポップアップメニューを表示するコードは以下のようになっています。
CMenu menu;
menu.CreatePopupMenu();
menu.AppendMenu(MF_STRING|MF_ENABLED,ID_EDIT_UNDO ,"xxx.exe");
//exeの名前が表示されるようにしている。ID_EDIT_UNDOは適当に入れただけ。他に何を入れたら良いのかは分からない。

ClientToScreen(&point);
menu.TrackPopupMenu(
TPM_LEFTALIGN | //クリック時のX座標をメニューの左辺にする
TPM_RIGHTBUTTON, //右クリックでメニュー選択可能とする
point.x,point.y, //メニューの表示位置
this //このメニューを所有するウィンドウ
);

ID_EDIT_UNDOを別のものに変えれば、exeを開くように出来るような気がして色々試したのですが、出来ませんでした。お力をお貸しください。

Comments

[takabosoft]

開発環境を書いてください。visual studio 2017等

[Grayi]

visual studio2015です。書いてなくてすみません。

[takabosoft]

右クリックメニューの項目からEXEを開く機能を付けたいのは理解しましたが、これはメニュー項目は１つだけですか？（例えばユーザーが自由に何個も登録できるようにしたいといった事はないですか？）

[Grayi]

メニュー項目は一つだけです。ユーザーが編集することができるような機能などは、今のところ予定していません。質問はちゃんと細かく書いた方が良いのですね。以後気を付けます。ご指摘ありがとうございます。

Answer 1

ベストアンサー付いちゃていますが、一応私がいつもやっている手順を書いておきます。

まず今回のように固定のポップアップメニューを表示するだけなら、
リソースとしてメニューを定義します。

ポップアップ専用のリソースは無いので（？）メニューバーの1つ目の項目をポップアップとして使うイメージです。
下の図のように、項目名とIDを指定してあげます。

で、リソースエディタでこのようにIDをしっかり付けることで、Visual StudioがこのIDをコマンドIDとして認識してくれます。これが重要です。
（※コマンドIDの番号とかは勝手に振ってくれます）

次にここで作ったポップアップメニューの表示方法ですが、以下のように実装してください。

	CMenu menu;
	if (menu.LoadMenu(IDR_POPUPMENU)) {
		CMenu* pPopupMenu = menu.GetSubMenu(0);
		if (pPopupMenu != NULL) {
			ClientToScreen(&point);
			pPopupMenu->TrackPopupMenu(TPM_LEFTALIGN | TPM_RIGHTBUTTON, point.x, point.y, this);
		}
	}

リソースを読み込んで表示しているだけです。
ここまでで、リソースで定義したポップアップメニューが正しく表示されることを確認してください。

---

次に、ダイアログベースなら自分のダイアログクラス（SDIとかMDIならViewまたはMainFrame）の「クラスウィザード」を開きます。

すると、さっき作ったID_BOOT_HOGEに対して「ハンドラーの追加」が実行できます。

ハンドラーがcppファイル側に作成されますので、中身を実装してください。
※コマンド「ID_BOOT_HOGE」が実行されたときにOnBootHoge()が呼ばれるようになります。

void CMFCApplication20Dlg::OnBootHoge()
{
	AfxMessageBox(_T("あとはここにEXEを起動するコードを書く"));
}

これでどうでしょうか。
可能であればMFC入門の書籍等を一冊買って読んでもらった方が良いかと思います。

Comments

[Grayi]

ご丁寧な回答、ありがとうございます。スクリーンショットまで見ることができ、とても分かりやすく、勉強になります。コマンドIDの設定の仕方などがなかなか分からず、結局resourse.hに手入力してしまったのですが、こんな便利な方法があったのですね。明日早速試してみます。
MFCの書籍は先輩から貸していただいたものを参考にしたりしているのですが、10年以上前に発行されたもののためか今回のケースでは参考にすることが出来ませんでした（私の理解力不足が原因かもしれません）。やっぱ購入するなどして、勉強した方がいいですね。色々教えてくださって、本当にありがとうございます。

Answer 2

StandAlyさんが書いたのは、ポップアップメニュー表示に関することまでです。
ポップアップメニューの選択した項目にコマンドIDを登録しているので、
そのコマンドID用の処理を別途書けということです。
それが、ttyp03さんの言っていることです。

Comments

[Grayi]

回答ありがとうございます。
教えて頂いた通りコマンドID用の処理を書いたら、右クリックでexeが起動できるようになりました。困ったことに、ポップアップメニューを選択しなくても、右クリックのみで起動してしまうのですが。

Answer 3

ID_EDIT_UNDOのイベント処理はどうなっているのでしょうか？
そこに実行する処理を書かない限りは何も起こりません。
手元に環境が無いのでサンプルを提示することができませんが、MFC AppendMenuで検索すれば色々出てくると思います。

Comments

[Grayi]

回答ありがとうございます。
ID_EDIT_UNDOのイベント処理は何も書いてませんでした。
頂いたアドバイスを参考に、以下のように書き直しました。

//CTest.h
#define IDS_STARTEXE ShellExcute(GetHwnd(),NULL,TEXT("exeの絶対パス"),NULL,（“exeが置いてあるディレクトリのパス”）SW_SHOWNORMAL)

//CTest.cpp
void CTest::OnRButtonDown(UINT nFlags, CPoint point)
{
// TODO: この位置にメッセージ ハンドラ用のコードを追加するかまたはデフォルトの処理を呼び出してください

CMenu menu;
menu.CreatePopupMenu();
menu.AppendMenu(MF_STRING|MF_ENABLED,IDS_STARTEXE ,"xxx.exe");

ClientToScreen(&point);
menu.TrackPopupMenu(
TPM_LEFTALIGN | //クリック時のX座標をメニューの左辺にする
TPM_RIGHTBUTTON, //右クリックでメニュー選択可能とする
point.x,point.y, //メニューの表示位置
this //このメニューを所有するウィンドウ
);
menu.DestroyMenu();
}

とりあえず、右クリックしたらexeを起動できるようになりました！ありがとうございます。
問題は、今のままだとポップアップメニューを選択しなくても、右クリックしただけでexeが起動してしまうことです。

[ttyp03]

AppendMenuの第二引数でShellExcuteしたらダメでしょう。
そりゃ右クリックしただけで実行されますよ。
ここに指定するのはあくまでもIDです。
そのIDのイベントハンドラを別に用意しないとダメです。
例えばここらへんのサイトなんか参考にならないでしょうか？
http://www.page.sannet.ne.jp/naoya/Windows/cmenu.html
https://qiita.com/tadnakam/items/13738da833801f297d9b

[Grayi]

教えて頂いたサイトを参考にしたら、ちゃんと
右クリック→ポップアップメニュー表示→左クリックメニューを選択→exeを起動　が出来るようになりました！
イベントハンドラを作るというのが分かっていなかったので、大変助かりました。ttype03さん、ありがとうございます。