0

こんにちは、プログラミングは初めてで、現在、ユーザーがテキストボックスを使用して登録データをローカルデータベースに入力できるようにするプロジェクトに取り組んでいます。コードは、アイテムをデータベースに追加するように機能し、「Show_users」ボタンを押すと、それらが listBox_users リストボックスに表示されます。

私の問題は、listBox_users から名前を選択すると、選択したユーザーに関するデータが上部のテキスト ボックスに表示されることです。最初に、listBox_users 用に作成したイベントを使用してデータを入力していましたが、 「既に閉じられているデータベースからデータを読み取れません」というエラー。

namespace Userform
{
public partial class Form1: Form
{
    SqlCeDataReader rdr;

    public Form1()
    {
        InitializeComponent();
    }

    // Some code between...

    private void button_ShowUsers_Click(object sender, EventArgs e) //code that shows users in listBox
    {

        var dt = new DataTable();
        string connectionString2 = @"Data Source=MyDatabase;Password=xxxxxx;";

        using (var cn = new SqlCeConnection(connectionString2))
        using (var cmd = new SqlCeCommand("Select * From Users", cn))
        {
            cn.Open();

            using (var reader = cmd.ExecuteReader())
            {
                dt.Load(reader);
                var results = (from row in dt.AsEnumerable()
                               select new
                               {
                                   //UserID = row.Field<int>("ID"),
                                   FirstName = row.Field<string>("Firsname"),
                                   LastName = row.Field<string>("Lastname"),
                                   FullName = row.Field<string>("Firstname") + " " + row.Field<string>("Lastname")
                               }).ToList();

                listBox_users.DataSource = results;
                listBox_users.DisplayMember = "FullName";

                rdr = cmd.ExecuteReader();
                }
            }
        }

    //I made an event for the listBox_users:

    private void listBox_users_SelectedIndexChanged(object sender, EventArgs e) 
    //event code that should show listbox selected data in the textBoxes
    {            
        if (listBox_inimesed.SelectedItem != null && rdr != null)
        {
            try
            {
                if (rdr.Read())
                {

                    textBox1_firstname.Text = rdr.GetString(1);
                    textBox2_lastname.Text = rdr.GetString(2);
                    textBox3_email.Text = rdr.GetString(3);
                    textBox4_address.Text = rdr.GetString(4);
                    dateTimePicker1.Value = rdr.GetDateTime(5);
                    richTextBox_info.Text = rdr.GetString(6);
                }
                else MessageBox.Show("Object not found");
            }
            finally
            {
                rdr.Close();
            }
        }
    }
}
4

1 に答える 1

0

使用しているリーダーが依存している接続は、button_ShowUsers_Click イベントで閉じられています。

イベント間で接続と DataReaders を共有しようとするのは良い方法ではありません。その結果、正しく破棄されない接続が開かれます。各イベント メソッドで Connection、Command、および DataReader を作成することをお勧めします。「using」ステートメントを使用することで、各メソッドで正しく閉じられ、破棄されます。クラス レベル変数「rdr」を削除することもできます。

namespace Userform
{
    public partial class Form1 : Form
    {
        const string connectionString2 = @"Data Source=MyDatabase;Password=xxxxxx;";

        public Form1()
        {
            InitializeComponent();
        }

        // Some code between...

        private void button_ShowUsers_Click(object sender, EventArgs e) //code that shows users in listBox
        {
            var dt = new DataTable();

            using (var cn = new SqlCeConnection(connectionString2))
            using (var cmd = new SqlCeCommand("Select * From Users", cn))
            {
                cn.Open();

                using (var reader = cmd.ExecuteReader())
                {
                    dt.Load(reader);
                    var results = (from row in dt.AsEnumerable()
                                   select new
                                   {
                                       //UserID = row.Field<int>("ID"),
                                       FirstName = row.Field<string>("Firsname"),
                                       LastName = row.Field<string>("Lastname"),
                                       FullName = row.Field<string>("Firstname") + " " + row.Field<string>("Lastname")
                                   }).ToList();

                    listBox_users.DataSource = results;
                    listBox_users.DisplayMember = "FullName";
                }
            }
        }

    //I made an event for the listBox_users:

        private void listBox_users_SelectedIndexChanged(object sender, EventArgs e)
        //event code that should show listbox selected data in the textBoxes
        {
            if (listBox_inimesed.SelectedItem != null)
            {
                using (var cn = new SqlCeConnection(connectionString2))
                using (var cmd = new SqlCeCommand("Select * From Users", cn))
                {
                    cn.Open();

                    using (var reader = cmd.ExecuteReader())
                    {
                        if (reader.Read())
                        {
                            textBox1_firstname.Text = reader.GetString(1);
                            textBox2_lastname.Text = reader.GetString(2);
                            textBox3_email.Text = reader.GetString(3);
                            textBox4_address.Text = reader.GetString(4);
                            dateTimePicker1.Value = reader.GetDateTime(5);
                            richTextBox_info.Text = reader.GetString(6);
                        }
                        else MessageBox.Show("Object not found");
                    }
                }
            }
        }
    }
}
于 2013-04-18T15:01:43.853 に答える