调用类进行数据库连接

Question

我在用C＃编程。 我正在尝试创建一个类，该类在被调用时将创建与数据库的连接。

我的数据库连接类在这里：

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Data.OleDb;

namespace HouseServer
{

    class db
    {

        // Variable to hold the driver and location of database
        public static OleDbConnection dbConnection;

        // Database connection
        public db()
        {

            // Define the Access Database driver and the filename of the database
            dbConnection = new OleDbConnection("Provider=Microsoft.Ace.OLEDB.12.0; Persist Security Info = False; Data Source=Houses.accdb");

            // Open the connection
            dbConnection.Open();
        }
    }
}

主程序在这里：

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Data.OleDb;

namespace HouseServer
{
    class Program : db
    {

        // List for holding loaded houses
        static List<house> houses = new List<house>();

        // Variable to hold "command" which is the query to be executed
        private static OleDbCommand query;

        // Variable to hold the data reader to manipulate data from the database
        static OleDbDataReader dataReader;

        static void Main(string[] args)
        {
            // Get the houses in a list
            List<house> c = getHousesFromDb();

            foreach (house yay in c)
            {
                // Show each house's full address
                Console.WriteLine(yay.house_number + " " + yay.street);
                Console.WriteLine(yay.house_town);
                Console.WriteLine(yay.postcode);
            }

            // Readline to prevent window from closing
            Console.ReadLine();
        }

        // Function which loads all of the houses from the database
        private static List<house> getHousesFromDb()
        {

            // Define the query to be executed
            query = new OleDbCommand("SELECT * FROM houses", dbConnection);

            // Execute the query on the database and get the data
            dataReader = query.ExecuteReader();

            // Loop through each of the houses
            while (dataReader.Read())
            {
                // Create a new house object for temporarily storing house
                house house = new house();

                // Create the house that we've just loaded
                house.house_id = Convert.ToInt32(dataReader["house_id"]);
                house.house_number = Convert.ToInt32(dataReader["house_number"]);
                house.street = dataReader["house_street"].ToString();
                house.house_town = dataReader["house_town"].ToString();
                house.postcode = dataReader["house_postcode"].ToString();

                // Now add the house to the list of houses
                houses.Add(house);
            }

            // Return all of the houses in the database as a List<house>
            return houses;
        }
    }
}

我以为放在class Program : db会在程序打开时调用db构造函数，但是当代码到达该行时dataReader = query.ExecuteReader(); ，出现错误“ ExecuteReader：连接属性尚未初始化”。

我想要实现的只是在另一个类中的数据库连接，我可以调用该数据库连接并将其用于所有代码。

我应该以其他方式调用数据库类吗？

Answer 1

不，什么都没有创建Program实例，什么也没有创建db实例。 但是，我强烈建议您完全更改设计：

• 没有用于数据库连接的静态字段。 需要时将其打开，使用并关闭。 您几乎不需要将其存储在除局部变量以外的任何其他内容中。
• 如果有帮助，请尽量不要使用静态变量。 它们使您的代码更难以测试，因为它们代表全局状态-比局部状态更难推理。 在您的程序中，我将完全使用局部变量。
• 不要对此类事情使用继承-您的Program类型在逻辑上不是从db派生的
• 对于方法，类和属性，请遵循.NET命名约定。 使您的代码像惯用的C＃一样“感觉”将大大提高其可读性。

Answer 2

这肯定看起来不合适，您的程序不应继承或扩展数据库类。 您的数据库类本身就是其自己的抽象数据类型。 您的程序应使用数据库类，但不能扩展它。

我会稍微改变一下

1. 摆脱继承
2. 使数据库类成为静态类（无需在此处实例化数据库实例）
3. 然后，您的程序可以使用DBClass.GetData();

那就是您的程序应该将数据库类用作黑匣子，它绝对不应从该类继承。 它应该在没有详细工作原理的情况下使用它。 在您的代码中：

// List for holding loaded houses
static List<house> houses = new List<house>();

// Variable to hold "command" which is the query to be executed
private static OleDbCommand query;

// Variable to hold the data reader to manipulate data from the database
static OleDbDataReader dataReader;

static void Main(string[] args)
{
    // Get the houses in a list
    List<house> c = getHousesFromDb();

您应该隐藏OleDbCommand和OleDbDatareader对象的详细信息，尽管并不需要在其他地方对其进行管理。 您的getHousesFromDB应该这样称呼：

MyDBClass.GetHousesFromDB()

其中MyDBClass是管理数据库读/写的静态类。 GetHousesFromDB的签名应该返回一些东西，以达到IList<House> GetHousesFromDB()的效果。

Answer 3

尽管乔恩·斯凯特（Jon Skeet）和乔恩（JonH）提出了正确的观点，但我会回答您为什么会得到例外的原因。 从那里开始，您应该听取他们的建议，并从头开始重做。

出现异常的原因是它是在db的构造函数中初始化的，并且永远不会被调用。

如果将此行添加到Main ，则程序应该可以运行。

new Program();

但要重申： 听取他们的建议，然后重新开始。 在许多情况下，这些玩具项目迅速发展为功能完善的企业应用程序，一旦您到达那里，一开始的错误就永远存在。