我是一家小公司(幾十名員工)的軟件開發人員。我們正在進行暑期實習。實習生是沒有專業經驗並且具有相當編程語言基礎知識的大學生。 (我沒有參與實習生的選擇)。將來,根據他們的表現,一些實習生將在公司聘用。
實習生正在開發一個簡單的應用程序(不適用於公司,而不是生產代碼),只是為了了解一些基本原理並相互了解,然後再進行更複雜的工作。
我每天都在幫助他們進行開發(快速會議以解決他們自己無法解決的問題),並進行代碼審查。他們遇到的主要問題之一是它們不遵循命名約定,並為方法創建了較差的簡稱(例如 convert
)。當然,我向他們解釋了正確的命名非常重要,他們不應該害怕使用更長的描述性方法名稱(例如 convertGallonsToMilliliters
)。不幸的是,其中一些人(我知道是誰,因為他們正在使用版本控制)顯然決定找點樂子(或嘲笑我),並開始創建諸如 convertToMillilitersBecauseIAmUsingSuchCleanCode
之類的愚蠢的方法名稱-並非一次,但很少。
我應該如何應對?我知道這不是生產代碼,但是我花了大量的時間進行審查,並且我會盡力而為-幫助實習生學習並讓他們學習有關乾淨代碼的最佳做法。
我應該以
- 笑出來(“是的,這很有趣,但是請刪除它”)
- 只是禮貌地要求刪除它
- 告訴我當某人浪費我的時間時我不喜歡,他們應該更認真地進行代碼審查
我知道這可能沒什麼大不了的,但這是我第一次幫助實習生,我想知道如何正確處理這種情況。儘管如此,他們在實習期間的表現仍會影響他們被錄用的機會,以後這種情況可能會發揮作用。或者我應該按照這些思路告訴他們一些東西,以使他們更有動力去實際學習一些東西?
編輯:謝謝您的出色回答。我與經理討論了這個問題,並與實習生進行了交談。我在白板上寫下了方法名稱,並詢問他們是否認為這是一個好名字。我還與他們簡短地討論了代碼審查的目的。我告訴他們,在實際項目中,我們有外部公司進行代碼審查審計-這種笑話可能會在以後給他們帶來麻煩。因此,對他們來說,在實習期間學習本課實際上更好。
我們進行了交談之後,他們承認不應提交此類代碼。他們還告訴我,他們感謝我花時間審查代碼並感謝我的幫助。但最棒的是,此後他們的工作質量得到了真正的改善。實際上,開玩笑的那個人已經開始提供小組中最好的代碼-我現在看到他(和其他實習生)更加認真地對待我的代碼審查筆記。