Génie logiciel code-smell

4

Manière la plus propre d'écrire des logiciels logiquement procéduraux dans un langage OO

Je suis ingénieur électricien et je ne sais pas ce que je fais. Veuillez enregistrer les futurs responsables de mon code. Récemment, j'ai travaillé sur un certain nombre de petits programmes (en C #) dont la fonctionnalité est logiquement "procédurale". Par exemple, l'un d'eux est un programme qui collecte des …

12 c# object-oriented code-quality code-smell procedural

8

L'activation et la désactivation des fonctionnalités de l'interface utilisateur (ou d'autres) sont-elles basées sur les dates, une odeur de code?

Nous avons un système horrible écrit en ASP.NET 2.0 auquel nous devons ajouter des fonctionnalités. Le problème est qu'un certain produit possède des fonctionnalités d'interface utilisateur qui doivent être activées pour les entreprises lancées après une certaine date (et d'autres désactivées), tandis que la page doit être identique pour les …

11 anti-patterns code-smell ui

5

Écrire du code testable vs éviter la généralité spéculative

Je lisais des articles de blog ce matin et je suis tombé sur celui-ci : Si la seule classe qui implémente l'interface client est CustomerImpl, vous n'avez pas vraiment de polymorphisme et de substituabilité car il n'y a rien en pratique à substituer au moment de l'exécution. C'est une fausse …

11 design unit-testing code-smell polymorphism

3

Vérification d'appel TDD Mock - est-ce un anti-modèle?

Je fais du TDD depuis un an maintenant, je m'en sens plutôt bien, j'adore mes suites de tests et tout. Cependant, j'ai remarqué que récemment, je faisais beaucoup de vérification des appels simulés. Par exemple, j'aurais un service qui aura un référentiel injecté - dans mon test unitaire, je passerais …

11 unit-testing tdd code-smell anti-patterns

4

cette façon d'appeler une fonction est-elle une mauvaise pratique?

J'ai le code suivant: public void moveCameraTo(Location location){ moveCameraTo(location.getLatitude(), location.getLongitude()); } public void moveCameraTo(double latitude, double longitude){ LatLng latLng = new LatLng(latitude, longitude); moveCameraTo(latLng); } public void moveCameraTo(LatLng latLng){ GoogleMap googleMap = getGoogleMap(); cameraUpdate = CameraUpdateFactory.newLatLngZoom(latLng, INITIAL_MAP_ZOOM_LEVEL); googleMap.moveCamera(cameraUpdate); } Je pense qu'avec cette façon j'élimine la responsabilité de savoir ce …

10 design design-patterns clean-code code-smell bad-code

4

Retravailler une fonction renvoyant un code entier qui représente de nombreux statuts différents

J'ai hérité d'un code horrible dont j'ai inclus un court échantillon ci-dessous. Y a-t-il un nom pour cet anti-modèle particulier? Quelles sont quelques recommandations pour refactoriser cela? // 0=Need to log in / present username and password // 2=Already logged in // 3=Inactive User found // 4=Valid User found-establish their …

10 coding-style code-quality coding-standards code-reviews code-smell

3

Nettoyer le code et les objets hybrides et l'envie de fonctionnalités

J'ai donc récemment fait quelques remaniements majeurs de mon code. L'une des principales choses que j'ai essayé de faire était de diviser mes classes en objets de données et objets de travail. Cela a été inspiré, entre autres, par cette section de Clean Code : Hybrides Cette confusion conduit parfois …

10 java object-oriented-design data-structures clean-code code-smell

4

Quelles sont les «odeurs de code» qui sont un symptôme qu'un modèle d'écouteur d'événement est requis?

Quels sont les symptômes dans une base de code qui indiquent qu'une approche d'écoute d'événements est requise? Il me semble que lorsqu'il y a des classes qui doivent être appelées par plusieurs, non définies au moment de la conception des autres classes, vous avez besoin d'une sorte de cadre de …

10 architecture design-patterns code-smell

1

Si vous avez un morceau de logique qui doit être partagé entre deux contrôleurs, où le conservez-vous?

J'ai un ensemble de fonctions à usage unique dont j'ai besoin dans deux contrôleurs distincts. Pour le moment, je n'ai qu'un code en double et je veux m'en débarrasser. Ce code fait partie du contrôleur et n'appartient pas à ma couche de service. Où le mettriez-vous?

10 mvc code-smell

2

Tests unitaires: «C'est une odeur de code si vous refactorez et qu'il n'y a pas de collaborateurs»?

Je lis The Art of Unit Testing par Roy Osherove. Je suis à la section 7.2 Écriture de tests maintenables où l'auteur a cette note sur l'odeur de code: REMARQUE: lorsque vous refactorisez l'état interne pour qu'il soit visible par un test externe, cela peut-il être considéré comme une odeur …

9 unit-testing code-quality code-smell

2

Est-ce une odeur de code si un objet connaît beaucoup de son propriétaire?

Dans notre application Delphi 2007, nous utilisons un grand nombre des constructions suivantes FdmBasic:=TdmBasicData(FindOwnerClass(AOwner,TdmBasicData)); FindOwnerClass parcourt la hiérarchie Propriétaire du composant actuel vers le haut pour rechercher une classe spécifique (dans l'exemple TdmBasicData). L'objet résultant est stocké dans la variable de champ FdmBasic. Nous l'utilisons principalement pour transmettre des modules …

9 code-smell

4

Pourquoi il n'est pas recommandé d'avoir une propriété set-only?

Aujourd'hui, au travail, un de mes collègues a examiné mon code et m'a suggéré de supprimer une propriété de type ensemble et d'utiliser une méthode à la place. Comme nous étions tous les deux occupés à autre chose, il m'a dit de regarder la Property Designsection du livre "Framework Design …

9 design code-quality code-reviews code-smell

12

Opérateur de fusion de propriétés pour C #

L'opérateur de coalescence nulle en c # vous permet de raccourcir le code if (_mywidget == null) return new Widget(); else return _mywidget; Jusqu'à: return _mywidget ?? new Widget(); Je continue de trouver qu'un opérateur utile que j'aimerais avoir en C # serait celui qui vous permettrait de renvoyer une …

9 c# code-smell null

Questions marquées «code-smell»