追加するパネル名を引数で設定する

Question

コンボボックスをパネルに追加する際、定数にパネル名を入れ、それを引数として受け取り、追加するパネルを判別したいのですがうまくいきません。(上手に説明できなくてすみません)
### 発生している問題・エラーメッセージ

エラーメッセージ
'string' に 'Constrols' の定義が含まれておらず、型 'string' の最初の引数を受け付けるアクセス可能な拡張メソッド 'Constrols' が見つかりませんでした。using ディレクティブまたはアセンブリ参照が不足しています。

該当のソースコード

pname.Controls.Add(comboBox);
``````ここに言語を入力
```ここに言語を入力

using System;
using System.Windows.Forms;
using System.Drawing;
class FormMain : Form
{
const string p1 = "panel1";
const string p2 = "panel2";
Panel panel1;
[STAThread]
public static void Main()
{
Application.EnableVisualStyles();
Application.SetCompatibleTextRenderingDefault(false);
Application.Run(new FormMain());
}
FormMain()
{
Text = "Button Click Sample";
ClientSize = new Size(600, 400);
Button btn1 = new Button();
btn1.Location = new Point(50, 50);
btn1.Text = "Create Panel";
btn1.Click += btn1_Click;
Controls.AddRange(new Control[] { btn1 });
this.Controls.Add(panel1);

}
public void CreateMyPanel( string pname )
{
panel1 = new Panel();
Panel panel2 = new Panel();
// Initialize the Panel control.
panel1.Location = new Point(56, 72);
panel1.Size = new Size(264, 152);
// Initialize the Panel control.
panel2.Location = new Point(356, 72);
panel2.Size = new Size(264, 152);
// Set the Borderstyle for the Panel to three-dimensional.
panel1.BorderStyle = System.Windows.Forms.BorderStyle.Fixed3D;
panel2.BorderStyle = System.Windows.Forms.BorderStyle.Fixed3D;
// Add the Panel control to the form.
this.Controls.Add(panel1);
this.Controls.Add(panel2);
ComboBox comboBox = new ComboBox();
comboBox.Items.Add("A");
comboBox.Items.Add("B");
comboBox.Items.Add("C");
comboBox.Items.Add("D");
comboBox.Items.Add("E");
pname.Controls.Add(comboBox);
}
void btn1_Click(object sender, System.EventArgs e)
{
CreateMyPanel(p1);
CreateMyPanel(p2);

}
}

### 試したこと
ここに問題に対して試したことを記載してください。
### 補足情報（FW/ツールのバージョンなど）
訳あってif文は使えないため、↓の方法以外でお願いいたします・
```ここに言語を入力
if (pname = p1 )
{panel1.Controls.Add(comboBox)}
else
{panel2.Controls.Add(comboBox)}
```
```

Comments

[退会済みユーザー]

ソースはコードの挿入（<code>アイコン）で```と```の間に記入してください。そのまま書くとインデントされないので読み辛いです。

[退会済みユーザー]

なんか```が多すぎて正しく表示出来てない気がします。プレビューで表示を確認してください。

[YeP95]

遅くなってしまい申し訳ございません。おそらく変更できていると思います。

[YAmaGNZ]

提示されたソースだと"panel1"といった名前のパネルはありません。
panel1という変数名に入ったパネルを操作したいのか、Nameプロパティにpanel1とセットされているパネルを操作したいのかが判別できません。

[YeP95]

後者です。

[Zuishin]

pname は string であって Panel ではありません。
したがって Controls プロパティもありません。
それがエラーの原因です。

それと、if 文を使えばいいじゃないですか。
たぶん、使えない訳の方がどうにかなると思います。

[YeP95]

CreateMyPanel(p1);として呼び出すことでp name に p1(=panel1)が格納され、pname.Controls.Add(comboBox)がpanel1.Controls.Add(comboBox)
となるような処理を考えていたのですがやはり不可能でしょうか。

[Zuishin]

不可能ではないけど、おそらくやる意味がない処理のように思います。
ユーザーコントロールを作れば済むのではないでしょうか。

[退会済みユーザー]

あとボタン連打するとPanelとComboBoxがどんどん増えていくでしょうね。
それが意図した動作なら別にいいんですが。

[Zuishin]

ああ、言葉通じない系か。

[TN8001]

[[Q&A] 追加するパネル名を引数で設定する - Qiita](https://qiita.com/YeP95/questions/5f81667756601d675ea8

> teratailでは、マルチポスト※の推奨はしていません。
> やむを得ず複数のサイトに質問を投稿された場合は、質問内容にマルチポストをする理由を書き、他のサイトの投稿へのリンクを貼ってください。また、解決した際には必ずteratail及びすべての投稿に解決した旨と、どのように解決したかを記載してください。

[ヘルプ｜teratail（テラテイル）](https://teratail.com/help#posted-otherservice

[退会済みユーザー]

マルチポストで質問消して逃亡か…

Answer 1

コンボボックスをパネルに追加する際、定数にパネル名を入れ、それを引数として受け取り、追加するパネルを判別したいのですがうまくいきません。

やりたいこととコードがあっていないように感じます。

コンボボックスをpanel1・panel2どちらに入れるか選びたいということだと思いますが、panel1・panel2は先にできていないとおかしくないですか？

radianさんのようにFindでもいいし、Formの子だとわかっているならインデクサも使えます。
Control.ControlCollection.Find(String, Boolean) メソッド (System.Windows.Forms) | Microsoft Docs

Control.ControlCollection.Item[] プロパティ (System.Windows.Forms) | Microsoft Docs

using System;
using System.Drawing;
using System.Windows.Forms;

namespace Questions360326
{
    class FormMain : Form
    {
        private const string p1 = "panel1";
        private const string p2 = "panel2";

        [STAThread]
        public static void Main()
        {
            Application.EnableVisualStyles();
            Application.SetCompatibleTextRenderingDefault(false);
            Application.Run(new FormMain());
        }

        FormMain()
        {
            Text = "Button Click Sample";
            ClientSize = new Size(600, 400);

            var btn1 = new Button { Location = new Point(50, 50), Text = "Add ComboBox", };
            btn1.Click += btn1_Click;

            var btn2 = new Button { Location = new Point(350, 50), Text = "Add ComboBox", };
            btn2.Click += btn2_Click;

            var panel1 = new Panel
            {
                Name = p1,
                Location = new Point(56, 72),
                Size = new Size(264, 152),
                BorderStyle = BorderStyle.Fixed3D,
            };

            var panel2 = new Panel
            {
                Name = p2,
                Location = new Point(356, 72),
                Size = new Size(264, 152),
                BorderStyle = BorderStyle.Fixed3D,
            };

            Controls.AddRange(new Control[] { btn1, btn2, panel1, panel2, });
        }

        private void btn1_Click(object sender, EventArgs e)
        {
            AddComboBox(p1);
            ((Button)sender).Visible = false;
        }
        private void btn2_Click(object sender, EventArgs e)
        {
            AddComboBox(p2);
            ((Button)sender).Visible = false;
        }

        private void AddComboBox(string pname)
        {
            var comboBox = new ComboBox();
            comboBox.Items.AddRange(new string[] { "A", "B", "C", "D", "E", });

            var panel = Controls[pname] as Panel; // インデクサ
            panel?.Controls.Add(comboBox);
        }
    }
}

Answer 2

例えば、Nameが"Panel1"というPanelがフォームにあって、そこにComboBoxを追加するなら、

    ComboBox comboBox = new ComboBox();
    comboBox.Items.Add("A");
    comboBox.Items.Add("B");
    comboBox.Items.Add("C");
    comboBox.Items.Add("D");
    comboBox.Items.Add("E");

    this.Controls.Find("Panel1", true).FirstOrDefault()?.Controls.Add(comboBox);

こんな感じで出来るのではないかと。

Comments

[YeP95]

void btn1_Click　を編集しました。
コードのように引数を設定して追加パネルを設定したいと考えております。

[退会済みユーザー]

私の書いたコードの意味は理解出来ます？
Formから文字列でコントロール（この場合はPanel1）探して、その子にComboBoxを追加してるだけです。引数の文字列あるんだから、あとはどうすればいいか判りますよね。

[YeP95]

this.Controls.Find("pname", true).FirstOrDefault()?.Controls.Add(comboBox);でしょうか？
visual studioでためしているのですがうまくいきません...

[退会済みユーザー]

それはpnameという変数に入っている名前ではなく、"pname"という名前のコントロールを探してます。あと、質問文のソースだとPanelのNameプロパティ設定してないから、Findで探すならNameプロパティを先に設定しないと見つからないと思います。1から10まで説明するのはさすがにお断りしたいので、最低限の文法を入門書やMicrosoftのC#ドキュメント等で勉強してから始めた方が良いと思います。