Discussion :

[Taiby]

Bonjour,

j'ai un fichier ressource qui me permet de gérer différents panneaux dont un où il y a 99 cases à cocher (gestion des voies d'un appareil de mesure).
Dans la procédure de gestion de ce panneau je vais initialiser ces cases à cocher et les relire lors de l'appui sur un bouton "OK". Je fais actuellement ces traitements case après case, les unes à la suite des autres, ex:

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
 
if (LOWORD(wParam) == ID_VALIDER )
                {
                nvoie=0;            
                if( IsDlgButtonChecked( hDlg, IDR_VOIE01 ) )
                    {
                    lvoie[nvoie]= 1;
                    nvoie++;
                    }
                if( IsDlgButtonChecked( hDlg, IDR_VOIE02 ) )
                    {
                    lvoie[nvoie]= 2;
                    nvoie++;
                    }
                    .
                    .
                    .
                if( IsDlgButtonChecked( hDlg, IDR_VOIE99 ) )
                    {
                    lvoie[nvoie]= 99;
                    nvoie++;
                    }
                    }

Avouez que c'est pas beau hein

Je voudrais donc faire cela dans une boucle mais je ne sais pas comment adresser les IDR_VOIExx dans celle-ci. Je sais le faire en VBA (enfin j'ai su) mais en C ... Il doit bien exister un moyen, aussi si quelqu'un peu éclairer ma lanterne cela serait sympa,

Merci d'avance
Taiby

[jeroman]

Salut

Code :

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

if( IsDlgButtonChecked( hDlg, IDR_VOIE01 ) )

Le problème, c'est la présence de ces constantes. Vérifie si les constantes VOIExx sont toutes incrémentées de 1 dans l'intervalle 1<=xx<=99. Si c'est le cas (et uniquement dans ce cas), alors il y a une méthode -que je qualifierais de pas très propre- pour s'en sortir :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
if (LOWORD(wParam) == ID_VALIDER )
{
	int i;
	nvoie = 0;
	for (i = 0 ; i < 99 ; i++)
	{
		if( IsDlgButtonChecked( hDlg , IDR_VOIE01 + i ) )
		{
			lvoie[nvoie] = i + 1 ;
			nvoie++;
		}
	}
}

En cas de modification de la ressource, il faudra TOUJOURS s'assurer que VOIExx va de 1 en 1. La valeur de VOIE01 n'a aucune importance ici.

Si VOIExx ne va pas partout de 1 en 1, il y a la méthode -plus propre mais plus fatigante- d'un tableau :

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
int tabRessourcesVoies[[99] = { VOIE01 , VOIE02 , VOIE03 , VOIE04 , ...... } ; /* jusqu'à VOIE99 */
 
if (LOWORD(wParam) == ID_VALIDER )
{
	int i;
	nvoie = 0;
	for (i = 0 ; i < 99 ; i++)
	{
		if( IsDlgButtonChecked( hDlg , tabRessourcesVoies[i] ) )
		{
			lvoie[nvoie] = i + 1 ;
			nvoie++;
		}
	}
}

L'intérêt, c'est que, quelles que soient les valeurs de VOIExx, ça marchera.

[Taiby]

Oui en effet, je voyais cela plus compliqué mais il n'y a pas de raison.
Cela marche parfaitement, j'ai opté pour la solution 1 sachant que mes numéros de voie se suivent et qu'il n'y a pas de raison que cela change.


Un grand merci !!