Gerer bouton pour décrémenter une page dans une MFC

Version imprimable

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:

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 } }

21/04/2009, 14h01
ram-0000

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

Code:

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

Ce test est toujours vrai car "n" est toujours soit différent soit égal à "nombreFoisAffich". :P
21/04/2009, 14h14
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!!!:)
21/04/2009, 14h27
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 !!
21/04/2009, 14h44
wolverine1987

Je comprend ce que vous me dîtes mais alors comment faire??????:marteau:
21/04/2009, 15h19
TheGzD

Citation:

Envoyé par wolverine1987

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

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.

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

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!!!

Code:

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);
   }
}

22/04/2009, 09h08
wolverine1987

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