Constantes
Uma enumeração é uma sequência de constantes. Em geral constantes devem ter seu estado imutável por todo ciclo de vida do projeto (as respostas nessa pergunta dão os detalhes). Eventualmente você pode adicionar algo novo no final, mas não mais que isso, mesmo assim isso nem sempre funciona bem, mesmo que muito bem documentado.
Se um dado pode ser modificado, mesmo que entre versões da aplicação o ideal é que não seja seja constante de fato, é melhor que seja apenas imutável.
π (Pi) é uma constante. Um código que depende de uma legislação não é uma constante. É um dado que no máximo não se altera durante a execução da aplicação, mas não durante todo o ciclo de vida do projeto, ele não é constante.
Dados que não são constantes já são problemáticos quando são tratados como se fossem. Fica mais complicado quando essas constantes estão em sequência. Fica mais difícil garantir o estado.
Enumerações são mecanismos de programação. Não mecanismos de compilação. Não servem para validar regras de negócio.
Esse exemplo pega parâmetros e os transforma em constantes. Grande erro.
Tem mais uma pergunta que fala sobre o abuso do enum
em banco de dados.
Controle do dado
Então uma enumeração tem que ser algo que você tem controle, que sabe que não irá mudar.
O tipo da pessoa se é jurídica ou física é algo interno seu? Você pode garantir que seja sempre isso? Não afeta nada externo? Então nenhum problema em fazer.
Ou é algo definido por legislação? Não há garantias que a legislação não vai mudar? Se você não tem controle, não opte por algo constante. Faça algo que explicitamente indique que pode mudar. Use uma estrutura de dados propícia para mudança. Mesmo que nunca mude, isso será um acaso, a semântica do dado é que ele pode ser mudado, então represente ele de forma apropriada.
O mesmo vale para o Item11
. Não tem contexto do que seja ele, mas parece ser algo que depende de legislação (a pergunta está sem contexto, muita gente não vai entender que o original fala em NFe (tem um caso pior).
A melhor explicação para isso é o tal do ID
que todo mundo usa em banco de dados. Porque você usa um ID
e não o CPF da pessoa, por exemplo? Porque nada garante que essa pessoa não vá trocar o CPF. Você cria um dado substituto para se referenciar àquilo. Você não usa o CPF porque não tem controle sobre isso. O mesmo vale para um código de produto do fornecedor, e-mail, ou coisas do tipo.
Mudanças
Se mudar e sua aplicação foi feita pensando que esses dados são constantes será um pesadelo dar manutenção nisso. Uma mudança pequenininha na legislação (e já houve mudança que alterou tudo) pode fazer com que todo o sistema pare de funcionar e para arrumar terá que refazê-lo.
Pode ser pior, uma mudança passar desapercebida. Pode parecer que está funcionando mas está gerando resultado todo errado. Imagine uma troca de ordem. Quando os dados podem mudar você precisa ter uma referência canônica dele dentro do seu sistema que não dependa de informações externas.
Faça o sistema preparado para mudanças.
Conceito correto
As pessoas precisam entender que sistemas não precisam apenas "funcionar". Eles precisam estar conceitualmente certos para suportar a manutenção futura. Se um dado faz parte de uma tabela mutável, represente-o como uma tabela mutável. Escolha um dicionário ou outra estrutura de dados mais adequada para isso. Pode até criar uma estrutura própria se for benéfico. Mas não use uma estrutura constante.
Note que esses dicionários possivelmente serão preenchidos com dados vindos do banco dedados. Ou até o banco de dados será usado para fornecer o que precisa de forma direta.
O fato de usar [System.Xml.Serialization.XmlEnumAttribute("11")]
já mostra que é o mecanismo errado. O dado real é um texto. Uma enumeração não representa textos, ela representa números. Isso é uma tentativa de arrumar o erro inicial. E é mais um erro. Daí concluímos que erros atraem erros. Só fica pior.
Nem falei sobre a legibilidade do código. Acha que uma enumeração chamada TNFeInfNFeDetImpostoICMSICMS00CST
serve para alguma coisa no código? Alguém consegue dizer do que se trata sem medo de errar?