Discussion :

[helpy2]

Bonjour à tous,

Voici mon souci.

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

Code :

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

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

[slim_java]

Salut,

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

Code :

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

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();

??

[helpy2]

Euh non pardon merci

Code :

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

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

[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é.

[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 ?

[thelvin]

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.

[helpy2]

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 ?

[thelvin]

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.

[helpy2]

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