C# 一定時間ごとの処理

Question

C#で一定時間ごとに処理させようと思ってます。
大まかな処理の流れは以下です。

処理の流れ

処理1→処理2→処理3→[1秒]→処理1→処理2→処理3→[1秒]→処理1→処理2→処理3→[3秒]→処理1→処理2→処理3→[1秒]→・・・

「処理1→処理2→処理3→[1秒]」が一つの処理の単位で、これが3回終われば3秒待つというような処理を書こうとしています。

上記を実現するのに、DispatcherTimerを二つ使って処理を書いたのですが、[3秒]にあたる待ち時間が、初めに実行している処理1からの経過時間となってしまい、処理3終了から3秒となりません。どのように修正すればよいかご教授ください。

using System;
using System.Windows;
using System.Windows.Threading;

namespace timerSamplePrj
{
    /// <summary>
    /// MainWindow.xaml の相互作用ロジック
    /// </summary>
    public partial class MainWindow : Window
    {
        DispatcherTimer timer= new DispatcherTimer();
        DispatcherTimer timer2 = new DispatcherTimer();

        string[] files = { "a", "b", "c" };
        int fileIter = 0;

        public MainWindow()
        {
            InitializeComponent();
            timer.Interval = TimeSpan.FromMilliseconds(1000);
            timer.Tick += send;

            timer2.Interval = TimeSpan.FromMilliseconds(3000);
            timer2.Tick += exec;
        }

        private void send(object sender, EventArgs e)
        {
            Console.WriteLine("send関数で" + files[fileIter++] + "を実行中");

            if (fileIter == files.Length)
            {
                timer.Stop();
                fileIter = 0;
            }
            
        }

        private void exec(object sender, EventArgs e)
        {
            Console.WriteLine("send中");
            textBox.Text = "実行中";
            timer.Start();
        }

        private void button_Click(object sender, RoutedEventArgs e)
        {
            timer2.Start();
        }

        private void button1_Click(object sender, RoutedEventArgs e)
        {
            timer2.Stop();
            timer.Stop();
        }
    }
}

[補足]
皆様、迅速なご回答ありがとうございます。適切な説明になっていない点もあり、改めて実現したい処理の流れを以下に記載させていただきます。

処理の流れ

func関数実行[1秒]→func関数実行→[1秒]→func関数実行→[3秒]→func関数実行→[1秒]→・・・

「func関数実行→[1秒]」が一つの処理の単位で、これがn回終われば3秒待つというような処理の流れです。

上記を実現するのに、以下ソースコードを書きました。ここで改めて質問させていただきたいのですが、timerのintervalプロパティを変更する、というのは適切なのでしょうか（intervalプロパティを変更し、待ち時間を変更させるというのは使い方が違う気がしてます）。
また、GUI上に配置するボタン押下で繰り返し実行処理を終了させたく、Zuishin様から頂いたTask.Delay()関数を使ったwhile文ループでは一度forループに入ると3回分の処理が実行されてしまうと思っており、途中で処理を終了させるようにもしたいです。

using System;
using System.Windows;
using System.Windows.Threading;

namespace timerSamplePrj
{
    /// <summary>
    /// MainWindow.xaml の相互作用ロジック
    /// </summary>
    public partial class MainWindow : Window
    {
        DispatcherTimer timer= new DispatcherTimer();

        string[] files = { "a", "b", "c" };
        int fileIter = 0;

        public MainWindow()
        {
            InitializeComponent();
            timer.Interval = TimeSpan.FromMilliseconds(0);
            timer.Tick += func;
        }

        private void func(object sender, EventArgs e)
        {
            Console.WriteLine("func関数で" + files[fileIter++] + "を実行中");
            
            if (fileIter == files.Length)
            {
                fileIter = 0;
                timer.Interval = TimeSpan.FromMilliseconds(3000);
            }
            else
            {
                timer.Interval = TimeSpan.FromMilliseconds(1000);
            }
        }

        private void button_Click(object sender, RoutedEventArgs e)
        {
            Console.WriteLine("exec開始");
            timer.Start();
        }

        private void button1_Click(object sender, RoutedEventArgs e)
        {
            Console.WriteLine("exec終了");
            fileIter = 0;

            timer.Stop();
            timer.Interval = TimeSpan.FromMilliseconds(0);
        }
    }
}

Comments

[Zuishin]

このような複雑な時間間隔の場合は、タイマーを使うよりも Reactive Extensions や非同期処理を使うほうが簡潔に書けます。

while (条件) {
for (int i = 0; i < 3; i++) {
処理1
処理2
処理3
await Task.Delay(1000);
}
await Task.Delay(2000);
}

[Zuishin]

なお、別スレッドから他のコントロールにアクセスすると例外が発生するので、その場合は次のように UI スレッドに処理を戻してください。

Dispatcher.Invoke(() =>
{
処理1
処理2
処理3
));
await Task.Delay(1000);

[Zuishin]

御託はいいから自分の望み通りのものを作れという意味ですか？

[senkei_river]

そのような意図ではございません。
y_waiwaiのご回答にもあるように、ソースコードと処理の対応もついておらず、おかしな説明となっていた点を修正した次第です。

不快にさせてしまったなら申し訳ございませんでした。

[Zuishin]

すでに付いている回答のどこが不満なのかをそれぞれの回答にコメントとして書いてください。

何の反応も無ければ回答の修正のしようもないでしょう。

不満がないのであればベストアンサーを選んでください。

[senkei_river]

補足の方法についてご指摘ありがとうございます。

以後気を付けます。。。

Answer 1

こんな構造体を作ってタスクをリスト化すればよいと思います。

struct task
{
    public string Name;
    public Action<task> Proc;
    public int Interval;
}

で、こんな感じに

using System;
using System.Collections.Generic;
using System.Diagnostics;
using System.Windows;
using System.Windows.Threading;

namespace WpfApp1
{
    struct task
    {
        public string Name;
        public Action<task> Proc;
        public int Interval;
    }

    public partial class MainWindow : Window
    {
        List<task> taskList;
        int taskPos;
        DispatcherTimer timer = new DispatcherTimer();

        public MainWindow()
        {
            InitializeComponent();
            timer.Tick += exec;
        }

        private void button_Click(object sender, RoutedEventArgs e)
        {
            taskPos = 0;
            taskList = new List<task>();
            taskList.Add(new task { Name = "Process1", Proc = Process1, Interval = 1 });
            taskList.Add(new task { Name = "Process2", Proc = Process2, Interval = 1 });
            taskList.Add(new task { Name = "Process3", Proc = Process3, Interval = 3 });
            taskList.Add(new task { Name = "Process1", Proc = Process1, Interval = 1 });
            taskList.Add(new task { Name = "Process2", Proc = Process2, Interval = 1 });
            taskList.Add(new task { Name = "Process3", Proc = Process3, Interval = 3 });
            Debug.WriteLine("タスクリスト開始");
            exec(this, EventArgs.Empty);
        }

        private void exec (object sender, EventArgs e)
        {
            timer.Stop();
            task task = taskList[taskPos];
            Debug.WriteLine(string.Format("{0} {1} 開始", DateTime.Now, task.Name));
            task.Proc.Invoke(task);
            Debug.WriteLine(string.Format("{0} {1} 終了", DateTime.Now, task.Name));
            timer.Interval = TimeSpan.FromSeconds(task.Interval);
            taskPos++;
            if (taskPos < taskList.Count) {
                timer.Start();
            } else {
                Debug.WriteLine("タスクリスト完了");
            }
        }

        private void Process1(task task)
        {
            Debug.WriteLine(string.Format("{0} {1} 実行中", DateTime.Now, task.Name));
        }

        private void Process2(task task)
        {
            Debug.WriteLine(string.Format("{0} {1} 実行中", DateTime.Now, task.Name));
        }

        private void Process3(task task)
        {
            Debug.WriteLine(string.Format("{0} {1} 実行中", DateTime.Now, task.Name));
        }
    }
}

Comments

[senkei_river]

ご回答ありがとうございます。大変わかりやすく、ソースコードも載せていただいたため、BAとさせていただきます。

Answer 2

たんに処理３終了でタイマをリセット、スタートすればいいんでは

＃提示のコードではどれが処理３かは不明ですが

Comments

[senkei_river]

質問文でも補足させていただきましたが、処理の流れについて適切な説明になっていない点がありました。補足として記載させていただいてます。