.net - 我应该使用 Resharper 来整理其他人的代码吗？

Question

我在工作中使用 Resharper。我的一些同事没有。

当我打开一些由其他人编写的代码时，屏幕上的橙色数量立即显而易见。

我不确定的是，我应该在多大程度上随意收拾那些在不知不觉中留下的烂摊子。就我所看到的大部分内容而言，它是草率但无害的，如果我没有使用过 Resharper，我也不会真的跳出来。

我想我广泛地认为我的选择是

1) 源代码更改的历史记录对于维护是必不可少的。尽可能少改变，否则下一个人就没有希望弄清楚发生了什么变化。无论如何，谁在乎无法访问的代码，不必要地使用 .ToString() 等。

2)更改无意义的东西，如包含，修复方法文档注释等。写它的人喜欢他的代码看起来像这样，所以让它处于他不会提示的状态，但去掉一些不必要的橙色

3)橙色只是红色但更轻。 F12 然后 Alt+Enter 直到绿色。

4)忘记橙色，看看那个怪物700线功能。这是什么1997？是时候开始忙碌了……如果您有时间，请将您的同事介绍给我们的好 friend 兼导师福勒先生。

我倾向于根据我有多少时间、我现在对代码负责的程度以及代码看起来有多复杂(通常可以让我选择 1 或 4)在选项之间切换。

似乎 4 个选项之一应该是我正在努力的一个，但我不知道哪一个

Answer 1

"Leave the campsite cleaner than you found it."

这是童子军原则。如果它是“他们的”代码并且他们维护它，那么引入小的清理更改不应该冒犯他们，但是走得太远可能看起来很粗鲁，或者您可能有效地获得了代码的所有权。