Discussion :

[master MI]

Je dois mettre à jour un code en appliquant les règles suivantes:
Si une exception est levée par s.execute() alors appeler c.rollback()
et propager l'exception,sinon appeler c.commit()
Dans tous les cas ,c.closest() doit être appelée avant de quitter la méthode a(Service s,Connection c)

le code est le suivant:

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
30
31
32
33
34
35
public class A {
	// doit executer le service avec les connections données
 
	void a(Service s,Connection c) {
		s.setConnection(c);
		s.execute();
	}
 
}
 
interface Service{
	void execute() throws Exception;
	void setConnection(Connection c);
}
 
interface Connection{
	void commit();
	void rollback();
}
 
 
 
public static void main(String[] args) {
		Service service=new Service() {
			public void execute()throws Exception{
				System.out.println("execute");
			}
 
 
			public void setConnection(Connection c) {
				System.out.println("setConnection");
 
			}
		};
}

En suivant ces instructions j'ai donc complété en faisant le code suivant:

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
void a(Service s,Connection c) throws Exception {
		s.setConnection(c);
		s.execute();
                c.rollback();//doit appeler rollback() si une exception est levée et propager l'exception
                c.commit();//Sinon appeler c.commit()
                c.close();//dans tous les cas appeler c.close()
	}
 
}
 
interface Service{
	void execute() throws Exception;
	void setConnection(Connection c);
}
 
interface Connection{
	void commit();
	void rollback();
}

j'ai suivi les instructions mais je ne sais pas si c'est juste
Qu'est ce que vous en pensez svp?

[thelvin]

Hello,

ben non c'est pas juste.

rollback() ne doit être appelé que s'il y a une une exception.
commit() ne doit être appelé que s'il n'y a pas eu d'exception.
et close() doit toujours être appelé, sous-entendu y compris s'il y a une exception. Donc avec ton code il n'est pas appelé s'il y a une exception.

[master MI]

Je ne comprends pas là,il y'a bien une exception avec "throws exception" donc
j'ai appelé rollback();qu'est ce qui est faux dans mon code?je ne vois pas svp.

Code :

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

1
2
3
4
5
6
7
8
 
void a(Service s,Connection c) throws Exception {
s.setConnection(c);
s.execute();
c.rollback();//doit appeler rollback() si une exception est levée et propager l'exception
c.commit();//Sinon appeler c.commit()
c.close();//dans tous les cas appeler c.close()
}

????

[Cincinnatus]

Je ne comprends pas là,il y'a bien une exception avec "throws exception" donc
j'ai appelé rollback();qu'est ce qui est faux dans mon code?je ne vois pas svp.

Que comprends-tu aux exceptions ??
La déclaration "throws Exception" dit juste que la méthode est susceptible de transmettre une exception de ce type au code appelant. Point.

Lit un peu la doc. Par exemple :
https://anisfrikha.developpez.com/tu...va/exceptions/

[master MI]

Merci mais Votre lien ne marche pas.
Sinon comment j'utilise c.rollback(),c.commit() et c.close()
dans mon code alors?

[thelvin]

Réessaie le lien, il marche parfaitement. Peut-être qu'il y a eu une panne temporaire.