當我的團隊負責人編寫意大利麪代碼時，我應該建議進行代碼審查嗎？

Question

當我inheret從我的一些同事的代碼在團隊（誰在例如度假去了），我簡直遭受修改自己的代碼，並修復它，我需要很長的時間做這件事：

• 錯誤代碼結構
• 很多寫得不好功能
• 誤導性變量的名稱，如xxxObj，我覺得後面包含一個列表！
• 同樣的事情，但在不同的狀態
• 的可能的代碼路徑
• 和當然，很少或無意見不可預知號碼太多的變數通過他們的叢林代碼來帶領我。

這裏是我的真正的問題，甚至我的團隊領導者的代碼（之前她成爲一個團隊領導，現在她幾乎從不代碼）使我遭受同樣，當我繼承了它出於同樣的原因。 ..

現在我很害怕，如果我建議代碼審查中實現，她將是一個做這件事，我不認爲這是非常有用的..

因爲，例如，當我被聘用 - 她在那裏 - 我們一起編碼，我發現她將x分配給y（複製值），其中o如果他們是對象，她不知道參考和值類型之間的區別。當然，我很好地向她解釋，但是......你明白我的觀點。

所以，在這種情況下，它是一個好主意，建議代碼審查，以提高代碼質量和減少維護時間？

如果不能，我能做些什麼來幫助提高代碼的質量？

PS：我認爲這是值得一提的是，一旦我工作的一個代碼，完全取決於我的同事的代碼，這是很難理解和處理一個的輸出，我從字面上告訴她本人親自「可能會增強代碼結構，所以我可以更容易處理我的代碼？我發現很難理解。」她說，你是來告訴我的代碼結構？然後她抱怨我領隊誰發送了一封電子郵件給整個團隊（針對我）說一些聽起來像這樣「大家好，只是你的工作任務，不多不少，如果有問題請諮詢我「。

Answer 1

它始終是最好的一個項目，以保持清潔，無重複的，可讀的代碼。你應該在任何環境中鼓勵這種最佳實踐。

只要確保尊重地解決問題並且不用指尖。如果她對此有疑問，則可能需要與高級經理交談。

這個問題應該是最好的項目，而不是某人的自我。

Answer 2

我想你在問什麼：「我如何讓團隊提高代碼質量？」

對這個問題有很多很好的答案。如何在不踩人腳的情況下做到這一點是另一個問題。

在提出解決方案之前，我先從團隊中解決問題開始。像你說的。凌亂的代碼會浪費你大量的時間。這是不可維護的，很難理解。讓人們理解這個問題將使他們更加開放的解決方案。 你甚至可以使用代碼示例來解釋問題。

一旦你完成了，團隊可以與解決方案一起來。你把他們轉向正確的方向。

我建議爲團隊定義代碼準則。指導方針將由整個團隊決定。這些也可以幫助你做代碼評論。因爲您可以在添加代碼評論評論時參考他們。

當建議將代碼審查作爲一種解決方案時，請不要關注您希望讓開發人員編寫更好的代碼的事實。儘管這是您想要進行代碼審查的原因之一。而是集中於其他許多被認爲不那麼敵對的原因。 像：知識分享，團隊合作，每個人都在一起工作，...可能有一些其他好的理由有一個代碼審查。

如果您認爲團隊不夠成熟，無法編寫乾淨的代碼。你可以建議圍繞乾淨的代碼書做一個閱讀組。例如在午餐休息期間。這對於更簡潔的代碼來說是一個很好的起點。

因此，在短期：

是的，你應該提出解決方案，以提高代碼。我寧願在我的團隊中有一個敢於發言的開發者，而不是那些即使我們走錯了路也會一直跟隨的開發者。

• 讓人理解的問題
• 與解決方案拿出作爲一個團隊一起。
• 實施這些解決方案
• 在實施的解決方案反饋
• 提高