C# - 将“硬”编码样式更改为可重用代码 [关闭]

Question

我的 C# 项目中有这个功能

public DataTable access2dt()
    {

        string myConnectionString = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=test.accdb";
        using (var con = new OleDbConnection(myConnectionString))
        {
            con.Open();

            using (var cmd = new OleDbCommand("EXEC OUTBOUND_FILTER",con))
            {
                cmd.Parameters.AddWithValue("prmORIGINCODE", "BDO");
                cmd.Parameters.AddWithValue("prmORIGIN", "\"*\"");
                cmd.Parameters.AddWithValue("prmSERVICECODE", "REG15");
                cmd.Parameters.AddWithValue("prmDESTCODE", "AMI");
                cmd.Parameters.AddWithValue("prmDESTINATION", "\"*\"");
                using (OleDbDataReader rdr = cmd.ExecuteReader())

                {
                    DataTable myTable = new DataTable();
                    myTable.Load(rdr);
                    return myTable;
                }


            }

        }
    }

上面的函数是用于在 Ms Access 中使用一些参数执行查询对象，该代码只适用于我的项目，但我对“代码样式”有问题。

我想更改代码，因此当我有另一个带有另一个不同参数的查询时，我不需要重写该代码（请参阅参数、查询名称和 db 文件名是“硬编码”以发挥作用）。

任何建议和 C# 代码都会有所帮助。

Answer 1

你是对的，你会想要重构它以使其更通用。不经常更改的一件事是该连接字符串。也许您可以将其抽象为 DataStore 类。然后你可以有一个ExecuteCommand 方法来接受命令名和参数：

public class DataStore
{
    private string _connectionStr;

    public DataStore(string connectionStr)
    {
        this._connectionStr = connectionStr;
    }

    public DataTable ExecuteCommand(string commandText, IDictionary<string,string> parameters)
    {
        using (var connection = new OleDbConnection(this._connectionStr))
        {
            connection.Open();
            using (var cmd = new OleDbCommand(commandText, con))
            {
                foreach (var pair in parameters)
                {
                    cmd.Parameters.AddWithValue(pair.Key, pair.Value);
                }

                using (var reader = cmd.ExecuteReader())
                {
                    var table = new DataTable();
                    table.Load(reader);
                    return table;
                }
            }
        }
    }
}

...但这仍然有一些限制，例如每个参数值都必须是一个字符串。您可以将 parameters 更改为 IDictionary<string, object>，但您会失去良好的类型安全性。

退后一步，您可能会想象 很多 开发人员在将数据库访问封装在抽象层中时遇到了完全相同的困难，并且可能已经解决了这个问题。事实上，像 Dapper 和 Entity Framework 这样的东西可以为你做这种事情。

请参阅How to use Entity framework for MS Access database，了解有关如何将 Entity Framework 与 MS Access 结合使用的一些提示。或者您可以寻找任何其他类型的 ORM（对象/关系映射器）

Answer 2

public DataTable access2dt(string filename, string namaQuery, IDictionary<string,string> keyValue)
{
    string myConnectionString = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=test.accdb";
    using (var con = new OleDbConnection(myConnectionString))
    {
        con.Open();

        using (var cmd = new OleDbCommand("EXEC OUTBOUND_FILTER",con))
        {
            foreach (var d in keyValue)
            {
                cmd.Parameters.AddWithValue(d.Key, d.Value);
            }

            using (OleDbDataReader rdr = cmd.ExecuteReader())
            {
                DataTable myTable = new DataTable();
                myTable.Load(rdr);
                return myTable;
            }
        }
    }
}