répétition d'un bloc try/catch

Version imprimable

Bonjour à tous,

J'ai longtemps hésité à posté ici ou sur débuter, donc excusez moi si je ne suis pas au même endroit.

Dans mon projet actuel, j'ai une classe java qui est appelée par le client et qui exécute des opérations.

Chacune des méthodes de cette classes ont des actions communes et séparées.

Mais j'ai quand même un point commun à toutes les méthodes, un même bloc try/catch.
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 try { //Mes actions } catch (InvalidCredentialsException error) { response.setStatusCode(ListOfStatusCode.INVALID_CREDENTIALS) ; response.setErrorCode(error) ; } catch (LoginException error) { response.setStatusCode(ListOfStatusCode.LOGIN_EXCEPTION) ; response.setErrorCode(error) ; } catch (ItemNotFoundException error) { response.setStatusCode(ListOfStatusCode.DOCUMENT_NOT_FOUND) ; response.setErrorCode(error) ; } catch (NoSuchWorkspaceException error) { response.setStatusCode(ListOfStatusCode.NO_SUCH_WORKSPACE) ; response.setErrorCode(error) ; } catch (KInvalidUuidException error) { response.setStatusCode(ListOfStatusCode.INVALID_UUID) ; response.setErrorCode(error) ; } catch (RepositoryException error) { response.setStatusCode(ListOfStatusCode.REPOSITORY_EXCEPTION) ; response.setErrorCode(error) ; } catch (Exception error) { response.setStatusCode(ListOfStatusCode.ANOTHER_EXCEPTION) ; response.setErrorCode(error) ; }
Je me demande donc s'il existe une méthode pour ne pas devoir répéter ce bloc à chacune des méthodes que j'ajoute à ma classe.

Actuellement j'ai 3 méthodes dans ma classe, et quand je modifie quelque chose dans mes exceptions, je dois à chaque fois aller modifier le bloc try/catch de chacune de mes méthodes.

Merci d'avances à ceux qui m'éclaireront :ccool:

Salut,

Utilises une méthode commune qui traitera les exception :

Code:

1
2
3
4
5
try {
	//Mes actions
} catch (Exception error) {
	setStatusErrorCode(response, error);
}

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
public static void setStatusErrorCode(Response response, Exception exception) {
	try {
		throw exception;
	} catch (InvalidCredentialsException error) {
		response.setStatusCode(ListOfStatusCode.INVALID_CREDENTIALS);
		response.setErrorCode(error);
	} catch (LoginException error) {
		response.setStatusCode(ListOfStatusCode.LOGIN_EXCEPTION);
		response.setErrorCode(error);
	} catch (ItemNotFoundException error) {
		response.setStatusCode(ListOfStatusCode.DOCUMENT_NOT_FOUND);
		response.setErrorCode(error);
	} catch (NoSuchWorkspaceException error) {
		response.setStatusCode(ListOfStatusCode.NO_SUCH_WORKSPACE);
		response.setErrorCode(error);
	} catch (KInvalidUuidException error) {
		response.setStatusCode(ListOfStatusCode.INVALID_UUID);
		response.setErrorCode(error);
	} catch (RepositoryException error) {
		response.setStatusCode(ListOfStatusCode.REPOSITORY_EXCEPTION);
		response.setErrorCode(error);
	} catch (Exception error) {
		response.setStatusCode(ListOfStatusCode.ANOTHER_EXCEPTION);
		response.setErrorCode(error);
	}
}

a++

il faudrait que tu te fasse une méthode commune:

Code:

1
2
3
4
5
6
7
8
9
10
void setResponse(Exception e){
  if (e instanceof InvalidCredentialsException) {
	response.setStatusCode(ListOfStatusCode.INVALID_CREDENTIALS);
  }
  if (e instanceof LoginException ) {
	response.setStatusCode(ListOfStatusCode.LOGIN_EXCEPTION);
  }
  ...
  response.setErrorCode(e)
}

comme cela tu pourras l'utiliser dans tous tes try/catch:

Code:

1
2
3
4
5
try {
	//Mes actions
} catch (Exception error) {
	setResponse(error) ;
}

edit: grillé par adiguba

03/12/2009, 15h47
hadler

C'est tellement simple comme solution que ça ne m'avait même pas effleuré l'esprit.

Je cherchais des manières compliquées pour réussir à faire une méthode qui gèrerait les exceptions.

Merci à vous pour votre rapidité et votre aide.

:ccool:

edit : au fait, les 2 méthodes que vous me donnez se valent, mais l'une d'elle est elle plus performante ou conseillée ?
03/12/2009, 16h09
tchize_

la première est plus facile à lire car utilise des try/catch. Mais elle relance l'exception, contrairment à la deuxième. J'aurais tendance à préférer la première.