Discussion :

[Davidbrcz]

Bonjour à tous.

J'ai un switch qui se présente comme ca:

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
 
switch(var)
{
 case 1:
//bidule
 break;
 
case 2:
//machin.
break;
 
//... j'ai 10 cas comme ca 
 
defaut:
break;
}

Et je souhaite appeller dans chacun des cas la même fonction foo().
Existe il un moyen pour éviter de copier-coller l'appel à foo dans chacun des case ?

Merci

[Vespasien]

Tu places l'appel à foo() avant ou après ton switch().

[Davidbrcz]

Je ne peux pas, car ce switch est une une sous partie d'un ensemble de test plus globale.

[Alp]

Explique un peu le contexte de ces tests.

[Davidbrcz]

Bon,voici le code que je souhaite éviter car les appels à updateImgPiece me semble trop redondant.

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
36
37
38
39
40
41
42
43
 
                if(event.Type == sf::Event::KeyPressed)
		 {
		     switch(event.Key.Code)
			 {
			 case sf::Key::A :
			     m_data->rotateL();
			     updateImgPiece();
			     //rotation vers la gauche (sens direct)
			     break;
 
			 case sf::Key::Z :
			     m_data->symmetryY();
			     updateImgPiece();
			     //symetrie vecticale
			     break;
 
			 case sf::Key::E :
			     m_data->rotateR();
			     updateImgPiece();
			     //rotation vers la droite (sens indirect)
			     break;		
 
			 case sf::Key::Q :
			     m_data->pieceActualDown();
			     updateImgPiece();
			     //piece suivante Down
			     break;			 
 
			 case sf::Key::S :
			     //symetrie horizontale
			     m_data->symmetryX();
			     updateImgPiece();
			     break;
 
			 case sf::Key::D:
			     m_data->pieceActualUp();
			     updateImgPiece();
			     //piece suivante up
			     break;			 
 
			 }
		 }

[Alp]

Dans le code que tu donnes, il serait plus simple d'appeler ta fonction juste après le dernier case, autrement dit juste à la fin du if.
En effet, raisonne un peu. Si tu as appuyé sur une touche, tu vas devoir mettre à jour ta pièce. Quelle que soit la touche.

Donc l'appel à updateImgPiece() juste avant la fin du if semble logique, au lieu de le répéter.