C# と VB.NET の質問掲示板

回答ありがとうございます。

> 生成し直し？
> 
> 通常、タイマーオブジェクト(DispatcherTimerに限らない)は、一度作ったらずーっとそのまま使えます。
> タイマーで通知を受け取った後、しばらく使わない(次の発行タイミングが確定するまで停止する)のであれば、
> Stop()メソッドを呼び出して止めておき、再びタイマーを発生させるときに Start() すればよいと思いますが？
申し訳ありません。説明が足りませんでした。
以下の理由によりタイマーオブジェクトを作り直しています。
・基底クラスと継承クラスを作成しており、タイマーオブジェクトは基底クラスに定義している
・継承しているクラスが複数あり、使用するクラスが変更されることがあるため継承クラスを生成し直すことがある。
・タイマーオブジェクトは継承クラスのコンストラクタでnewしている。
タイマーオブジェクトを保持するクラスを生成しなおすため、結果的にタイマーオブジェクトも再生成されているといった具合です。


> ところで、生成しなおしている部分というのは具体的にどういうコードを書いていますか？
以下のように記述しています。

base._timer =
                new DispatcherTimer(TimeSpan.FromMilliseconds(1000),
                DispatcherPriority.Send, new EventHandler(base.timerCallback),
                Dispatcher.CurrentDispatcher);
base._timer.IsEnabled = false;
base._timer.Stop();
                
保持しているクラスを生成し直さなければ動作するので、記述自体は合っている気がします・・・。
ちなみに、newした後は別の処理を行い、最後に以下も記述しています。

_timer.Interval = TimeSpan.FromMilliseconds(1000);
_timer.IsEnabled = true;
_timer.Start();

クラスの再生成をし直す前のDispatcherTimerが残っていたりするのでしょうか？
普通のTimerと違い破棄する命令が無いのが気になるところです・・・。

    //継承クラス
    public class ExpandClass : BaseClass
    {
        public ExpandClass()
        {
            clsTimer =
                new DispatcherTimer(TimeSpan.FromMilliseconds(1000),
                DispatcherPriority.Send, new EventHandler(base.dispatcherTimer_Tick),
                Dispatcher.CurrentDispatcher);
            clsTimer.Stop();
            iCounter = 0;
        }

        //タイマー作り直し
        public void RemakeTimer()
        {
            //これを消すと多重に動く
            clsTimer.Tick -= new EventHandler(base.dispatcherTimer_Tick);
            //再作成
            clsTimer =
                new DispatcherTimer(TimeSpan.FromMilliseconds(1000),
                DispatcherPriority.Send, new EventHandler(base.dispatcherTimer_Tick),
                Dispatcher.CurrentDispatcher);
            clsTimer.Stop();
            iCounter = 0;
        }

        //タイマー開始
        public void StartTimer()
        {
            if (clsTimer.IsEnabled == false)
            {
                clsTimer.Interval = TimeSpan.FromMilliseconds(1000);
                clsTimer.Start();
            }
        }

        //タイマー終了
        public void StopTimer()
        {
            if (clsTimer.IsEnabled == true)
            {                
                clsTimer.Stop();
            }
        }

        //タイマーが実行中かどうか
        public bool IsTimerStarted()
        {
            return clsTimer.IsEnabled;
        }
    }

    //
    public class BaseClass
    {
        protected DispatcherTimer clsTimer = null;
        protected int iCounter = 0;
        private Label clsBindLabel = null;

        //ラベルにカウンタ表示
        protected void dispatcherTimer_Tick(object sender, EventArgs e)
        {
            iCounter++;
            if(clsBindLabel != null)
            {
                clsBindLabel.Content = iCounter.ToString();
            }
        }

        public void SetLabel(Label clsLabel)
        {
            clsBindLabel = clsLabel;
        }
    }

＞とっちゃん様、howling様

度々ありがとうございます。
返信が遅くなり申し訳ありません。

提示して頂いたコードを参考にさせて頂き多少処理を改良しました。
ですが、相変わらず動作しない状態です・・・。

> えっと。。。生成しなおすのは、継承クラスですか？
> それとも、基底クラスが保持しているタイマーオブジェクトですか？
生成しなおしているのは継承クラスになります。（また言葉足らずでした・・・）
少しソースを修正したので、各クラスの重要な部分を以下に抜き出してみました。


■上位クラス(←前述の継承クラスをnewしている)
�@継承クラス生成
BaseClass Expand = new ExpandA();

�A継承クラス再作成（�@とは別メソッドで、�@よりも後に動作）
Expand = null;
Expand = new ExpandB();

■継承クラス

�@コンストラクタ
if (_timer != null)
{
　　_timer.Tick -= new EventHandler(timerCallback);
    _timer = null;
}

base._timer =
　　new DispatcherTimer(TimeSpan.FromMilliseconds(1000),
    DispatcherPriority.Send, new EventHandler(base.timerCallback),
    Dispatcher.CurrentDispatcher);
base._timer.Stop();

■基底クラス

�@メインの処理末尾
_timer.Interval = TimeSpan.FromMilliseconds(1000);
_timer.Start();

�Aコールバック関数（Tickイベントで動作）
protected void timerCallback(object sender, EventArgs e)
{
	// 処理いろいろ
}

検証したところ、上位クラスの�Aを行うとその後のコールバック関数が
呼ばれなくなってしまうようです（�Aを省けば動きます）。

> ちなみに、環境は、Win8,VS2012(.NET 4.5 のWPFアプリ。C#で記述)です。
自分はWin7で、後は同じ環境です。

頂いたサンプルソースでは無事動いたので、DispatcherTimer自体には問題なさそうですね・・・。
やはり別のバグという線が濃厚でしょうか？

shu様

返信ありがとうございます。
色々調べたところ、無事解決することができました。

どうやら、UIスレッド以外でDispatcherTimerを使用していたのが原因だったようです。
インスタンス生成、Start、StopはUIスレッドで呼び出さなければいけないようですね・・・。
子スレッドでも動きはしていたので気付くのに遅れてしまいました。

UIスレッドのコンストラクタでインスタンス生成をしてStartおよびStopするだけのメソッドを
用意し、それを子スレッドで呼び出すようにしたところ、無事継承クラス再生成後も動作するようになりました。

UIスレッド以外で使用できないのは仕様なのか、それとも.NETの不具合なのかが気になるところです。

▼参考
http://w.livedoor.jp/wpf/d/DispatcherTimer

> base._timerと_timerは同じもののような気がしますがなぜ記述を変えてあるのでしょう？
> おなじくbase.timerCallbakとtimerCallbackも
記述ミスですね・・・すみません。

色々お騒がせして申し訳ありませんでした。