Discussion :

[behess]

Attends, mais tu sais que c'est pas con en fait

J'étais resté sur le fait de faire appel à mon type déléguant, mais si je pointe directement sur un type déléguateur, ça le fait (sauf dans l'IDE où il va sans cesse me proposer de créer ma classe et où je n'aurais pas de complétion sur les constructeurs puisque inexistants) :

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
18
19
20
21
22
23
24
25
26
27
28
29
 
[...]
WithDelegation_MaClasse toto=new MaClasse(); // là l'IDE gueule et pas de complétion pour le constructeur (inconnu alors)
[...]
 
[...]
abstract class WithDelegation_MaClasse extends Bidule implements InterfaceMachin, InterfaceTruc {
  @Delegate
  protected InterfaceMachin fieldMachin;
  @Delegate
  protected InterfaceTruc fieldTruc;
 
  WithDelegation_MaClasse(InterfaceMachin p_machin, InterfaceTruc p_truc){
     fieldMachin=p_machin;
     fieldTruc=p_truc;
  }
 
  [...]
 
}
[...]
 
@GeneratedDelegateClass
class MaClass extends WithDelegation_MaClasse  {
  //méthodes déléguées
 
  //réplication des constructeurs avec appel à super(...);
 
}

Du coup j'ai juste à imposer une syntaxe, vérifier les cas litigieux et à utiliser un Filer

Merci maître adiGuba, toi au moins tu sais expliquer les choses !

Reste plus qu'à voir si ça marche (le plus dur sera de savoir si une méthode est implémentée dans MaClasse, Bidule, ou un autre parent).

Remarque : pour le problème de l'éditeur je devrais sans doute conserver la syntaxe

Code :

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

WithDelegate_MaClass toto=new WithDelegate_MaClass(...)

qui serait remplacé à la volée par

Code :

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

MaClass toto=new MaClass(...)

mais c'est encore super complexe...

Edit : non en fait ça ne change rien car en mettant la classe déléguante abstract j'ai un autre message d'erreur puisque ma classe est abstraite... donc finalement il me faudra supporter les erreurs de classe introuvable par l'IDE et l'absence de complétion ; avec des classes incomplètes (mon idée initiale) j'aurais eu d'autres messages d'erreur, le moindre mal est sûrement celui que tu proposes.

[Alain Defrance]

Pour l'IDE j'utilise IntelliJ qui permet de configurer un processing d'annotation et ainsi te faire de la completion sur les types générés, mais j'imagine que c'est une feature répandue.

Si tu veux rendre complètement opaque la génération de code et éviter de te trimballer 2 types tu peux tenter un truc du genre :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
public class DelegateClassLoader extends ClassLoader
{
   @Override
   public Class<?> loadClass(String name) throws ClassNotFoundException
   {
      Class<?> clazz = super.loadClass(name);
      if (clazz.isAnnotationPresent(Delegate.class))
      {
         clazz = super.loadClass("WithDelegate_" + name);
      }
      return clazz;
   }
}

C'est du sucre mais ca rend plus sexy l'utilisation de tes types générés.

[behess]

Merci Alain pour cette suggestion.

Tu sembles vouloir dire que c'est la classe déléguante (WithDelegation_MaClasse) qui serait chargée si j'écris :

Code :

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

WithDelegate_MaClasse toto=new MaClasse(...);

Cependant c'est bien la classe déléguatrice que je veux, pas la classe déléguante...

Je ne comprends donc pas à quoi ça peut servir et comment s'en servir un exemple serait le bien venu

A la limite, je comprendrai mieux de pouvoir écrire :

Code :

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

WithDelegate_MaClasse toto=new WithDelegate_MaClasse(...);

Et que ça me charge la classe délégatrice...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
public class DelegatorClassLoader extends ClassLoader
{
   @Override
   public Class<?> loadClass(String name) throws ClassNotFoundException
   {
      Class<?> clazz = super.loadClass(name);
      if (clazz.isAnnotationPresent(Delegate.class) && name.startWith("WithDelegate_"))
      {
         clazz = super.loadClass(name.substring("WithDelegate_".length+1);
      }
      return clazz;
   }
}

ou un truc comme ça

J'aurai toujours les warning sur le fait que c'est une classe abstraite, mais peut-être que c'est plus sympa à utiliser.

Par contre je ne sais pas comment utiliser ce ClassLoader...

[professeur shadoko]

sauf erreur ou omission de ma part: si tu charges la classe englobante qui fait de la délégation la classe sur laquelle la délégation opère sera chargée .... donc je ne vois pas l'intérêt de ta version de code.

[behess]

En fait je crois que j'ai compris... tu as une logique inverse de la mienne ; elle est sûrement mieux d'ailleur : je pourrais écrire :

Code :

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

MaClasse toto=new MaClasse(...);

avec :

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
18
19
20
21
22
23
public abstract class MaClasse implements List {
 
    @Delegate protected List machin;
 
    public MaClasse(List p_machin){
        this.machin=p_machin;
    }
 
}
 
@GeneratedDelegateClass
public class DelegatorMaClasse extends MaClasse {
 
    public MaClasse(List p_machin){
        super(p_machin)
    }
 
    @Override
    public boolean Add(Object p_element){
       machin.Add(p_element);
    }
 
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public class DelegatorClassLoader extends ClassLoader
{
   @Override
   public Class<?> loadClass(String name) throws ClassNotFoundException
   {
      Class<?> clazz = super.loadClass(name);
      if (clazz.isAnnotationPresent(Delegate.class))
      {
         clazz = super.loadClass("Delegator"+name);
      }
      return clazz;
   }
}

Comment je peux rendre l'instantiation du ClassLoader transparente ?

[behess]

sauf erreur ou omission de ma part: si tu charges la classe englobante qui fait de la délégation la classe sur laquelle la délégation opère sera chargée .... donc je ne vois pas l'intérêt de ta version de code.

Je veux bien mais comprends rien... tu peux expliciter s'il te plait ?

[behess]

Pour l'IDE j'utilise IntelliJ qui permet de configurer un processing d'annotation et ainsi te faire de la completion sur les types générés, mais j'imagine que c'est une feature répandue.

j'utilise aussi IntelliJ, c'est quoi cette fonctionnalité ?

[Alain Defrance]

Tiens une doc IntelliJ/JPA qui fait ça : http://blogs.jetbrains.com/idea/2009...-20-metamodel/

Pour le classloader soit tu l'appel soit, soit tu peux le définir comme classloader system.
Je te conseil de lire ça : http://download.oracle.com/javase/1....assLoader.html. Tu as une propriété system.class.loader qui t'aidera

[behess]

En fait le classloader ne fonctionne qu'à l'exécution...

Si je mets MonDeleguant abstract et
si je fais un "new MonDeleguant(...)",
à la compilation il me dira "MonDeleguant ne peut pas être instancié".

Si je mets MonDeleguant concret et
si je fais un "new MonDeleguant(...)",
à la compilation il me dira "MonDeleguant n'implemente pas complètement les interfaces".

Si je mets MonDeleguant abstract et
si je fais un "new DelegatorMonDeleguant(...)",
lors de l'édition il me dira "DelegatorMonDeleguant n'existe pas".

Donc finalement, j'aurai quoi qu'il arrive une erreur et je ne peux pas me débarrasser des problèmes de compilation sans faire appel uniquement à la classe générée

[behess]

Tiens une doc IntelliJ/JPA qui fait ça : http://blogs.jetbrains.com/idea/2009...-20-metamodel/

Pour le classloader soit tu l'appel soit, soit tu peux le définir comme classloader system.
Je te conseil de lire ça : http://download.oracle.com/javase/1....assLoader.html. Tu as une propriété system.class.loader qui t'aidera

merci pour la doc ! je vais lire ça

[behess]

Finalement, j'ai l'impression qu'il est impossible de faire de la delegation propre par les annotations.

Mais ce fut une belle aventure et j'ai appris plein de trucs sympa qui pourront toujours me resservir...

Merci pour votre participation

[Alain Defrance]

Tout est possible.

J'ai eu un peu le temps aujourd'hui en fin de journée. Je t'ai écrit le coeur du problème : https://github.com/defrancea/delegationdemo--dvp-

Il te reste quand même un TODO : la génération du fichier (qui actuellement est écrit) mais le méchanisme autours de ça est en place.

EDIT : Bien entendu c'est pas une véritable implémentation c'est juste un petit bout de code pour te montrer comment je vois les choses.

Alain.

[behess]

c'est bien ce que j'avais compris (et que j'ai aussi implémenté en ce sens sur la base de tes conseils) je me permets juste quelques remarques sur ton implémentation (mais tu as fait ça vite fait) :
- j'avais supposé une délégation d'interface, mais ton approche est intéressante
- d'où sort ce new B() ? dans l'idéal le délègué est passé par constructeur
- tu n'as qu'un constructeur par défaut ; quid d'un vrai constructeur avec ta factory ?
- être obligé de passer par une factory, en mode classe utilitaire en plus, c'est pas userfriendly. c'est déjà pas si mal, mais ça ne me plait pas trop

La conclusion disait simplement que faire une delegate userfriendly c'est pas possible : on peut toujours faire du delegate avec annotations mais c'est trop gras pour être assez pratique. Mais ça reste intéressant ! La je n'ai plus le temps d'approfondir, mais si tu arrives a quelque chose de finalisé, tiens moi au courant, sinon je me repencherai dessus dans quelques temps.

Edit: un truc me chiffonne en fait : si je veux étendre A (hors délégation), comment je fais ?

[Alain Defrance]

Salut,

c'est bien ce que j'avais compris (et que j'ai aussi implémenté en ce sens sur la base de tes conseils) je me permets juste quelques remarques sur ton implémentation (mais tu as fait ça vite fait) :

En effet.

- j'avais supposé une délégation d'interface, mais ton approche est intéressante

Peu importe, ça marcherait pareil.

- d'où sort ce new B() ? dans l'idéal le délègué est passé par constructeur

Dans l'idéal ca pourrait interessant de n'avoir pas d'instance et d'injecter le délégué et/ou le passer par constructeur mais peut importe cela ne changerai rien.

- tu n'as qu'un constructeur par défaut ; quid d'un vrai constructeur avec ta factory ?

C'est quoi le problème ? On pourrait compier les constructeurs vers le délégué, on peut également invoquer un constructeur ayant des paramètres avec de la reflexion.

- être obligé de passer par une factory, en mode classe utilitaire en plus, c'est pas userfriendly. c'est déjà pas si mal, mais ça ne me plait pas trop

On pourrait se passer de la factory en overridant le classloader system il me semble (j'ai eu à le faire). Sinon si tu préfère tu peux toujours avoir :

Code :

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

A a = new A_WithDelegation();

Pour ma part je n'aime pas rendre visible des types qui ne sont pas présent avant la compilation, et mon code ne pose pas de problèmes de completion même sans configuration de processor au niveau de l'IDE.

La conclusion disait simplement que faire une delegate userfriendly c'est pas possible : on peut toujours faire du delegate avec annotations mais c'est trop gras pour être assez pratique. Mais ça reste intéressant ! La je n'ai plus le temps d'approfondir, mais si tu arrives a quelque chose de finalisé, tiens moi au courant, sinon je me repencherai dessus dans quelques temps.

J'ai pris le temps d'écrire ca très rapidement hier soir, tu te doutes bien que j'ai du travail et que je ne peux pas m'occuper de "de faire un code source full ok, productisé & co".
Si ca t'interesse de polir/améliorer c'est à toi de jouer

Edit: un truc me chiffonne en fait : si je veux étendre A (hors délégation), comment je fais ?

Code :

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

1
2
A2 extends A
A_WithDelegation extends A2

Rien ne t'empêche de déterminer la première classe de ta hiérarchie qui délègue et la dernière.
Rien ne t'empêche non plus de créer une annotation facultative sur un type qui permet de customiser ce comportement.

Comme tu le vois (ou tu le sait), développer de la lib/framework est compliqué car ton utilisateur est un développeur et il sera tenté de faire n'importe quoi, il faut donc penser à tout et prévoir le plus de cas possible pour pas que ca chie/bride le développeur.
Néanmoins c'est le jeu et il faut avancer petit à petit et régler les problèmes les uns derrières les autres à base de test unitaire pour éviter les régressions.
Je n'ai pas vu de truc "bloquant" dans ce que tu m'a dit, le sucre que tu veux poser par dessus ou les features/use case que tu m'a donné plus haut ne sont que des implémentations supplémentaires.
Courage