-2

Estou desenvolvendo uma aplicação web onde um fotografo faz upload de uma imagem. Após upar a imagem, vou varrer todo o metadata da imagem afim de capturar algumas informações dos diretórios (Exif,IPTC,XMP,JFIF,PNG,BMP,JPG etc.), estou utilizando a biblioteca Metadata Extractor. Ok, aprendi a usá-la. Primeiramente eu preciso receber os diretórios:

 IEnumerable<MetadataExtractor.Directory> directories = iptcReader.ReturnMetadataFile(Path.Combine(StorageRootPOOL, imagePool.File_name[i]));

Aqui eu já tenho todos os diretórios da minha imagem, após isso preciso declarar variáveis de cada diretório que preciso capturar no meu caso:

var IPTCDirectory = directories.OfType<IptcDirectory>().FirstOrDefault();
var JPEGDirectory = directories.OfType<JpegDirectory>().FirstOrDefault();
var JFIFDirectory = directories.OfType<JfifDirectory>().FirstOrDefault();
var EXIFDirectory = directories.OfType<ExifIfd0Directory>().FirstOrDefault();
var EXIFSubDirectory = directories.OfType<ExifSubIfdDirectory>().FirstOrDefault();

Todos os valores dos diretórios NÃO SÃO NULOS mesmo que não exista o diretório na imagem, o problema está nas tags que existem dentro deles. Para capturar as tags dentro dos diretórios eu utilizo o seguinte código:

imageFileInfo.Keywords = IPTCDirectory.GetDescription(IptcDirectory.TagKeywords);

imageFileInfo é a classe onde estarei guardando todos os dados. O meu problema é, se por exemplo não existir a TAG IptcDirectory.TagKeywords eu irei receber um NullException, até ai nada fora do comum, porém se eu vou prevenir esse erro com um if por exemplo:

 if (IPTCDirectory.GetDescription(IptcDirectory.TagKeywords) == null)
 {
 }

Eu recebo um outro NullException, aí está o problema, eu queria definir um valor padrão para as tags que não existem, por exemplo:

if (IPTCDirectory.GetDescription(IptcDirectory.TagKeywords) == null)
{
      imageFileInfo.Keywords = "valor padrão";
}

Lembrando que não adianta eu utilizar o seguinte código, porque eu preciso saber quais tags são nulas e não quais diretórios, pois os diretórios sempre existe, mas algumas tags não:

if (EXIFDirectory != null && ExifIfd0Directory != null) {
    //Os diretórios nunca são nulos e sim as tags, então esse exemplo não iria se aplica ao meu problema.
}
0

3 Respostas 3

7

Não faça em hipótese alguma o que está na outra resposta engolindo a exceção. Isto não resolve problema algum, só piora a situação. Isto é até mais terrível que capturar Exception. E usar goto é pior ainda.

Só capture exceção se precisar mesmo. Ela é lenta e só deve ser usada em situações excepcionais, não para controle de fluxo.

Se está recebendo um NullReferenceException, tem um erro de programação aí. Tem que consertar o erro. Capturar e engolir as exceções não está fazendo o código ficar certo, a solução é ele ser consertado.

Verifique com um if o que pode ser nulo e trata da forma que deseja, se quer atribuir um valor padrão, faça isto. Então pode nem precisar do if.

Você tem que ver o que está gerando a exceção e verificar exatamente isto antes do erro acontecer. Que fique muito claro sempre que recebe NullReferenceException é um erro de programação que deve ser resolvido, não se deve capturá-la. E verificar o erro antes dele ocorrer é a solução. Por isso ainda considero que a resposta aceita agora não é adequada, apesar de não ser o absurdo que a outra propunha.

Talvez seja isso que deseja:

for (int i = 0; i < numberOfProcessAvailable; i++) {
    var IptcDirectory = directories.OfType<IptcDirectoryBase>().FirstOrDefault();
    imageFileInfo.Keywords = IptcDirectory.TagKeywords == null ? "padrao keywords" : IptcDirectory.GetDescription(IptcDirectory.TagKeywords);
    imageFileInfo.LensModel = IptcDirectory.TagLensModel == null ? "padrao lensmodel" : IptcDirectory.GetDescription(IptcDirectory.TagLensModel);
    imageFileInfo.TagMake = IIptcDirectory.TagMake == null ? "padrao make" : IptcDirectory.GetDescription(IptcDirectory.TagMake);
}

Coloquei no GitHub para referência futura.

Leia isto: Como melhor tratar exceções em Java?. Mesmo sendo outra linguagem, vale igual.

20
  • Bigown, tudo bem, irei rever o que está correto ou não, porém o problema nesse caso é que existem certas tags por exemplo do EXIFDirectory que não existe, e eu caio nesse problema entende? Tem imagem que tem e imagem que não tem, para as que não tem por exemplo a tag "DateTimeOriginal" eu quero definir um valor padrão no meu imageFileInfo, você tem alguma ideia de como posso resolver isso sem usar o tratamento citado como errado? Commented 21/07/2017 às 12:14
  • O que eu quero dizer é, o diretório em sí nunca é null, mas algumas tags dentro dele sim, e eu não consigo trata-las com if pois dentro da expressão do if eu recebo uma exception null. Commented 21/07/2017 às 12:15
  • Nada disto está na sua pergunta, se dissesse o valor padrão eu colocaria no meu código. Eu dei uma base deixando de processar o item que está nulo. Seu if está completamente errado, o meu provavelmente está certo, só não garanto porque a pergunta não tem detalhes, não tem como eu testar. Conserte o if em vez de usar o mecanismo errado.
    – Maniero
    Commented 21/07/2017 às 12:22
  • 1
    Não tem nada de mais trabalhoso, a não ser que esteja se referindo ao trabalho de aprender fazer o certo, o que recomendo fortemente, exceção quase nunca é a solução, a maioria dos programadores não sabem disto, a maioria acredita ser boa prática botar exceção em tudo, o que já questiono a ideia de boa prática (pt.stackoverflow.com/search?q=user%3A101+boas+pr%C3%A1ticas), quanto mais isso ser bom.
    – Maniero
    Commented 21/07/2017 às 12:26
  • 1
    Você deu uma bela mudada na pergunta, não só adicionou detalhes. Tem coisas novas que não existiam antes e até sumiu o que tinha. Estou pensando em fechar como não clara, porque está ficando mais confuso, talvez o problema seja a confusão.
    – Maniero
    Commented 21/07/2017 às 13:33
1

Havia entendido sua pergunta a primeira vez de uma maneira diferente.

No seu caso o motivo da NullReferenceException é claro e conhecido por você e é um comportamento esperado quando uma tag não existe. Nesse caso é de seu interesse capturar a exceção para atribuir um valor padrão às tags inexistentes.

Não há nenhum problema nesse caso em você usar:

try {
    imageFileInfo.Keywords = IPTCDirectory.GetDescription(IptcDirectory.TagKeywords);
} catch (NullReferenceException e) {
    imageFileInfo.Keywords = "valor padrao"
}

Inclusive é melhor usar try catch do que if para testar se o erro existe, desse modo o processamento é reduzido quando não há erro.

Observação

Você está recebendo NullException na linha

if (IPTCDirectory.GetDescription(IptcDirectory.TagKeywords) == null)

provavelmente porque o objeto IptcDirectory não possui a propriedade TagKeywords, o jeito certo de testar seria:

if (IptcDirectory.GetProperty('TagKeywords') == null) {
    imageFileInfo.Keywords = "valor padrao";
} else {
    imageFileInfo.Keywords = IPTCDirectory.GetDescription(IptcDirectory.TagKeywords);
}

Porem esse código com if é menos eficiente que o try catch.

EDIT

Como apontado pelo Maniero, na verdade Exceções em C# são muito caras, então a estrutura if else nao é menos eficiente a princípio (tudo depende do quão caro GetProperty é em relação ao lançamento de exceções)

7
  • 1
    Por que é menos eficiente?
    – Maniero
    Commented 21/07/2017 às 13:42
  • No caso do try catch, quando não há erro, apenas a instrução imageFileInfo.Keywords = IPTCDirectory.GetDescription(IptcDirectory.TagKeywords); é executada. Já no caso do if sempre serão executadas no minimo duas instruções, a do if e a de atribuição. Commented 21/07/2017 às 13:46
  • 1
    Mas quando dá a exceção é absurdamente (ordens de magnitude) menos eficiente e ele diz que espera que tenha exceções, então não é mais eficiente. Por isso que eu sempre digo, não use exceção para controlar fluxo, use só para situações excepcionais. Toda vez que se captura NullReferenceException está tentando consertar com código algo que deve ser consertado no código. È erro de programação.
    – Maniero
    Commented 21/07/2017 às 13:52
  • 1
    @bigown, tenho mais experiencia em python onde geralmente usamos a filosofia: it's easier to ask forgiveness than it is to get permission, mas você tem razão para o caso de C# as exceções são muito caras. Vou alterar a resposta. Commented 21/07/2017 às 14:10
  • Agora sim ficou mais claro, até o momento acredito que a sua resposta seja a mais adequada. Eu não sei o peso de um try catch, mas provavelmente o if seja menos pesado então vou usar o if. Aliás, sua resposta foi muito satisfatória na duvida do "porque o if indicava que era null sendo que eu estava apenas perguntando se ele era null" Commented 21/07/2017 às 14:19
0

Nesse caso você deveria separar cada uma das instruções em blocos de try catch específicos.

for (int i = 0; i < numberOfProcessAvailable; i++) {
    var EXIFDirectory = directories.OfType<ExifDirectoryBase().FirstOrDefault();

    try {
        imageFileInfo.LensModel = Convert.ToString(EXIFDirectory.GetDescription(ExifIfd0Directory.TagLensModel));
    } catch (NullReferenceException e){ //tratar excecao }

    try {    
        imageFileInfo.Make = Convert.ToString(EXIFDirectory.GetDescription(ExifIfd0Directory.TagMake));
    } catch (NullReferenceException e){ //tratar excecao }

    try {
        imageFileInfo.Model = Convert.ToString(EXIFDirectory.GetDescription(ExifIfd0Directory.TagModel));
    } catch (NullReferenceException e){ //tratar excecao }
}

Dessa forma todos os blocos seriam sempre executados.

PS.: Você poderia tratar esse problema também utilizando a instrução goto, mas isso geralmente torna o código menos legível e mais complicado.

Ps2.: Nao tinha deixado explicito por achar obvio.. Mas as exceções devem sempre ser tratadas e ser o mais específicas o possível

6
  • 1
    Danilo, isso é uma boa prática? Commented 20/07/2017 às 19:01
  • Ate onde sei, sim. Dessa forma fica explicito que cada uma das instruções pode falhar, mas que as outras devem ser executadas. Esse código é bastante eficiente uma vez que ele não processa nada a mais caso nenhuma exceção ocorra. Commented 20/07/2017 às 19:07
  • 1
    Caso o numero de variáveis não seja conhecido ou seja muito grande você pode colocar o bloco try catch dentro de um loop que itera sobre elas. Commented 20/07/2017 às 19:10
  • 5
    Esta resposta é péssima.
    – Jéf Bueno
    Commented 20/07/2017 às 19:40
  • 3
    LINQ, uma critica mais construtiva me ajudaria a melhorar. Commented 20/07/2017 às 20:03

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