Comment codez-vous sans offenser?
Ce que je veux dire par là est, comment allez-vous développer sur une base de code que vous partagez avec les développeurs qui y travaillent depuis des années et sont très familiers?
Je ne veux pas marcher sur des orteils d'autre, mais je ne reçois pas si subtiles plaintes sur la façon dont je fais des choses, que ce soit comment je blanchis mon code, ou à quel point je commence souvent à SVN (trop souvent). Donc, alors que je peux changer ces choses facilement - je veux être un meilleur développeur d'équipe en général.
Je ne suis pas sûr de quoi faire, autre que de demander, mais peut-être que vous avez des pensées que je pouvais mettre en pratique.
MISE À JOUR
Il n'y a pas de guide de style pour parler - ce sont juste des gens ne sont pas utilisés pour partager le codebase. Tout le monde a leur propre petit monde siloné.
C'est un atelier de perl, mais je suis sûr que ceux-ci s'appliquent à n'importe quelle langue
Mise à jour 2
Le CTO qui est devenu plus tard le PDG était un mégalomane complet et était la principale source de ces plaintes. Si vous n'avez pas fait les choses exactement comment il a aimé, qu'il s'agisse d'un Mac, d'emacs, ou de 4 espaces d'onglets au lieu de 2, ou habillez-vous d'une certaine manière, vous étiez inférieur. C'était une situation horrible que j'ai essayé de corriger, mais la seule réponse correcte pour moi était de partir.
Je suis convaincu que c'était une instance de intimidation sur un lieu de travail et, par la suite, je suis plus conscient de ce qui pourrait être une intimidation subtile et un comportement inapproprié dans un environnement de travail.
à tout développeur à la recherche de réponses à une situation comme celle-ci, laissez-le immédiatement. Vous ne pouvez pas vous rendre à votre sortie d'une mauvaise équipe.
Demandez. C'est, demandez aux personnes avec lesquelles vous travaillez. Faites de votre mieux pour coller au style établi du code existant. Demandez surtout s'il existe une liste de documents des normes de codage et suivez-la. S'il n'y en a pas, écrivez un premier projet en fonction de ce que vous observez dans le code, puis demandez aux autres membres de l'équipe de la critiquer. Vous allez faire de l'entreprise (et les nouveaux développeurs qui viennent après vous) un service en commençant à documenter les pratiques de codage acceptées. Le seul risque est éventuellement se faire prendre au milieu s'il s'avère que les anciens ne sont pas d'accord sur ce qui est ou n'est pas acceptable.
En outre, n'ayez pas peur de soyez vous-même. Vous serez peut-être le nouveau gars, mais vous êtes membre de l'équipe et vos opinions sont valides. Si vous pouvez penser à de meilleures façons de faire quelque chose, suggère-le. Respectez les autres membres de l'équipe et la manière établie de faire des choses, mais ne les laissez pas vous pousser. La société n'aurait pas embauché si elle n'a pas valorisé votre contribution.
Il vous aidera beaucoup si vous pouvez trouver quelqu'un de l'équipe qui semble amicale et surtout prêt à répondre aux questions. (Si c'est une bonne équipe, cela devrait être tout le monde, mais les équipes ne sont pas toujours comme ça.) Votre patron peut avoir assigné à quelqu'un pour vous aider à démarrer. Utilisez cette personne en tant que ressource. Notez des questions comme elles vous appartiennent, puis demandez à cette personne utile de leur répondre de temps en temps.
En ce qui concerne la vérification du code "trop souvent", pourquoi ne pas créer votre propre branche pour vos engagements périodiques, puis fusionner à la coffre lorsque votre code est prêt? Il n'y a pas de mal à personne d'autre à faire, et quand vos collègues que vous voyez obtenir des avantages de SVN qu'ils aimeraient, ils pourraient suivre votre exemple.
En ce qui concerne WhitSpace: faites-le simplement le code le fait déjà. Aller avec le flux.
En ce qui concerne Svn Checkins: les documenter très clairement. Cela aide les gens à comprendre ce qui se passe dans le code. (Suivi: Quelles sont leurs objections aux checkins fréquents?)
En général: Commencez à construire un document standard de codage. N'essayez pas de le remplir avec 100 règles. Il suffit d'ajouter des règles à mesure qu'ils montent.
Aussi: poser des questions de vos collègues développeurs. Cela leur donne l'occasion de peser avant de faire quelque chose qu'ils n'aimeront pas. De plus, il construit des relations. De plus, vous en apprendrez plus sur la façon dont le magasin fonctionne.
En ce qui concerne le style de formatage du code (espaces blanche, onglets, où vont les accolades, etc.), vous devez suivre la norme dominante dans le code. S'il n'y en a pas un, je ne pense pas qu'ils ont beaucoup à se plaindre. Lorsque cela revient, que vous mettiez des bretelles sur leur propre ligne ou non, mettez des espaces autour des listes de paramètres de méthode, etc. est la préférence personnelle et vous devriez céder au style dominant car à long terme, il Peu importe pas. Ce qui compte est cohérent.
En ce qui concerne la vérification du code dans SVN, j'essaierais d'évangéliser ce que je ressens est la bonne façon de faire des choses, plutôt que de me laisser steamrold. Je ne vérifie pas mon code à moins que cela ne construit et ne passe pas des tests, et si je fais plusieurs changements non liés, je brise mon travail en plusieurs engagements. Si quelque chose sera cassé pendant un moment, je crée une succursale et fais mon travail là-bas. Commettre des commentaires descriptifs. Cela fonctionne mieux dans mon expérience que le "chèque dans une pile de changements le vendredi à 5 h 00", et il semble être la "meilleure pratique" dominante selon la plupart de ce que j'ai lu ici et ailleurs.
Lisez d'abord leur document de conventions de codage (s'ils n'en ont pas, demandez-leur d'en écrire un afin que vous puissiez le suivre)
Puis prenez notification et faites un effort conscient de suivre cela et de ce qu'ils disent. Il peut vous sembler qu'ils sont difficiles, mais les normes de codage sont importantes, il est préférable de souligner maintenant ce qui ne va pas plutôt que de la laisser se développer dans un problème plus tard lorsque vous apportez des changements plus importants.
Je suis sûr que vous allez faire la même chose dans un an ou deux fois où un nouveau-shirt marche sur vos orteils :)
Demandez les normes de code de l'entreprise. Portez plus d'attention aux détails. Si vous voyez d'autres suivent une forme très spécifique d'espace blanc et de motifs de support, puis suivez-les. En tant que Sr. Vous pourriez faire valoir cela pour être Nit-Picky, mais comme un Jr. ou un nouveau gars sur un projet, vous devez montrer que vous pouvez suivre avant de pouvoir mener.
En outre, comprenez que tous les nouveaux développeurs d'un projet auront effectivement un nécessaire "Ramp Up". Alors ne vous inquiétez pas sur cela.
La meilleure chose à faire est de suivre les conseils qu'ils vous donnent. Il n'y a aucun moyen de dire à l'avance ce qu'ils veulent. Sauf si vous êtes psychique.
Cependant, je suggérerais que les gens vous donnent des conseils, vous le documez. Avez-vous un wiki? Si oui, utilisez-le. Sinon, un fichier texte enregistré avec le code source pourrait être une bonne idée. Construire un guide de programmation bien organisé. Cela vous aidera à vous rappeler comment faire des choses, et si quelqu'un contredit des conseils antérieurs, cela vous donne un point de référence pour discuter de l'incohérence. De plus, lorsque la personne suivante rejoint l'équipe, elles n'auront pas à parcourir ce que vous traversez.
Je suggérerais que vous n'attribuiez pas les conseils du document aux individus (afin que les blocs de code doivent être en retrait par trois espaces ", ne me dit pas que" Bill m'a dit que les blocs de code devraient être en retrait par trois espaces "). Toutefois, si vous pouvez enregistrer cette information de manière discrète (par exemple dans le commentaires de commettre, écrivez "une règle ajoutée sur l'indentation basée sur les conseils de projet de loi"), il peut être utile de résoudre les contradictions, car vous pouvez immédiatement avoir deux points de vue. sur quelque chose. Ce que je pense ici, c'est que si vous recevez des conseils contradictoires, vous devez éviter de devenir un football à frapper par deux collègues qui sont en désaccord sur quelque chose. C'est un peu une approche de couverture-votre-cul, mais cela pourrait être prudent.
Un facile est de trouver le guide de style et de le suivre. La plupart n'ont rien de trop offensant et vous empêcheront d'offenser les autres.