Dois-je être gêné si mon ratio LOC / jour est trop élevé? [fermé]

Je travaille actuellement sur un projet indépendant, donc je n'ai pas exactement le luxe de tout au long des tests humains ou de la révision du code externe - cependant, je ne vois aucun bogue difficile dans mon code actuel (je les corrige comme je les vois) , et la plupart du temps ce ne sont que des noms de champs erronés et des choses que vous corrigez en une minute ou deux), et je le teste après avoir implémenté une fonctionnalité avant de la pousser. Dernièrement, mon numéro LOC était d'environ 400 par jour (pour mémoire, c'est C #), et je n'implémente pas seulement de nouveaux systèmes, mais aussi réécris des choses que j'ai déjà écrites et corrige quelques bugs.

Dois-je être dérangé? Est-ce le signe que je dois m'arrêter et revoir tout le code que j'ai écrit jusqu'à cette date et le refactoriser?

LOC est probablement l'une des mesures les plus utilisées, et en conséquence est probablement l'une des mesures les plus inutiles de la qualité du code et une mesure encore plus inutile de l'effort de programmation.

Oui, c'est une déclaration audacieuse pour moi de faire, et non, je ne peux pas vous indiquer des études prouvant mon point. Cependant, je peux affirmer avec une expérience durement acquise que lorsque vous commencez à vous soucier de la quantité de code que vous avez écrit, vous vous inquiétez probablement des mauvais problèmes.

Vous devez d'abord vous demander ce que vous essayez de mesurer ou de prouver, et si cette preuve est simplement hors d'intérêt, ou pour soutenir une amélioration de la qualité plus large et où vous devez utiliser ces informations pour obtenir l'adhésion de votre équipe. / la direction pour faire quelque chose.

L'une des choses pour lesquelles j'ai tendance à utiliser LOC est un peu de santé mentale. Si je me retrouve à écrire beaucoup de code, je m'intéresse davantage au LOC par méthode, ou au LOC par classe, plutôt qu'au LOC dans l'ensemble. Ces mesures peuvent être des indicateurs que vous devez refactoriser davantage si vous vous sentez un petit trouble obsessionnel-compulsif sur la façon dont votre code devrait être factorisé. Les classes très volumineuses peuvent avoir besoin d'être refactorisées en quelques classes plus petites, et les longues méthodes multilignes peuvent devoir être décomposées en plusieurs méthodes, d'autres classes, ou peuvent même indiquer une répétition qui pourrait être supprimée. Remarquez que j'ai utilisé le mot «pourrait» plusieurs fois là-bas.

La réalité est que LOC ne fournit qu'un indicateur possible et aucune garantie réelle que votre code puisse avoir besoin de changer. La vraie question à se poser est de savoir si le code se comporte comme requis et comme prévu. Si c'est le cas, votre prochaine question est de savoir si vous serez en mesure de maintenir le code facilement et si vous aurez le temps, maintenant ou à l'avenir, d'apporter des modifications au code de travail pour réduire vos frais de maintenance à l'avenir.

Souvent, beaucoup de code signifie que vous aurez plus à gérer plus tard, mais parfois même un code bien factorisé peut s'étendre sur des centaines de lignes de code, et oui, vous pouvez parfois vous retrouver à écrire des centaines de lignes de code en une journée. L'expérience me dit cependant que si je soutiens une sortie de centaines de lignes de nouveau code chaque jour, il y a souvent un risque qu'une grande partie du code soit copiée et collée de manière inappropriée ailleurs, et cela en soi peut indiquer des problèmes avec la duplication et la maintenance, mais là encore ce n'est pas une garantie, j'ai donc tendance à compter sur ce que mon expérience et mon instinct me disent en fonction de la façon dont les tâches à accomplir ont été accomplies.

La meilleure façon d'éviter le dilemme posé dans votre question à mon humble avis est d'oublier le LOC et de refaçonner TOUT le temps. Écrivez d'abord votre test de code, implémentez pour échouer, refactorisez pour passer, puis voyez ce qui peut être refactorisé et ensuite améliorez le code. Vous quitterez la tâche en sachant que vous avez déjà revérifié votre travail, et vous ne serez pas si inquiet à l'idée de vous remettre en question à l'avenir. De manière réaliste, si vous utilisez une approche de test d'abord comme je l'ai décrit, toute mesure LOC / jour sur votre code terminé signifie vraiment que vous avez écrit 3 à 5 fois la quantité mesurée, cet effort étant masqué avec succès par votre refactoring en cours efforts.

Pas du tout - certains jours, vous corrigez un bug difficile à trouver et ne changez qu'une ligne. D'autres jours, vous ajoutez du nouveau code et écrivez plusieurs milliers de lignes.

LOC LOC ne vous dit rien, sauf que les tâches de cette journée pourraient être accomplies avec 400 lignes de code.

Certaines de ces réponses manquent de raison, vous n'utilisez pas LOC comme mesure de la productivité (si vous l'étiez, vous ne vous inquiéteriez pas d'être trop `` productif ''), ce que vous faites en réalité, c'est de vous soucier de la qualité de votre code car le code est l'ennemi c'est une bonne chose à craindre.

Malheureusement, la seule façon de connaître la qualité du code est la révision du code, car vous êtes une équipe individuelle, ce sera difficile, même si vous vous êtes arrêté pour réviser votre code (et vous ne voulez pas vraiment vous arrêter, non?) son propre code ne révélera pas autant qu'un pair examinant votre code. Je suggère d'essayer de demander à quelqu'un d'autre de revoir au moins une partie de votre code afin que vous puissiez savoir si votre 400 LOC / jour produit du charabia ou non. Même un examen indépendant du code d'un jour vous aidera ici

Vous ne devriez pas être dérangé par le nombre de LOC que vous produisez par jour.

Mais vous devriez être dérangé:

• si votre code n'est pas testé (si par exemple, vous n'avez pas de tests unitaires)
• si vous commencez à avoir des difficultés à ajouter de nouvelles fonctionnalités ou à changer les fonctionnalités implémentées (cela signifie que votre refactoring n'était pas correct)
• si votre expérience n'est pas importante et que votre code n'est pas révisé (une paire d'yeux supplémentaire est susceptible de détecter des problèmes)

LOC est une mesure `` officielle '' de la productivité, mais les arguments contre sa valeur pourraient être longs (un ORM pourrait générer 50000 lignes de code en 3 minutes, cependant, si la conception de la base de données est incorrecte, tout ce code peut aller dans le bac).

Je vous suggère de mesurer vos progrès en suivant le% de tâches terminées par rapport au temps vs% de tâches planifiées. C'est ça qui compte. Les clients paient pour un code de travail qui fournit une valeur commerciale et non pour les LOC.

Quelques références sur LOC

• Contre: LOC-mesure la plus dénuée de sens

• Neutre dans une certaine mesure: Wiki-LOC

• Pro (je suis en désaccord avec la plupart des choses mais, vous pouvez y jeter un œil): déséconomies d'échelle et lignes de code

Mesurez-vous également le nombre de doublons de code ?

Si le rendement élevé est dû au fait que vous avez beaucoup de copier-coller dans votre code, vous devez vous inquiéter.

Raison: dans le cas où il y avait une erreur dans votre source copier-coller, il est difficile et sujet aux erreurs de corriger toutes les utilisations du copier-coller

Si vous croyez au beau code fonctionnel, cela devrait être votre seule mesure

"Est-ce que ça coule? Est-ce que c'est beau?"