c# - 在代码中使用大量硬编码字符串

Question

当我查看我的代码时，我正在编写诸如...

if (role == "Customer")
{
    bCustomer = true;
}
else if (role == "Branch")
{
    bIsBranch = true;
}

或者

foreach(DataRow as row in myDataSet.Tables[0].Rows)
{
    row["someField"]=somefield.Tostring()
}

你们在做这个吗？什么时候可以这样做，什么时候不应该这样做？如果有的话，写这篇文章的更好方法是什么？

感谢您的评论:我想我应该添加如果(为了这个例子的目的)我只使用这个角色比较一次怎么办？创建一个全新的类(class)仍然是一个更好的主意吗？我还应该有 1 个名为“常量”的类是多个包含特定常量的类，例如“角色”类吗？

Answer 1

没有。不要使用“magic strings”。而是创建一个带有常量的静态类，或者如果可以的话创建一​​个枚举。

例如:

public static class Roles
{
  public const string Customer = "Customer";
  public const string Branch = "Branch";
}

用法:

if (role == Roles.Customer)
{

}
else if (role == Roles.Branch)
{

}

这是一个 good discussion on various solutions .