Discussion :

[zmatz]

Salut!
Je débute en C#, et j'ai besoin de gérer des exceptions dans mon programme:
Je fais quelque chose comme ça, afin de détecter les noms de "SceneNode" qui existe deja dans ma "_scene" et ainsi créer un nouveau nom lors de la création d'un nouveau SceneNode:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
  int i = 0;
            try
            {
                while (true)
                {
                    scene_.GetSceneNode(NomForme + i + "Node");
                    i++;
                }
            }
            catch (Exception)
            {
                nomElement_ = NomForme + i;
            }

Bien qu'il fonctionne, est ce que ce code est correct?
GetSceneNode renvoi une exception d'après sa documentation, mais je n'arrive pas à savoir quelle est son nom(C'est pourquoi j'ai mis "Exception" dans le "catch". Est-ce qu'il ya moyen de le trouver (par le debbuger par exemple)?
Le gros probleme est que d'autres parties de mon code me renvoi maintenant des exceptions, alors qu'avant ils n'en renvoyaient pas... Comme si la portée du "try" s'étendait a tout mon code...

[Thomas Lebrun]

Ben dans la doc, c'est pas écrit le nom de l'exception que cela renvoit

[zmatz]

La doc me dit
Note:
Throws an exception if the named instance does not exist

http://www.ogre3d.org/docs/api/html/...ceneManagera24
je pense qu'il s'agit de ERR_ITEM_NOT_FOUND en regardant la liste des Codes d'Erreurs:
http://www.ogre3d.org/docs/api/html/...edExceptionw10
Je me demandait donc si il y avait un moyen de récuperer le nom de cette exception autrement...

Mais en essayant

Code :

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

catch(ERR_ITEM_NOT_FOUND)

il me dit qu'il ne connait pas ERR_ITEM_NOT_FOUND, bien que j'ai inclus sa librairie avec un using....

Sinon pour le code ca te parait correct?

[zeavan]

Salut!

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
  int i = 0;
            try
            {
                while (true)
                {
                    scene_.GetSceneNode(NomForme + i + "Node");
                    i++;
                }
            }
            catch (Exception)
            {
                nomElement_ = NomForme + i;
            }

il me semble que tu es un probleme a la ligne :

scene_.GetSceneNode(NomForme + i + "Node");

tu melange integer et string :
tu devrais plutot faire ca :
scene_.GetSceneNode(NomForme + i.ToString() + "Node");

sinon pour le catch:

catch (Exception e)
{
string messageError = e.Message;// ca te donnera plus d'info.


}

[zmatz]

Merci pour ta réponse Zeavan. J'ai corrigé le int...
pour ce qui est du message d'erreur j'obtient ça dans la variable messageError :
"Un composant externe a levé une exception."

Ce qui m'embête le plus c'est que je récupère des beugs sur une autre partie de mon programme, alors qu'il n'y en avait pas avant que je mette en place ce traitement d'exception....

J'obtient un "Tentative de lecture ou d'écriture de mémoire protégée. Cela indique souvent qu'une autre mémoire est endommagée."} System.Exception {System.AccessViolationException}"
C'est un probleme de pointeur ça non?

[zeavan]

lorsque tu debug , tu rentre dans le catch ???

si oui il faut que tu t'arranges pour que ton code ne rentre pas dans le catch , une fois fait tiens nous au courant , je pense que ca resoudra pas mal de chose.

[zmatz]

Oui je rentre dans le catch...

En fait mon algo c'est a peu près ca:
-J'ai une liste d'element dans une scene
-J'ai une methode qui me permet de renvoyer une exception si j'essai de récuperer un element qui n'existe pas.

Donc pour creer une nouvel element:
-Je fait tourner dans mon while tous les elements un par un en incrementant le suffix du nom.
-Si en ayant incrementer je me retrouve avec l'impossibilité d'obtenir un objet=> ca me genere une exception=> Dans l'exception je crée un nouvel element avec le suffixe bien incrémenté.
-Je sort du try et du catch (c'est cette derniere commande que je suis pas sur qui se réalise, du moins si à la vue du debbuger, mais j'ai des effets colateraux dans d'autres parties de mon code)...

Voila est-ce que c'est correct comme algo ou bien il faut s'y prendre autrement?

[Chubyone]

A la premiere Exception dans le try, tu entre dans le catch. Mais ensuite ca ne se poursuit pas.
A la limite si tu as besoin de finir quelque chose tu peux continuer avec "finally"

[Maniak]

Voila est-ce que c'est correct comme algo ou bien il faut s'y prendre autrement?

C'est un algo qui se base sur une exception pour sortir d'une boucle. Ce n'est jamais une bonne pratique. Les exceptions c'est pour gérer les erreurs/évènements exceptionnels, pas pour contrôler le flux du code.

La classe que tu utilises doit bien avoir une méthode pour tester si le nom de ta node existe non ? Si oui, appelle ça avant d'y accéder. Plus besoin d'exception, tu sais quand tu as un nom inexistant, tu peux le gérer directement dans la boucle. S'il n'y a pas de méthode du genre dans cette classe, donc qu'elle est faite pour que le code client se repose sur les exceptions pour gérer son comportement... selon le cas, tu peux soit frapper le développeur qui a pondu ça, soit lui demander de rajouter une méthode pour faire ça proprement, soit chercher une autre classe mieux faite.

Sinon de manière générale, dès qu'il y a un 'while ( true )', ça devrait activer le flag 'oula va falloir nettoyer ça' :)