Discussion :

[wolverine1987]

Bonjour,

J'ai un problème, lorsque j'appui sur mon bouton "précédente" qui se trouve sur une boite de dialog celui-ci décrémente les pages, sauf que il décrémente toujours?? je ne c pas comment l'arrêter?
voici mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
//FONCTION QUI PERMET DE PASSER A LA PAGE PRECEDENTE
void CIndicDlg::OnPrecedente() 
{
	if (nombreFoisAffich>1)	//si le nombre de page est sup à 1, cad (1/n) si n est sup à 1
	{
		if(n!=nombreFoisAffich || n==nombreFoisAffich) // si n est different du nombre de page affiché ou n est égal à la page affiché
		{			
			n=n-1;				// on décrémente n
		}
		numeroPage.Format("%d/%d",n,nombreFoisAffich); //on affiche la page
	}
}

[ram-0000]

Heu ... là, tu as un problème

Code :

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

if(n!=nombreFoisAffich || n==nombreFoisAffich)

Ce test est toujours vrai car "n" est toujours soit différent soit égal à "nombreFoisAffich".

[wolverine1987]

Je suis obligé car quand ma page arrive a 5/5 par exemple est bien la variable n sera égale a nombreFoisAffich donc je ne pourrai pas décrémenter ma page si je ne met pas cette condition!!!

[ram-0000]

Oui mais ce test ne peux JAMAIS être faux, regarde le encore un peu et mets des valeurs à tes variables
si (4!=12 || 4==12), c'est toujours vrai donc ce test ne sert à rien !!

[wolverine1987]

Je comprend ce que vous me dîtes mais alors comment faire??????

[TheGzD]

Je comprend ce que vous me dîtes mais alors comment faire??????

en enlevant le test totalement inutile, tu ne crois pas ?

Il y a surtout un pb de logique dans ton code, car tu es sensé utiliser une variable pour stocker le nombre de pages tournées et une variable pour le numéro de la page courante : là tu n'as qu'une seule variable, d'où le problème.

[wolverine1987]

ok j'ai vraiment compris c'est vrai que cette fonction ne sert à rien!!!

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
void CIndicDlg::OnPrecedente() 
{
	if (nombreFoisAffich>1)	//si le nombre de page est sup à 1, cad (1/n) si n est sup à 1
	{
		n--;				// on décrémente n
		numeroPage.Format("%d/%d",n,nombreFoisAffich); //on affiche la page
 
			if(n==1)
			{
				GetDlgItem(IDC_PRECEDENTE)->EnableWindow(FALSE);
			}
			else
			{
				GetDlgItem(IDC_PRECEDENTE)->EnableWindow(TRUE);
			}
	}
}

Le problème est que lorsque je clic sur le bouton précédent celui-ci décrémente comme je le souhaite, mais lorsque j'arive à la page 1/7 par exemple et que je clic sur le bouton celui-ci décrémente alors qu'il ne devrait pas.

Infos pour les variables : ex : n/nombreFoisAffich , 1/7, 1=n et 7=nombreFoisaffich.

Pour contourner le problème de la décrémentation,lorsque j'arive à n=1, alors je grise le bouton précédent. mais si j'incrémente le nombre de page alors celui-ci reste grisé. Où doit-je réinitialiser cette partie?

PS : dans le code ci-dessus le else ne sert à rien!!!

[ram-0000]

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
void CIndicDlg::OnPrecedente()
{
   GetDlgItem(IDC_SUIVANTE)->EnableWindow(TRUE);
   if(n == 1)
   {
      GetDlgItem(IDC_PRECEDENTE)->EnableWindow(FALSE);
   }
}
 
void CIndicDlg::OnSuivante()
{
   GetDlgItem(IDC_PRECEDENTE)->EnableWindow(TRUE);
   if(n == 7) // ou 8, je sais pas, a toi de voir
   {
      GetDlgItem(IDC_SUIVANTE)->EnableWindow(FALSE);
   }
}

[wolverine1987]

ça fonctionne je te remercie pour ton aide!!!