関数呼び出しの遅延

スレッドの実行を継続させながら関数呼び出しを遅らせる簡単な方法はありますか？

例えば

public void foo()
{
    // Do stuff!

    // Delayed call to bar() after x number of ms

    // Do more Stuff
}

public void bar()
{
    // Only execute once foo has finished
}

タイマーとイベントハンドラーを使用してこれを実現できることは承知していますが、これを実現する標準のc＃方法があるかどうか疑問に思っていましたか？

誰かが好奇心を持っている場合、これが必要な理由は、foo（）とbar（）が異なる（シングルトン）クラスにあり、例外的な状況でお互いを呼び出す必要があるためです。これは初期化時に行われるため、fooは、作成されているfooクラスのインスタンスを必要とするbarを呼び出す必要があるという問題があります。悪いデザインのほとんどスマック！

編集

悪いデザインについてはアドバイスをいただきながらポイントを取っていきます！私はシステムを改善できるかもしれないと長い間考えていましたが、この厄介な状況は例外がスローされたときにのみ発生し、それ以外の場合は2つのシングルトンが非常にうまく共存します。私は厄介な非同期パターンでメッセージを書くつもりはなく、クラスの1つの初期化をリファクタリングするつもりだと思います。

最新のC＃5/6に感謝します:)

public void foo()
{
    Task.Delay(1000).ContinueWith(t=> bar());
}

public void bar()
{
    // do stuff
}

私は自分でこのようなものを探していました-タイマーを使用しますが、最初の遅延のために一度だけ使用し、Sleep呼び出しを必要としないので、次のことを思いつきました...

public void foo()
{
    System.Threading.Timer timer = null; 
    timer = new System.Threading.Timer((obj) =>
                    {
                        bar();
                        timer.Dispose();
                    }, 
                null, 1000, System.Threading.Timeout.Infinite);
}

public void bar()
{
    // do stuff
}

（ありがとう コールバック内でタイマーを破棄するアイデアを提供して Fred Deschenesに）

以前のコメンターの設計観察に同意することを除いて、解決策はどれも私にとって十分にきれいではありませんでした。.Net 4は、現在のスレッドでの実行の遅延を非常に簡単にするクラスDispatcherとTaskクラスを提供します。

static class AsyncUtils
{
    static public void DelayCall(int msec, Action fn)
    {
        // Grab the dispatcher from the current executing thread
        Dispatcher d = Dispatcher.CurrentDispatcher;

        // Tasks execute in a thread pool thread
        new Task (() => {
            System.Threading.Thread.Sleep (msec);   // delay

            // use the dispatcher to asynchronously invoke the action 
            // back on the original thread
            d.BeginInvoke (fn);                     
        }).Start ();
    }
}

コンテキストについては、これを使用ICommandして、UI要素上でマウスの左ボタンに関連付けられているボタンを上に跳ね返してデバウンスしています。ユーザーはダブルクリックしてあらゆる種類の混乱を引き起こしています。（私はClick/ DoubleClickハンドラーも使用できることを知っていますが、ICommand全面的にs で機能するソリューションが必要でした）。

public void Execute(object parameter)
{
    if (!IsDebouncing) {
        IsDebouncing = true;
        AsyncUtils.DelayCall (DebouncePeriodMsec, () => {
            IsDebouncing = false;
        });

        _execute ();
    }
}

これらのオブジェクトの作成の制御とそれらの相互依存性は、クラス間ではなく外部で制御する必要があるように思えます。

これは確かに非常に悪い設計です。シングルトンだけでなく、それ自体が悪い設計です。

ただし、実際に実行を遅らせる必要がある場合は、次のようにします。

BackgroundWorker barInvoker = new BackgroundWorker();
barInvoker.DoWork += delegate
    {
        Thread.Sleep(TimeSpan.FromSeconds(1));
        bar();
    };
barInvoker.RunWorkerAsync();

ただし、これはbar()別のスレッドで呼び出されます。bar()元のスレッドで呼び出す必要がある場合は、bar()呼び出しをRunWorkerCompletedハンドラーに移動するか、で少しハッキングする必要がありますSynchronizationContext。

まあ、私は「設計」の点に同意する必要があります...しかし、おそらくモニターを使用して、他のユーザーがクリティカルセクションを通過したときに通知することができます...

    public void foo() {
        // Do stuff!

        object syncLock = new object();
        lock (syncLock) {
            // Delayed call to bar() after x number of ms
            ThreadPool.QueueUserWorkItem(delegate {
                lock(syncLock) {
                    bar();
                }
            });

            // Do more Stuff
        } 
        // lock now released, bar can begin            
    }

public static class DelayedDelegate
{

    static Timer runDelegates;
    static Dictionary<MethodInvoker, DateTime> delayedDelegates = new Dictionary<MethodInvoker, DateTime>();

    static DelayedDelegate()
    {

        runDelegates = new Timer();
        runDelegates.Interval = 250;
        runDelegates.Tick += RunDelegates;
        runDelegates.Enabled = true;

    }

    public static void Add(MethodInvoker method, int delay)
    {

        delayedDelegates.Add(method, DateTime.Now + TimeSpan.FromSeconds(delay));

    }

    static void RunDelegates(object sender, EventArgs e)
    {

        List<MethodInvoker> removeDelegates = new List<MethodInvoker>();

        foreach (MethodInvoker method in delayedDelegates.Keys)
        {

            if (DateTime.Now >= delayedDelegates[method])
            {
                method();
                removeDelegates.Add(method);
            }

        }

        foreach (MethodInvoker method in removeDelegates)
        {

            delayedDelegates.Remove(method);

        }


    }

}

使用法：

DelayedDelegate.Add(MyMethod,5);

void MyMethod()
{
     MessageBox.Show("5 Seconds Later!");
}

完璧な解決策は、タイマーに遅延アクションを処理させることですが。FxCopは、間隔が1秒未満の場合は好ましくありません。DataGridが列による並べ替えを完了するまで、アクションを遅らせる必要があります。私はワンショットタイマー（AutoReset = false）が解決策になると考え、それは完全に機能しました。そして、FxCopは警告を抑制させません！

これは古いバージョンの.NETで動作します
短所：独自のスレッドで実行されます

class CancelableDelay
    {
        Thread delayTh;
        Action action;
        int ms;

        public static CancelableDelay StartAfter(int milliseconds, Action action)
        {
            CancelableDelay result = new CancelableDelay() { ms = milliseconds };
            result.action = action;
            result.delayTh = new Thread(result.Delay);
            result.delayTh.Start();
            return result;
        }

        private CancelableDelay() { }

        void Delay()
        {
            try
            {
                Thread.Sleep(ms);
                action.Invoke();
            }
            catch (ThreadAbortException)
            { }
        }

        public void Cancel() => delayTh.Abort();

    }

使用法：

var job = CancelableDelay.StartAfter(1000, () => { WorkAfter1sec(); });  
job.Cancel(); //to cancel the delayed job

タイマーとイベントを使用する以外に、関数の呼び出しを遅延させる標準的な方法はありません。

これは、メソッドの呼び出しを遅延させるGUIアンチパターンのように聞こえるので、フォームのレイアウトが完了したことを確認できます。良い考えではありません。

David O'Donoghueからの回答に基づいて、ここに遅延デリゲートの最適化バージョンがあります。

using System.Windows.Forms;
using System.Collections.Generic;
using System;

namespace MyTool
{
    public class DelayedDelegate
    {
       static private DelayedDelegate _instance = null;

        private Timer _runDelegates = null;

        private Dictionary<MethodInvoker, DateTime> _delayedDelegates = new Dictionary<MethodInvoker, DateTime>();

        public DelayedDelegate()
        {
        }

        static private DelayedDelegate Instance
        {
            get
            {
                if (_instance == null)
                {
                    _instance = new DelayedDelegate();
                }

                return _instance;
            }
        }

        public static void Add(MethodInvoker pMethod, int pDelay)
        {
            Instance.AddNewDelegate(pMethod, pDelay * 1000);
        }

        public static void AddMilliseconds(MethodInvoker pMethod, int pDelay)
        {
            Instance.AddNewDelegate(pMethod, pDelay);
        }

        private void AddNewDelegate(MethodInvoker pMethod, int pDelay)
        {
            if (_runDelegates == null)
            {
                _runDelegates = new Timer();
                _runDelegates.Tick += RunDelegates;
            }
            else
            {
                _runDelegates.Stop();
            }

            _delayedDelegates.Add(pMethod, DateTime.Now + TimeSpan.FromMilliseconds(pDelay));

            StartTimer();
        }

        private void StartTimer()
        {
            if (_delayedDelegates.Count > 0)
            {
                int delay = FindSoonestDelay();
                if (delay == 0)
                {
                    RunDelegates();
                }
                else
                {
                    _runDelegates.Interval = delay;
                    _runDelegates.Start();
                }
            }
        }

        private int FindSoonestDelay()
        {
            int soonest = int.MaxValue;
            TimeSpan remaining;

            foreach (MethodInvoker invoker in _delayedDelegates.Keys)
            {
                remaining = _delayedDelegates[invoker] - DateTime.Now;
                soonest = Math.Max(0, Math.Min(soonest, (int)remaining.TotalMilliseconds));
            }

            return soonest;
        }

        private void RunDelegates(object pSender = null, EventArgs pE = null)
        {
            try
            {
                _runDelegates.Stop();

                List<MethodInvoker> removeDelegates = new List<MethodInvoker>();

                foreach (MethodInvoker method in _delayedDelegates.Keys)
                {
                    if (DateTime.Now >= _delayedDelegates[method])
                    {
                        method();

                        removeDelegates.Add(method);
                    }
                }

                foreach (MethodInvoker method in removeDelegates)
                {
                    _delayedDelegates.Remove(method);
                }
            }
            catch (Exception ex)
            {
            }
            finally
            {
                StartTimer();
            }
        }
    }
}

デリゲートに一意のキーを使用することで、クラスを少し改善できます。最初のデリゲートが起動する前に同じデリゲートを2回追加すると、辞書に問題が発生する可能性があるためです。

private static volatile List<System.Threading.Timer> _timers = new List<System.Threading.Timer>();
        private static object lockobj = new object();
        public static void SetTimeout(Action action, int delayInMilliseconds)
        {
            System.Threading.Timer timer = null;
            var cb = new System.Threading.TimerCallback((state) =>
            {
                lock (lockobj)
                    _timers.Remove(timer);
                timer.Dispose();
                action()
            });
            lock (lockobj)
                _timers.Add(timer = new System.Threading.Timer(cb, null, delayInMilliseconds, System.Threading.Timeout.Infinite));
}