Discussion :

[MatMeuh]

Bonjour

J'ai une servlet qui peut être bloqué pour un grand nombre de raison et d'erreur. J'ai mis des try/catch comme il faut et pour l'instant dans mes catch je fais un truc du genre

Code :

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

1
2
3
4
catch (Exception e) 
{
out.println("AUTRE ERREUR : " + e.getMessage());
}

ce que j'aimerais c'est générer une liste ou un tableau des erreurs
et ne les afficher qu'a la fin de mon traitement avec des

Code :

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

1
2
3
4
out.println("html");
out.println("body");
out.println("erreur 1 = " + taberreur[0]);
....

est-ce possible ? si oui comment "stocker les erreurs" pour les réutiliser plus loin ? avec quelle structure de données ? y'en existe t'il des déjà tout établie ?

merci d'avance !

[Grabataire]

Salut,

Dans ton cas je stockerais mes erreurs dans un java.util.Vector.

Pour l'affichage tu récupère la liste de tes éléments du vector pour les afficher.

j'espère avoir répondu correctement a ton interrogation.

A+

[MatMeuh]

ok je vais essayer

mais un doute me taraude

c'est un code que j'ai repris parfois j'ai des PrintStackError (ou quelquechose du genre, j'ai pas le code sous les yeux) et j'aurais aimé qu'ils passent aussi dans ce vector

est-ce possible ?

[christopheJ]

Généralement j'utilise un ArrayList.
J'ajoute les Exceptions dedans, et au moment de renvoyer la page (où sur une tache critique) je teste la taille de l'ArrayList. S'il contient un objet je renvoie une page d'erreur où j'affiche les StackTrace des Exceptions.
Attention c'est valable pendant la réalisation et le débugage, avant le passage en prod, la gestion est changée parce que ca parle pas à l'utilisateur un StackTrace. Il faut mieux le logguer et lui envoyer un message plus clair.

[MatMeuh]

ok d'accord... j'avais vaguement commencé comme ca... mais j'ai mis les getMessage dedans... ce qui finalement revenait a creer un buffer...

pour les StackTrace, je sais pas trop a quoi ca correspond, puisque ce bout de code n'était pas le mien, je l'ai laissé pour pas "tout casser" mais je ne suis jamais tmbé encore sur une erreur "stacktracée"

merci et je m'attele a cela ce matin !


EDIT: y'a aussi un détail qui me chiffone, en effet suivant l'erreur j'aurais aimé mettre un message particulier pour chaque type d'exception, comment faire ? il faut que dans mon ArrayList je teste a quel type d'exception elle appartient ?

par ex :

Code :

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

1
2
3
4
5
6
7
8
9
catch (TransformerException te) 
{
out.pintln("<td width=\"100%\" class=\"libelleTexte\">Erreur de type Transformation "+ te.getMessage()+"</td>");
}
....
catch (SQLException e)
{
out.println("<td width=\"100%\" class=\"libelleTexte\">ERREUR SQL : " + e.getMessage()+"</td>");
}

[wace]

Je suis d'accord a peu pres sur tout,

mais je serais toi je ne ferai pas dans la servlet

Code :

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

1
2
3
4
catch (SQLException e)
{
out.println("<td width=\"100%\" class=\"libelleTexte\">ERREUR SQL : " + e.getMessage()+"</td>");
}

mais dans une classe metier independante de ta servlet

Code :

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

1
2
3
4
catch (SQLException e)
{
throw new MyException("<td width=\"100%\" class=\"libelleTexte\">ERREUR SQL : " + e.getMessage()+"</td>");
}

et tu catch ton exception MyException dans ta servlet et dans ta servlet tu fait un sendError avec le contenu de l'exception que tu as catché

[MatMeuh]

ouah !
j'ai été laché sur le throw la...

en fait pour l'instant j'ai suivi les instructions qu'il y avait plus haut :

Code :

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

1
2
3
4
5
6
7
8
9
10
ArrayList errorList = new ArrayList();
...
catch (SQLException e)
{
errorList.add(e);			
}
catch (Exception e) 
{
errorList.add(e);			
}

puis a la fin je fais

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
for(int i=0;i<errorList.size();i++)
{
Object e = errorList.get(i);
if (e instanceof Exception) 
{	
	out.println("erreur1 " + e.getMessage());
}
else
{
	out.println("erreur2");
} 
}

ce qui pose problème a cause du getMessage()... il faudrait pouvoir caster
mais je n'y arrive pas

merci !

ps: si cette solution est du grand n'importe quoi, vous pouvez le dire aussi !

[Grabataire]

Salut,

Je pense, a regarder ton code que tu devrais caster ton retour pour éviter les erreurs.

Je modifierais ton code de la sorte :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
for(int i=0;i<errorList.size();i++) 
{ 
Object e = errorList.get(i); 
if (e instanceof Exception) 
{    
   out.print("erreur1 " + e.getClass().getName()); // permet de connaître le type l'Exception générée
   out.println(" " + ((Exception)e).getMessage()); 
} 
else 
{ 
   out.println("erreur2 "+e.getClass().getName()); 
} 
}

C'est a tester bien sur, mais j'espère que ça t'aideras

A+

[MatMeuh]

nicek chrome !!! merci

ps: j'ai pas utilisé le getClass getName... c'est pour un affichage "user" donc ils s'en "battent" un peu