インターン生のプロ意識のない行動にどう対処する?
私は小さな会社(社員数十人程度)でソフトウェア開発者をしています。夏のインターンシップを実施しています。インターン生は、専門的な経験がなく、プログラミング言語のかなり基本的な知識を持った大学生です。(インターン生の選定には私は関与していません)。インターン生の中には、将来的に彼らのパフォーマンスに基づいて会社に採用される人もいます。
インターン生は、より複雑なものに移る前に、いくつかの基本的な原理を得て、お互いを知るために簡単なアプリケーションを開発しています。彼らが抱えている主な問題の一つは、命名規則に従わず、メソッドの名前を貧弱で短いものにしてしまうことです(convert
のような)。もちろん、私は適切な命名が非常に重要であること、長い、説明的なメソッド名を使うことを恐れてはいけないことを彼らに説明しました(convertGallonsToMilliliters
のような)。不幸なことに、彼らのうちの何人か(そして、バージョン管理を使っているので、私は誰か知っています)は、どうやら、ちょっとした楽しみ(あるいは私をあざ笑うこと)を見つけて、convertToMillilitersBecauseIAmUsingSuchCleanCode
のようなバカげたメソッド名を作り始めました - 一度きりではなく、いくつか。本番のコードではないことはわかっていますが、私はかなりの時間をレビューに費やしていますし、最善を尽くしています - インターンの学習を助け、クリーンなコードになるとベストプラクティスをピックアップしてもらうために。
- 笑い飛ばす("Yeah, it’s funny but please remove it")
- 丁寧に削除をお願いする
- 誰かが私の時間を無駄にするのは好きではないし、彼らはコードレビューをもっと真剣に受け止めるべきだと伝える
大したことではないのはわかっているのですが、インターンを手伝うのは初めてなので、このような状況にどう対処したらいいのかを知りたいです。でも、インターン中の成績が採用の可能性を左右しますし、後々のことを考えると、こういうこともありますよね。それとも、実際に何かを学ぶためのモチベーションを高めるために、これらのラインに沿って何かを伝えるべきでしょうか?
EDIT: 素晴らしい回答をありがとうございます。上司と相談して、インターン生にも相談してみました。私はホワイトボードにメソッド名を書き、彼らはそれが良い名前だと思うかどうかを尋ねました。また、コードレビューの目的についても簡単に話しました。実際のプロジェクトでは、外部の会社がコードレビューの監査を行っていることもあり、このような冗談を言っていても後でトラブルになる可能性があります。このような冗談は後々トラブルになる可能性があるので、実際にはインターンシップの間に学んだ方がいいと思います。また、私がコードのレビューに時間を割いてくれたり、助けてもらったりしていることに感謝しているとも言ってくれました。しかし、一番良いのは、それ以来、彼らの仕事の質がreally向上しているということです。実際、そのジョークをコミットした人は、グループの中で最高のコードを提供し始めました。