catcher n'importe quelle exception genere par une fonction

bonjour a tous,
j'ai plusieurs fonction faisant differents trts sur une base de donnee
lors de l'appel a ces fonctions, je veux catcher toutes les exceptions qu'ils peuvent generer.

voici un exemple de fonction :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29

public boolean deleteHierarchieDAO(Hierarchie hierarchie) throws RuntimeException{ Transaction tx = sessionFactory.getCurrentSession().beginTransaction(); boolean result = false; try { sessionFactory.getCurrentSession().delete(hierarchie); tx.commit(); result = true; } catch (RuntimeException re) { if (tx != null) { tx.rollback(); result = false; } throw re; //log.error("deleting hierarchie failed", re);   //throw re; } catch (Exception e) { System.out.println("General Exception occurred"); e.printStackTrace(); if (tx != null) tx.rollback();   throw e; } finally { sessionFactory.getCurrentSession().close(); return result; } }

---

et voila ce que je fais lors du catch

Code:

---

1 2 3 4 5 6 7 8

try{ dao.deleteHierarchieDAO(hierarchie); } catch (Exception e) { JOptionPane.showMessageDialog(null, "exception"); // autre trt   }

---

mais la , l'exception n'est pas catchee
que dois-je faire pour catcher tous types d'exception generee par deleteHierarchieDAO?


merci pour l'aide !
Sallemel

Salut,


Ton bloc finally est incorrect : il ne doit pas contenir de return !

:arrow: Lorsque tu lèves une exception, tu passes dans le finally qui stoppe sa remonté en renvoyant un résultat :?

De plus ta méthode est étrange car on dirait que tu veux à la fois utiliser la valeur de retour et l'exception en cas d'erreur :koi:


Perso je te conseillerais d'utiliser deux try/finally distincte pour fermer la session et gérer le rollback :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

public void deleteHierarchieDAO(Hierarchie hierarchie) { boolean result = false; Transaction tx = sessionFactory.getCurrentSession().beginTransaction(); try { try { sessionFactory.getCurrentSession().delete(hierarchie); tx.commit(); result = true; } finally { // Si le code ne s'est pas exécuté jusqu'à la fin // on effecute un rollback : if (!result) { tx.rollback(); } }   } finally { sessionFactory.getCurrentSession().close(); } }

---

a++

merci AdiGuba pour votre reponse,

le pb c'est que avec ca , on ne "catche" pas d'exception
moi a part le traitement de suppression
je veux recuperer l'exception, si jamais elle se produit, et la cause de cette erreur dans :

Code:

---

1 2 3 4 5 6 7 8

try{ dao.deleteHierarchieDAO(hierarchie); } catch (Exception e) {   // recuperation de la cause de l'erreur ds e.getMessage()   }

---

et au vu de la modification de deleteHierarchieDAO que vous avez propose
la methode deleteHierarchieDAO (il n'y a plus de bloc catch) , elle ne "throws" aucune exception

A+
Sallemel

Citation:

---

Envoyé par sallemel

et au vu de la modification de deleteHierarchieDAO que vous avez propose
la methode deleteHierarchieDAO (il n'y a plus de bloc catch) , elle ne "throws" aucune exception

---

Puisqu'il y a un try/catch dans votre code, j'ai supposé que les exception était remonté par les méthodes delete() ou commit(). Ce n'est pas le cas ?

a++

Je pense qu'il faut que tu ajoutes la clause throws Exception au prototype de ta fonction.

Hello,
merci a vous deux, j'ai definit la fonction que me l'a decrit adiGuba et j'ai ajoute throws exception au proptotype
et c ok, je catche bien l'exception
encore merci

Citation:

---

j'ai ajoute throws exception au proptotype

---

Ne faites surtout pas ça sinon vous devrez décorer toutes vos méthodes jusqu'au main de throws Exception.
Il ne faut catcher QUE les exceptions concernées, sinon vous risquez de catcher des erreurs critiques ingérables.