Estou criando uma classe relativamente simples, mas incluí algumas validações em suas propriedades.
public class Sala : EntidadeBase
{
#region Enums
public enum StatusSala
{
Disponivel,
Reservada,
Inativa,
Bloqueada
};
#endregion
#region Propriedades
public string Nome { get; private set; }
public int Capacidade { get; private set; }
public StatusSala Status { get; private set; }
#endregion
#region Contrutores
public Sala(string nome, int capacidade, StatusSala status) : base()
{
SetNome(nome);
SetCapacidade(capacidade);
SetStatus(status);
}
#endregion
#region Métodos
#region SettersPrivados
private void SetNome(string nome)
{
ValidacaoDominio.Validar()
.Quando(string.IsNullOrEmpty(nome), "O nome não pode ser vazio")
.Quando(!string.IsNullOrEmpty(nome) && nome.Length < 3, "o nome deve ter pelo menos 3 caracteres")
.DispararValidacao();
Nome = nome;
}
private void SetCapacidade(int capacidade)
{
ValidacaoDominio.Validar()
.Quando(capacidade < 1, "Capacidade da sala Inválida")
.DispararValidacao();
Capacidade = capacidade;
}
private void SetStatus(StatusSala status)
{
ValidacaoDominio.Validar()
.Quando(!Enum.TryParse(typeof(StatusSala), status.ToString(), out var retorno), "Status Inválido")
.DispararValidacao();
Status = status;
}
#endregion
#region Publicos
public void AlterarNome(string nomeAlterado)
{
SetNome(nomeAlterado);
}
public void AlterarCapacidade(int capacidadeAlterada)
{
SetCapacidade(capacidadeAlterada);
}
public void AlterarStatus(StatusSala statusAlterado)
{
SetStatus(statusAlterado);
}
#endregion
#endregion
Por uma questão de organização, não gostaria de deixar uma string fixa nas mensagens de validação, pois algumas podem ser repetidas (exemplo: Nome inválido. Isso pode ser referenciado para qualquer entidade que possua a propriedade Nome) e a possibilidade de fugir do padrão é grande.
Até aonde sei, posso fazer isso via resources ou criar uma classe auxiliar para concentrar essas mensagens.
Inicialmente, concentrei as mensagens em classes estáticas, separando-as por categorias (das mais gerais para as mais especificas), mas nada impede de fazer de outra forma.
Classe para mensagens "gerais":
public class GeralMsgs
{
public const string NomeInvalido = "O Nome é invalido";
public const string NomeMenorTresCaracteres = "O nome deve ter pelo menos três caracteres";
}
Classe para mensagens especifica da classe Sala
:
public class SalaMsgs
{
public const string CapacidadeInvalida = "A capacidade da sala é inválida";
public const string StatusInvalido = "O status da sala é inválido ";
}
Dessa forma, as validações iriam referenciar as constantes
ValidacaoDominio.Validar()
.Quando(string.IsNullOrEmpty(nome), GeralMsgs.NomeInvalido)
.Quando(!string.IsNullOrEmpty(nome) && nome.Length < 3, GeralMsgs.NomeMenorTresCaracteres)
.DispararValidacao();
...
ValidacaoDominio.Validar()
.Quando(capacidade < 1, SalaMsgs.CapacidadeInvalida)
.DispararValidacao();
...
ValidacaoDominio.Validar()
.Quando(!Enum.TryParse(typeof(StatusSala), status.ToString(), out var retorno), SalaMsgs.StatusInvalido)
.DispararValidacao();
Do ponto de vista de saúde da aplicação, pensando em escalabilidade e boas práticas, qual é o local correto para concentrar essa mensagens? Deixo dessa forma ou jogo tudo para um arquivo resource do projeto ou tanto faz?
Meu objetivo é sempre tentar fazer da melhor maneira possível, mas sem criar uma arma de raio laser para matar uma formiga, mas também não quero no futuro olhar para o 147 que o Maniero sempre posta e perceber que minha aplicação ou meu mindset me levou para o "Importante é funcionar".