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

Question

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

当我打开一些已写入的代码而不是其他代码时,我的屏幕上的橙色数量就会立即显而易见.

我不确定的是,我应该在多大程度上自由地整理那些在不知不觉中留下的混乱.对于我所看到的大部分内容,它是草率但无害的,如果我从未使用过Resharper,就不会真的跳出来.

我想我大致看到了我的选择

1)源代码更改的历史记录对于维护至关重要.尽可能少地改变,或者下一个人不希望弄清楚改变了什么.谁关心无法访问的代码,无论如何都不必要地使用.ToString()等.

2)更改无意义的东西,如包括,修复方法文档注释和类似的东西.写它的人喜欢他的代码看起来像这样,所以把它留在他不会抱怨的状态,但摆脱一些不必要的橙色

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

4)忘记橙色,看看怪物700线功能.这是什么1997年？是时候忙碌......如果你有时间,请把你的同事介绍给我们的好朋友和导师福勒先生.

我倾向于在选项之间进行过滤,具体取决于我有多少时间,我现在负责代码的程度,以及代码看起来有多复杂(这通常可以让我选择1或4).

看起来四个选项中的一个应该是我正在努力的那个,但我不知道哪一个

Answer 1

"Leave the campsite cleaner than you found it."

这是boyscout的原则.如果它是"他们的"代码并且他们维护它然后引入一些清理 - 改变不应该冒犯它们,但是走得太远可能看起来很粗鲁或者你可能有效地获得了代码的所有权.

Answer 2

您的团队应该就标准达成一致.如果其他人使用其他工具,您可能会发现自己处于无意的编辑战争中.

但如果你们都同意,那么是的.随时清理代码.

Answer 3

在更改任何实际逻辑之前,我会进行"重新格式化"检查,这样您就可以看到发生了哪些变化.

Answer 4

不必要的重构只是 - 不必要的.它使存储库历史记录日志变得混乱,您可以引入错误.

如果"无意义"的东西(文档,注释等)应该以某种方式格式化,并且它不符合您的开发标准,那么我会尽可能少地签入所有内容.

当您实际处理这些代码并有机会测试您的更改时,请进行重构.Resharper将随时为您提供指示.