Skip to main content
adicionou 121 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

É. Péssimo :)

Primeiro precisaria ver se aquilo realmente é necessário lançar a exceção. Vejo muitos casos onde a exceção não é o mais adequado (ver mais aqui e aqui).

Talvez seja um caso de exceção vexatória/ruidosa (ver o link aí em cima). Talvez seja uma exceção estrangeira, o que aí seria normal.

Se está tudo normal com o lançamento então tem que perguntar-se porque não fazer nada. Esta é a ação esperada? Não vai avisar, não vai logar, não vai tomar um rumo diferente, escolher uma alternativa, nada assim? Vai fazer de conta que nada aconteceu?

Se considerou tudo isto e tem certeza que fazer nada é o certo então está tudo ok. Como tudo, regras fixas valem nada, tem que fazer o que é certo naquele caso.

É difícil dizer que exista um caso onde engolir a exceção sem fazer algo seja o certo, mas pode existir. Quando isso é normal talvez a exceção seja o mecanismo errado. O nome exceção não é assim por acaso, deve ser algo excepcional, não normal. Eu já vi gente defendendo que exceção não deve ser usada para situações excepcionais, bota camisa de força em uma pessoa dessas.

Esse caso parece estranho que possa ser ignorado. "Não era tão importante enviar o e-mail quanto que a solicitação fosse cadastrada", jura? Então não mande-o.

Pode ser chamado de anti-padrão, mas eles estão aí para serem usados também. Isto se chama exception swallowing ou error hiding.

Tem o risco de estar fazendo algo errado :P Se for o correto para essa situação não há problema. Tenha certeza que em nenhum momento o fato da falha não ter um tratamento não vai causar um outro problema.

Tem um problema neste caso específico, está capturando Exception. Isso é muito errado, porque ele engolirá exceções que precisam ser tratadas, que podem até serem erros de programação (trapalhada), aí é grave. Se é para engoli-la é mais importante ainda que capture uma bem específica.

E se a exceção lançada não for tão específica assim? Ache outro fornecedor. Esse não presta :P

Coloquei no GitHub para referência futura.

É. Péssimo :)

Primeiro precisaria ver se aquilo realmente é necessário lançar a exceção. Vejo muitos casos onde a exceção não é o mais adequado (ver mais aqui e aqui).

Talvez seja um caso de exceção vexatória/ruidosa (ver o link aí em cima). Talvez seja uma exceção estrangeira, o que aí seria normal.

Se está tudo normal com o lançamento então tem que perguntar-se porque não fazer nada. Esta é a ação esperada? Não vai avisar, não vai logar, não vai tomar um rumo diferente, escolher uma alternativa, nada assim? Vai fazer de conta que nada aconteceu?

Se considerou tudo isto e tem certeza que fazer nada é o certo então está tudo ok. Como tudo, regras fixas valem nada, tem que fazer o que é certo naquele caso.

É difícil dizer que exista um caso onde engolir a exceção sem fazer algo seja o certo, mas pode existir. Quando isso é normal talvez a exceção seja o mecanismo errado. O nome exceção não é assim por acaso, deve ser algo excepcional, não normal. Eu já vi gente defendendo que exceção não deve ser usada para situações excepcionais, bota camisa de força em uma pessoa dessas.

Esse caso parece estranho que possa ser ignorado. "Não era tão importante enviar o e-mail quanto que a solicitação fosse cadastrada", jura? Então não mande-o.

Pode ser chamado de anti-padrão, mas eles estão aí para serem usados também. Isto se chama exception swallowing ou error hiding.

Tem o risco de estar fazendo algo errado :P Se for o correto para essa situação não há problema. Tenha certeza que em nenhum momento o fato da falha não ter um tratamento não vai causar um outro problema.

Tem um problema neste caso específico, está capturando Exception. Isso é muito errado, porque ele engolirá exceções que precisam ser tratadas, que podem até serem erros de programação (trapalhada), aí é grave. Se é para engoli-la é mais importante ainda que capture uma bem específica.

E se a exceção lançada não for tão específica assim? Ache outro fornecedor. Esse não presta :P

É. Péssimo :)

Primeiro precisaria ver se aquilo realmente é necessário lançar a exceção. Vejo muitos casos onde a exceção não é o mais adequado (ver mais aqui e aqui).

Talvez seja um caso de exceção vexatória/ruidosa (ver o link aí em cima). Talvez seja uma exceção estrangeira, o que aí seria normal.

Se está tudo normal com o lançamento então tem que perguntar-se porque não fazer nada. Esta é a ação esperada? Não vai avisar, não vai logar, não vai tomar um rumo diferente, escolher uma alternativa, nada assim? Vai fazer de conta que nada aconteceu?

Se considerou tudo isto e tem certeza que fazer nada é o certo então está tudo ok. Como tudo, regras fixas valem nada, tem que fazer o que é certo naquele caso.

É difícil dizer que exista um caso onde engolir a exceção sem fazer algo seja o certo, mas pode existir. Quando isso é normal talvez a exceção seja o mecanismo errado. O nome exceção não é assim por acaso, deve ser algo excepcional, não normal. Eu já vi gente defendendo que exceção não deve ser usada para situações excepcionais, bota camisa de força em uma pessoa dessas.

Esse caso parece estranho que possa ser ignorado. "Não era tão importante enviar o e-mail quanto que a solicitação fosse cadastrada", jura? Então não mande-o.

Pode ser chamado de anti-padrão, mas eles estão aí para serem usados também. Isto se chama exception swallowing ou error hiding.

Tem o risco de estar fazendo algo errado :P Se for o correto para essa situação não há problema. Tenha certeza que em nenhum momento o fato da falha não ter um tratamento não vai causar um outro problema.

Tem um problema neste caso específico, está capturando Exception. Isso é muito errado, porque ele engolirá exceções que precisam ser tratadas, que podem até serem erros de programação (trapalhada), aí é grave. Se é para engoli-la é mais importante ainda que capture uma bem específica.

E se a exceção lançada não for tão específica assim? Ache outro fornecedor. Esse não presta :P

Coloquei no GitHub para referência futura.

conteúdo editado
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

É. Péssimo :)

Primeiro precisaria ver se aquilo realmente é necessário lançar a exceção. Vejo muitos casos onde a exceção não é o mais adequado (ver mais aqui e aqui).

Talvez seja um caso de exceção vexatória/ruidosa (ver o link aí em cima). Talvez seja uma exceção estrangeira, o que aí seria normal.

Se está tudo normal com o lançamento então tem que perguntar-se porque não fazer nada. Esta é a ação esperada? Não vai avisar, não vai logar, não vai tomar um rumo diferente, escolher uma alternativa, nada assim? Vai fazer de conta que nada aconteceu?

Se considerou tudo isto e tem certeza que fazer nada é o certo então está tudo ok. Como tudo, regras fixas valem nada, tem que fazer o que é certo naquele caso.

É difícil dizer que exista um caso onde engolir a exceção sem fazer nadaalgo seja o certo, mas pode existir. Quando isso é normal talvez a exceção seja o mecanismo errado. O nome exceção não é assim por acaso, deve ser algo excepcional, não normal. Eu já vi gente defendendo que exceção não deve ser usada para situações excepcionais, bota camisa de força em uma pessoa dessas.

Esse caso parece estranho que possa ser ignorado. "Não era tão importante enviar o e-mail quanto que a solicitação fosse cadastrada", jura? Então não mande-o.

Pode ser chamado de anti-padrão, mas eles estão aí para serem usados também. Isto se chama exception swallowing ou error hiding.

Tem o risco de estar fazendo algo errado :P Se for o correto para essa situação não há problema. Tenha certeza que em nenhum momento o fato da falha não ter um tratamento não vai causar um outro problema.

Tem um problema neste caso específico, está capturando Exception. Isso é muito errado, porque ele engolirá exceções que precisam ser tratadas, que podem até serem erros de programação (trapalhada), aí é grave. Se é para engoli-la é mais importante ainda que capture uma bem específica.

E se a exceção lançada não for tão específica assim? Ache outro fornecedor. Esse não presta :P

É. Péssimo :)

Primeiro precisaria ver se aquilo realmente é necessário lançar a exceção. Vejo muitos casos onde a exceção não é o mais adequado (ver mais aqui e aqui).

Talvez seja um caso de exceção vexatória/ruidosa (ver o link aí em cima). Talvez seja uma exceção estrangeira, o que aí seria normal.

Se está tudo normal com o lançamento então tem que perguntar-se porque não fazer nada. Esta é a ação esperada? Não vai avisar, não vai logar, não vai tomar um rumo diferente, escolher uma alternativa, nada assim? Vai fazer de conta que nada aconteceu?

Se considerou tudo isto e tem certeza que fazer nada é o certo então está tudo ok. Como tudo, regras fixas valem nada, tem que fazer o que é certo naquele caso.

É difícil dizer que exista um caso onde engolir a exceção sem fazer nada seja o certo, mas pode existir. Quando isso é normal talvez a exceção seja o mecanismo errado. O nome exceção não é assim por acaso, deve ser algo excepcional, não normal. Eu já vi gente defendendo que exceção não deve ser usada para situações excepcionais, bota camisa de força em uma pessoa dessas.

Esse caso parece estranho que possa ser ignorado. "Não era tão importante enviar o e-mail quanto que a solicitação fosse cadastrada", jura? Então não mande-o.

Pode ser chamado de anti-padrão, mas eles estão aí para serem usados também. Isto se chama exception swallowing ou error hiding.

Tem o risco de estar fazendo algo errado :P Se for o correto para essa situação não há problema. Tenha certeza que em nenhum momento o fato da falha não ter um tratamento não vai causar um outro problema.

Tem um problema neste caso específico, está capturando Exception. Isso é muito errado, porque ele engolirá exceções que precisam ser tratadas, que podem até serem erros de programação (trapalhada), aí é grave. Se é para engoli-la é mais importante ainda que capture uma bem específica.

E se a exceção lançada não for tão específica assim? Ache outro fornecedor. Esse não presta :P

É. Péssimo :)

Primeiro precisaria ver se aquilo realmente é necessário lançar a exceção. Vejo muitos casos onde a exceção não é o mais adequado (ver mais aqui e aqui).

Talvez seja um caso de exceção vexatória/ruidosa (ver o link aí em cima). Talvez seja uma exceção estrangeira, o que aí seria normal.

Se está tudo normal com o lançamento então tem que perguntar-se porque não fazer nada. Esta é a ação esperada? Não vai avisar, não vai logar, não vai tomar um rumo diferente, escolher uma alternativa, nada assim? Vai fazer de conta que nada aconteceu?

Se considerou tudo isto e tem certeza que fazer nada é o certo então está tudo ok. Como tudo, regras fixas valem nada, tem que fazer o que é certo naquele caso.

É difícil dizer que exista um caso onde engolir a exceção sem fazer algo seja o certo, mas pode existir. Quando isso é normal talvez a exceção seja o mecanismo errado. O nome exceção não é assim por acaso, deve ser algo excepcional, não normal. Eu já vi gente defendendo que exceção não deve ser usada para situações excepcionais, bota camisa de força em uma pessoa dessas.

Esse caso parece estranho que possa ser ignorado. "Não era tão importante enviar o e-mail quanto que a solicitação fosse cadastrada", jura? Então não mande-o.

Pode ser chamado de anti-padrão, mas eles estão aí para serem usados também. Isto se chama exception swallowing ou error hiding.

Tem o risco de estar fazendo algo errado :P Se for o correto para essa situação não há problema. Tenha certeza que em nenhum momento o fato da falha não ter um tratamento não vai causar um outro problema.

Tem um problema neste caso específico, está capturando Exception. Isso é muito errado, porque ele engolirá exceções que precisam ser tratadas, que podem até serem erros de programação (trapalhada), aí é grave. Se é para engoli-la é mais importante ainda que capture uma bem específica.

E se a exceção lançada não for tão específica assim? Ache outro fornecedor. Esse não presta :P

adicionou 113 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

É. Péssimo :)

Primeiro precisaria ver se aquilo realmente é necessário lançar a exceção. Vejo muitos casos onde a exceção não é o mais adequado (ver mais aqui e aqui).

Talvez seja um caso de exceção vexatória/ruidosa (ver o link aí em cima). Talvez seja uma exceção estrangeira, o que aí seria normal.

Se está tudo normal com o lançamento, então tem que se perguntar-se porque não fazer nada. Esta é a ação esperada? Não vai avisar, não vai logar, não vai tomar um rumo diferente, escolher uma alternativa, nada assim? Vai fazer de conta que nada aconteceu?

Se considerou tudo isto e tem certeza que fazer nada é o certo, então está tudo ok. Como tudo, regras fixas não valem nada. Tem, tem que fazer o que é certo naquele caso.

É difícil dizer que exista um caso onde engolir a exceção sem fazer nada seja o certo, mas pode existir. Quando isso é normal, talvez a exceção seja o mecanismo errado. O nome exceção não é assim por acaso, deve ser algo excepcional, não normal. Eu já vi gente defendendo que exceção não deve ser usada para situações excepcionais, bota camisa de força em uma pessoa dessas.

Esse caso parece estranho que possa ser ignorado. "Não era tão importante enviar o e-mail quanto que a solicitação fosse cadastrada", jura? Então não mande-o.

Pode ser chamado de anti-padrão, mas eles estão aí para serem usados também. Isto se chama exception swallowing ou error hiding.

Tem o risco de estar fazendo algo errado :P Se for o correto para essa situação, não há problema. Tenha certeza que em nenhum momento o fato da falha não ter um tratamento não vai causar um outro problema.

Claro que temTem um problema neste caso específico. Está, está capturando Exception. Isso é muito errado, porque ele vai engolirengolirá exceções que precisam ser tratadas, que podem até serem erros de programação (trapalhada). Aí, aí é grave. Se é para engoli-la, é mais importante ainda que capture uma bem específica.

E se a exceção lançada não for tão específica assim? Ache outro fornecedor. Esse não presta :P

É. Péssimo :)

Primeiro precisaria ver se aquilo realmente é necessário lançar a exceção. Vejo muitos casos onde a exceção não é o mais adequado (ver mais aqui e aqui).

Talvez seja um caso de exceção vexatória/ruidosa (ver o link aí em cima). Talvez seja uma exceção estrangeira, o que aí seria normal.

Se está tudo normal com o lançamento, então tem que se perguntar porque não fazer nada. Esta é a ação esperada? Não vai avisar, não vai logar, não vai tomar um rumo diferente, escolher uma alternativa, nada? Vai fazer de conta que nada aconteceu?

Se considerou tudo isto e tem certeza que fazer nada é o certo, então está tudo ok. Como tudo, regras fixas não valem nada. Tem que fazer o que é certo naquele caso.

É difícil dizer que exista um caso onde engolir a exceção sem fazer nada seja o certo, mas pode existir. Quando isso é normal, talvez a exceção seja o mecanismo errado. O nome exceção não é assim por acaso, deve ser algo excepcional, não normal.

Esse caso parece estranho que possa ser ignorado. "Não era tão importante enviar o e-mail quanto que a solicitação fosse cadastrada", jura? Então não mande-o.

Pode ser chamado de anti-padrão, mas eles estão aí para serem usados também. Isto se chama exception swallowing ou error hiding.

Tem o risco de estar fazendo algo errado :P Se for o correto para essa situação, não há problema. Tenha certeza que em nenhum momento o fato da falha não ter um tratamento não vai causar um outro problema.

Claro que tem um problema neste caso específico. Está capturando Exception. Isso é muito errado, porque ele vai engolir exceções que precisam ser tratadas, que podem até serem erros de programação (trapalhada). Aí é grave. Se é para engoli-la, é mais importante ainda que capture uma bem específica.

E se a exceção lançada não for tão específica assim? Ache outro fornecedor. Esse não presta :P

É. Péssimo :)

Primeiro precisaria ver se aquilo realmente é necessário lançar a exceção. Vejo muitos casos onde a exceção não é o mais adequado (ver mais aqui e aqui).

Talvez seja um caso de exceção vexatória/ruidosa (ver o link aí em cima). Talvez seja uma exceção estrangeira, o que aí seria normal.

Se está tudo normal com o lançamento então tem que perguntar-se porque não fazer nada. Esta é a ação esperada? Não vai avisar, não vai logar, não vai tomar um rumo diferente, escolher uma alternativa, nada assim? Vai fazer de conta que nada aconteceu?

Se considerou tudo isto e tem certeza que fazer nada é o certo então está tudo ok. Como tudo, regras fixas valem nada, tem que fazer o que é certo naquele caso.

É difícil dizer que exista um caso onde engolir a exceção sem fazer nada seja o certo, mas pode existir. Quando isso é normal talvez a exceção seja o mecanismo errado. O nome exceção não é assim por acaso, deve ser algo excepcional, não normal. Eu já vi gente defendendo que exceção não deve ser usada para situações excepcionais, bota camisa de força em uma pessoa dessas.

Esse caso parece estranho que possa ser ignorado. "Não era tão importante enviar o e-mail quanto que a solicitação fosse cadastrada", jura? Então não mande-o.

Pode ser chamado de anti-padrão, mas eles estão aí para serem usados também. Isto se chama exception swallowing ou error hiding.

Tem o risco de estar fazendo algo errado :P Se for o correto para essa situação não há problema. Tenha certeza que em nenhum momento o fato da falha não ter um tratamento não vai causar um outro problema.

Tem um problema neste caso específico, está capturando Exception. Isso é muito errado, porque ele engolirá exceções que precisam ser tratadas, que podem até serem erros de programação (trapalhada), aí é grave. Se é para engoli-la é mais importante ainda que capture uma bem específica.

E se a exceção lançada não for tão específica assim? Ache outro fornecedor. Esse não presta :P

replaced http://pt.stackoverflow.com/ with https://pt.stackoverflow.com/
Fonte Link
Carregando
removeu 6 caracteres do conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando
adicionou 14 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando
adicionou 401 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando