我继承了一些广泛且不必要地使用 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 的不必要使用还有什么破坏性的?
我特别关心维护。最好有合理的答案和例子。
也欢迎您争论清理是没有必要的。
最佳答案
我想说最大的危险是如果参数由于某种原因在函数内部被设置为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.
如果代码现在按预期工作并且您的开发人员理解其中的区别,那么将它们全部删除可能不值得付出努力。
关于c# - 引用滥用 : Worth Cleaning Up?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/748878/