Quel problème de code le plus simple évitable?

Mes premières pensées sont trop de lignes de code dans une méthode. Dans la plupart des cas, il est facile de séparer quelque chose en plusieurs méthodes.Quel problème de code le plus simple évitable?

J'ai beaucoup plus, mais cela semble être l'un des plus simples à éviter. Cela me dérange vraiment de voir une méthode avec 1000 lignes de code, cela semble aussi être un problème courant. Tout ce dont vous avez besoin de faire défiler vers le haut/bas est que l'OMI devrait vous supplier de le démembrer. Mais avoir des problèmes de liste pourrait aider beaucoup de nouveaux développeurs à apprendre à identifier les problèmes et à fournir quelques bonnes pratiques.

Source

2009-06-10 eschneider

Je préfère poser la question inverse, c'est-à-dire quel problème de code laid est le plus difficile à éviter? Parce que la plupart des problèmes sont assez faciles à éviter. –

Quelle est votre question? –

Je suppose que je suis curieux de savoir ce que les autres pensent sont des problèmes faciles à prévenir. Like Trop de blocs d'if/loop inclus. – eschneider

Commentaires dans le code: P

Mais sérieusement, oui, les méthodes trop longues sont un cauchemar. La réutilisation de code incorrecte, ou l'absence totale de là, est probablement ma plus grande bête cependant.

Source

2009-06-10 08:32:28 Phil

Mon animal de compagnie préféré a de nombreux niveaux d'indentation, c'est-à-dire de grandes quantités d'ifs et de boucles imbriquées.

Pire que de faire défiler vers le haut et vers le bas doit défiler sur le côté!

Ma règle générale est généralement de 5 niveaux MAX. Plus et ils devraient être refactorisés dans de nouvelles méthodes.

Source

2009-06-10 07:48:15 Cogsy

Ne pas utiliser d'exceptions pour la gestion des erreurs. Il est tellement plus simple d'utiliser des exceptions, et si moche de vérifier chaque code de retour de la fonction (et en oublier la moitié).

Source

2009-06-10 07:55:04 Jem

Je ne suis pas d'accord. http://www.joelonsoftware.com/items/2003/10/13.html –

Je le savais et je suis totalement en désaccord avec cela. – Jem

Je suis d'accord avec vous. Les méthodes longues sont les plus simples à éviter et essentielles pour un bon code. Ma règle générale est que ce que la méthode fait devrait être une chose évidente. Si un autre développeur doit demander ce que fait une méthode après avoir lu un nom de méthode, en général, cela en fait trop. Bien sûr, il peut aussi s'agir d'un mauvais nom :)

Source

2009-06-10 08:02:22 Ankur

L'injection SQL est si facilement aviodable que je ne comprends pas pourquoi elle apparaît à l'écran.

Et c'est moche, aussi.

Et je suis abasourdi par les non-solutions, comme l'appel de guillemets ou le remplacement de INSERT/UPDATE/DELETE dans l'instruction. Merde, utilisez simplement des espaces réservés!

Source

2009-06-10 08:05:02 alamar

Principe de responsabilité unique, si facile mais si souvent ne pas utilisé.

Source

2009-06-10 12:43:23

Quel problème de code le plus simple évitable?

Répondre

Questions connexes