C# と VB.NET の質問掲示板

VS2008・C#で、Windowsアプリケーションを作成しています。
コントロールのフォントスタイル変更について質問です。

ラジオボタンが２つ(radioButtonA,radioButtonB)フォームにあって、
チェックが入っているほうのラジオボタンのテキストを「太字かつ下線」にして、
チェックなしは「標準テキスト」にしたいです。
CheckedChangedイベントに、以下のように記述しています。

/// <summary>
/// ラジオボタンA,BのCheckedが変更された場合に発生するイベント
/// </summary>
private void radioButtonAB_CheckedChanged(object sender, EventArgs e)
{
     if (sender.Equals(radioButtonA))
     {
         // 色を変更して、下線
         radioButtonA.ForeColor = Color.Red;
         radioButtonB.ForeColor = SystemColors.ControlText;

         radioButtonA.Font = new Font(radioButtonA.Font, FontStyle.Bold | FontStyle.Underline);
         radioButtonB.Font = new Font(radioButtonB.Font, FontStyle.Regular);
      }
      else if (sender.Equals(radioButtonB))
      {
          radioButtonA.ForeColor = SystemColors.ControlText;
          radioButtonB.ForeColor = Color.Red;

          radioButtonA.Font = new Font(radioButtonA.Font, FontStyle.Regular);
          radioButtonB.Font = new Font(radioButtonB.Font, FontStyle.Bold | FontStyle.Underline);
       }
}

radioButtonB.Font = new Font(radioButtonB.Font, FontStyle.Regular)のように、
フォントスタイルの変更のためにはFontをnewする必要があると思いますが、
「Fontはnewしたら解放するものだ」と覚えているので、このままで良いのかなと、
ちょっと違和感が残ります...。

解放に関して知識が足りないので、これで問題ないのか分かりません...。
宜しくお願い致します。

なるほど。
解放しなくても特に問題はないけれど、
・Fontをnewする前にDisposeすればOK.
だし、
・「チェック時フォント」と「通常フォント」を用意しておいて、切り替える
のもすっきりしますね！
よねＫＥＮさんのコードから、以下のようにしました。
// --------------------------------------------------------------------------------
private Font selectFont;   // チェック時フォント
private Font regularFont;  // 通常フォント

private void Form1_Load(object sender, EventArgs e)
{
    selectFont = new Font("MS UI Gothic", 9.0f, FontStyle.Bold | FontStyle.Underline);
    regularFont = new Font("MS UI Gothic", 9.0f, FontStyle.Regular);
}

private void radioButtonAB_CheckedChanged(object sender, EventArgs e)
{
    if (sender.Equals(radioButtonA))
    {
        radioButtonA.Font = selectFont;
        radioButtonB.Font = regularFont;
    }
    else if (sender.Equals(radioButtonB))
    {
        radioButtonA.Font = regularFont;
        radioButtonB.Font = selectFont;
    }
}
// --------------------------------------------------------------------------------

とても助かりました！
皆様、ありがとうございました！

■No45936 (かおりQ さん) に返信
>private void radioButtonAB_CheckedChanged(object sender, EventArgs e)
>{
>    if (sender.Equals(radioButtonA))
>    {
>        radioButtonA.Font = selectFont;
>        radioButtonB.Font = regularFont;
>    }
>    else if (sender.Equals(radioButtonB))
>    {
>        radioButtonA.Font = regularFont;
>        radioButtonB.Font = selectFont;
>    }
>}

フォントの開放とは別の所で気になったのだが
同じグループボックス内にradioButtonA と radioButtonB
があった場合、両方からイベント通知されてきますよね？

radioButtonBが選択されている状態で、radioButtonAを選択した場合、
radioButtonB Checked = false になる
radioButtonA Checked = true  になる
イベント通知される順番って保障されてるのかしら。。。

選択されているラジオボタンのフォントを変えるだけなら以下のほうがよいかも

private void radioButtonAB_CheckedChanged(object sender, EventArgs e)
{
    if (((RadioButton)sender).Checked)
    {
        ((RadioButton)sender).Font = selectFont;
    }
    else
    {
        ((RadioButton)sender).Font = regularFont;
    }
}

--