2016-06-08 54 views
1

通常會要求您提供某些功能。但是,在進行功能更改時,您會注意到其他地方的許多糟糕的編碼。隨着編碼技能的提高,它總是會發生。你想立刻解決它們。理想的是立即清理代碼,因爲它非常煩人,並且您知道完成該功能後會忘記它。您還希望加強您的編碼最佳實踐,將其應用於您認爲合適的地方。你會懶得去尋找一些不是眼神的東西。但是,您希望將所有非特徵方面的問題放在一邊,並且在下一個特徵被請求之前,您不會再回頭查看代碼,這意味着沒有地方可以修復不好的編碼。這是最容易的,但在以後的時間會很痛苦。將格式化與功能更改相結合

我想生成所有不相關的小格式更改,並將它們收集到單獨的提交中,因爲我忙於功能性編碼。這樣做的最佳做法是什麼?

回答

1

總是做單獨的重構提交。如果您發現一些錯誤代碼,請在開始使用功能之前重新格式化它。這將有助於追蹤您完成任務所做的工作以及您在後面的代碼審查中作爲重構部分所做的工作。

如果重構與您正在實施的事情非常接近,請立即重構它。如果沒有直接觸摸你的工作,請考慮其他git分支進行這些更改或至少另一次提交。

要做單獨的提交,您可以使用git add --patch並通過重構/功能實現/ etc對您的更改進行分組。

+0

您的意思是說最少的麻煩是我首先修復編碼問題,就像我做我的功能一樣,然後過濾更改,將它們分成兩個「patch」選項提交? –

+0

@LittleAlien是的,這是一種選擇,但它不會每次都有效。它只有在你的重新格式化/重構不在你當前工作的某些功能的情況下才有效 – kTT

+0

這正是我需要的!因爲與我的功能相混淆的更改被固定爲功能的一部分。認爲他們是有效的,因爲我需要爲我的功能重寫那段代碼,我們可以合法地假裝它沒問題 - 我只需要爲我的功能更新它。所以,我的問題只涉及遙遠的,絕對不相關的代碼片斷。我只需要它,因爲每個人都需要這個選項:) –