独自のクラス型のListをDateGridViewに表示する方法

Question

C#、VisualStudio初心者です。

データベースから特定の列のみ抜き取ったものをリストに格納し、DateGridViewに表示したいのですが、反映されません。

---Listの型になる独自のクラス Model---
private string date;
private string time;
//以下、コンストラクタ生成、またアクセッサメソッド（割愛）

---DBからリストに格納 Control---
public List<Model> Show(string day)
{
 List<Model> list = new List<Model>();
 command = connection.CreateCommand();
 SqlTransaction transaction = connection.BeginTransaction();
 command.Connection = connection;
 command.Transaction = transaction;
 command.CommandText = @"SELECT date,time from testtable Where date Like '" + day + "%' order by date asc;";
 var reader = command.ExecuteReader();
 while (reader.Read())
 {
  date = (string)reader.GetValue(1);
  time = (string)reader.GetValue(2);
 }
 reader.Close();
 transaction.Commit();
 return list;
}

---Modelからリストを受け取りDateGridViewに表示をする View---
private void Search_Click(object sender, EventArgs e)
{
 try
 {
  string year = comboBox1.SelectedItem.ToString();
  string month = comboBox2.SelectedItem.ToString();
  string day = year +"/"+ month;
  list =control.Show(day);
  for (int i = 0; i < list.Count(); i++)
  {
   list.ForEach(o=>dataGridView1.Rows.Add(o.Date,o.Starttime,o.Finishtime,o.Sumtime));
  }
 }
 catch (NullReferenceException ex)
 {
  Console.WriteLine(ex.Message);
  MessageBox.Show("未入力項目があります。", "エラー", MessageBoxButtons.OK, MessageBoxIcon.Error);
 }
}

//インスタンス作成、初期化等は省略しています。

---

SQL文、Listの格納の仕方、DateGridViewに表示するくだり、
等間違えていそうなところはところどころあるのですが全くの無知なのでわかりません。
ご教授お願いいたします。

Answer 1

簡単な書き方としては、下記の様にDataGridViewのDataSourceにバインドするのも一つのやり方かと思います。
検討してみてはいかがでしょうか。

Form側

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

        private void Form1_Load(object sender, EventArgs e)
        {
            getSpecifiedData gsf = new getSpecifiedData();
            BindingList<myModel> data = new BindingList<myModel>( gsf.show());
            //データソースを設定する
            dgv1.DataSource = data;
        }
    }

モデル側の処理例

    //特定のデータを取得する処理
    class getSpecifiedData
    {
        public List<myModel> show()
        {
            List<myModel> returnList = new List<myModel>();
            //ここで、Selectだと思うのですが、簡単に固定値を記述してます。
            returnList.Add(new myModel("20170714", "15:00:00"));
            returnList.Add(new myModel("20170715", "16:00:00"));
            returnList.Add(new myModel("20170716", "17:00:00"));
            return returnList;
        }
    }

独自に作成しているクラス

    public class myModel
    {
        private string _date;
        private string _time;
        public myModel(string id, string name)
        {
            this.date = id;
            this.time = name;
        }

        public string date
        {
            get { return this._date; }
            set { this._date = value; }
        }
        public string time
        {
            get { return this._time; }
            set { this._time = value; }
        }
    }

Comments

[ygoooo]

BindingListを使ったデータバインドで無事表示できました！
ありがとうございます、助かりました。

Answer 2

気になったところ。二重ループになっているので一つにしておきましょう。
もっと言えば一行ずつ入れるよりも、Rows.AddRange()を使ったほうが速いです。

foreach (var o in list)
{
    dataGridView1.Rows.Add(o.Date, o.Starttime, o.Finishtime, o.Sumtime);
}

Answer 3

• readerのループで希望の値が取れているか
• Modelクラス(返すlistの1行のデータ)に正しく格納されているか
• list =control.Show(day);の中身が正しく格納されているか

ひとつずつ確認していけば、解決すると思いますよ。