Discussion :

[sallemel]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[adiGuba]

Salut,


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

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


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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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++

[sallemel]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[adiGuba]

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++

[muad'dib]

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

[sallemel]

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

[_skip]

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.