Un collègue a récemment quitté notre entreprise. Avant de partir, il a codé un composant qui avait une grave fuite de mémoire qui a provoqué une panne de production ( OutOfMemoryErroren Java). Le problème était essentiellement un HashMapqui a grandi et n'a jamais supprimé les entrées, et la solution était de remplacer le HashMappar une implémentation de cache.

D'un point de vue professionnel, je pense que je devrais lui faire part du défaut afin qu'il puisse apprendre de l'erreur. D'un autre côté, une fois que les gens quittent une entreprise, ils ne veulent souvent pas entendre parler des projets hérités qu'ils ont laissés derrière pour des choses plus grandes et meilleures.

Quel est le protocole général pour ce genre de situation?

Vous ne traquez pas un ancien collègue pour lui dire qu'il a fait une erreur. Vous pouvez dire à votre ami qu'il a fait une erreur.

Que ce soit un ami ou un ancien collègue dépend de vous.

Ne fais rien.

1. Contacter quelqu'un uniquement pour lui dire qu'il a foiré, mais nous l'avons corrigé, n'est pas professionnel et peu importe à quel point vous essayez, il est peu probable qu'il soit reçu positivement.
2. Parler suffisamment en profondeur pour qu'une conversation soit utile à distance sur le code à des non-employés est mauvais quels que soient les problèmes potentiels de NDA.

Si vous êtes sous NDA, alors c'est un grand non-non de parler à quelqu'un en dehors de votre entreprise de tout problème lié à la propriété intellectuelle, qu'il s'agisse d'anciens employés ou non.

Si vous n'êtes pas sous un NDA, j'oserais dire qu'il / elle ne s'en souciera pas.

Cela mis à part, cette personne était-elle mécontente? Était-ce quelque chose qui aurait pu être intentionnel?

Avec une erreur aussi simple, les chances sont bonnes si cela dérange le collègue, ils ont probablement réalisé le problème quelques jours plus tard en réfléchissant. Je sais que je suis rentré du travail et j'ai réalisé "... merde, cet algorithme est totalement défectueux, je vais devoir le refaire demain" tout en me relaxant et en me remémorant ma journée.

Ce collègue est votre AMI avec qui vous continuez à avoir des contacts étroits après le départ? Si oui, parlez-en si / quand vous prenez des bières au bar.

Sinon, pourquoi s'embêter?

PS: Concernant le NDA, quel est le secret ici? M. X est de toute façon celui qui a écrit le code et si le départ est récent, le logiciel continue au même niveau de divulgation.

Les choses seraient différentes si cette conversation se produisait 3 ans après le départ et que vous disiez des choses qu'il n'aurait pas besoin de savoir à part vous ...

Cela dépend de la façon dont cette personne est partie et de votre relation avec elle.

De plus, que vous importez? Je vois que vous voulez l'aider à "apprendre de l'erreur", mais êtes-vous vraiment? Allez-vous lui montrer les journaux * et la trace de la pile *? Allez-vous lui montrer les étapes que vous avez suivies pour diagnostiquer le problème? Allez-vous lui montrer la source * pour qu'il puisse voir où était le problème?

Sinon, vous perdez probablement son temps et le vôtre.

* Allez-vous avoir des problèmes pour divulguer les actifs / données de l'entreprise à un non-employé?

Si vous décidez de le lui dire, assurez-vous de le dire à tous les réviseurs de son code! Ils sont également responsables! Pour moi, il semble que vous ne vous entendiez pas avec ce gars et que vous vouliez le fouiller. Laisse tomber, il ne s'en souciera probablement pas.

Probablement pas

Cela me semble presque inutile, que ce soit des amis ou des collègues. Et, dans certaines circonstances, peut-être nocif pour eux, pour vous et pour votre relation avec eux.

Nous faisons tous des erreurs occasionnelles.

En fait, le seul facteur qui me donnerait envie de dire à mes collègues est ceci: est-ce une erreur que je sais qu'ils ne feraient pas habituellement / une situation que je sais qu'ils sauraient gérer?

Si la réponse est oui, il n'est pas nécessaire de les bogue car il n'y a probablement aucune valeur éducative pour eux, donc je ne vois pas le devoir de les informer. Si vous les rencontrez un jour ou prévoyez de prendre un verre le dernier jour et que vous entretenez de bonnes relations avec eux en tant que pairs et collègues professionnels, bien sûr, vous pouvez le mentionner, davantage pour nourrir des plaisanteries amicales ou inoffensives qu'autre chose.

Si la réponse est non, il pourrait y avoir une obligation (ne l'appellerait pas "professionnelle", cependant) de tendre la main et de l'aider à comprendre son erreur.

Gardez-le civil

La plupart des gens n'aiment pas les critiques sur leur travail en général, les développeurs / programmeurs encore moins, et les programmeurs sortants auraient probablement une tolérance encore plus faible. Pourquoi prendre le risque de les agacer et de leur donner l'impression qu'ils partent sur une mauvaise note?

Bien sûr, s'ils étaient de mauvais employés partout, cela ne s'applique pas, mais s'ils étaient par ailleurs des collègues progammeurs suffisamment qualifiés, je ne vois pas pourquoi je ferais tout mon possible pour souligner leurs erreurs, sauf si je peux être sûr que nous peut à la fois rire. Encore une fois, en supposant qu'ils n'apprendraient pas grand-chose de cela et seraient simplement mortifiés d'avoir laissé cela derrière.

Légal?

Sous un angle d'approche différent, s'ils ont quitté l'entreprise, cela dépend vraiment de votre contrat et des politiques de sécurité de votre entreprise. Vous ne serez peut-être pas autorisé à transmettre le code (ou d'autres choses, d'ailleurs) à d'anciens collègues.

Penser positivement

Enfin, je pense que les seules situations où j'ai contacté un ancien collègue pour discuter d'une base de code qu'ils ont laissée étaient:

• pour demander une confirmation sur quelque chose de louche tout en recherchant une zone particulière du code,
• pour les féliciter pour un morceau de code que j'ai trouvé particulièrement magistral et qui aurait empiré ma vie s'il n'y avait pas,
• de partager avec eux la bonne nouvelle d'un lancement réussi s'ils partaient avant que cela ne se produise (ou de grandes annonces similaires concernant un produit sur lequel ils travaillaient).

Apprenez de leurs erreurs

Ce que vous pouvez sûrement faire, c'est signaler l'erreur au reste de l'équipe, pour vous assurer qu'elle ne se reproduise plus avec les autres membres. Pas besoin de signaler l'erreur réelle dans SCM ou l'auteur, ce n'est pas un jeu de blâme.

Cela sort du cadre de la question, mais je précise tout de même que vous devez vous assurer de corriger l'erreur, documenter ses origines, ses impacts et ses résolutions, et mettre en œuvre un test pour qu'elle ne réapparaisse pas, si possible.

Il n'est peut-être pas légal de le dire à quelqu'un. À moins que le code ne soit open source, laissez les chiens endormis mentir.