如何处理实习生的不专业行为?
我是一家小公司(几十名员工)的软件开发人员。我们正在进行暑期实习。实习生是大学生,没有专业经验,对编程语言有相当基础的知识。我没有参与选择实习生)。
实习生们在开发一个简单的应用(不是为公司开发的,不是生产代码),只是为了掌握一些基本的原理和了解对方,然后再去做更复杂的东西。他们的主要问题之一就是不遵守命名规则,给方法起了个又差又短的名字(比如convert
)。当然,我向他们解释过,正确的命名非常重要,他们不应该害怕使用较长的、描述性的方法名(如convertGallonsToMilliliters
)。不幸的是,他们中的几个人(我知道是谁,因为他们使用的是版本控制)显然决定找点乐子(或者说是嘲笑我),开始创建像convertToMillilitersBecauseIAmUsingSuchCleanCode
这样的愚蠢的方法名–不是一次出现,而是几次出现。我知道这不是生产型代码,但我花了相当多的时间来复习,我也在尽力而为–帮助实习生学习,让他们在清洁代码的时候捡起最佳实践。
我应该通过
- 一笑置之("是的,这很有趣,但请删除它")
- 只是礼貌地要求删除它
- 告诉他们我不喜欢有人浪费我的时间,他们应该更认真地对待代码审查
我知道这可能不是什么大事,但这是我第一次帮助实习生,我想知道如何正确处理这种情况。不过,他们在实习期间的表现还是会影响到他们被录用的机会,像这样的情况在以后会起到一定的作用。还是说我应该跟他们说一些这样的事情,让他们更有动力去实际学习一些东西?
EDIT:谢谢你的精彩回答。我和我的经理讨论了这个问题,也和实习生谈了一下。我在白板上写下了方法的名字,问他们觉得这个名字好吗?我又和他们简单地讨论了一遍代码评审的目的。我告诉他们,在实际项目中,我们有外部公司来做代码评审审核—-这样的玩笑话,以后真的会给他们带来麻烦。所以,其实在实习的时候,他们最好还是在实习的时候就吸取这个教训。
经过我们的交谈,他们承认,他们不应该提交这样的代码。他们还告诉我,他们很感谢我花时间审核他们的代码,感谢我的帮助。但最棒的是,他们的工作质量从那以后真的有了很大的提高。实际上,提交笑话的那个人已经开始提供了小组里最好的代码–我看到他(还有其他实习生)现在更认真地对待我的代码审查笔记了。