Sim, é. Mostra que o código está grande demais, difícil demais de organizar.
Ele existe porque precisam separar código gerado pelo IDE (Visual Studio) do código que o programador deveria mexer. Perceberam o erro e criaram uma solução melhor, o partial
, desta forma você separa de forma mais adequada e segura, prefira ele.
Se as classes são muito grandes provavelmente ela está fazendo coisas demais, provavelmente ela tem baixa coesão. Da mesma forma que métodos não devem ter muitas linhas porque isto indica que ele está fazendo coisas demais, uma classe também não deve ser tão grande.
A maioria das vezes quando as pessoas organizam por funcionalidade mostra claramente que a classe tem funções demais.
#Formas de agrupamento
Alguns agrupam a implementação de uma interface, o que muitas vezes só tem um membro (aí fica ridículo usar #region
mas as pessoas usam por consistência). Mesmo quando há vários membros, não há ganho. Não deixa de ser um agrupamento por funcionalidade.
Há o agrupamento do campo com propriedade ou evento com delegado. Também é agrupar por funcionalidade. Eles devem estar próximos, mas só.
Quando se organiza por visibilidade pode até ser um pouco melhor mas ainda mostra que tem coisas demais. Mas quando isto é necessário, em geral o partial
pode ser uma saída. Nem todos gostam.
#Sem abuso
Mas este recurso em si não melhora a qualidade do código nem faz nada adicional. Ele soluciona um problema causado pelo próprio código que dificulta a leitura do código.
Agrupar código é válido, delimitar onde eles estão é exagero.
Não é que não deva usar, não causa problemas, mas é que seu uso indica algo errado no seu código que você está querendo solucionar com isto. Fica tentador fazer errado porque existe uma "solução".
Como tudo, tem momentos para usar. Eu costumo dizer que é algo que não deveria estar na linguagem porque no fundo é um facilitador para o IDE, mesmo quando serve para colapsar código, então neste caso um comentário é suficiente, ainda que provavelmente usado para resolver o tal problema de código muito grande.
Mas se você me perguntam onde eu uso: em lugar algum. Eu prefiro reduzir o código ou quando não é possível mesmo, separo o que é API e o que é detalhe de implementação em arquivos separados.
Já é ruim para agrupar métodos, fica pior se usar dentro de métodos. Embora eu já ache que as pessoas costumam usar regions brancas dentro de métodos. Quando se costuma pular linhas em branco para separar partes de código do método, está se criando regiões e provavelmente elas não são necessárias ou indicam que o método está grande demais, mas parece ser um gosto da maioria das pessoas.
StyleCop SA1124: you should not use regions
Minha experiência pessoal de ficar abrindo e fechando as regiões foi terrível, atrapalhava mais que ajudava. Já pedi demissão porque exigiam o uso deste recurso (ok, não foi o único motivo). Eu aprendi a pensar melhor no código quando eu vejo necessidade de uso de #region
. E vejo com frequência. Prefiro evitar o caminho mais curto.
Faz parte da especificação da linguagem. Se fosse só um recurso do IDE poderia usar um comentário. No fundo o resultado é o mesmo mas o compilador tem que tratar isto.