複数フォルダを監視する場合のfilesystemwatcherの使い方は？

Question

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

###前提・実現したいこと
filesystemwatcherの使い方で悩んでいます。

複数のfilesystemwatcherを使い、相互に状態を見たいです。
複数のwatcherはいくつ使うかわからないので、動的に追加しています。

###試したこと
下記のようにwatcherを設定すると、それぞれのwatcherはfieldでないので参照できません。
こういうときはどのようにするものでしょう。

(1)rootを監視対象にして、すべてのサブフォルダを監視する。今回ならC:\。使うfilesystemwatcherはひとつ。パフォーマンスが心配。

(2)fieldにする。List<FileSystemWatcher>とか？
Createdでforeachする。

(3)その他。

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

		private void button1Click(object sender, EventArgs e) {
			string folder = @"c:\tmp1";
			var watcher1 = new FileSystemWatcher();
			watcher1.Path = folder;
			watcher1.Created += Watcher1_Created;
			watcher1.EnableRaisingEvents = true;
		}

		private void button2Click(object sender, EventArgs e) {
			string folder = @"c:\tmp2";
			var watcher2 = new FileSystemWatcher();
			watcher2.Path = folder;
			watcher2.Created += Watcher2_Created;
			watcher2.EnableRaisingEvents = true;
		}

		private void Watcher1_Created(object sender, FileSystemEventArgs e) {
			string createdfilepath = e.FullPath;
			//ここでbutton2で作ったwatcher2を表示したい。
			MessageBox.Show(watcher2.Path);//<private methodなのでない。
		}

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

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

Answer 1

目的がよくわかりませんが（"c:\tmp1"にファイルかフォルダが生成されたときに"c:\tmp2"のFileSystemWatcherを参照する目的）、とりあえず field にしたら良いのではないでしょうか。

インスタンスを生成するタイミング（この場合だとボタンのクリック時？）で field の List に Add するとかで。

ちなみに「button1_Click」じゃなくて「button1Click」なのは意味がありますか？
イベントハンドラではなくてただのメソッドとか。

Comments

[cancat]

コメントありがとうございます。
目的はうまく説明できません。すみません。
button1Clickには意味はありません。まあ、おっしゃるとおり、ただのMethodと考えていただいてよいと思います。それは（うまくいえませんが）質問の本質とは無関係な細部です。_がないことなんて、いままで気づいていませんでした。

とりあえずfieldですか。
watcher1, 2をフィールドとして宣言することはできません。
watcherの数は2個ではない（いくつかわからない）ためです。
宣言するとしたら、List<FileSystemWatcher> watchers = new List<FileSystemWatcher>();となります。
foreach使うのかと思うと気乗りしないです。

コードが少なそうなので、（１）のinclude subdrectoriesに魅力を感じてます。

Answer 2

アプリの設計を少し変更したほうがよさそうです。

現状の設計だと、button1やbutton2がクリックされるたびに FileSystemWatcher のインスタンスが無限に作られて行ってしまいます。
以下のように修正されてはいかがでしょう。

1. watcher1 と watcher2 をフォームのフィールドとして宣言する。
2. watcher1 と watcher2 の初期化とイベントハンドラの登録処理は、フォームのコンストラクタ内で行うようにする。
3. イベントハンドラ内でwatcher1とwatcher2を対象とした処理を行う。

こんな感じにすると整理できるのではないでしょうか。
ご参考になれば。

Comments

[cancat]

コメントありがとうございます。
変更は必須と考えています。

button1,2は1回しか押さないのでインスタンスは無限にはできません。

watcher1, 2をフィールドとして宣言することはできません。
watcherの数は2個ではない（いくつかわからない）ためです。
宣言するとしたら、List<FileSystemWatcher> watchers = new List<FileSystemWatcher>();となります。
なので、設計変更については同意できるものの、細部の方向性はちがっているようでした。
わたしの質問も悪いのかなと思います。ご容赦ください。