2016-06-08 37 views
1

通常会要求您提供某些功能。但是,在进行功能更改时,您会注意到其他地方的许多糟糕的编码。随着编码技能的提高,它总是会发生。你想立刻解决它们。理想的是立即清理代码,因为它非常烦人,并且您知道完成该功能后会忘记它。您还希望加强您的编码最佳实践,将其应用于您认为合适的地方。你会懒得去寻找一些不是眼神的东西。但是,您希望将所有非特征方面的问题放在一边,并且在下一个特征被请求之前,您不会再回头查看代码,这意味着没有地方可以修复不好的编码。这是最容易的,但在以后的时间会很痛苦。将格式化与功能更改相结合

我想生成所有不相关的小格式更改,并将它们收集到单独的提交中,因为我忙于功能性编码。这样做的最佳做法是什么?

回答

1

总是做单独的重构提交。如果您发现一些错误代码,请在开始使用功能之前重新格式化它。这将有助于追踪您完成任务所做的工作以及您在后面的代码审查中作为重构部分所做的工作。

如果重构与您正在实施的事情非常接近,请立即重构它。如果没有直接触摸你的工作,请考虑其他git分支进行这些更改或至少另一次提交。

要做单独的提交,您可以使用git add --patch并通过重构/功能实现/ etc对您的更改进行分组。

+0

您的意思是说最少的麻烦是我首先修复编码问题,就像我做我的功能一样,然后过滤更改,将它们分成两个“patch”选项提交? –

+0

@LittleAlien是的,这是一种选择,但它不会每次都有效。它只有在你的重新格式化/重构不在你当前工作的某些功能的情况下才有效 – kTT

+0

这正是我需要的!因为与我的功能相混淆的更改被固定为功能的一部分。认为他们是有效的,因为我需要为我的功能重写那段代码,我们可以合法地假装它没问题 - 我只需要为我的功能更新它。所以,我的问题只涉及遥远的,绝对不相关的代码片断。我只需要它,因为每个人都需要这个选项:) –