Fazia tempo que não via uma pergunta tão importante no site.
No geral o mais comum é capturar as exceptions específicas e fazer algum tratamento com isso, certo?
Se fizer sentido naquele cenário, é sim.
Porém, quando o programador não cria um catch
específico para sua exception, ela acaba sendo capturada pelo Exception
genérico.
Isso acaba dando problemas, porque em alguns casos é gravado log dessas exceptions em uma tabela no banco de dados, e a informação não fica completa...
Não necessariamente, essa é uma possibilidade. Se tiver uma específica e uma genérica juntas, tem chance de ter algo errado aí.
No local específico o comum é ter tratamentos específicos. Para tratamentos genéricos o local costuma ser mais geral.
O que me veio a cabeça seria implementar uma classe que lidasse com cada uma das exception, assim eu poderia chama um método especifico via reflection.
Existem várias formas de fazer, quase ninguém foge do arroz com feijão porque não costuma ser necessário.
Inclusive você pode não usar exceção na sua API e evitar até onde dá as exceções na API de terceiros (até do .NET), tem vantagens.
Não me parece ser uma boa ideia e talvez nem tenho muito o que se fazer... O que acham?
Não vi vantagem em fazer exatamente assim, na verdade parece uma grande gambiarra para não ficar pensando na solução específica. Não é a filosofia do C#.
Assim parece mais adequado (não estou dizendo que é o ideal para qualquer caso):
catch (MyException ex) {
ex.Handle();
}
Se quiser pode ter outros catch
es, inclusive de Exception
*se fizer mesmo sentido), mas faça explicitamente, mesmo que todos tenham códigos idênticos dentro (duvido porque alguns desses métodos receberão parâmetros). Pode parecer que essas repetições são uma violação do DRY, mas DRY não é isso.
try {
//Algum trecho de código
} catch (MyException myException) {
ex.Handle();
} catch (Exception ex) {
ex.Handle();
}
Apesar de ex
parecer ser o mesmo código olhando a sintaxe, a semântica dele é diferente porque eles são de tipos diferentes.
Ter uma informação canônica de como manipular uma exceção é algo não só bom, como também fundamental. Erradas são as pessoas que repetem a mesma manipulação por todo o código. Mas só a chamada é normal.
Só não acho ideal usar reflexão. Quase sempre que usa reflexão está fazendo algo errado para economizar digitação ou fazer gambiarra para resolver um problema que não existia de fato.
Dá para fazer com switch
. Ele deve ser usado como um pattern matching de forma muito simples, o compilador lida direitinho para você sem reflexão. Não vou mostrar porque ainda é errado, é trocar uma forma de switch que tem consequências adequadas para a situação por outro que funciona, mas cria alguns problemas para alguns cenários (esse foi até o motivo de terem criado as tais filter exceptions).
O catch
sempre foi uma forma de pattern matching com uma semântica mais especializada.
E claro, se tiver algum caso que a manipulação deve ter um comportamento diferente desse padrão que criou na sua classe que centraliza as manipulações, você deve tratar isso. Então deve ter como existir uma versão diferente que não seja o Handle(MyException)
. Precisa ser um método que manipule essa mesma exceção mas de uma forma diferente:
public static void Handle(this MyException exp) {
//Aqui faço alguma coisa
}
public static void Handle(this MyException exp, string message) {
//Aqui faço alguma coisa
}
public static void HandleSpecific(this MyException exp) {
//Aqui faço alguma coisa
}
Coloquei no GitHub para referência futura.
Também poderia ter um parâmetro que indique o que fazer e internamente terá um switch
decidindo o que fazer (usando pattern matching). Ou pode ter outros mecanismos, estou apenas dando exemplos de como tratar essa situação.
Muitos dirão que nem deve centralizar isso e ter uma classe de manipulação para cada exceção que for tratar, e tem lá seu sentido, só não acho que deva ser assim sempre. Mas muita gente boa faz porque na maioria dos cenários não é problema. Não vou me estender nisso porque não é o foco, existem outras discussões sobre isso no site e em outros lugares, isto é ortogonal ao que está perguntado aqui.
Desenvolver software é mais complicado do que parece
É o que eu sempre falo, para tomar todas decisões certas precisa entender todos os conceitos de desenvolvimento e combinar todos eles de forma correta em cada caso. É bem complicado fazer isso, muita coisa não é ortogonal.
Eu sou da opinião que as exceções deveriam quase obrigatoriamente ter uma manipulação padrão, mas não foi assim que aconteceu em nenhuma linguagem, eu entendo porque (mas tem solução para o problema que seria isso), mas tem soluções melhores. Também não vou me estender.
Tirando a parte da exceção é algo até bem básico, havia uma época que quase todo programador sabia fazer a abstração e generalização de uma ação em uma função. Hoje as pessoas aprendem por receita de bolo, então elas leem um tutorial que fala para fazer de um jeito muito simples, sem pensar em todas consequências, sem estar se preocupando com o caso concreto, com as interações daquele mecanismo com outros, e aí a pessoa aprende errado. E ela faz um código errado que outras pessoas usarão como referência, e talvez ela faça um outro tutorial ou livro ou vídeo errado. Chega um ponto que a pessoa nem se dá conta que ela está ensinando errado. É como a Covid-19. Contamina tanta gente que o certo começa ser considerado errado por muitos.
É por isso que eu falo para aprender de forma estruturada. Eu tenho várias respostas aqui, especialmente sobre exceções. Se você as ler isoladamente aprenderá errado. Não foi minha intenção, mas a resposta era isolada praquela questão, não pra tudo o que precisava aprender sobre o assunto. Até agora não tinha uma pergunta sobre o que foi postado aqui, ninguém aprendeu nada sobre o DRY de manipulações de exceções.
Note que essa resposta não dá toda informação possível sobre o assunto porque a pergunta não é sobre isso, mas mostra a importância de não repetir código de manipulação quando ele é igual. Tem outros pontos a serem analisados aqui, não tome a resposta como um guia definitivo de como lidar com isso. E leia diversas outras respostas sobre o assunto.
Conclusão
Eu acho que deve esquecer essa ideia de sempre só capturar Exception
. Mas é uma solução, é criativa, se feita certinha não acho ela inviável, só não é ideal.