c# - 引用滥用 : Worth Cleaning Up?

Question

我继承了一些广泛且不必要地使用 ref 关键字的代码。最初的开发人员显然担心如果不使用 ref 对象会像原始类型一样被克隆，并且在编写 50k+ 行代码之前没有费心去研究这个问题。

这与其他不良编码实践相结合，造成了一些表面上非常危险的情况。例如:

---

Customer person = NextInLine(); 
//person is Alice
person.DataBackend.ChangeAddress(ref person, newAddress);
//person could now be Bob, Eve, or null

您能想象走进一家商店更改地址，然后以完全不同的方式走出去吗？

---

可怕，但在实践中在此应用程序中使用 ref 似乎是无害的多余。我无法证明清理它需要花费大量时间。为了帮助推销这个想法，我提出了以下问题:

对 ref 的不必要使用还有什么破坏性的？

我特别关心维护。最好有合理的答案和例子。

也欢迎您争论清理是没有必要的。

Answer 1

我想说最大的危险是如果参数由于某种原因在函数内部被设置为null:

public void MakeNull(ref Customer person)
{
    // random code
    person = null;
    return;
}

现在，你不仅仅是一个不同的人，你已经完全从存在中消失了!

只要开发此应用程序的人明白:

By default, object references are passed by value.

和:

With the ref keyword, object references are passed by reference.

如果代码现在按预期工作并且您的开发人员理解其中的区别，那么将它们全部删除可能不值得付出努力。