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

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

ただいまの
回答率

87.36%

WPFでButtonのEventが存在するか知りたい

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 3,317

score 249

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

前提・実現したいこと

event handlerを.csのコードで割り当てる場合、複数回割り当てるとeventを複数回実行してしまいます。
if (button.Clickのeventhandlerの設定をしていなければ)と条件づけたいです。

試したこと

button.Click -= Button_Click;
回数が不明なので、これでは不十分。

発生している問題・エラーメッセージ

event handlerを.csのコードで割り当てる場合、複数回割り当てるとeventを複数回実行してしまう。

該当のソースコード

<Window x:Class="MultiEvent.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:local="clr-namespace:MultiEvent"
        mc:Ignorable="d"
        Title="MainWindow" Height="350" Width="525" Loaded="Window_Loaded">
    <Grid>
        <Button x:Name="button" Content="Button" HorizontalAlignment="Left" Margin="229,260,0,0" VerticalAlignment="Top" Width="75"/>

    </Grid>
</Window>
using System.Windows;

namespace MultiEvent {
    public partial class MainWindow : Window {
        public MainWindow() {
            InitializeComponent();
        }

        private void Window_Loaded(object sender, RoutedEventArgs e) {
            button.Click += Button_Click;
        }

        int counter = 0;
        private void Button_Click(object sender, RoutedEventArgs e) {
            //if (button.Clickのeventhandlerの設定をしていなければ)
                button.Click += Button_Click;
            MessageBox.Show(counter.ToString());
        }
    }
}

補足情報(言語/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 

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

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+3

eventに対して可能な操作は+=/-=のみですが、自分はそれを「その操作だけあればeventへの操作は十分であると言語仕様策定者が想定している」と解釈します。つまりeventへハンドラーを登録する契機はそのイベントを持つオブジェクトが活動を開始する以前のタイミングで一度だけ登録すれば十分であり、登録されていなければ登録するという操作は典型的な使い方では必要ないという判断なのだと思います。

ハンドラーが登録済みかどうかの判断機能は提供されてないので、必要ならば自分自身で管理するためのコードを書くことになろうかと思います。


多重にハンドラーを登録してしまうことを防ぐ管理コードの一例。

あくまで実装のイメージを示すだけのものです。本来は登録解除の機能やDisposeしたコントローラーに対して(WeakReferenceを使って)自動的に登録解除したりする仕組みを実装しないと使い物にならないと思います。下記の例では簡単のためTargetEventをstructにしてますがvalur type/classのどちらが適切かは最終実装によってかわるかもしれません。また当然ながらxamlで登録されたハンドラーについては対処のしようがないため、この仕組みを使うコントローラー(インスタンス)のイベントはxamlで定義しないことが前提になります(xamlで静的に記述したインスタンスのイベントは動的にハンドラーを追加することはないと思えるのでおそらく質問者さんが取り組んでおられるプログラムでは問題にならないと推測します)。

先にも書いたようにeventは+=/-=の左辺にしか使えず(他のメソッドの引数に渡すことさえできない)のでどうしてもこのような迂遠な実装になると思います。(リフレクションとunsafeコードを使うともう少しましにできるのかも知れませんが自分には可能かどうかわかりません)

public enum EventCode {
  CLICKED,
  ...
}
public static class HandlerManager {
  private struct TargetEvent {
    public Control TargetControl;
    public EventCode EventCode;
  }

  private static Dictionary<TargetEvent, EventHandler> registory
    = new Dictionary<TargetEvent, EventHandler>();

  public static bool RegisterIfAny(Control c, EventCode ec, EventHandler handler) {
    TargetEvent te = new TargetEvent { TargetControl = c, EventCode = ec };
    EventHandler h;
    if (registory.TryGetValue(te, out h)) {
      return false;
    } else {
      registory.Add(te, handler);
      return true;
    }
  }
}

// 上記を使ったハンドラー登録の例

public partial class Form1 : Form {
  public Form1() {
    InitializeComponent();
    // 以下のように2回登録しても最初しか登録されない
    if (HandlerManager.RegisterIfAny(button1, EventCode.CLICKED, button1_Click))
      button1.Click += button1_Click;
    if (HandlerManager.RegisterIfAny(button1, EventCode.CLICKED, button1_Click))
      button1.Click += button1_Click;
    }
  }
  ...
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/11/08 12:18 編集

    なるほど目的がわかりました。これから設計するならハンドラーの動的追加しないようにするのがよいと思ったのですが、問題の焦点はそこではなく「既にそうしてしまっているソースをいかに直すか」ということなのですね。わかってみるとなかなかに難物です。あまりすっきりした解決法はなさそうですが。うーむ・・・
    ヒントになるかどうかわかりませんが管理コードのアイデアを追記してみました。

    キャンセル

  • 2016/11/08 16:28

    ありがとうございます。
    おっしゃるとおり、これから設計するなら、そういう挙動/仕様だとわかって作れるのです。
    実におっしゃるとおりで、直面している課題は多重起動していて大変困っている、ということなのです。
    アイデア、ありがとうございます。すぐに役立てられそうにないのですが、試行錯誤してみます。

    キャンセル

  • 2016/11/08 19:00

    EventTargetにhandlerのメンバー入ってませんでした・・・うっかりしてました。

    キャンセル

0

質問文のコードだとButton_Clickが呼ばれている時点ですでに
button.Click に Button_Clickが登録されていると思うので、
こういうことでいいのか大変不安ですが、

if(button.Click == null)
    button.Click += Button_Click;

そういえばクラスの外側ですね。
すみません。

C#的に単純には無理です。
ある言語でやりづらいことは、
その言語的にはおかしいコードを組んでいるということだと思っているので、
可能ならコードをバッサリ投げ捨てますが、それが無理なら

一つしか登録できないボタンを継承して作るか
イベントの登録はボタンを生成するタイミングだけにするよう気をつけるか、
イベントを登録する側がイベントを追加したかを覚えておくかです。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/11/07 14:49

    単にnullを見るだけよいのですね!

    キャンセル

  • 2016/11/07 15:00

    とおもったら、確認したところ、イベントButtonBase.Clickは+=または-=の左側にのみ配置できます、とエラーになります。

    キャンセル

  • 2016/11/07 15:10

    単に2行書けばいいんじゃないでしょうか
    nullで"-="しても例外は発生しません

    button.Click -= Button_Click;
    button.Click += Button_Click;

    キャンセル

  • 2016/11/07 17:09

    単に2行書くだけでは応用できないのでだめです。

    キャンセル

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

  • ただいまの回答率 87.36%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る