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

■No34500 (tamaboyo さん) に返信
>>(ちなみにAutoSizeプロパティをfalseに設定するコードもないようなので、
>>このままだとSizeプロパティの設定はあまり意味がありません。)

AutoSize プロパティの規定値は false のようなので、設定せずとも大丈夫かも。
(デザイナから貼りつけた場合は、最初に true に編集されるというだけで)


> public void Contorol_Load(int nMax) 
基本的には、それで問題ないと思います。下記に示すような問題はありましたが、
当方環境でパネルに登録され、Click イベントにも反応することを確認できましたし。

------------

>     int i = 0;
ループカウンタ変数のスコープを出来る限り短くするためにも、
　int i;
　for (i = 0; i < length; i++) { ･･･ }
の構文では無く、
　for (int i = 0; i < length; i++) { ･･･ }
の構文を利用されることをお奨めします。


>     this.SuspendLayout();
対応する ResumeLayout が書かれていないようです。


>     Lb = new System.Windows.Forms.Label[64];
>     for (i = 0; i < Bit.Length; i++)
>     {
>         //インスタンス作成
>         Lb[i] = new System.Windows.Forms.Label();

デバッグのため、配列変数 Lb の要素数を 64固定にしているのは分かりますが、
その後のループ条件が、「i < 64」や「i < Lb.Length」ではなく、
「i < Bit.Length」にしているのは何故でしょうか?
最初のコードでは、Lb.Length を使っていましたよね。

もし、Bit.Length が 0 だとしたら、Lb[] の各要素はすべて null であるため、
ラベルが追加される事は無いかと思います。


>         //フォームにコントロールを追加
>         Panel1.Controls.AddRange(Lb);
C# の開発環境では、パネル名の規定値は「Panel1」ではなく「panel1」かと。

提示のコードをもとに私の方で試してみましたが、ラベルのクリックイベントは普通に起きました。

私の方で手を加えたのは以下のあたりだけです。
○Form1の作成
・新規にWindows Formsのプロジェクトを作成した。
・Form1にはパネルを追加した。(デフォルト名はpanel1なのでPanel1に修正した)
・Lbの定義が提示のコードにはないので、適当に追加した。
・Bit.Lengthは定義が存在しないため、とりあえず64固定にした。
○呼び出し側
 Form1 f = new Form1();
 f.Contorol_Load(1);  // 引数の1は適当
 f.Show();

-- 以下はその実験コードのうち、Form1の部分のみ--

public partial class Form1 : Form
{
    private Label[] Lb;
    public Form1()
    {
        InitializeComponent();
    }

    public void Contorol_Load(int nMax)
    {
        int i = 0;

        this.SuspendLayout();

        Lb = new System.Windows.Forms.Label[64];
        for (i = 0; i < 64; i++)
        {
            //インスタンス作成
            Lb[i] = new System.Windows.Forms.Label();
            //プロパティ設定
            Lb[i].Name = "Bit" + i.ToString();
            Lb[i].Size = new Size(29, 36);
            Lb[i].Location = new Point(i * 30, 10);
            Lb[i].Visible = true;
            Lb[i].BackColor = Color.Gray;
            //イベントハンドラに関連付け
            Lb[i].Click += new EventHandler(this.LB_Click);
        }

        //フォームにコントロールを追加
        Panel1.Controls.AddRange(Lb);
    }

    private void LB_Click(object sender, EventArgs e)
    {
        System.Diagnostics.Debug.WriteLine("LB_Click");
    }
}

■No34508 (魔界の仮面弁士 さん) に返信
> ■No34500 (tamaboyo さん) に返信
> >>(ちなみにAutoSizeプロパティをfalseに設定するコードもないようなので、
> >>このままだとSizeプロパティの設定はあまり意味がありません。)
> 
> AutoSize プロパティの規定値は false のようなので、設定せずとも大丈夫かも。
> (デザイナから貼りつけた場合は、最初に true に編集されるというだけで)

あれ？そうでしたっけ？

Label.AutoSizeプロパティ
http://msdn.microsoft.com/ja-jp/library/system.windows.forms.label.autosize.aspx

には既定値 trueと書かれているので、trueだとばかり信じてました。
以下で試したら確かにfalseが返ってきましたorz

class Test
{
	static void Main()
	{
		System.Console.WriteLine(new System.Windows.Forms.Label().AutoSize);
	}
}

> >>        protected override void OnLbClicked(object sender, EventArgs e)
>>必ずしもオーバーライドさせる必要は無いかと思いますが、オーバーライドさせるならば、
>>継承元のベースメソッドを呼びだすコードも含めておくべきかと。
> 
> 申し訳ないのですが、ここの意味が理解できませんでした。
> 継承先の宣言でnew eventで新しく割り付け直したつもりだったのですが、違うのでしょうか？

protected override void OnLbClicked(object sender, EventArgs e) {
    base.OnLbClicked(sender,e); // ←これ入れとけ、ってこっちゃないすか。
    // なんかする。 場合によっては base.... の前に。
}