遇到string.Replace / Regex.Replace问题

Question

我有一个这样的SQL查询：

UPDATE StockUpdateQueue SET Synced = 1, SyncedAt = GETDATE() WHERE Id IN (@p0,@p1);

该查询是动态生成的。 我想做的是通过一个函数运行它，该函数将有效地将所有@pN参数替换为其相应的值。

我试图用标准的string.Replace以及Regex.Replace做到这一点，但运气不好-替换没有发生。

这是我到目前为止尝试过的：

class Program
{
    static string _lastQuery;

    static void Main(string[] args)
    {
        var sqlQuery = "UPDATE StockUpdateQueue SET Synced = 1, SyncedAt = GETDATE() WHERE Id IN (@p0,@p1);";
        var sqlParamters = new Dictionary<string, object>()
        {
            { "@p0", 12345 },
            { "@p1", 65432 }
        };

        LogLastQuery(sqlQuery, sqlParamters);
    }

    static void LogLastQuery(string sqlQuery, Dictionary<string, object> sqlParamters = null)
    {
        _lastQuery = sqlQuery;
        if (sqlParamters != null && sqlParamters.Count > 0)
            foreach (KeyValuePair<string, object> sqlParamter in sqlParamters)
                _lastQuery = Regex.Replace(
                    _lastQuery, 
                    "\\@" + sqlParamter.Key,
                    sqlParamter.Value.GetType() == typeof(int) || sqlParamter.Value.GetType() == typeof(decimal)
                        ? sqlParamter.Value.ToString() 
                        : "'" + sqlParamter.Value.ToString() + "'");
    }
}

我希望函数执行参数替换并理想地输出如下内容：

UPDATE StockUpdateQueue SET Synced = 1, SyncedAt = GETDATE() WHERE Id IN (12345,65432);

有任何想法吗？

---

更新资料

我使用sqlQuery和sqlParamters就像用户un-lucky已经显示出，即

{
    mySqlCommand.Parameters.AddWithValue(sqlParamter .Key, sqlParamter.Value);
}

事实证明，我一直在生成没有@的字典键，并且.NET已自动为我添加它们（当它们丢失时-如用户juharr ）。

有时，我会动态生成sql查询和参数-在for / foreach循环中生成键列表及其对应的值。 这样就导致了混合使用的情况。

因此，为了解决这个问题-我已经像这样更新了我的函数，它可以按预期工作：

internal void LogLastQuery(string sqlQuery, Dictionary<string, object> sqlParamters = null)
{
    _lastQuery = sqlQuery;
    if (sqlParamters != null && sqlParamters.Count > 0)
        foreach (KeyValuePair<string, object> sqlParamter in sqlParamters)
            _lastQuery = Regex.Replace(
                _lastQuery,
                (sqlParamter.Key.ToString()[0] != '@' ? "\\@" : "") + sqlParamter.Key,
                sqlParamter.Value.GetType() == typeof(int) || sqlParamter.Value.GetType() == typeof(decimal)
                    ? sqlParamter.Value.ToString()
                    : "'" + sqlParamter.Value.ToString() + "'");
}

现在处理这两种情况。 我不确定这是否是最好的方法，但目前效果很好。

Answer 1

您在字典中的键已经有“ @”符号，请尝试使用sqlParamter.Key代替"\\\\@" + sqlParamter.Key 。

Answer 2

我认为最好使用该查询创建SqlCommand并将这些值添加为参数，这将是您执行查询的最佳选择，而不是替换。 这意味着您可以尝试如下操作：

 var sqlQuery = "UPDATE StockUpdateQueue SET Synced = 1, SyncedAt = GETDATE() WHERE Id IN (@p0,@p1);";
 var sqlParamters = new Dictionary<string, object>()
 {
      { "@p0", 12345 },
      { "@p1", 65432 }
 };
 SqlCommand mySqlCommand = new SqlCommand(sqlQuery , conSQL); // conSQL is the connection
 foreach (KeyValuePair<string, object> sqlParamter in sqlParamters)
 {
    mySqlCommand.Parameters.AddWithValue(sqlParamter .Key, sqlParamter.Value);
 }
 mySqlCommand .ExecuteNonQuery();