Discussion :

[Arnaoud]

Bonjour,

J'ai plusieurs méthodes que je souhaite encapsuler dans une autre.
Ceci me permettrai d'éviter de dupliquer du code non fonctionnel.

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
27
28
29
30
31
32
33
34
35
 
public void MaMethodeAEncapsuler_1(int i)
{
  // Code avec entrées sorties
}
 
public void MaMethodeAEncapsuler_2(string adress)
{
  // Code avec entrées sorties
}
 
public void runWithRetry(method)
{
   int nbEssai = 3;
   bool isok = false;
 
  While (!isok && nbEssai > 0)
  {
    try
    {
        // Exécuter la méthode "method"   
        isok = true;
    }
    catch
    {
        nbEssai--; 
    }
  }
}
 
static void main(string[] args)
{
   runWithRetry(MaMethodeAEncapsuler_1(1))
   runWithRetry(MaMethodeAEncapsuler_2("blabla"))
}

Est il possible de faire cela?
Merci d'avance.

[Graffito]

La procedure RunWithRetry doit utiliser 2 delegates ("method") de même type, ce qui n'est pas le cas puisque la première methode a comme paramètre un int et la 2ème une string.

On peut résoudre ce conflit en utilisant comme paramètre un objet dans les 2 cas et en castant cet objjet sur un int dans la la première methode et sur une string dans la 2ème.

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
delegate void MyMethodDelegate(object param);
 
public void MaMethodeAEncapsuler_1(object param_i) {int i=(int)param_i ; // Code avec entrées sorties }
 
public void MaMethodeAEncapsuler_2(object param_adress) { string adress=(string)param_adress ; // Code avec entrées sorties }
 
public void runWithRetry(MyMethodDelegate method,object param)
{
  int nbEssai = 3; bool isok = false;
  while (!isok && nbEssai > 0) { try { method(param) ; isok = true; } catch { nbEssai--; } }
}
 
static void main(string[] args) { runWithRetry(MaMethodeAEncapsuler_1,1) ; runWithRetry(MaMethodeAEncapsuler_2,"blabla) ; }

[Arnaoud]

Merci Graffito pour cette réponse.
Ca fera beaucoup plus propre.

[chamamo]

j'opterai plutôt pour une seule méthode générique plutôt que deux:

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
27
28
29
30
 
public void MaMethodeAEncapsuler<T>(T param) 
        {
            if (param is string)
            {
                MessageBox.Show("c'est un string");
            }
            else if(param is int)
            {
                MessageBox.Show("c'est un int");
            }
        }
 
        public void runWithRetry<T>(Action<T> method, T param)
        {
            int nbEssai = 3;
            bool isok = false;
            while (!isok && nbEssai > 0)
            {
                try
                {
                    method(param);
                    isok = true;
                }
                catch
                {
                    nbEssai--;
                }
            }
        }

Utilisation:

Code :

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

1
2
runWithRetry<int>(MaMethodeAEncapsuler, 1);
runWithRetry<string>(MaMethodeAEncapsuler, ""blabla"");