根据调用的上下文不同地使用静态类

Question

我有一个静态类，它的唯一工作是通用 SQL 调用的包装器。但是，根据调用静态类的类，我想相应地设置 connectionString（它是包装类中的公共静态字符串）。如果可能的话，我希望它根据使用的上下文推断要使用的连接字符串，而不必传递它。

有没有办法按照我的要求去做？

以这种方式使用静态类是不好的做法吗？

我不太关心有一个只包装 sql 调用的非静态类，维护该对象似乎是额外的工作。

包装器：

public static class SqlServer
{
    private static readonly string SqlConnectionString = ConfigurationManager.ConnectionStrings["foo"].ConnectionString;

    /// <summary>
    ///     Single point of entry for SQL queries that need to return a LIST of well-defined objects
    /// </summary>
    public static IEnumerable<T> ExecuteQuery<T>(SqlConnection sqlConnection, string sql, object param, int? timeout = null)
    {
        if (sqlConnection != null)
        {
            return sqlConnection.Query<T>(sql, param, commandTimeout: timeout).ToList();
        }

        using (var tempSqlConnection = new SqlConnection(SqlConnectionString))
        {
            tempSqlConnection.Open();
            return tempSqlConnection.Query<T>(sql, param, commandTimeout: timeout).ToList();
        }
    }
}

应该使用“bar”连接字符串：

public class BarController : ApiController
{
    [HttpGet]
    public int getUserId(string username)
    {
        var args = new
        {
            USERNAME = username
        };

        const string sql = @"SELECT id FROM users WHERE username = @USERNAME";

        return  SqlServer.ExecuteQuery<int>(null, sql, args).Single();
    }
}

应该使用“foo”连接字符串：

public class FooController : ApiController
{
    [HttpGet]
    public int getUserId(string username)
    {
        var args = new
        {
            USERNAME = username
        };

        const string sql = @"SELECT id FROM users WHERE username = @USERNAME";

        return  SqlServer.ExecuteQuery<int>(null, sql, args).Single();
    }
}

Answer 1

有没有办法做我所要求的？ 以这种方式使用静态类是不好的做法吗？

总有办法，但有些事情不应该做，除非你了解后果。这是不应该做的事情之一。强制静态类表现得像一个可变对象几乎没有充分的理由。所以，是的，这是不好的做法。

静态类包装的 SQL 方法需要连接。该类可以使用从调用者传入的参数提供与这些 SQL 方法的连接，这就是您现在正在做的事情（一种可接受的做法）。或者该类可以包含提供连接本身，这意味着它将把它存储为一个属性，并在调用 SQL 方法时使用它。

使用静态类意味着您希望连接字符串在初始化后保持不变；除非您公开一个方法（公共属性）来更改它，否则它不会在进程的生命周期内更改，这会使您容易受到并发问题的影响。通过根据调用者进行更改，您将使其成为非常量。

您要做的是让静态类从调用者那里提取连接的值。这违反了OOP的封装原则。

更好的做法是做您认为“额外工作”的事情。让每个调用者实例化这个包装类的一个对象，在构造函数参数中提供连接字符串。

Answer 2

您可以将对象传递给您正在调用的静态方法，然后打开该对象的类型以设置连接字符串。

例如

public static ConnectToDb(object _Object)
{
    switch(typeof(_Object))
    {
       //Set connection string...
    }
    // Do some stuff
}    

通过 ConnectToDb(this)... 或其他方式调用