Discussion :

[Sake_reflex]

Bonjour,
J'ai une classe représentant une activité d'environs 700 lignes. Je trouve que cela fait beaucoup même si j'ai déclaré dedans deux adapter, un wrapper et que je n’hésite pas à sauter des lignes pour mettre en forme le code.
Je sais qu'il n'existe pas vraiment de moyenne et que tout dépend du projet mais vous avez combien de lignes un peu près ?
Vous trouvez vous aussi que > 700 lignes ça commence à faire beaucoup ?

[Mako 5013]

Bonjour,

Vous trouvez vous aussi que > 700 lignes ça commence à faire beaucoup ?

J'ai envie de dire qu'il vaut mieux 700 lignes claires et aérées plutôt que 350 qui soient un sac de nœuds, avec des lignes indescriptibles car réduites à l'extrême.

Le nombre de lignes en soi (700) ne me choque pas plus que cela, même si dans l'absolu, cela ne veut pas dire grand chose. Si c'est pour au final additionner deux entiers, c'est un peu beaucoup, par contre, si c'est pour une tâche un peu complexe et qui les nécessite, pourquoi pas.

En bref, perso, cela ne me semble pas "faire beaucoup" même si cela dépend très fortement du fonctionnel associé à ta classe.

Mako.

[Agriesean]

ce qui compte ce n'est pas le nombre de lignes mais la lisibilité des lignes. Je suis étudiant stagiaire mais je code toujours de telle sorte que si un collègue doit prendre mon code demain et que je ne suis pas là, il puisse comprendre et s'en sortir. Je fais certe plus de lignes mais pour la lisibilité c'est nettement plus agréable.

[Feanorin]

Je fais certe plus de lignes mais pour la lisibilité c'est nettement plus agréable.

Pas tout le temps, par exemple lorsque une fonction fait 500 lignes, cela n'est pas forcément agréable à la décrypter ( je ne peux pas afficher ces 500 lignes en une fois sur mon écran ), c'est quand même mieux de segmenter son code, pour qu'un prochain développeur puisse retrouver rapidement les grandes lignes de la fonction.

Pour revenir sur l'exemple

J'ai une classe représentant une activité d'environs 700 lignes. Je trouve que cela fait beaucoup même si j'ai déclaré dedans deux adapter, un wrapper et que je n’hésite pas à sauter des lignes pour mettre en forme le code.

A toi de voir si tu penses que tes adapter méritent toujours d'être dans cette classe ou si tu dois créer un nouveau fichier les incorporant en vue d'une meilleure maintenance de ceux-ci .

[Sake_reflex]

Bonsoir,
Merci pour vos réponses, je vais laisser le code comme cela mais en mettant un peu plus de commentaires

[antoine-renard]

Hello,

Plus que de vouloir à tout prix réduire le nombre de lignes, réfléchis toujours à ce qui pourrait être factorisable ou bien réutilisable. D'expérience, lorsqu'on écrit de longs bouts de code et que l'on prend un peu de recul, on parvient souvent à identifier (quitte à remodifier le code) des points capitalisable (un helper, un composant générique...) ayant vocation à être de fait sorti de ton fichier initial, et qui t'éviteront de réinventer la roue par la suite

Beaucoup de lignes peut parfois aussi vouloir dire réinventer la roue alors qu'il existe en standard quelque chose dans le sdk où une librairie qui pourrait simplifier ton code (ce qui arrive fréquemment quand on débute sur une techno, comme moi avec Androïd

Privilégie la lisibilitè et la simplicité pluuôt qu'un code trop concis et ésothérique (sauf lorsqu'il y a une raison précise comme un impératif de performances/ optimisation etc...).Dans ce cas, de bons commentaires sont indispensables.
Bref du bon sens
bonne continuation à toi