Discussion :

[samball]

Bonsoir à tous,

derrière ce titre occulte se cache en fait une question très simple.

Imaginons le code suivant:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
void BeginUpdate()
{
	if (count == 0)
		my_ctrl.Enabled = false;
	count++;
}
 
void EndUpdate()
{
	count--;
	if (count == 0)
		my_ctrl.Enabled = true;
}

Utilisé dans le cas suivant:

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
 
void Toto()
{
	BeginUpdate();
 
	if (condition)
	{
		// do something
		EndUpdate();
		return true;
	}
	else
	{
		// do something else
		EndUpdate();
		return true;
	}
	else
	{
		EndUpdate();
		return false;
	}
}

Pour éviter cette écriture disgracieuse j'ai pensé aux deux méthodes suivantes:

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
void Toto()
{
	try
	{
		BeginUpdate();
 
		if (condition)
		{
			// do something
			return true;
		}
		else
		{
			// do something else
			return true;
		}
		else
		{
			return false;
		}
	}
	finally
	{
		EndUpdate();
	}
}

Et une autre méthode un peu plus 'hack' avec une classe encapsulant une delegate et l'executant dans le DoDispose:

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
void Toto()
{
	BeginUpdate();
 
	using (ScopeFinalizer finalizer = new ScopeFinalizer(delegate() { EndUpdate(); })
	{
		if (condition)
		{
			// do something
			return true;
		}
		else
		{
			// do something else
			return true;
		}
		else
		{
			return false;
		}
	}
}

Quelqu'un connait-il un autre manière plus 'propre' de faire? Ou sinon, laquelle de ces trois methodes est la plus propre?

Merci d'avance pour vos suggestion!

Bonne soirée.

[SaumonAgile]

Bah tout simplement,

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
 
bool returnValue;
 
if (...)
{
    // Code
    returnValue = true;
}
else if (...)
{
    // Code
    returnValue = true;
}
else
{
    // Code
    returnValue = false;
}
 
EndUpdate();
return returnValue;

Tu as aussi le try/finally

[harz62]

Moi je dirais le 2e...

[smyley]

Je préfère moi aussi le try-finally... pourquoi ?
- dans le code 1, tu n'est pas sure que EndUpdate soit appelé si une exception est levée par do something
- dans le 2, EndUpdate est toujours appelé, quelque soit l'exécution de do something
- dans le 3 ... pourquoi prendre un bazooka pour tuer une fourmi ?

[samball]

Merci pour vos réponses.

Après avoir fait mon post, j'ai continué à fouiller, et je suis tombé sur un article qui expliquait que implicitement les codes avec du "using" etaient automatiquement transformés en try / finally avec un appel au DoDispose dans le finally.

Etant donné que j'aime pas trop la méthode de tout écrire à la main, je pense que je vais me tourner vers le try / finally qui a le mérite d'etre Exception-Safe.

Merci à vous !

Bonne journée.