Discussion :

[foetus]

Je travaille avec C++ Builder Xe , donc un compilateur C++03 pas trop standard avec des extensions: enum typés, et pleins de mots clef pour l'IDE/ la bibliothèque VCL
Et par exemple Boost ne compile pas toujours.

J'ai une IHM avec un menu à gauche avec 6 entrées

Lorsque je clique sur une entrée, un écran/ contrôleur s'affiche sur les 6/ 10ième droit.
Donc à partir d'un écran, j'ai accès aux 5 autres écrans: soit une matrice de transitions (ce n'est pas terrible comme terme )

Donc le code est le suivant :

Code :

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

1
2
3
4
5
6
7
  current_controller->set_parent(NULL);
  je déselectionne l'entrée courante
 
  new_controller->set_parent(&main_form);
  je sélectionne la nouvelle entrée
 
  current_controller = new_controller // <- actuellement c'est avec un état enum

Mais, parce que sinon c'est trop facile :
1) Sur l'écran 1 et l'écran 3, j'ai une liste de XXX que je partage entre les 2 écrans

Si (current_controller != 1) && (current_controller != 3) && ((new_controller == 3) || (new_controller == 1)) -> j'affiche la liste
Si (current_controller == 1) && (new_controller == 3) -> je repositionne la liste et je sélectionne la première ligne
Si (current_controller == 3) && (new_controller == 1) -> je repositionne la liste et je sélectionne la première ligne

Mais il faut aussi penser à la cacher

Si ((current_controller == 1) || (current_controller == 3)) && (new_controller != 3) && (new_controller != 1) -> je cache la liste

2) l'écran 6 est l'écran de "settings" (langue et couleur)

Code :

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

1
2
3
4
5
6
7
8
9
10
  Si (current_controller == 6)
  alors
     controller_06->set_parent(NULL);
     je déselectionne l'entrée 06
 
     je sélectionne la nouvelle entrée
 
     si (il n'y a pas de changements) -> j'affiche le nouvel écran
     sinon -> je lance un thread pour répercuter les changements partout, et écrire le fichier de configuration
  fsi

3) Extra : on voit clairement que 1 entrée (un label texte) est associée à un contrôleur (mais dans mon code je n'ai pas créé de classe d'association)
Sauf pour l'écran 4 , où l'entrée est sur 2 labels


J'ai codé cela avec 6 switch cases de 6 cas (avec le cas diagonal vide à chaque fois) : cela fonctionne parfaitement.


Mais j'ai un autre menu ... un menu à 33 entrées (soit 33*32 cas = 1056) : avec 33 switch cases de 32 cas + 1 cas diagonal vide (7000 - 8000 lignes le machin )
cela pique fortement
Et là je vais avoir 3 à 5 listes partagées, et sûrement des tests réseau "au début"


Est-ce que vous avez une idée de coder un truc dynamique avec des pointeurs de fonctions/ méthodes, machines à états, ... ?


En gros, mon code: j'ai une classe "groupe de contrôleurs" qui contiennent tous les contrôleurs (6 ou 33) et c'est ce groupe qui fait l'affichage du menu.

[Et actuellement, je n'ai pas d'héritage, pas de virtuel, pas de pointeurs, juste un état enum de 6 ou 33 valeurs: de la composition la plus simple ]
[au début, j'avais séparé ce groupe et le menu, mais pour plus de facilité, j'ai fusionné les 2 classes]

[ternel]

En premiere idée, je dirai une map (ou autre) d'enum vers pointeurs de fonctions

En mieux, il y a boost::function.
S'il ne compile pas, tu dois pouvoir la reconstruire, mais tu va galérer quelque peu.

[jo_link_noir]

Je ferrais probablement cela à gros coup de template. Même si j'ai l'impression qu'à part les cas 1/3, 4 et 6, le code est toujours identique, non ?

C'est possible de voir le switch 6*6 pour se faire une meilleure idée de la forme du code ?

[foetus]

C'est possible de voir le switch 6*6 pour se faire une meilleure idée de la forme du code ?

L'écran 1 -> STATE_SITE_LOG_IN
L'écran 3 -> STATE_SITE_DELETE
L'écran 6 -> STATE_SETTINGS_CHANGE

le booléen bypass_test sert à désactiver le test puisqu'une fois le thread fini, on relance l'évènement (c'est une sécurité de ne pas avoir une boucle infinie impromptue)

select_admin_change_pwd et STATE_ADMIN_CHANGE_PWD doivent gérer une entrée sur 2 labels (mais ce sera sûrement la seule entrée comme cela du logiciel)

Tous les switch ont leurs case dans l'ordre et sans trou pour [plus ou moins] forcer le compilateur à créer des jump tables

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
void __fastcall Group_Login::select_admin_change_pwd(TObject* sender) {
	clicked_entry = ((Stdctrls::TLabel*) sender);
 
	switch(state) {
	case STATE_ADMIN_CHANGE_PWD:
		break;
 
	case STATE_ADMIN_LOG_OUT:
		frame_admin_log_out.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_admin_log_out, &select_admin_log_out)
 
		DISPLAY_SCREEN_ADMIN_CHANGE_PWD
		break;
 
	case STATE_SITE_CREATE:
		frame_site_create.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_create, &select_site_create)
 
		DISPLAY_SCREEN_ADMIN_CHANGE_PWD
		break;
 
	case STATE_SITE_DELETE:
		frame_site_delete.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_delete, &select_site_delete)
 
		list_sites->Parent = NULL;
 
		DISPLAY_SCREEN_ADMIN_CHANGE_PWD
		break;
 
	case STATE_SITE_LOG_IN:
		frame_site_log_in.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_log_in, &select_site_log_in)
 
		list_sites->Parent = NULL;
 
		DISPLAY_SCREEN_ADMIN_CHANGE_PWD
		break;
 
	case STATE_SETTINGS_CHANGE:
		frame_settings_change.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_settings_change, &select_settings_change)
 
		b_lang_has_changed  = (lang  != IHM_GET_LANG());
		b_theme_has_changed = (theme != bundle.current_theme);
 
		if (bypass_test || (!b_lang_has_changed && !b_theme_has_changed)) {
			DISPLAY_SCREEN_ADMIN_CHANGE_PWD
		} else {
			SELECT_MENU(label_admin_change_pwd1)
			SELECT_MENU(label_admin_change_pwd2)
			LAUNCH_WAIT_SETTING_CHANGE(&Group_Login::select_admin_change_pwd)
		}
		break;
	}
}
 
 
void __fastcall Group_Login::select_admin_log_out(TObject* sender) {
	clicked_entry = ((Stdctrls::TLabel*) sender);
 
	switch(state) {
	case STATE_ADMIN_CHANGE_PWD:
		frame_admin_change_pwd.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_admin_change_pwd1, &select_admin_change_pwd)
		UNSELECT_MENU(label_admin_change_pwd2, &select_admin_change_pwd)
 
		DISPLAY_SCREEN(frame_admin_log_out)
		break;
 
	case STATE_ADMIN_LOG_OUT:
		break;
 
	case STATE_SITE_CREATE:
		frame_site_create.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_create, &select_site_create)
 
		DISPLAY_SCREEN(frame_admin_log_out)
		break;
 
	case STATE_SITE_DELETE:
		frame_site_delete.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_delete, &select_site_delete)
 
		list_sites->Parent = NULL;
 
		DISPLAY_SCREEN(frame_admin_log_out)
		break;
 
	case STATE_SITE_LOG_IN:
		frame_site_log_in.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_log_in, &select_site_log_in)
 
		list_sites->Parent = NULL;
 
		DISPLAY_SCREEN(frame_admin_log_out)
		break;
 
	case STATE_SETTINGS_CHANGE:
		frame_settings_change.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_settings_change, &select_settings_change)
 
		b_lang_has_changed  = (lang  != IHM_GET_LANG());
		b_theme_has_changed = (theme != bundle.current_theme);
 
		if (bypass_test || (!b_lang_has_changed && !b_theme_has_changed)) {
			DISPLAY_SCREEN(frame_admin_log_out)
		} else {
			SELECT_CLICKED_MENU()
			LAUNCH_WAIT_SETTING_CHANGE(&Group_Login::select_admin_log_out)
		}
		break;
	}
}
 
 
void __fastcall Group_Login::select_site_create(TObject* sender) {
	clicked_entry = ((Stdctrls::TLabel*) sender);
 
	switch(state) {
	case STATE_ADMIN_CHANGE_PWD:
		frame_admin_change_pwd.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_admin_change_pwd1, &select_admin_change_pwd)
		UNSELECT_MENU(label_admin_change_pwd2, &select_admin_change_pwd)
 
		DISPLAY_SCREEN(frame_site_create)
		break;
 
	case STATE_ADMIN_LOG_OUT:
		frame_admin_log_out.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_admin_log_out, &select_admin_log_out)
 
		DISPLAY_SCREEN(frame_site_create)
		break;
 
	case STATE_SITE_CREATE:
		break;
 
	case STATE_SITE_DELETE:
		frame_site_delete.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_delete, &select_site_delete)
 
		list_sites->Parent = NULL;
 
		DISPLAY_SCREEN(frame_site_create)
		break;
 
	case STATE_SITE_LOG_IN:
		frame_site_log_in.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_log_in, &select_site_log_in)
 
		list_sites->Parent = NULL;
 
		DISPLAY_SCREEN(frame_site_create)
		break;
 
	case STATE_SETTINGS_CHANGE:
		frame_settings_change.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_settings_change, &select_settings_change)
 
		b_lang_has_changed  = (lang  != IHM_GET_LANG());
		b_theme_has_changed = (theme != bundle.current_theme);
 
		if (bypass_test || (!b_lang_has_changed && !b_theme_has_changed)) {
			DISPLAY_SCREEN(frame_site_create)
		} else {
			SELECT_CLICKED_MENU()
			LAUNCH_WAIT_SETTING_CHANGE(&Group_Login::select_site_create)
		}
		break;
	}
}
 
 
void __fastcall Group_Login::select_site_delete(TObject* sender) {
	clicked_entry = ((Stdctrls::TLabel*) sender);
 
	switch(state) {
	case STATE_ADMIN_CHANGE_PWD:
		frame_admin_change_pwd.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_admin_change_pwd1, &select_admin_change_pwd)
		UNSELECT_MENU(label_admin_change_pwd2, &select_admin_change_pwd)
 
		DISPLAY_SCREEN(frame_site_delete)
 
		DISPLAY_LIST_DELETE
		break;
 
	case STATE_ADMIN_LOG_OUT:
		frame_admin_log_out.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_admin_log_out, &select_admin_log_out)
 
		DISPLAY_SCREEN(frame_site_delete)
 
		DISPLAY_LIST_DELETE
		break;
 
	case STATE_SITE_CREATE:
		frame_site_create.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_create, &select_site_create)
 
		DISPLAY_SCREEN(frame_site_delete)
 
		DISPLAY_LIST_DELETE
		break;
 
	case STATE_SITE_DELETE:
		break;
 
	case STATE_SITE_LOG_IN:
		frame_site_log_in.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_log_in, &select_site_log_in)
 
		frame_site_delete.list_set_position(list_sites, main_form_top_offset,
				(main_form_left_offset + MENU_LEFT_MARGIN + menu_width), area_width);
 
//		if (node_count_changed) {
//			list_sites->RootNodeCount = app_data.nb_sites;
//			node_count_changed = false;
//		}
 
//		Select first
		list_sites->Selected[list_sites->GetFirst()] = true;
		on_dbl_click(NULL); // <- update selected_site_id
		list_sites->ScrollIntoView(list_sites->GetFirst(), true);
 
		DISPLAY_SCREEN(frame_site_delete)
		break;
 
	case STATE_SETTINGS_CHANGE:
		frame_settings_change.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_settings_change, &select_settings_change)
 
		b_lang_has_changed  = (lang  != IHM_GET_LANG());
		b_theme_has_changed = (theme != bundle.current_theme);
 
		if (bypass_test || (!b_lang_has_changed && !b_theme_has_changed)) {
			DISPLAY_SCREEN(frame_site_delete)
			DISPLAY_LIST_DELETE
		} else {
			SELECT_CLICKED_MENU()
			LAUNCH_WAIT_SETTING_CHANGE(&Group_Login::select_site_delete)
		}
		break;
	}
}
 
 
void __fastcall Group_Login::select_site_log_in(TObject* sender) {
	clicked_entry = ((Stdctrls::TLabel*) sender);
 
	switch(state) {
	case STATE_ADMIN_CHANGE_PWD:
		frame_admin_change_pwd.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_admin_change_pwd1, &select_admin_change_pwd)
		UNSELECT_MENU(label_admin_change_pwd2, &select_admin_change_pwd)
 
		DISPLAY_SCREEN(frame_site_log_in)
 
		DISPLAY_LIST_LOG_IN
		break;
 
	case STATE_ADMIN_LOG_OUT:
		frame_admin_log_out.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_admin_log_out, &select_admin_log_out)
 
		DISPLAY_SCREEN(frame_site_log_in)
 
		DISPLAY_LIST_LOG_IN
		break;
 
	case STATE_SITE_CREATE:
		frame_site_create.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_create, &select_site_create)
 
		DISPLAY_SCREEN(frame_site_log_in)
 
		DISPLAY_LIST_LOG_IN
		break;
 
	case STATE_SITE_DELETE:
		frame_site_delete.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_delete, &select_site_delete)
 
		frame_site_log_in.list_set_position(list_sites, main_form_top_offset,
					(main_form_left_offset + MENU_LEFT_MARGIN + menu_width), area_width);
 
//		if (node_count_changed) {
//			list_sites->RootNodeCount = app_data.nb_sites;
//			node_count_changed = false;
//		}
 
//		Select first
		list_sites->Selected[list_sites->GetFirst()] = true;
		on_dbl_click(NULL); // <- update selected_site_id
		list_sites->ScrollIntoView(list_sites->GetFirst(), true);
 
		DISPLAY_SCREEN(frame_site_log_in)
		break;
 
	case STATE_SITE_LOG_IN:
		break;
 
	case STATE_SETTINGS_CHANGE:
		frame_settings_change.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_settings_change, &select_settings_change)
 
		b_lang_has_changed  = (lang  != IHM_GET_LANG());
		b_theme_has_changed = (theme != bundle.current_theme);
 
		if (bypass_test || (!b_lang_has_changed && !b_theme_has_changed)) {
			DISPLAY_SCREEN(frame_site_log_in)
			DISPLAY_LIST_LOG_IN
		} else {
			SELECT_CLICKED_MENU()
			LAUNCH_WAIT_SETTING_CHANGE(&Group_Login::select_site_log_in)
		}
		break;
	}
}
 
 
void __fastcall Group_Login::select_settings_change(TObject* sender) {
	clicked_entry = ((Stdctrls::TLabel*) sender);
 
	switch(state) {
	case STATE_ADMIN_CHANGE_PWD:
		frame_admin_change_pwd.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_admin_change_pwd1, &select_admin_change_pwd)
		UNSELECT_MENU(label_admin_change_pwd2, &select_admin_change_pwd)
 
		DISPLAY_SCREEN(frame_settings_change)
		break;
 
	case STATE_ADMIN_LOG_OUT:
		frame_admin_log_out.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_admin_log_out, &select_admin_log_out)
 
		DISPLAY_SCREEN(frame_settings_change)
		break;
 
	case STATE_SITE_CREATE:
		frame_site_create.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_create, &select_site_create)
 
		DISPLAY_SCREEN(frame_settings_change)
		break;
 
	case STATE_SITE_DELETE:
		frame_site_delete.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_delete, &select_site_delete)
 
		list_sites->Parent = NULL;
 
		DISPLAY_SCREEN(frame_settings_change)
		break;
 
	case STATE_SITE_LOG_IN:
		frame_site_log_in.set_parent_ui(NULL);
 
		UNSELECT_MENU(label_site_log_in, &select_site_log_in)
 
		list_sites->Parent = NULL;
 
		DISPLAY_SCREEN(frame_settings_change)
		break;
 
	case STATE_SETTINGS_CHANGE:
		break;
	}
 
	theme = bundle.current_theme;
	lang  = IHM_GET_LANG();
 
	bypass_test = false;
}

[jo_link_noir]

Avec un tableau tu te prendrais vachement moins la tête, il n'y aurait qu'un seul switch et une condition supplémentaire dans chaque case pour les valeurs en diagonales (si l'extérieur ne le gère pas déjà).
Il y a la différence entre DISPLAY_SCREEN_ADMIN_CHANGE_PWD et DISPLAY_SCREEN(truc) à prendre en compte aussi, mais j'ai l'impression que le premier est l'équivalent d'un DISPLAY_SCREEN(frame_admin_change_pwd).

Si un tableau n'est pas possible à cause de type différent pour les variables frame_*, j'utiliserais un tuple (ou des fonctions get(std::integral_constant<le type enum, la valeur de l'enum>{}) pour récupérer la bonne variable membre si on ne veut pas de tuple) et un template du genre:

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
template<class... I, F f>
cswitch(unsigned i, std::index_sequence<I...>, F f) {
  (void)std::initializer_list<int>((void(
    i == I{} ? f(I{}) : void()
  ), 1)...);
}
 
using Seq33 = std::make_index_sequence<33>;
cswitch(idx_current_controller, Seq33{}, [new_controller](auto idx_current_controller){
  /* ici idx_current_controller est un std::integral_constant*/
  if (idx_current_controller == 6) {
    ...
  }
  else if condition 1 et 3 {
    ...
  }
  else {
    cswitch(idx_new_controller, Seq33{}, [idx_current_controller](auto idx_new_controller){
      /* ici idx_new_controller est un std::integral_constant*/
      truc commuin
    });
  }
})

À adapter en C++03 en remplaçant initializer_list + seq33 du cswitch par un vrai switch et les lambdas par des structures.

[foetus]

J'ai rajouté des choses: sur les écrans 2, 3, et 4 j'ai un panel (un cadre) pour afficher des informations: type et taille de saisie, message de suppression qui efface tout, ...

Et c'est dans le bousin que je parente/ déparente ce panel pour ainsi ne le faire qu'au bon moment - après c'est le contrôleur qui l'affiche ou le cache et le configure.

Il y a la différence entre DISPLAY_SCREEN_ADMIN_CHANGE_PWD et DISPLAY_SCREEN(truc) à prendre en compte aussi, mais j'ai l'impression que le premier est l'équivalent d'un DISPLAY_SCREEN(frame_admin_change_pwd).

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
#define DISPLAY_SCREEN(CONTROLLER) \
	SELECT_CLICKED_MENU() \
	 \
	CONTROLLER.will_appear(); \
	CONTROLLER.set_parent_ui(&main_form); \
	CONTROLLER.did_appear(); \
	 \
	state = (Group_Login::STATE) clicked_entry->Tag;
 
 
#define DISPLAY_SCREEN_ADMIN_CHANGE_PWD \
	SELECT_MENU(label_admin_change_pwd1) \
	SELECT_MENU(label_admin_change_pwd2) \
	 \
	frame_admin_change_pwd.will_appear(); \
	frame_admin_change_pwd.set_parent_ui(&main_form); \
	frame_admin_change_pwd.did_appear(); \
	 \
	state = (Group_Login::STATE) ((Stdctrls::TLabel*) sender)->Tag;

En premiere idée, je dirai une map (ou autre) d'enum vers pointeurs de fonctions

C'est une idée que j'ai eu effectivement. Mais,

• Le nombre d'évènements va fortement grandir. Ici, j'ai 6 évènements [menu de X entrées -> X évènements fixes]. Mais je dirais 70% des cas parmi 30 sont différents, soit au moins 12-15 évènements.
• Avec une matrice de 33*33 le bout de code qui va initialiser cette matrice va être tout autant imbuvable. Surtout si tu as 70% des cas qui sont différents

Avec un tableau tu te prendrais vachement moins la tête, il n'y aurait qu'un seul switch et une condition supplémentaire dans chaque case pour les valeurs en diagonales (si l'extérieur ne le gère pas déjà).

Je vais étudier ton code ... C++14 (je n'ai pas encore l'habitude) ou

À vue de nez, j'avais déjà fait comme cela avant (avant de fusionner mon groupe et mon menu).
Mais tu vas avoir plusieurs tests dans tes cas, mais chaque if peut être du code assez long.
Donc un seul switch cases mais de 1000-2000 lignes.

Si j'ai fait X switch cases c'est justement pour éviter d'avoir des tests dans les cas (même si ce n'est pas réellement vrai) et ainsi limiter la taille de mes cas: pas plus de 10 lignes (et un commentaire au niveau de chaque cas pour rappeler l'évènement pour le menu à 33 entrées)