Mon MainViewModel
doit faire face à beaucoup de commandes avec des actions complexes à l'intérieur, donc son extension est devenue inévitable. Pour garder le code organisé, j'ai tendance à utiliser #region
pour regrouper des tâches similaires. Cela est-il considéré comme un abus de cette fonctionnalité ou est-ce normal? Vous pouvez dire que cela dépend de moi, si je me sens bien à ce sujet. Je pense que ça aide beaucoup mais j'aimerais savoir ce que font les autres. Voici une capture d'écran de la façon dont le code ressemble:L'utilisation "massive" de #region est-elle considérée comme une mauvaise pratique?
2
A
Répondre
7
La mauvaise pratique est une classe massive unique. #region est simplement en train de se cacher/d'y faire face. S'il existe des groupes de méthodes, déléguez-les à une autre classe.
Questions connexes
- 1. L'utilisation de Perl externe dans une application Coco serait-elle considérée comme une mauvaise pratique?
- 2. La substitution d'une méthode pour passer une constante statique est-elle considérée comme une mauvaise pratique?
- 3. Pourquoi la création de propriétés protégées dans les haricots est considérée comme une mauvaise pratique?
- 4. L'inclusion à la volée de .js (en utilisant jQuery) serait-elle considérée comme une mauvaise pratique?
- 5. L'utilisation de "NOT EXISTS" est-elle considérée comme une mauvaise pratique SQL?
- 6. La définition de propriétés sur l'objet Window est-elle considérée comme une mauvaise pratique?
- 7. QScriptEngine: mauvaise expression considérée comme valide
- 8. Javascript: $$ Variable globale considérée comme mauvaise?
- 9. Partial-moqueur considéré comme une mauvaise pratique? (Mockito)
- 10. Est-ce considéré comme une mauvaise pratique en C#?
- 11. est @unlink une mauvaise pratique?
- 12. Bonne pratique ou mauvaise pratique
- 13. L'instanciation d'une collection dans un modèle de domaine est-elle considérée comme une bonne pratique?
- 14. L'utilisation d'une ImageMap pour un menu de navigation est-elle considérée comme une bonne pratique?
- 15. Node.js route paramétrée - bonne ou mauvaise pratique?
- 16. Mauvaise pratique en PHP?
- 17. rel = "tooltip", est-ce une mauvaise pratique?
- 18. Pourquoi SendKeys est-il une mauvaise pratique?
- 19. est une interface implémentée considérée comme une superclasse de l'implémentation?
- 20. Téléphone-maison pour la vérification de la licence considérée comme mauvaise
- 21. Est-ce une mauvaise pratique? C++
- 22. href avec "#", est-ce une mauvaise pratique?
- 23. Est-ce que l'encapsulation de tout mon site Web php dans isLoggedIn() et hasPermission() est considérée comme une mauvaise pratique de codage?
- 24. Est-il considéré comme une mauvaise pratique de placer une boucle infinie à l'intérieur de TThread.Execute?
- 25. Implémente la mauvaise pratique OnSharedPreferenceChangeListener?
- 26. Le tri dans une définition de carte est-il considéré comme une mauvaise pratique?
- 27. MVC bonne ou mauvaise pratique
- 28. Ma demande est-elle considérée comme reposante?
- 29. Java attribuant une nouvelle valeur à un paramètre, est-ce considéré comme une mauvaise pratique?
- 30. En Python est-il considéré comme une mauvaise pratique d'importer quelque chose uniquement pour l'exposer de manière plus pratique?
Si vous avez besoin de tant de «# régions», alors vous avez un objet de Dieu. Et c'est pire que d'avoir un tas de '# région '. Divisez ce 'MainViewModel'! –
Personnellement, je trouve qu'il encombre le code inutilement. Normalement, vous pouvez lire le code de haut en bas, comme un roman, sans problèmes. Les régions dans ce cas rendent seulement difficile à lire. En outre, cela signifie que votre classe fait beaucoup trop de choses. Quand je pense à faire une région, je la sépare dans une classe différente pour suivre le principe SRP. –
aimerait voir ces numéros de ligne dans la capture d'écran :) –