C#. NET组织SQL查询并拥有干净代码的最佳方法是什么

Question

我和我的同事面临一个问题。我们有几个SQL查询作为字符串。下面是一个例子：

    public class Query
{

    public static string CreditTransferId(string expectedValue, string table, int statusId, int messageTypeId, int destination103, int destination202, string StoreStatus202Id)
    {
        return $"SELECT top 1 Id from {table} where MessageId = '{expectedValue}' and FlowId=3 and StatusId={statusId} and MessageTypeId={messageTypeId} and " +
          $" Destination103={destination103} and Destination202={destination202} and StoreStatus103Id is null and StoreStatus202Id {StoreStatus202Id}";
    }
}

目前，我们将它们作为字符串从查询类中的方法返回。我们希望重构代码，使其干净，因为我们有一个方法有3个以上的参数，这是很难使用的任何方式。

你会怎么做？组织需要大量参数的SQL查询的最干净的方法是什么？我非常感谢任何可能改善我们代码当前状态的答案。

谢谢

Answer 1

EF很棒，你必须决定这是否是你想要的。在某些情况下，这是不合适的。如果你决定坚持使用纯文本查询，那么把查询分成几个部分如何：FromBuilder JoinBuilder GroupBuilder Condition builder等等

前：

return @"
"+new TableIdSelectorBuilder(table).Get() +@"
"+new FromBuilder().Get(table) +@"
"+new TableConditionByIdBuilder(table).Get(I'd)+@"
";

编辑：存储过程允许在不发布新应用程序版本的情况下更改查询，但在活体上工作有点麻烦。至少有时候是这样。

Answer 2

动态SQL是一个非常糟糕的开始，因为这些是开放SQL注入，您应该使用参数化查询并返回一个字符串。

"例如：从[表]中选择顶部1 id，其中[MessageId]=@MessageId"etc

因此，您不需要传入任何值，您可以将这些值添加到SQLParameter的列表中

表名可能毫无意义，因为它与sql有关，所以可能只需将其添加到sql字符串中即可

这实际上不需要额外的类，只需在调用它的地方创建sql变量，如果需要的话，它就在那里？

…或者使用存储过程

…或使用实体框架