c# - 我应该保留这个静态变量，还是重构它？

Question

之前的开发者在我继承的一个项目的“god”类中放置了一个名为“Qry”的静态字符串。

开发人员然后在整个程序中构建和使用单个数据库查询字符串的每个地方都使用了这个静态变量。

例如:

SomeGodClass.Qry = "select count(1) from AddressBook where Name = '" + txtName.Text.Trim(' ') +
                            "' and Group_Name = '" + txtGroupName.Text.Trim(' ') + "'";

int count = sqlHelper.ExecuteScalar(SomeGodClass.Qry);

因此，这个变量被精确引用了 626 次，大多数都分配了不同的查询。他还使用了其他类似的静态变量 - 可能有 50 个 - 但这是最​​主要的。

我的第一直觉是删除这个静态字符串并重新处理所有 626 种用法。但是，我不知道这种做法是否足够糟糕，需要花时间去做。

因此，我的问题是:这是对静态字符串的可接受使用吗，尤其是考虑到重构所需的工作量时？

Answer 1

至少从我看到的代码来看，在这里使用static 没有发现任何问题。但不要使用 TextBox 连接!

使用 Parameters 代替。

这是我在查看提供的代码时能想到的最重要的重构。

Why do we always prefer using parameters in SQL statements?