Discussion :

[Bolamix]

Bonjour,

Quelqu'un pourrait-il m'expliquer pourquoi la méthode suivante ne compile pas (cannot find symbol: class beanClass) ?

Code :

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

1
2
3
4
5
6
 
private void printBeanId(String className, Object obj) throws ClassNotFoundException {
    Class beanClass = Class.forName(className);
    beanClass bean = (beanClass) obj;
    System.out.println("Bean Id: " + bean.getId());
}

Cette méthode est juste un exemple, mais j'aimerais utiliser ce principe (classe dynamique) afin de supprimer du code à double (ou la seule chose qui change est le type de mes objets) et j'aimerais faire cela sans interface...

Merci d'avance pour vos réponses

[LittleBean]

Code :

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

1
2
 
beanClass bean = (beanClass) obj;

Le compilo ne connait pas cette classe : beanClass

Tu ne peux pas travailler au niveau dynamique ET demander au compilateur de connaître ce que tu fais .... c'est dans deux phase différentes

par exemple pour faire un cast tu dois :

Code :

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

1
2
3
4
5
6
7
private void printBeanId(String className, Object obj) throws ClassNotFoundException {
    Class beanClass = Class.forName(className);
    if (beanClass.isInstance(obj)){
 /// tu as bien un objet de type className
}
    System.out.println("Bean Id: " + bean.getId());
}

Bonne chance

[Bolamix]

Merci

J'ai finalement réussi en utilisant getMethod() et invoke():

Code :

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

1
2
3
4
public void printBeanId(Object bean) throws Exception {
  System.out.println("Bean(" + bean.getClass().getName() + ") ID : " +
    bean.getClass().getMethod("getId", new Class[0]).invoke(bean, new Object[0]));
}

ce qui me permet d'utiliser la même méthode pour tous mes beans qui ont une méthode getId() sans implémenter d'interface:

Code :

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

1
2
3
printBeanId(new Account());
printBeanId(new Sales());
printBeanId(new Channel());

Et ca marche

Code :

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

1
2
3
Bean(beans.Account) ID : 1
Bean(beans.Sales) ID : 1
Bean(beans.Channel) ID : 1

[Invité]

juste pour info :
http://java.sun.com/javase/6/docs/ap...lang.Object%29
je ne sais pas si ça aurait pu être utile ...

[Grimly_old]

Une interface tuerais-t-il votre code ?
Une simple "interface IndentifiedBean { int getId(); }" suffirais.

[LittleBean]

Une interface tuerais-t-il votre code ?
Une simple "interface IndentifiedBean { int getId(); }" suffirais.

Je pense qu'il a pour instruction : " faut toucher le moins possible au code existant ... on a peur que ça explose de partout "

[Invité]

c'est vrai que l'interface est ce qu'il y a de mieux dans ce cas,
Mais si ce n'est pas possible, je pense qu'on peut combiner le isInstance() et Class.cast().
Ca fait moins "bricolage" que le invoke (personellement).

[adiGuba]

Salut,

Mais si ce n'est pas possible, je pense qu'on peut combiner le isInstance() et Class.cast().
Ca fait moins "bricolage" que le invoke (personellement).

Sauf qu'en restant générique et sans type de base définissant une méthode getId() (interface ou classe), il est impossible d'appeler la méthode sans passer par la reflection et invoke()...


a++

[Bolamix]

Je sais qu'avec des interfaces ça serait plus simple et plus "joli", mais comme les beans et les jpacontrollers sont générés par NetBeans d'après la base de données, j'ai pas trop envie de devoir les modifier (si la BDD est modifiée, je préfère regénérer le bean et le controller sans devoir remettre mes modifs).
Pour les id, il suffirait effectivement d'implémenter une interface comme proposée par Grimly.
Pour les jpacontrollers par contre, implémenter une interface ne suffit pas car les méthodes prennent le bean en question en paramètre et pas un Object, il faudrait alors modifier les méthodes du controller avec un paramètre Object et le typecaster dans le type du bean dans la méthode => modification du code généré par NetBeans...

Voila actuellement le code auquel je suis arrivé:

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
 
  private void deleteSelectedBean(List beans, JList list, Object jpaCtrl, String idConfirmMessageI18n, String idConfirmTitleI18n) {
    try {
      if (Tools.questionBox(idConfirmMessageI18n, idConfirmTitleI18n)) {
        Object del = beans.get(list.getSelectedIndex());
        jpaCtrl.getClass().getMethod("destroy", new Class[]{Long.class}).invoke(jpaCtrl, new Object[]{
          del.getClass().getMethod("getId", new Class[0]).invoke(del, new Object[0])
        });
        beans.remove(del);
        list.setSelectedIndex(beans.size()-1);
        list.ensureIndexIsVisible(list.getSelectedIndex());
      }
    } catch (Exception ex) {
      System.err.println("Exception: " + ex.toString());
    }
  }
 
  private void btnDelAccountActionPerformed(java.awt.event.ActionEvent evt) {                                              
    deleteSelectedBean(accountList, lstAccounts, JPA.ctrlAccount, "messageDeleteAccount.message", "messageDeleteAccount.title");
  }                                             
 
  private void btnDelSalesTeamActionPerformed(java.awt.event.ActionEvent evt) {                                                
    deleteSelectedBean(salesList, lstSalesTeam, JPA.ctrlSales, "messageDeleteSalesTeam.message", "messageDeleteSalesTeam.title");
  }                                               
 
  private void btnDelChannelActionPerformed(java.awt.event.ActionEvent evt) {                                              
    deleteSelectedBean(channelList, lstChannels, JPA.ctrlChannel, "messageDeleteChannel.message", "messageDeleteChannel.title");
  }

Sachant que j'ai encore une quinzaine de beans à mettre, ca m'évite d'avoir du code à double pour tous ces beans...

Après si quelqu'un a une meilleure idée sans modifier les beans et les controllers, je suis preneur

Merci