Problème pour retrourner une valeur

Version imprimable

Bonjour,

Voila deux jour que j'essaie de faire une fonction "ERREUR" mais j'ai un problème et je ne trouve rien.

Donc désespérément je me tourne vers vous.

Je vous explique donc mon problème.
Je souhaite définir des erreurs :
Code:

1 2 3 4 5 6 7 8 #define NO_ERR 0x00000001 #define ERROR_1 0x00000002 #define ERROR_2 0x00000004 #define ERROR_3 0x00000008 . . . #define ERROR_31 0x80000000
Dans l'application d'un bouton je vérifie les valeurs contenues dans 2 EDIT
Si la valeur contenu dans l'EDIT dépasse une certain valeur alors je lui affecte une erreur
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 void __fastcall TMainForm::btnValiderClick(TObject *Sender) { int ERR=NO_ERR; . . . if (duree < 1 || duree > 1000) ERR |= ERROR_3; . . . et puis a la fin je met un return (ERR); }
Mais je ne peux pas... pkoi??
PS : je traville sous C++ Builder 6

21/01/2009, 15h25
Médinoc

Parce que ta fonction est marquée comme ne retournant rien (void) ?
21/01/2009, 15h26
fmr130

oui je suis d'accord mais si je mets un int TMainForm::btnValiderClick(TObject *Sender)
il me dis que ça ne marche pas comme ça !!!
21/01/2009, 15h29
fmr130

Faut il changer le ModalResult du boutton??
21/01/2009, 15h32
Médinoc

Mais cette fameuse erreur, tu cherches à la retourner à qui exactement ?
À la fonction qui appelle la boîte de dialogue modale ? Directement à l'affichage ?

Directement à l'affichage...

Exemple que l'on me dise ERREUR Valeur trop grande.

Voila la solution de facilité en utilisant des if, else... et puis switch case.
Mais j'ai 31 erreur possible... donc ça serait vraiment pas joli lol.

Code:

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
void __fastcall TForm1::ValiderClick(TObject *Sender)
{
   double amplitude, duree, total;
   int erreur;
 
   amplitude = Amplitude->Text.ToDouble();
   duree = Duree->Text.ToDouble();
 
   if (amplitude < 0 || amplitude > 12)
   {
      if (duree < 1 || duree > 1000)
      erreur = 3;
      else
      erreur = 1;
   }
   else if (duree < 1 || duree > 1000)
   erreur = 2;
   else erreur = 0;
 
   switch(erreur)
   {
      case 1:
      {
         ShowMessage ("L'amplitude doit être comprise entre"
         " 0 et 12 Volts");
         break;
      }
      case 2:
      {
         ShowMessage ("La durée des impulsions doit être comprise entre"
         " 0 et 1000 ms");
         break;
      }
      case 3:
      {
         ShowMessage ("L'amplitude doit être comprise entre 0 et 12 Volts"
         "\net la durée des impulsions doit être comprise entre"
         " 0 et 1000 ms");
         break;
      }
      default:

21/01/2009, 15h44
Médinoc

Dans ce cas, tu n'as aucun return à faire: Tu affiches directement dans ton handler de message.

Le mieux, c'est de faire une string, regarder tous les bits de ERR, et y ajouter le message correspondant à chaque bit. Puis, afficher le tout...
21/01/2009, 15h52
fmr130

Bonne idée... je vais essayer mais le résultat n'est pas assurer lol

Donc l'idéal est de créer un String ERR puis d'ajouter toutes les erreurs et
à la fin je fais un ShowMessange(ERR)

Cette technique marche si j'ai 2 boutons différents qui vérifient des valeurs différentes?? donc ils donnent des erreurs différentes style valeur1 bouton 1 faux et valeur 2 bouton 2 faux dans le meme message??
21/01/2009, 15h55
Médinoc

Ben non, tu sauras seulement que les deux erreurs sont présentes, mais pas quelle champ a provoqué quelle erreur...
21/01/2009, 16h01
fmr130

C'est donc la, la faiblesse de cette méthode.
Car j'aimerai bien indiquer sur quel champs la personne c'est trompée
et non avoir qu'un message identique pour tous.

Comment je serais au fait que les 2 erreurs sont présentes??

Vois tu une autre solution sinon??
21/01/2009, 16h11
Médinoc

Citation:

Envoyé par fmr130

Comment je serais au fait que les 2 erreurs sont présentes??

Ben, parce que ce sont des flags

Citation:

Vois tu une autre solution sinon??

Une liste de couples { erreur, champ } ou un truc du genre...
21/01/2009, 16h11
fmr130

Si j'obtiens un truc comme Erreur Durée et Erreur Amplitude sur l'affichage, cela me va!!!

Merci je vais pas plus t'embeter...
je vais voir ça...

J'ai trouver une solution...

Il faut créer une methode dans la classe en question

style:
Code:

1 2 public: // Déclarations de l'utilisateur void __fastcall GestionErreur(error);
puis faire un

Code:

enum Erreur {ERROR_Amplitude, ERROR_Duree, ERROR_Ampli_Duree};

Dans ma méthode je met
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 void __fastcall TMainForm::GestionErreur(error) { switch(error) { case ERROR_Amplitude: ShowMessage ("Amplitude\n"); break; case ERROR_Duree:ShowMessage ("Durée\n"); break; case ERROR_Ampli_Duree: ShowMessage ("Amplitude et Durée\n"); break; default: ShowMessage ("Erreur inconnue."); break; } }
et enfin dans mon bouton je fais:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 void __fastcall TMainForm::btnValiderClick(TObject *Sender) { double amplitude, duree, total; amplitude = edtAmplitude->Text.ToDouble(); duree = edtDuree->Text.ToDouble(); if ((amplitude < 0 || amplitude > 12) && (duree < 1 || duree > 1000)) GestionErreur(ERROR_Ampli_Duree); if (duree < 1 || duree > 1000) GestionErreur(ERROR_Duree); else if ((amplitude < 0) || (amplitude > 12)) GestionErreur(ERROR_Amplitude);
il reste néanmoins un problème... la il n'y a que 2 boites Edit donc 3 possibilités. Mais s'il y a 4 boites Edit il y a 24 solutions je crois!!! bref facile, d'oublier un cas... Quelqu'un à une idée??? pour simplifier... faire apparaitre toutes les erreurs dans la même fenêtre et en même temps?
Par exemple pour le cas de 2 boites Edit faire une instruction pour ERROR_Duree et une pour ERROR_Amplitude. La troisième c'est à dire ERROR_Ampli_Duree... doit apparaitre sans mettre le cas :
Code:

1 2 if ((amplitude < 0 || amplitude > 12) && (duree < 1 || duree > 1000)) GestionErreur(ERROR_Ampli_Duree);