c# 画面コントロール生成時の処理速度改善

Question

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

###前提・実現したいこと
c#にて、
事前に設定した質問と解答を
dbから取得し動的にコントロールを
配置するシステムを作っています。
コントロール生成に時間がかかってしまい
ここの速度をあげたいです。
現在、100コントロールで3秒程度。
これを1秒程度にしたい。
また、速度アップが無理であれば
他の仕様に変更することも可能。

###発生している問題・エラーメッセージ
画面にはtabcontrolの上に
layoutflowpanelを乗せ
その上にコントロールを追加しています。

追加するコントロールは
その都度dbから取得しています。
基本的には例えば以下のようなコントロールを
追加しています。

氏名 ラベル
氏名解答欄 テキストボックス
生年月日 ラベル
生成月日解答欄 ラジオボタン
（元号 大．昭，平）
テキストボックス3つ
（年、月、日）

ソースとしては以下のような形です
FlowLayoutPanel flowPanel = new FlowLayoutPanel();
flowPanel.Dock = DockStyle.Fill;
flowPanel.FlowDirection = FlowDirection.TopDown;
flowPanel.AutoScroll = true;

this.Controls.Add(flowPanel);
※下記は1つのボタンですが
ここはdb内容を見ながら都度
コントロールインスタンス生成を
しながら動的に作成。
下記はプロパティ設定はtextのみだが
実際にはsizeなど
他のプロパティ設定もある。

Button button = new Button();
button.Text = i.ToString();

flowPanel.Controls.Add(button);

###試したこと
速度アップのため、ソースの見直しをしましたが
なかなかうまくいかないため、
マルチスレッド化を検証。

上記ソースにて画面を動的生成するのは
2つ目のタブである。
1つ目のタブに関しては
画面は固定で生成している。

そのため、2つ目の画面生成を
マルチスレッド化して
2つ目画面生成をしている間に
1つ目タブの固定画面に対して
ユーザーからの入力を受け付けようとしたが
マルチスレッド側にて画面生成をしているため
1つ目のタブのテキストボックスなどに
入力ができない（画面が触らない）
※お客様としては、2つ目のタブ画面生成時に
1つ目のタブ画面のコントロールに
入力が出来れば最悪仕様としては良い。

###補足情報(言語/FW/ツール等のバージョンなど)
より詳細な情報

言語 c# 2014
画面はlayout flow panelで動的に作成

Answer 1

現行はこんなようなコードなわけですね。

// DBからコントロールの情報を取得？
// 勝手にDataTableにしていますが適宜読み替えてください
DataTable dt = /* DBからデータを取得する処理 */

FlowLayoutPanel flowPanel = new FlowLayoutPanel();
flowPanel.Dock = DockStyle.Fill;
flowPanel.FlowDirection = FlowDirection.TopDown;
flowPanel.AutoScroll = true;

this.Controls.Add(flowPanel);   // ★

// DBから取得した内容を参照しながら複数のコントロールを生成
// 勝手にDataTableとDataRowを使っていますが適宜読み替えてください
foreach (DataRow row in dt.Rows)
{
    Button button = new Button();
    button.Text = row[0].ToString();
    // button のプロパティを row の情報を参照して適宜変更

    flowPanel.Controls.Add(button);
}

単なるアイディアで未検証ですが、★ の行を諸々のコントロール生成後に持ってきたら、速くなったりしませんか？

DataTable dt = /* DBからデータを取得する処理 */

FlowLayoutPanel flowPanel = new FlowLayoutPanel();
flowPanel.Dock = DockStyle.Fill;
flowPanel.FlowDirection = FlowDirection.TopDown;
flowPanel.AutoScroll = true;

// this.Controls.Add(flowPanel);   // ★

// DBから取得した内容を参照しながら複数のコントロールを生成
// 勝手にDataTableとDataRowを使っていますが適宜読み替えてください
foreach (DataRow row in dt.Rows)
{
    Button button = new Button();
    button.Text = row[0].ToString();
    // button のプロパティを row の情報を参照して適宜変更

    flowPanel.Controls.Add(button);
}

this.Controls.Add(flowPanel);   // ★

※ SuspendLayout が入っていても遅いようですと、↑は意味ないかもしれませんね……。

Comments

[alg]

あるいは `flowPanel.Controls.Add` ではなく `flowPanel.Controls.AddRange` を使う、とか。

[enenta]

ありがとうございます。

★試して見ます！
また、addrangeも試して見ます！

あと、少し早くなれば理想の速さまで行きそうです。

Answer 2

たぶん、追加ごとに描画の計算をしているとかそういうことがパフォーマンスの足を引っ張っているじゃないかな。

大量に追加するときは、計算を止めて、最後に計算させるというのをしないとだめ。

flowPanel.SuspendLayout();

をしています？

普通に作るときに生成される、
InitializeComponent()
を参考にするといいですよ。

あとは、一気に１００個のコントロールを見せるのがいけないのであって、
スクロールをすると読み込まれる機構をつくるとか（面倒そう）、
UIとして、カテゴリーごとにわける（簡単！）とかするといいんじゃないですかね。

Comments

[enenta]

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

flowPanel.SuspendLayout
→これはしています。

回答頂きました
描画の計算がいまいち分からないのですが、教えて頂けますでしょうか？

また、スクロールやカテゴリごとに分けるアイディアありがとうございます。
現在、タブごとにカテゴリで分けています。お客様からは、タブ1は固定画面、タブ2は動的画面のため、タブ1入力中にタブ2の動的画面（フローレイアウトパネルへのcontrol.add）が作成できないか？と言われています。
僕としては、画面生成時には、画面への入力が出来ないと思っています。

[kiichi54321]

flowPanel.SuspendLayout じゃないとなると、お手上げかな。レイアウト決定のための計算が何度も回ると、遅くなるので、それかな、と思ったのですが。まぁ、あとできることは、処理速度改善は、計測が基本なので、ちゃんと計測することですかね。VSには、色々揃っていますよ。メソッド単位の呼び出し回数の計測とか。VS2014にあるのか、しりませんが。「画面生成時には、画面への入力が出来ない」非同期構文を使えば、原理的にはできると思うのですけどね。　http://ufcpp.net/study/csharp/sp5_async.html

[enenta]

アドバイス頂いた通り
InitializeComponentを参考にして見ました。

現在、layoutflowpanelの上に
パネルやスプリットコンテナを利用していますので、それに対して、suspendlayoutを指定したところ、劇的に早くなりました。

もう少しソースを見直せば
納品レベルになりそうです。

ありがとうございます。