6

Tenho uma classe para fazer buscas pelo DisplayName das minhas entidades, onde passo a entidade e classe retorna uma lista com os valores e nome reais de cada atributo da entidade.

Meu problema em si é quando faço a busca e algum atributo na entidade não possui a anotação DisplayName. recebo o erro de 'System.NullReferenceException'.

Eu tentei colocar where para quando encontrar algum valor null não listar, mas meu erro ocorre quando instancio o DisplayName:

.Where(p => p.GetCustomAttribute<DisplayNameAttribute>().DisplayName != null)

Como posso resolver esse problema? pois terei entidades com alguns atributos sem displayname e não quero que sejam selecionados.

Segue os códigos:

PesquisaAvançada

public class PesquisaAvancada
{
    public String Valor { get; set; }
    public String Texto { get; set; }

    public static List<PesquisaAvancada> camposPesquisa<T>()
    {
        return typeof(T).GetProperties(BindingFlags.Instance | BindingFlags.Static | BindingFlags.Public |
                       BindingFlags.NonPublic | BindingFlags.FlattenHierarchy).Where(p => p.GetCustomAttribute<DisplayNameAttribute>().DisplayName != null).
                                                                             Select(p => new PesquisaAvancada()
                                                                            {
                                                                                Valor = p.Name,
                                                                                Texto = p.GetCustomAttribute<DisplayNameAttribute>().DisplayName
                                                                            }).ToList();
    }
}

Entidade Cliente

public class Cliente
{
    [Key]
    [DisplayName("Identificador do Cliente")]
    public Guid ClienteId { get; set; }

    [Required]
    // aqui removi o display name e acusa o erro
    public String Nome { get; set; }
    [Required]
    [DisplayName("Nome Fantasia")]
    public String Fantasia { get; set; }
}

Erro:

An unhandled exception of type 'System.NullReferenceException' occurred in Model.dll

Imagem para melhor entendimento:

inserir a descrição da imagem aqui

3 Respostas 3

6

É só usar o null-conditional operator do C#.

Se o retorno de p.GetCustomAttribute<DisplayNameAttribute>() for null não será feita a tentativa de acesso à propriedade DisplayName e o resultado será null.

.Where(p => p.GetCustomAttribute<DisplayNameAttribute>()?.DisplayName != null)

Duas perguntas que pode ser interessante dar uma olhada:

  1. O que é o operador "?."?

  2. Qual o significado do operador "??"

Veja o exemplo abaixo funcionando no .NET Fiddle.

using static System.Console;

public class Program
{
    public static void Main()
    {
        var customAttrName = GetCustomAttribute()?.DisplayName; 
        // Isto não estoura erro, ao invés disto, 'customAttrName' recebe null

        WriteLine(customAttrName ?? "Null"); //Null será a saída
    }

    public static CustomAttr GetCustomAttribute()
    {
        return null;
    }
}

public class CustomAttr
{
    public string Name;
    public string DisplayName;
}
5
  • O downvoter pode me dizer o que está errado na resposta pra que eu possa arrumar?
    – Jéf Bueno
    Commented 25/04/2017 às 15:21
  • Engraçado esses downvoter sem justificativa, poderia ter um meio de evitar isso; Downvoter só ser valido com um comentário justificando e etc... Commented 25/04/2017 às 15:48
  • É, mas é complicado. Isso já foi bastante discutido aqui no site, mas realmente é uma situação bem delicada. De qualquer forma, eu aguardo feedback pra poder arrumar a resposta.
    – Jéf Bueno
    Commented 25/04/2017 às 15:50
  • @jbueno entendi sua colocação, quando coloco o operado ? não consigo encontrar mais o atributo disponível DisplayName utilizando o comando Ctrl+Espaço. .Where(p => p.GetCustomAttribute<DisplayNameAttribute>()?.DisplayName != null) OBS: não fui eu que negativei sua resposta. Agradeço a ajuda
    – Aprendiz
    Commented 25/04/2017 às 22:29
  • @Aprendiz Qual a versão da linguagem que você está usando?
    – Jéf Bueno
    Commented 26/04/2017 às 18:54
5

O que está ocorrendo é que você está tentando acessar uma propriedade de um objeto nulo

p.GetCustomAttribute<DisplayNameAttribute>().DisplayName != null

O certo é verificar se o GetCustomAttribute retorna um objeto válido, sendo assim a sua condição where fica:

.Where(p => p.GetCustomAttribute<DisplayNameAttribute>()!= null)

No código acima todas as propriedade que possuem o atributo DisplayName são retornadas.

2
  • Certo Tiago entendi, mas na hora do Select busco novamente o DisplayName e irá ocorrer NullReferenceException. Como posso pegar o valor na variável texto? desse local: Texto = p.GetCustomAttribute<DisplayNameAttribute>().DisplayName
    – Aprendiz
    Commented 25/04/2017 às 11:18
  • 4
    Depois que você fez esta condição where, terá uma coleção de DisplayNameAttribute que não são nulos, assim, você pode pegar a propriedade DisplayName no Select.
    – Tiago S
    Commented 25/04/2017 às 11:28
1

Esse DisplayName retorna uma string ou estou enganado ?

Tenta substituir:

p => p.GetCustomAttribute<DisplayNameAttribute>().DisplayName != null

por:

p => !string.IsNullOrEmpty( p.GetCustomAttribute<DisplayNameAttribute>().DisplayName));
1
  • 4
    Ainda assim vai ser feita uma tentativa de acesso a um membro no retorno nulo. O que vai estourar um erro.
    – Jéf Bueno
    Commented 25/04/2017 às 11:54

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .