在非控制情况下使用 SqlDataSource

Question

作为我在所有业务应用程序中使用的常用实用程序的一部分，我有这段代码...

using System.Web.UI.WebControls;

public class Database
    {
    /// <summary>
    /// Creates a DataView object using the provided query and an SqlDataSource object.
    /// </summary>
    /// <param name="query">The select command to perform.</param>
    /// <returns>A DataView with data results from executing the query.</returns>
    public static DataView GetDataView(string query)
        {
        SqlDataSource ds = GetDBConnection();
        ds.SelectCommand = query;
        DataView dv = (DataView)ds.Select(DataSourceSelectArguments.Empty);
        return dv;
        }

     /// <summary>
     /// Creates a SqlDataSource object with initialized connection string and provider
     /// </summary>
    /// <returns>An SqlDataSource that has been initialized.</returns>
    public static SqlDataSource GetDBConnection()
        {
        SqlDataSource db = new SqlDataSource();
        db.ConnectionString = GetDefaultConnectionString(); //retrieves connection string from .config file
        db.ProviderName = GetDefaultProviderName(); //retrieves provider name from .config file
        return db;
        }
   }

然后，在我的项目中，为了从数据库中检索数据，我将编写一些代码，例如..

DataView dv=Database.GetDataView("select mycolumn from my table");
//loop through data and make use of it

我因以这种方式使用 SqlDataSource 而受到了人们的关注。人们似乎不喜欢我纯粹从代码中使用 Web 控件，而不是将其放在 ASPX 页面上。这对他们来说看起来不对，但他们无法告诉我不利的一面。那么，有缺点吗？这是我的主要问题。因为如果有很多缺点，我可能不得不改变我开发许多内部应用程序的方式。

只要我添加 System.Web 程序集，我的 Database 类甚至可以在非 ASP.NET 情况下工作。我知道包大小略有增加，但我觉得对于我正在编写的应用程序类型来说这是值得的。从 WPF/Windows 窗体/控制台程序中使用 SqlDataSource 是否有缺点？

Answer 1

嗯，没有硬性规则阻止任何人进行此类实施。

但是，在执行该实施之前，需要回答以下几个问题。

1. 这种使用线程安全吗？ （因为同一个调用很可能由多个消费应用程序进行。
2. 是否会有分层区分（UI.Control 用于数据层）？
3. 如果该控件在下一个框架版本中变得过时/受到限制怎么办？

Answer 2

鉴于替换此代码非常容易，同时消除了使用动态 SQL 查询来传递参数的诱惑，我认为问题应该是：保持代码原样有什么好处吗？

例如：

public static class Database
{
    private static readonly Func<DbCommandBuilder, int, string> getParameterName = CreateDelegate("GetParameterName");
    private static readonly Func<DbCommandBuilder, int, string> getParameterPlaceholder = CreateDelegate("GetParameterPlaceholder");

    private static Func<DbCommandBuilder, int, string> CreateDelegate(string methodName)
    {
        MethodInfo method = typeof(DbCommandBuilder).GetMethod(methodName, BindingFlags.Instance | BindingFlags.NonPublic, Type.DefaultBinder, new Type[] { typeof(Int32) }, null);
        return (Func<DbCommandBuilder, int, string>)Delegate.CreateDelegate(typeof(Func<DbCommandBuilder, int, string>), method);
    }

    private static string GetDefaultProviderName()
    {
        ...
    }

    private static string GetDefaultConnectionString()
    {
        ...
    }

    public static DbProviderFactory GetProviderFactory()
    {
        string providerName = GetDefaultProviderName();
        return DbProviderFactories.GetFactory(providerName);
    }

    private static DbConnection GetDBConnection(DbProviderFactory factory)
    {
        DbConnection connection = factory.CreateConnection();
        connection.ConnectionString = GetDefaultConnectionString();
        return connection;
    }

    public static DbConnection GetDBConnection()
    {
        DbProviderFactory factory = GetProviderFactory();
        return GetDBConnection(factory);
    }

    private static void ProcessParameters(
        DbProviderFactory factory, 
        DbCommand command, 
        string query, 
        object[] queryParameters)
    {
        if (queryParameters == null && queryParameters.Length == 0)
        {
            command.CommandText = query;
        }
        else
        {
            IFormatProvider formatProvider = CultureInfo.InvariantCulture;
            DbCommandBuilder commandBuilder = factory.CreateCommandBuilder();
            StringBuilder queryText = new StringBuilder(query);

            for (int index = 0; index < queryParameters.Length; index++)
            {
                string name = getParameterName(commandBuilder, index);
                string placeholder = getParameterPlaceholder(commandBuilder, index);
                string i = index.ToString("D", formatProvider);

                command.Parameters.AddWithValue(name, queryParameters[index]);
                queryText = queryText.Replace("{" + i + "}", placeholder);
            }

            command.CommandText = queryText.ToString();
        }
    }

    public static DataView GetDataView(string query, params object[] queryParameters)
    {
        DbProviderFactory factory = GetProviderFactory();

        using (DbConnection connection = GetDBConnection(factory))
        using (DbCommand command = connection.CreateCommand())
        {
            command.CommandType = CommandType.Text;
            ProcessParameters(factory, command, query, queryParameters);

            DbDataAdapter adapter = factory.CreateDataAdapter();
            adapter.SelectCommand = command;

            DataTable table = new DataTable();
            adapter.Fill(table);
            return table.DefaultView;
        }
    }
}

使用此版本，您现在可以简单安全地传入参数，而无需依赖自定义代码来尝试阻止 SQL 注入：

DataView dv = Database.GetDataView(
   "select mycolumn from my table where id = {0} and name = {1}",
   1234, "Robert');DROP TABLE Students;--");

编辑
在this answer 的帮助下更新以支持不同提供商的参数。

Answer 3

我看到的唯一问题是

(1) 这就像重新发明轮子。 FW3.5 的企业库 v5 和 FW4.5 的 v6 具有数据访问组件。使用它。

使用 EL，您可以拨打电话并在 Dataset 中加载 2、3、4 个表。用你的方法这是不可能的，一次只有一个。

Enterprise library 是 Microsoft 提供的完整数据访问套件。它会处理所有的小细节，您所需要的只是调用您的数据。这是完整的数据访问层。如果你看得更深，EL 允许集成数据和缓存，以及其他东西。但是你不必使用你不需要的东西。如果您需要数据访问，则只能使用它。

并且 (2) 通常，在参考中编写低级程序集和高级程序集并不是一个好主意。任何System.Web.... 都是 UI 和客户端相关的东西。在分层蛋糕设计中，这就像它的顶部，而数据访问位于底部。所有引用 [除了“common”] 都应该从底部到顶部，并且方向相反。

看这张图：

这是来自微软。你会看到“蛋糕”的层次。所有引用都在上升。你做了什么——你拿了 UI 相关的组件并在里面写了 Data Access。

您可以将其称为基于意见的 - 但这种意见是软件开发中的标准做法和模式。您的问题也是基于意见的。因为您可以在单个文件、单个类中编写所有内容，并且它会起作用。如果需要，您可以在 Asp.net 应用程序中设置对 System.Windows.Forms 的引用。 从技术上讲，这是可能的，但这是非常糟糕的做法。

您的应用程序现在的可重用性有限。如果您编写需要使用相同数据访问的 WPF 组件或服务怎么办。非得把System.Web都拖进去吗？