Hoe ga je om met het onprofessionele gedrag van stagiaires?
Ik ben een softwareontwikkelaar in een klein bedrijf (een paar dozijn medewerkers). We hebben een zomerstage. De stagiaires zijn hogeschoolstudenten zonder professionele ervaring en met een vrij elementaire kennis van de programmeertaal. (Ik was niet betrokken bij de keuze van de stagiaires). Sommige stagiaires zullen in de toekomst worden aangenomen in het bedrijf op basis van hun prestaties.
De stagiaires ontwikkelen een eenvoudige applicatie (niet voor het bedrijf, geen productiecode) alleen maar om wat basisprincipes te krijgen en elkaar te leren kennen voordat ze overgaan tot complexere dingen.
Ik help hen in de ontwikkeling op een dagelijkse basis (snelle vergaderingen om problemen op te lossen die ze niet zelf kunnen oplossen) en doe code review. Een van de belangrijkste problemen die ze hebben is dat ze geen naamgevingsconventies volgen en slechte en korte namen maken voor methoden (zoals convert
). Natuurlijk heb ik hen uitgelegd dat een goede naamgeving erg belangrijk is, dat ze niet bang moeten zijn om langere, beschrijvende methodieknamen te gebruiken (zoals convertGallonsToMilliliters
). Helaas hebben een paar van hen (en ik weet wie, omdat ze versiebeheer gebruiken) blijkbaar besloten om wat plezier te hebben (of mij te bespotten) en zijn ze begonnen met het maken van domme methode namen zoals convertToMillilitersBecauseIAmUsingSuchCleanCode
- niet een enkele gebeurtenis, maar een paar.
Hoe moet ik hier op reageren? Ik weet dat het geen productiecode is, maar ik besteed een behoorlijke hoeveelheid tijd aan het nakijken en doe mijn best - om de stagiaires te helpen leren en ze best practices te laten oppikken als het gaat om schone code.
Moet ik reageren door
- het aflachen (“Ja, het is grappig maar verwijder het alsjeblieft”)
- gewoon beleefd vragen om het te verwijderen
- vertellen dat ik het niet leuk vind als iemand mijn tijd verspilt en ze de code review serieuzer moeten nemen
Ik weet dat het waarschijnlijk niet zo erg is maar het is mijn eerste keer dat ik de stagiaires help en ik wil graag weten hoe ik goed met zo'n situatie om kan gaan. Toch zal hun prestatie tijdens de stage van invloed zijn op hun kansen om aangenomen te worden en situaties als deze kunnen later een rol spelen. Of moet ik hen iets in die zin vertellen om hen meer gemotiveerd te maken om daadwerkelijk iets te leren?
- *
EDIT: Dank u voor uw geweldige antwoorden. Ik heb de kwestie met mijn manager besproken en met de stagiaires gesproken. Ik schreef de naam van de methode op het whiteboard en vroeg of zij het een goede naam vinden. Ik heb ook nog eens kort met hen gesproken over het doel van de code reviews. Ik vertelde hen dat we in echte projecten externe bedrijven hebben die code review audits doen - dit soort grappen kunnen hen later echt in de problemen brengen. Het is dus eigenlijk beter voor hen om deze les te leren tijdens de stage.
Nadat we spraken, gaven ze toe dat ze zo'n code niet moeten vastleggen. Ze vertelden me ook dat ze dankbaar zijn voor de tijd die ik besteed aan het nakijken van hun code en voor mijn hulp. Maar het mooiste is dat de kwaliteit van hun werk sindsdien reëel is verbeterd. Eigenlijk is de man die de grap maakte begonnen met het leveren van de beste code in de groep - ik zie hem (en andere stagiaires, ook) mijn aantekeningen van de code review nu veel serieuzer nemen.