Discussion :

[thelpi]

salut tout le monde,

j'ai un problême qui me parait étrange avec mon code c#:

voila une classe ProcessThread, dans son constructeur je l'abonne à l'évenement "configChange", qui est déclenché quand je change la configuration via un panneau de configuration sur l'interface graphique (le code ici est exécuté en parallèle).
Quand un évenement configChange survient, j'appelle la méthode OnChange qui lève une exception

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
 
class ProcessThread
{
public ProcessThread()
{
pagConfig.configChange += newpagConfig.ConfigChangeEventHandler(OnChange);
}
 
public void ExecuteLoop
{
try
{
// faire quelque chose
}
catch (ConfigChangeException ex)
{
// faire quelque chose d'autre
}
}
 
public void OnChange(object o, ConfigChangeEventsArgs é)
{
throw new ConfigChangeException("exception: la configuration a changé")
}
}

pendant l'exécution, je met un point d'arrèt dans le try de la méthode ExecuteLoop pour être certain d'y être rentré. à ce moment la je change ma configuration, l'évenement configChange est enclenché, je rentre dans la méthode OnChange et la j'ai le droit à un superbe message d'erreur lié à la levée de l'exception, alors que j'aurais du tomber dans le catch de ma méthode ExecuteLoop. Je ne comprend vraiment pas ou j'ai fait une erreur ...

merci.

[Pol63]

si le code qui modifie la configuration n'est pas dans le try, c'est normal que ca aille pas dans le catch

[thelpi]

merci.

Maintenant le problème est que malgré tout je doit absolument sortir immédiatement du try si la configuration a changé.

Si mon code ne marche pas, je ne sais pas comment faire,a moins de créer un booléen "configChange" de lui mettre true au passage dans "OnChange" et de mettre un if(!configChange) sur toutes les lignes de mon try... il doit y avoir une solution 100 fois meilleure mais elle m'échappe pour le moment

[Pol63]

c'est en effet une solution, et envisageable en plus

enfin si c'est dans un thread, tu peux t'en sortir autrement, avec thread.abort ou thread.interrupt

interrupt ca arrete tout (ca ne passe ni à la ligne d'après, ni dans le catch)

thread.abort génère une exception, qui se propage à travers tous les bloc catch, en partant de la ligne ou est actuellement le thread
c'est risqué, car si t'es sur une ligne de code qui appelait une méthode du framework, c'est la méthode du framework qui récupère en premier l'erreur, donc ca peut faire un comportement étrange ...

[thelpi]

ah oui mais la je doit préciser mon code alors

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
 
public void ExecuteLoop()
{
     while (unBooléenQuelconque)
     {
          // code A
          try
          {
               // code B
          }
          catch
          {
               // gestion exceptions
          }
          // code C
     }
}

quand la config change, le code B ne doit pas être exécuté, mais le code code A et C si...

merci de t'être penché sur mon cas

[Pol63]

veux tu rollbacker ce que tu fais carrément ?

enfin il faudrait meme préciser beaucoup plus, que fais tu réellement
et qu'est ce qui change dans la config de quoi qui t'horripiles ?

[thelpi]

dans ma configuration, on peut modifier les identifiants de connection SQL server, l'url d'une boutique en ligne, etc...

mais bon je pense que j'ai une solution correcte:

un booléen configChange reçoit true quand la configuration change

ensuite je fait

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
 
while (quelqueChose)
     try
     {
          // code A
          if (configChange)
          {
               throw new ConfigChangeException("La configuration a été modifié");
          }
     }
     catch (configChangeException)
     {
          // gestion des changements de configuration
          configChange = false;
     }
     catch (SqlException)
     {
          // gestion des échecs de connection
     }
     // code B
}

c'est pas le top car effectivement j'aurais préféré que le code A s'arrète immédiatement mais bon visiblement j'ai pas le choix...

bonne journée à toi!