SQL Severから取得した値を WindowsFormsAppのｃlass Form1のlistboxに表示するようなシステムについて

Question

C#のlistについての質問です。
質問を投稿するのも初めてなので、至らない点が多々あると思いますがよろしくお願いします。

Windows 10 の Visual Studio 2022 で Windows Forms アプリをターゲットフレームワーク .NET Framework 4.8 で作っています
SQL Server のバージョン・エディションは、2022でlocalDBで作っております

SQL Severから取得した値をフィールドのListに格納し、
WindowsFormApplicationのｃlass Form1のlistboxに表示するようなシステム
を考えております。

質問事項：
SQL Severから取得した値をフィールドのListに格納できても、
(☆class1.Select();での処理ではうまく格納できている)
★への処理に移ると、データがリセットしてしまいます。
(ID.Countが0になるため、データを表示できない)

どうしたら☆class1.Select();での処理したフィールド変数の値を保持したまま
★の処理に移行できますか。

以下コード：

ｃlass Form1
　　　//フィールド変数の定義
　　 public string abc = "{0,-10}{1,-20}";
       private List<string> id { get; set; } = new List<string>();
       public List<string> ID { get => id; set => id=value; }
       private List<string> country { get; set; } = new List<string>(); 
       public List<string> Country { get => country; set => country=value;} 
    
　　　
         private void Form1_Load(object sender, EventArgs e)
       {
           listBox1.Items.Add(string.Format(abc, "id", "country"));
          
　　　　☆// SQL Serverからデータの抽出処理
           Class1 class1 = new Class1();
           class1.Select();
　　　　
　　　　★//Form１のlistboxに追加
           for (int i = 0; i < ID.Count; i++)
           {
               listBox1.Items.Add(string.Format(abc, id[i], country[i]));
           }
　　　　
       }

class Class1
internal class Class1:Form1
   {
       public void  Select()
       {
             //SQL Serverに接続処理
　　　　　 
             //抽出処理
              cmd.CommandText = "SELECT * FROM Countrys";

               SqlDataReader dr = cmd.ExecuteReader();
　　　　　
　　　　　 while (dr.Read())
               {
                   string aa = dr["country_id"].ToString();
                   string bb = dr["country"].ToString();
　　　　　　//フィールド変数に格納
                   this.ID.Add(aa);
                   this.Country.Add(bb);
               }

　　　　　SQL Serverに接続切断処理
　　　}

Comments

[matukeso]

class1のID、Countryは定義が書いてないけど、どこから生えてきたの？Form1のインスタンスを指す処理がどこかについてるの？
それとも、単に同名な別インスタンス？

[退会済みユーザー]

まず、何はともあれコードは上下を ``` と ``` で囲ってください（``` はバッククォート 3 つ）。インデントされて読みやすくなるので。

何を何で作っているか書いてください。（例: Windows 10 の Visual Studio 2022 で Windows Forms アプリをターゲットフレームワーク .NET Framework 4.8 で作っています・・・とか）

SQL Server のバージョン・エディションを書いてください。LocalDB でないとすると、アプリからの接続のための設定、ログインのための設定などいろいろ事前準備が必要ですが、ちゃんとできてますか？

[Erizabesu_Queen]

matukesoさん
見てくださり、ありがとうございます。
Form1のインスタンスを指しています。

internal class Class1:Form1
{

を追記いたしました

[Erizabesu_Queen]

SurferOnWwwさん
見てくださりありがとうございます。
初めての質問で至らない点多々あり、申し訳ございません。

ご指摘いただいた点は追記いたしました。
よろしくお願いいたします。

Answer 1

Class1がForm1を継承しているなら、Form1_Loadの実行主体(this)の持つIDと、new　Class1してできるclass1の持つIDは、別のインスタンスですよ。

こうすれば、違いがわかるとは思います。thisのIDはカラで、class1のIDは埋まっているわけです

  Class1 class1 = new Class1();
  class1.Select();
  for (int i = 0; i < class1.ID.Count; i++)
  {
     listBox1.Items.Add(string.Format(abc, class1.ID[i], class1.Country[i]));
  }

根本的には、Class1でForm1の継承をやめてID,Countryをコンストラクタでもらうようにするとか、
class1が持つとして、Select後にID,Countryをコピーするとかのほうがいいとは思いますけど。

internal class Class1
{
   private readonly List<string> ID;
   private readonly List<string> Country;
   internal Class1(List<string> id, List<string> country){
      ID=id; Country=country;
   }
…
}

Comments

[Erizabesu_Queen]

解決しました。ありがとうございます！

[Erizabesu_Queen]

実装してみて、質問事項出来たので、質問させていただきます。

```
Class class1
internal class Class1
{
private readonly List<string> ID;
private readonly List<string> Country;
internal Class1(List<string> id, List<string> country){
ID=id; Country=country;
}
…
}
```
と実装した際に、なぜForm１のid、countryのlistに追加できるのでしょうか。
Class1内の「readonly」 を使うと読み取り専用になるし、そもそもForm１のlist
に追加する記述がないように感じます。
初心者的な質問であり申し訳ございません。よろしくお願いいたします。

[matukeso]

コンストラクタでForm1のidへの参照をわたす前提なのでform1.idとclass1.IDは同じList Objectを指すようになります。
Class1のIDにつくReadonlyはIDの指す先を変えられない、という意味です。

[Erizabesu_Queen]

同じList Objectを指すというところってつまり下記のコードがあるからってことの認識でよろしいでしょうか。

internal Class1(List<string> id, List<string> country){
ID=id; Country=country;
}

[matukeso]

そうですね

[Erizabesu_Queen]

なるほど、、、
わかりやすく丁寧な回答していただきありがとうございました。

Answer 2

SQL Server のテーブルから DataTable を作成して、それを ListBox.DataSource プロパティに代入するようにしてはいかがですか？

例えば以下のような SQL Server のテーブルから、

以下のようなコードで SqlDataAdapter を利用して DataTable を作成し、それを ListBox.DataSource プロパティに代入すると、

using System;
using System.Data;
using System.Data.SqlClient;
using System.Windows.Forms;

namespace WindowsFormsApp1
{
    public partial class Form12 : Form
    {
        DataTable table;

        public Form12()
        {
            InitializeComponent();
        }

        private void Form12_Load(object sender, EventArgs e)
        {
            var connString = "Data Source=lpc:(local)\\sqlexpress;Initial Catalog=NORTHWIND;Integrated Security=True";
            var selectQuery = "SELECT CategoryID, CategoryName FROM Categories";

            this.table = new DataTable();
            using (var connection = new SqlConnection(connString))
            {
                using (var command = new SqlCommand(selectQuery, connection))
                {
                    var adapter = new SqlDataAdapter(command);
                    adapter.Fill(table);
                }
            }

            this.listBox1.DataSource = table;
            this.listBox1.DisplayMember = "CategoryName";
            this.listBox1.ValueMember = "CategoryID";
        }
    }
}

結果は以下のようになります。

Comments

[Erizabesu_Queen]

こんな技術があるなんて知りませんでした。
教えてくださり、ありがとうございます！