2013-07-01 2 views
2

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?

enter image description here

+1

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'! –

+2

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. –

+0

aimerait voir ces numéros de ligne dans la capture d'écran :) –

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