質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Q&A

解決済

2回答

1682閲覧

C#でMethodを引数にしたい。

cancat

総合スコア313

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

0グッド

0クリップ

投稿2016/09/23 07:16

こんにちは。
Windows10でC#のアプリケーションを開発しています。
Visual Studio 2015 Communityを使っています。

###前提・実現したいこと
Methodを引数にできませんか?

###該当のソースコード

C#

1private void test() { 2 ContextMenu contextMenu = new ContextMenu(); 3 { 4 contextMenu.Items.Add(new Separator()); 5 MenuItem item = new MenuItem(); 6 item.Header = "MenuA"; 7 item.Name = "MenuA"; 8 item.Click += (s, e) => MenuA_Click(s, e); 9 contextMenu.Items.Add(item); 10 } 11 { 12 contextMenu.Items.Add(new Separator()); 13 MenuItem item = new MenuItem(); 14 item.Header = "MenuB"; 15 item.Name = "MenuB"; 16 item.Click += (s, e) => MenuB_Click(s, e); 17 contextMenu.Items.Add(item); 18 } 19}

このようなコードで、繰り返し多くていやだなーと感じました。

そこで、共通部分を切り出すと、次のようになります。

C#

1private void NewMethod(ContextMenu contextMenu) { 2 Dictionary<string, Method> dic = new Dictionary<string, Method>(); 3 dic.Add("MenuA", MenuA_Click); 4 dic.Add("MenuB", MenuB_Click); 5 foreach(var item in dic) 6 addContextMenu(contextMenu, item.Key, item.Value); 7} 8 9private void addContextMenu(ContextMenu contextMenu, string menu, Method method) { 10 contextMenu.Items.Add(new Separator()); 11 MenuItem item = new MenuItem(); 12 item.Header = menu; 13 item.Name = menu; 14 item.Click += (s, e) => method(s, e); 15 contextMenu.Items.Add(item); 16} 17

こういうことはできないのでしょうか?

###補足情報(言語/FW/ツール等のバージョンなど)
Microsoft Visual Studio Community 2015
Version 14.0.25424.00 Update 3
Microsoft .NET Framework
Version 4.6.01038

インストールしているバージョン:Community

Visual C# 2015 00322-20000-00000-AA575
Microsoft Visual C# 2015

です。
よろしくお願いします。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

C#

1item.Click += (s, e) => method(s, e);

これのようにラムダ式を定義すれば使えます。

C#

1 private void addContextMenu(ContextMenu contextMenu, string menu, Action<S,E> method) {

Action<S, E>は型Sと型Eを引数とする戻り値なしの関数を定義しています。(適宜型を変更してください。)ただし、method(s, e)は実行時の変数の値になるので注意が必要です。

詳しくは下記
http://ufcpp.net/study/csharp/sp_delegate.html?key=anonymous#anonymous
http://ufcpp.net/study/csharp/sp3_lambda.html

投稿2016/09/23 07:30

編集2016/09/23 07:48
iwamoto_takaaki

総合スコア2883

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

cancat

2016/09/23 07:45

ありがとうございます。 コードはできました。 夜か週末に動作を確認します。 すごいです。
iwamoto_takaaki

2016/09/23 07:51

はい、うまく使えるようになるとすごいです。オブジェクト指向の考え方がだいぶ変ります。
cancat

2016/10/05 09:31

遅くなりました。ID/PWを失念して探しておりました。 無事動きました。 codeがとてもスマートになりました。 感謝します。
guest

0

delegateを使うと実現できそうなので、試して頂いても良いですか?
http://ginpro.winofsql.jp/article/292539736.html

投稿2016/09/23 07:24

ijufumi

総合スコア276

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

cancat

2016/09/23 07:46

情報ありがとうございます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問