画面遷移の際に生じる ちらつきをなくしたいです。

Question

解決したいこと
・下記の画像の通り、メニューバーを作成したのですが画面遷移をする際に、遷移後のFormが開くことによって生じる画面のちらつきをなくしたいです。

ちらつきをなくしたいのですが、なるべくコードは、基底クラスでメニューバー生成処理を記述して、そのメニューバーを派生クラスで利用する形にしたいです。
自分で試してみたこととして、DoubleBufferedを有効にするためにコンストラクタに、「this.DoubleBuffered = true;」や「SetStyle(ControlStyles.OptimizedDoubleBuffer, true);　SetStyle(ControlStyles.AllPaintingInWmPaint, true);」を追加してみたのですが、画面のちらつきはなくなりませんでした。何か良い方法があればご教授いただけると幸いです。

コードは下記の通りとなります。

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace WindowsFormsApp2
{
    // 基底Formクラス
    public partial class FormBase : Form
    {
        private Label menuberLbl;      // メニューバー表示に利用するラベル
        public static int LBL_CNT = 4; // メニューバーで表示するラベルの数
        private Panel menuPnl;         // メニューバー生成時にラベルを張り付けるためのパネル
                
        public FormBase()
        {
            InitializeComponent();
            // 全画面表示
            this.WindowState = FormWindowState.Maximized;

            // メニューバー作成処理
            // パネル生成処理
            menuPnl = new Panel(); 
            menuPnl.Location = new Point(90,50);
            menuPnl.Size = new Size(240, 30);
            menuPnl.BackColor = Color.FromArgb(255, 255, 255);
            this.Controls.Add(menuPnl);

            // ラベル生成処理
            List<Label> lblList = new List<Label>();
            for (int i = 0; i < LBL_CNT; i++)
            {
                menuberLbl = new Label();
                menuberLbl.Name = "lbl" + (i+1);
                menuberLbl.Font = new Font("MS UI Gothic", float.Parse("9.75"), FontStyle.Bold);
                menuberLbl.TextAlign = ContentAlignment.MiddleLeft;
                menuberLbl.BorderStyle = System.Windows.Forms.BorderStyle.Fixed3D;
                menuberLbl.AutoSize = false;
                menuberLbl.Size = new Size(60, 30);
                menuberLbl.Location = new Point(1+(60 * i), 1);
                menuberLbl.BackColor = Color.FromArgb(255, 255, 255);
                menuberLbl.ForeColor = Color.FromArgb(0, 0, 0);
                menuberLbl.Text = "メニュー" + (i + 1);
                menuberLbl.Click += new EventHandler(sideLbl_Click);
                lblList.Add(menuberLbl);
            }
            foreach (Label lbl in lblList) 
            {
                menuPnl.Controls.Add(lbl);
                menuberLbl.BringToFront();
            }

        }

        // 画面遷移
        private void sideLbl_Click(object sender, EventArgs e) 
        {
            if (((Label)sender).Name == "lbl1")
            {
                Form1 f1 = new Form1();
                f1.Show();
                this.Visible = false;
            }
            else if (((Label)sender).Name == "lbl2")
            {
                Form1 f1 = new Form1();
                f1.Show();
                this.Visible = false;
            }
            else if (((Label)sender).Name == "lbl3")
            {
                Form1 f1 = new Form1();
                f1.Show();
                this.Visible = false;
            }
            else if (((Label)sender).Name == "lbl4")
            {
                Form1 f1 = new Form1();
                f1.Show();
                this.Visible = false;
            }
        }
    }
}

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace WindowsFormsApp2
{
    public partial class Form1 : FormBase
    {
        public Form1()
        {
            InitializeComponent();
        }
    }
}

お手数をおかけしますがどうぞよろしくお願いいたします。

Comments

[KOZ6.0]

このコードだとラベルをクリックするたびに Form1 がどんどん生成されて、画面を閉じても終了できないんですが、その辺はどう考えてますか？

[aya5kishi]

返信が遅くなってしまい申し訳ございません。
重要なご指摘ありがとうございます。
ご指摘の通り、画面を閉じても終了しない点については考慮が欠けていました。
自分でも調べてみて下記のように画面遷移のコードを変更してみました。
下記のコードで考慮が足りない場合は、ご指摘いただけると幸いです。
if (((Label)sender).Name == "lbl1")
{
Form1 f1 = new Form1();
f1.Owner = this;
f1.Show();
this.Hide();
f1.Owner.Show();
f1.Close();
}

[KOZ6.0]

それだと f1 のインスタンスを立ち上げて、一瞬表示した後閉じてるので意味がありません。

[aya5kishi]

Runメソッドの引数を変更するようにコードを修正してみようと思います。

Answer 1

新しいフォームを表示後、古いフォームを即座に非表示にすると、新しいフォームの描画が完了する前に古いフォームの表示が消えるのでチラツキが発生します。

新しいフォームの表示を完了してから、少し遅延を掛けて元のフォームを閉じればマシにはなるでしょう。例えば、下記のような拡張メソッドを作成し、

public static class FormExtention
{
    private static ApplicationContext _context = new();

    public static void Transition(this Form formOld, Form formNew)
    {
        //古いフォームの位置とサイズを引き継ぐ
        formNew.Location = formOld.Location;
        formNew.StartPosition = FormStartPosition.Manual;
        formNew.Size = formOld.Size;
        formNew.WindowState = formOld.WindowState;

        // 新フォーム表示後に、時間差で古いフォームを閉じる
        EventHandler handler = null;
        handler = async (sender, e) =>
        {
            formNew.VisibleChanged -= handler;
            _context.MainForm = formNew;
            await Task.Delay(20);
            formOld.Close();
        };
        formNew.VisibleChanged += handler;

        formNew.Show();
    }

    public static void Run(this Form form)
    {
        _context.MainForm = form;
        Application.Run(_context);
    }
}

次に、Program.cs の Application.Run(new フォーム名()) を、new フォーム名().Run() に書き換えます。

internal static class Program
{
    [STAThread]
    static void Main()
    {
        ApplicationConfiguration.Initialize();
        new Form1().Run();
    }
}

あとは、Form.Show の代わりに Transition を呼び出します。
（thisを省略すると拡張メソッドと認識してくれません）

public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();
    }

    private void button1_Click(object sender, EventArgs e)
    {
        this.Transition(new Form2());
    }
}

完全な解決策という訳ではなく、最大化状態から次の画面遷移してサイズを元に戻した時に、前の画面の位置の復元をどうするか、フォームをクローズせずに非表示で維持したい場合はどうするかとか、他にも考慮すべき点はありますので、そのへんは工夫してください。

Answer 2

ウィンドウ内遷移？は、UserControlでやるのがいいんじゃないでしょうか？
【C#】ウィンドウ内での画面遷移をコントロール切り替えで実装 - いつもの作業の備忘録

【C#】画面遷移1（ユーザーコントロールを使う） - memo

Formの中にFormを入れることもできるようですが、単独でも出したいとかそういうときなんでしょうかね？？
【C#】Formの中にFormを表示して良い感じに切り替える | エンジニアが送る穴倉生活のすゝめ

UserControlにするまでもないような簡単な内容なら、TabControlで作っておいてタブを消すような手もあります。
C# - C#での画面遷移の仕方を教えてください｜teratail

UserControlを作った場合でも、TabControlは遷移処理に便利に使えます。

---

あげた例はどれもインスタンスを保持する作りですが、都度newするような作りでもいいでしょう（どちらがいいかは場合によるので何とも言えませんが^^;

Answer 3

やりたいことがいまいち不明ですが、ウインドウ(Form) を切り替えるとどうしてもちらつくので、中身を初期化する方向で考えてみては？

FormBase(Form1 のクライアント領域に張り付ける)

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

public partial class FormBase : Form
{
    private Label menuberLbl;      // メニューバー表示に利用するラベル
    public static int LBL_CNT = 4; // メニューバーで表示するラベルの数
    private Panel menuPnl;         // メニューバー生成時にラベルを張り付けるためのパネル

    public FormBase() : this(-1) { }

    public FormBase(int index) {
        InitializeComponent();

        //// 全画面表示
        //this.WindowState = FormWindowState.Maximized; ←やめる
        
        // パネル生成処理
        menuPnl = new Panel();
        menuPnl.Location = new Point(90, 50);
        menuPnl.Size = new Size(240, 30);
        menuPnl.BackColor = Color.FromArgb(255, 255, 255);
        this.Controls.Add(menuPnl);

        List<Label> lblList = new List<Label>();
        for (int i = 0; i < LBL_CNT; i++) {
            menuberLbl = new Label();
            menuberLbl.Name = "lbl" + (i + 1);
            menuberLbl.Font = new Font("MS UI Gothic", float.Parse("9.75"), FontStyle.Bold);
            menuberLbl.TextAlign = ContentAlignment.MiddleLeft;
            menuberLbl.AutoSize = false;
            menuberLbl.Size = new Size(60, 30);
            menuberLbl.Location = new Point(1 + (60 * i), 1);
            // どこをクリックしたかわかるように
            if (i == index) {
                menuberLbl.BorderStyle = BorderStyle.None;
                menuberLbl.BackColor = Color.Black;
                menuberLbl.ForeColor = Color.White;
            } else {
                menuberLbl.BorderStyle = System.Windows.Forms.BorderStyle.Fixed3D;
                menuberLbl.BackColor = Color.White;
                menuberLbl.ForeColor = Color.Black; 
            }
            menuberLbl.Text = "メニュー" + (i + 1);
            menuberLbl.Click += new EventHandler(sideLbl_Click);
            lblList.Add(menuberLbl);
        }
        foreach (Label lbl in lblList) {
            menuPnl.Controls.Add(lbl);
            menuberLbl.BringToFront();
        }

        // Control と同様に扱う
        TopLevel = false;
        FormBorderStyle = FormBorderStyle.None;
        Dock  = DockStyle.Fill;
    }

    private void sideLbl_Click(object sender, EventArgs e) {
        // 新しいインスタンスを作成
        string[] labelNames = new string[] { "lbl1", "lbl2", "lbl3", "lbl4" };
        int index = Array.IndexOf(labelNames, ((Label)sender).Name);
        Form f = new FormBase(index) { Visible = true };
        Parent.Controls.Add(f);
        f.BringToFront();

        // （前の状態が不要なら）自分を破棄する
        Parent.Controls.Remove(this);
        Dispose();
    }
}

using System.Windows.Forms;

public partial class Form1 : Form
{
    public Form1() {
        InitializeComponent();
        Controls.Add(new FormBase() { Visible = true});
    }
}

Comments

[aya5kishi]

ご回答ありがとうございます。
画面のちらつきがなくなりとても助かりました。中身を初期化する方向でコードを書いていこうと思います。
自分の技術レベルが低いこともあり、まだコード内容を理解できていませんが、明日ご教授いただいたコードを参考にして調べていこうと思います。

[aya5kishi]

1点ご教授いただきたいのですが、全画面表示は何故削除したほうが良いのでしょうか？
Formアプリケーション開発では、全画面表示をせずに、画面を固定して開発することが多いのでしょうか？
私は実務経験がないため、開発現場ではどうなのかとても気になりました。(開発するアプリや現場にもよるとは思いますが、一般的にはどうなのでしょうか？)

[KOZ6.0]

FormBase は Form1 のコントロールになるので最大化は意味ありません。
やるなら Form1 を最大化させることになりますね。
OnParnetChanged で Parent != null なら最大化させることになるでしょう

Answer 4

画面がちらつく理由としてはメニューバーが毎回newされているからだと思います。
メニューバーに使うコントロールはすべてstaticで宣言してFormBaseが作られる最初の1回だけnewするようにすると改善するかもしれません。
そのためには、コンストラクタ中のメニューバー作成処理をどうにかしないといけないのですが、簡単な方法としてはbool型のstatic変数を用意して初めてメニューバーが作成されたら変数をtrueにて2回目以降のnewをしないようにすればいいと思います。

public partial class FormBase : Form
    {
        private static Label menuberLbl;      // メニューバー表示に利用するラベル
        public static int LBL_CNT = 4; // メニューバーで表示するラベルの数
        private static Panel menuPnl;         // メニューバー生成時にラベルを張り付けるためのパネル
        public static bool menu_initialized = false; //メニューバーが初期化されたかどうか

        public FormBase()
        {
          if(!menu_initialized){
            // メニューバー作成処理
            
            menu_initialized = true; // メニューバー初期化済み
          }
        }

Comments

[aya5kishi]

返信が遅くなってしまい申し訳ございません。
ご回答頂きましてありがとうございます。
今からvann_2921さんのご回答をもとに自分でコードを記述してみて、わからない点があるかを確認いたしますので申し訳ないのですが、ベストアンサーボタンは明日には押せるようにいたします。