Discussion :

[sat83]

Bonjour a tous!

Une petite question qui me turlupine, pour l'illustrer prenons un exemple bateau:

Code :

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

1
2
3
4
5
6
7
void Exchange( AnsiString &s1, AnsiString &s2 )
{
  AnsiString tmp ;
  tmp = s1;
  s1  = s2;
  s2  = tmp;
}

Je veux maintenant échanger le contenu de 2 TEdits :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void __fastcall TForm1::BitBtn1Click(TObject *Sender)
{
  Exchange( Edit1->Text, Edit2->Text ) ;
}
//---------------------------------------------------------------------------
void __fastcall TForm1::BitBtn2Click(TObject *Sender)
{
  AnsiString tmp1 = Edit1->Text ;
  AnsiString tmp2 = Edit2->Text ;
  Exchange( tmp1, tmp2 ) ;
  Edit1->Text = tmp1 ;
  Edit2->Text = tmp2 ;
}

Pourquoi est ce que la première version ne marche pas, et qu'en passant par des variables temporaires (2ème version) ça fonctionne??

[pottiez]

Salut,
je pense, mais je ne suis absolument pas sur, que c'est parce que derrière la propriété de ton composant il n'y as pas directement une variable, mais un accesseur qui va lui écrire ou lire dans une variable, et donc lorsque tu prend la référence, c'est celle de l'accesseur et non celle de la variable

[sat83]

C'est ce que j'avais aussi conclu, que la propriété Text devait être un truc du genre :

Code :

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

__property AnsiString Text = { read=GetText, write=SetText};

Ce qui empêche d'utiliser directement une référence sur cette propriété.

[kmaniche]

Ta première version de programme, elle utilise deux variable AnsiString, alors que la deuxième utilise les propriétés des deux conposants. Ce que je peux te proposer :

Code :

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

1
2
3
4
5
6
7
8
9
 
void Exchange(TEdit *edt1, TEdit *edt2)
{
  AnsiString tmp ;
  tmp = edt1 -> Text ;
  edt1 -> Text  = edt2 -> Text;
  edt2 -> Text  = tmp;
}

[pottiez]

Oui mais ta méthode ne peut plus servir que au TEdit alors que celle de sat83 peut servir pour tous ce qui est AnsiString

[kmaniche]

je pense qu'il veux faire permuter le contenu des deux TEdit !

[pottiez]

Oui, la en l'occurrence c'est le contenu de deux TEdit, mais le but de faire une méthode est tout de même de pouvoir la réutiliser pour le plus de cas possible, donc si ta méthode ne sert qu'une fois elle ne sert à rien

[sat83]

je pense qu'il veux faire permuter le contenu des deux TEdit !

Dans mon cas l'exemple de permutation de chaîne de caractère était simplement fais pour illustrer le problème de passage de propriété par référence lors d'un appel à une fonction.

[henderson]

Salut !

Ca fonctionne avec des données membres mais pas avec des propriétés.
Notons qu'avec des propriétés, les alias sont bien modifiés (AnsiString& donne lieu à une copie d'objet).
Par contre, le contenu de l'alias n'est pas répercuté sur la donnée cible !

A plus !

[kmaniche]

le but de faire une méthode est tout de même de pouvoir la réutiliser pour le plus de cas possible, donc si ta méthode ne sert qu'une fois elle ne sert à rien

Pour satisfaire le besoin, je propose une implementation en "template" ...

[pottiez]

Je n'irai pas jusque la, mais pourquoi pas

[kmaniche]

C'est une solution plus générale au problème, celle-ci permettra de permuter des chaine de caractères, nombres et d'autres types pré ou définie par l'utilisateur.

[pottiez]

Je le sais bien, mais bon après tout dépend de tes besoins

[Invité]

En fait, la propriété Text est définie, dans le header Controls.hpp, de la façon suivante :

Code :

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

1
2
3
__property AnsiString Text = {read=GetText, write=SetText};
AnsiString __fastcall GetText();
void __fastcall SetText(const AnsiString Value);

Quand tu écris

Code :

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

AnsiString &s=Edit1->Text;

Le compilateur va en fait appeler Edit1->GetText(), qui renvoie une chaine sur laquelle s fait référence. Jusque là tout va bien...

Mais changer ladite référence n'appellera pas pour autant Edit1->SetText(s) (ce que fait Edit1->Text=s

Du coup :

Code :

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

1
2
3
4
5
6
7
8
9
 
void __fastcall TForm1::Button1Click(TObject *Sender)
{
String &s=Edit1->Text;
String &t=Edit2->Text;
String tmp=s;
s=t;
t=tmp;
}

Ne fonctionne pas: à la sortie de la fonction, s,t, et tmp ont bien les bonnes valeurs, mais leur affectation n'appelle pas Edit1->SetText();

En revanche...

Code :

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

1
2
3
4
5
6
7
8
9
 
void __fastcall TForm1::Button1Click(TObject *Sender)
{
String &s=Edit1->Text;
String &t=Edit2->Text;
String tmp=s;
Edit1->Text=t;
Edit2->Text=tmp;
}

Fonctionnera...Et, comme

Code :

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

String &s=Edit1->Text;

crée en fait une copie de la chaine,

Code :

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

1
2
3
4
5
6
7
void __fastcall TForm1::Button1Click(TObject *Sender)
{
String &s=Label2->Caption;
String &t=Label3->Caption;
Label2->Caption=t;
Label3->Caption=s;
}

Fonctionnera, contre toute attente !

Enfin bon, une référence (ou un pointeur) sur une propriété, faut pas !

Francois