Netbeans warning sur classe abstraite

Version imprimable

16/10/2010, 17h40
helpy2

Netbeans warning sur classe abstraite
Bonjour à tous,

Voici mon souci.

J'ai crée une classe abstraite contenant 2 methodes abstraites :
Code:

1 2 3 4 5 6 7 8 public abstract class A extends B { this.method1(); this.method2(); } public abstract void method1(); public abstract void method2();
Le souci, c'est que Netbeans me signale ceci :
-Make the class A final
-Make the method method1 "final" ou "static" ou "private"
-(Même chose pour method2)

Et je ne comprends pas pourquoi il me propose ces corrections.
Je ne veux pas mettre ma classe final, de même pour les méthodes (si je l'ai mise abstraite, c'est pour être dérivable)
Static, je ne vois pas l'intérêt.
Private, c'est incompatible avec une méthode abstraite non ?

Merci :)

Salut,

c'est ca le code que tu as mis dans ta classe ?

Code:

1
2
3
4
5
6
7
8
public abstract class A extends B
{
    this.method1();
    this.method2();
}
 
public abstract void method1();
public abstract void method2();

Euh non pardon merci :mouarf:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 public abstract class A extends B { public A() { this.method1(); this.method2(); } public abstract void method1(); public abstract void method2(); }
Je sens que j'ai fait un truc minable dans le code :mrgreen:

16/10/2010, 18h21
thelvin

Ce ne sont pas des erreurs de compilation. Netbeans te donne ces conseils, parce que cette construction est un nid à bugs.

J'explique : ces méthodes sont appelées dans le constructeur de ta classe abstraite. Ce constructeur va donc être appelé au début des constructeurs de ses classes filles. Et il va donc appeler l'implémentation de ces méthodes par ces classes filles, avant que leur initialisation ne commence. Tu avais réalisé ça ? La plupart ne le font pas, et de toute façon ça se voit pas tout de suite.
En général, les erreurs arrivent là.

Si ces méthodes n'étaient ni abstraites, ni redéfinissables (si elles étaient final, donc,) alors le problème n'existerait pas. D'où le conseil donné.
16/10/2010, 18h36
helpy2

Ok merci je comprends.
Je me suis d'ailleurs taper un nullpointerexception après le lancement.
Dès que je les retire de la classe, ça fonctionne.
Le truc qui me chagrine, c'est que pour chacune des filles, j'appelle ces méthodes à leur construction.
Vu comme je compte faire plusieurs méthodes en appliquant ce même mécanisme, c'est la raison pour laquelle je souhaite faire ceci dans la mère.
Vous avez une alternative à me proposez ?
16/10/2010, 20h10
thelvin

Citation:

Envoyé par helpy2

Le truc qui me chagrine, c'est que pour chacune des filles, j'appelle ces méthodes à leur construction.
Vu comme je compte faire plusieurs méthodes en appliquant ce même mécanisme, c'est la raison pour laquelle je souhaite faire ceci dans la mère.
Vous avez une alternative à me proposez ?

Fais qu'il n'y ait qu'une seule méthode à appeler dans le constructeur des classes filles. Cette méthode-là peut être implémentée dans la classe abstraite, puisque c'est pareil pour toutes, et reposer sur l'appel de méthodes abstraites.
16/10/2010, 22h18
helpy2

Citation:

Envoyé par thelvin

Fais qu'il n'y ait qu'une seule méthode à appeler dans le constructeur des classes filles. Cette méthode-là peut être implémentée dans la classe abstraite, puisque c'est pareil pour toutes, et reposer sur l'appel de méthodes abstraites.

Ah oué bonne idée !
Mais ça reste une méthode dissimulée si tu vois ce que je veux dire.
Ca cache le warning, mais ça ne résout pas le problème.
Donc si j'ai bien compris je n'ai pas le choix non ?
16/10/2010, 22h44
thelvin

Citation:

Envoyé par helpy2

Ca cache le warning, mais ça ne résout pas le problème.

Ben si. Réfléchis. Le problème de base était qu'on appelait methode1() et methode2() des classes filles avant leur initialisation. Là le principe est de le faire à l'intérieur de leur propre constructeur, donc après les initialisations et de préférence à un moment où tout est prêt. Problème réglé, donc.

Il reste deux problèmes, c'est vrai :
- c'est qu'en principe toutes les classes filles ont besoin d'appeler cette méthode dans leur constructeur et qu'on ne peut rien faire pour imposer à une classe fille de le faire, il faut y penser chaque fois qu'on écrit un nouveau constructeur d'une de ces classes filles.
- Les classes petite-filles : ce n'est pas faisable. Dès qu'une classe fille doit avoir elle-même des classes filles, il faut lui déclarer les méthodes en question final, ou alors refaire la même astuce avec elle.
Mais ce sont d'autres problèmes, et je les trouve moins graves.
17/10/2010, 00h26
helpy2

Merci, c'est parfait, j'ai tout compris ! :mrgreen:
Je vous remercie à tous pour votre aide :ccool: