Discussion :

[hadler]

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 :

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

[adiGuba]

Salut,


Utilises une méthode commune qui traitera les exception :

Code :

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

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

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

[Uther]

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

Code :

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

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 :

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

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

edit: grillé par adiguba

[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.



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 ?

[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.