c# - const int 列表而不是枚举

Question

我开始处理大型 C# 代码库，发现使用带有多个 const int 字段的静态类。这个类的行为与枚举完全一样。

我想将类转换为实际的枚举，但被拒绝的权力。我想转换它的主要原因是我可以将枚举作为数据类型而不是 int。这对可读性有很大帮助。

有什么理由不使用枚举而是使用常量整数吗？目前代码是这样的:

public int FieldA { get; set; }
public int FieldB { get; set; }

public static class Ids
{
    public const int ItemA = 1;
    public const int ItemB = 2;
    public const int ItemC = 3;
    public const int ItemD = 4;
    public const int ItemE = 5;
    public const int ItemF = 6;
}

不过，我觉得应该是下面这样:

public Ids FieldA { get; set; }
public Ids FieldB { get; set; }

Answer 1

我认为这里的许多答案都忽略了枚举语义的含义。

• 如果预先知道所有有效值 (Id) 的整个集合，并且小到足以在程序代码中声明，您应该考虑使用枚举。

• 当一组已知值是所有可能值的子集时，您应该考虑使用 int - 并且代码只需要知道这个子集。

关于重构 - 在时间和业务限制允许的情况下，当新设计/实现比以前的实现具有明显优势并且风险得到充分理解时，清理代码是个好主意。在 yield 低或风险高(或两者都有)的情况下，采取“不伤害”而不是“持续改进”的立场可能更好.只有您能够判断哪种情况适用于您的情况。

顺便说一句，既不枚举或常量整数必然是个好主意的情况是，当 ID 表示外部存储中记录的标识符(如数据库)。在程序逻辑中对此类 ID 进行硬编码通常是有风险的，因为这些值在不同环境(例如，测试、开发、生产等)中实际上可能不同。在这种情况下，在运行时加载值可能是更合适的解决方案。