Discussion :

[Nhaps]

Bonjour,

J'ai un getter

Code :

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

1
2
3
4
 
PROCEDURE PUBLIQUE p_pclRefMaVariable() : CMACLASSE
 
RENVOYER :m_pclRefMaVariable

Et je veux utiliser cette valeur, pour l'utiliser dans un constructeur de copie

Code :

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

:m_pclRefMaVariable = allouer un CMACLASSE(:m_pclAutreVariable:p_pclRefMaVariable)

Sauf que je me retrouve avec une erreur :

Code :

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

Aucune syntaxe de 'Constructeur' ne correspond à l'appel.

Or, j'ai bien un constructeur de type

Code :

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

CMACLASSE(:m_pclCOPIE est un CMACLASSE dynamique)

Si vous avez des solutions pour ce probleme...car j'ai du repasser mon membre en public et je l'appelle directement pour que ca fonctionne. Mais je me demande si il y a une solution pour ma première méthode.

Nhaps

[dsr57]

Bonjour


Je ne sais pas si la version 19 de WinDev le permet, peux tu créer un deuxième constructeur de la classe CMACLASSE sans paramètre.

Bon dev

[Nhaps]

Bonjour


Je ne sais pas si la version 19 de WinDev le permet, peux tu créer un deuxième constructeur de la classe CMACLASSE sans paramètre.

Bon dev

Dans quel but créer un deuxième constructeur par défaut ?

Merci.

[tunizar]

Bonjour

Code :

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

1
2
 
CMACLASSE(:m_pclCOPIE est un CMACLASSE dynamique)

Code :

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

1
2
 
m_pclRefMaVariable = allouer un CMACLASSE(:m_pclAutreVariable:p_pclRefMaVariable)

Est ce que p_pclRefMaVariable est de type CMACLASSE ?

[Nhaps]

Bonjour

Code :

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

1
2
 
CMACLASSE(:m_pclCOPIE est un CMACLASSE dynamique)

Code :

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

1
2
 
m_pclRefMaVariable = allouer un CMACLASSE(:m_pclAutreVariable:p_pclRefMaVariable)

Est ce que p_pclRefMaVariable est de type CMACLASSE ?

Comme écrit dans le post originel

Code :

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

1
2
3
PROCEDURE PUBLIQUE p_pclRefMaVariable() : CMACLASSE
 
RENVOYER :m_pclRefMaVariable

D'où le problème incompréhensible

[dsr57]

Bonjour

Dans quel but créer un deuxième constructeur par défaut ?

Dans la ligne de code suivante :

Code :

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

PROCEDURE PUBLIQUE p_pclRefMaVariable() : CMACLASSE

et plus précisément : CMACLASSE je suppose que WinDev appelle le constructeur de la classe à ce moment là et comme tu as seulement un constructeur avec paramètre, ce qui explique le message d'erreur.

C'est pour cette raison que je proposais d'effectuer le test avec un deuxième constructeur sans paramètre.
J'ai déjà eu un problème similaire avec les paramètres de procédures, d’où l'utilisation du mot clé dynamique.

BOn dev

[tunizar]

Comme écrit dans le post originel

Code :

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

1
2
3
PROCEDURE PUBLIQUE p_pclRefMaVariable() : CMACLASSE
 
RENVOYER :m_pclRefMaVariable

D'où le problème incompréhensible

Bonjour,

Ce code renvoi un objet de la classe CMACLASSE qui n'est pas dynamique

Code :

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

1
2
3
4
 
PROCEDURE PUBLIQUE p_pclRefMaVariable() : CMACLASSE
 
RENVOYER :m_pclRefMaVariable

cependant le constructeur attend un objet de la classe CMACLASSE qui est dynamique

Code :

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

1
2
 
CMACLASSE(:m_pclCOPIE est un CMACLASSE dynamique)

Il me semble que j'ai vu ça auparavant

[tatayo]

Bonjour,
Dans la déclaration du getter, le nom de la classe n'est là que pour typer la valeur de retour, il ne s'agit pas d'une instance.
Le seul indice ici est le nom de la propriété renvoyée: :m_pclRefMaVariable
Je parie donc que le membre est un objet dynamique, sinon le nom sera :m_clRefMaVariable.

Tatayo.

[Nhaps]

Bonjour,
Dans la déclaration du getter, le nom de la classe n'est là que pour typer la valeur de retour, il ne s'agit pas d'une instance.
Le seul indice ici est le nom de la propriété renvoyée: :m_pclRefMaVariable
Je parie donc que le membre est un objet dynamique, sinon le nom sera :m_clRefMaVariable.

Tatayo.

Oui tout à fait.
Donc je renvoie un objet dynamique.
Mais quand cette valeur renvoyé est utilisée dans le constructeur de CMACLASSE (pour une copie)
Cela ne fonctionne pas...

[tunizar]

Oui tout à fait.
Donc je renvoie un objet dynamique.
Mais quand cette valeur renvoyé est utilisée dans le constructeur de CMACLASSE (pour une copie)
Cela ne fonctionne pas...

Bien venu dans le monde merveilleux du copie d'objet WinDev !!
un objet dynamique vers un objet instancié non dynamique ne marche pas correctement qu'en utilisant la compilation dynamique
c'est ce que je fais toujours

[tatayo]

Tunizar: sauf qu'ici, d'après ce que je vois du code, nous n'avons que des objets dynamiques...
Bref dans le code un instance d'un objet dynamique, renvoyé par une procédure (ici une propriété), est utilisé comme paramètre dans une autre procédure (le constructeur) qui attend en paramètre... un objet dynamique de la même classe.
C'est d'ailleurs un peu le principe qui est utilisé pour faire un Décorateur ou un Adaptateur.
Bref rien dans le code présenté n'explique le dysfonctionnement.

Nhaps: Est-ce que tu as essayé de passer par une variable intermédiaire ?
J'ai eut des cas où l'affectation depuis une propriété d'une instance vers une propriété d'une autre instance ne fonctionnait pas (alors que le deux traitaient des chaines), et WinDev m'indiquait qu'il ne pouvait pas convertir une chaine... en chaine.
J'avais été obligé de passer par une fonction, du genre Remplace(MonInstance:MaPropriété,"",""). Ca ne fait rien, mais du coup mon code fonctionne. C'était avec la version 18, mais j'avoue que je n'y ai pas touché depuis (quand ça fonctionne, ne touche à rien !).

Tatayo.

[Nhaps]

Nhaps: Est-ce que tu as essayé de passer par une variable intermédiaire ?
J'ai eut des cas où l'affectation depuis une propriété d'une instance vers une propriété d'une autre instance ne fonctionnait pas (alors que le deux traitaient des chaines), et WinDev m'indiquait qu'il ne pouvait pas convertir une chaine... en chaine.
J'avais été obligé de passer par une fonction, du genre Remplace(MonInstance:MaPropriété,"",""). Ca ne fait rien, mais du coup mon code fonctionne. C'était avec la version 18, mais j'avoue que je n'y ai pas touché depuis (quand ça fonctionne, ne touche à rien !).
Tatayo.

Oui j'ai essayé de passer par une variable intermédiaire, mais cela n'a pas résolu le problème.

LE bug est facile à reproduire chez vous, suffit de récupérer la variable dynamique via un getter, et d'utiliser ce getter en paramètre d'un constructeur de copie (voir même autre type de fonction)

Ça m’embête un peu, car j'essaye de faire une belle encapsulation des données, mais des soucis comme ça, m'empêche de faire un vrai code propre, et au final on est facilement tenté de mettre tout en public...

[yoshi570]

Je suis en train de me dire que je suis un con sur le coup.
Tout le monde a compris ou c'est juste moi?
Quand tu dis:

J'ai un getter

Code :

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

1
2
3
4
 
PROCEDURE PUBLIQUE p_pclRefMaVariable() : CMACLASSE
 
RENVOYER :m_pclRefMaVariable

pclRefMaVariable est un CMACLASSE dynamique ... DANS CMACLASSE ?
Ca n'as peut-être aucune utilité dans ce que tu veux faire, mais il y a le mot clé "objet" pour faire référence a l'objet/instance en cours.

Or, j'ai bien un constructeur de type

Code :

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

CMACLASSE(:m_pclCOPIE est un CMACLASSE dynamique)

Pareil, c'est quoi cette syntaxe ?
Pour quoi le ":" dans la liste des paramètres ??
En 20 et 21 il n'accepte pas cette syntaxe.

[Nhaps]

Le premier code se situe dans une autre classe.

Et en effet dans le second code, j'ai mis les ":" en trop en voulant reproduire l'exemple, petite erreur de ma part.
D'ailleurs c'est même

Code :

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

PROCEDURE Constructeur(pclCOPIEMAVARIABLE est un CMACLASSE dynamique)

Mais non tu n'es pas fou
C'est mon problème qui me rend fou :p

[tatayo]

Peut-être une piste...
Est-ce que tu as essayé ainsi:

Code WinDev :

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

1
2
 
CMACLASSE(LOCAL m_pclCOPIE est un CMACLASSE dynamique)

D'après l'aide en ligne, avec un paramètre local la propriété est "lue" au moment de l'appel, et non dans le code lors de la lecture du paramètre.

Tatayo.

[Nhaps]

Peut-être une piste...
Est-ce que tu as essayé ainsi:

Code WinDev :

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

1
2
 
CMACLASSE(LOCAL m_pclCOPIE est un CMACLASSE dynamique)

D'après l'aide en ligne, avec un paramètre local la propriété est "lue" au moment de l'appel, et non dans le code lors de la lecture du paramètre.

Tatayo.

Je viens d'essayer, mais cela ne fonctionne pas.

[yoshi570]

Mais non tu n'es pas fou

Merci

Bon, je viens d'essayer aussi. En effet, en utilisant directement la propriété comme tu le montre cela ne marche pas.

Mais, j'ai trouvé deux alternatives, de contournement...

1. Utilise un paramètre non typé dans le constructeur
   Pour éviter les surprises tu peut vérifier "manuellement" le type
   

   Code :

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

   1
   2
   3
   4
   5
   PROCEDURE Constructeur(unobjet)
   MaDéfinition est une Définition =  RécupèreDéfinition(unobjet)
   SI MaDéfinition..Nom = "CMACLASSE" ALORS
   	m_pclCopie <- unobjet
   FIN

2. Utilise un objet intermédiaire
   

   Code :

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

   1
   2
   3
   4
   clCopieTemp est un CMACLASSE dynamique
   clCopieTemp = allouer un CMACLASSE
   clCopieTemp <- pclTest1.p_pclRefMaVariable
   MaVariable = allouer un CMACLASSE(clCopieTemp)

   Biensur il te faut un constructeur sans arguments

[Nhaps]

...

Merci. En effet tes solutions fonctionnent et permettent de garder une encapsulation propre, bien que les détournements ne soient pas forcement très agréables mais bon il y a pire.

Encore une fois, un détournement de trouvé dans l'utilisation de Windev...pas le premier, ni le dernier ^^