[design] ou et comment dois je ecrire ma methode?

Version imprimable

09/01/2007, 12h58
Malo

[design] ou et comment dois je ecrire ma methode?

bonjour a tous,
j'ai un petit soucis au niveau de la conception des classes en java qui me revient souvent, et je souhaitais avoir votre avis la dessus. En plus de ca on m'a posé cette colle un jour lors d'un entretiens d'embauche et j'ai pas su repondre.

Par exemple, j'ai une super classe qui s'appelle "vehicule".
3 classes heritent de cette classe, a savoir "moto", "voiture", "bateau".
D'autre part:
"kawazaki","yamaha", "honda" herite de "moto".
"peugeot", "renault" herite de "voiture".
"Mariner","Riamar" herite de "bateau".

le probleme est le suivant :
j'ai une methode tres tres longue a ecrire, que je retrouve dans la classe "Kawazaki", "renault" et "Mariner" mais pas dans les autres.
OU dois je ecrire cette methode??????

Suis je obligé de l'ecrire 3 fois (une fois dans la classe "kawazaki", une fois dans la classe "renault" et une fois dans la classe "Mariner"), ou existe t il une astuce pour que je n'ai a l'ecrire qu'une fois et que je puisse l'utiliser dans chacune des classes interressées?

C'est un probleme tres general et je suis sur que nombreux sont ceux qui ont rencontré ce genre de dilemme, alors j'espere que quelqu'un a une solution élegante.

merci beaucoup par avance.
09/01/2007, 13h12
thibaut

Si ce sont toutes des classes, pour ne pas avoir à définir ta méthode tu dois la définir au niveau de vehicule et la rédéfinir dans les autres cas où elle autres cas où elle ne doit pas être appelée.

Ou intercaler une classe commune entre Moto et Kawasaki, entre Voiture et Renault ...

A+
09/01/2007, 13h20
Janitrix

Tu utilises une interface. Cela semble le choix le plus judissieux dans ton cas. Tu définis ta méthode dans l'interface que tu implemantes. Mais je suis pas sur car si tu veux la même dans toutes les classes, l'interface devient un peu inutile car tu dois redéfinir la méthode à chaque fois.
09/01/2007, 13h30
Malo

Citation:

Si ce sont toutes des classes, pour ne pas avoir à définir ta méthode tu dois la définir au niveau de vehicule et la rédéfinir dans les autres cas où elle autres cas où elle ne doit pas être appelée.

Je ne pense pas que cela soit satisfaisant, car cela revient a redefinir la methode pour toute les classes qui n'utilise pas la methode, ce qui est tout aussi contraignant.

Citation:

Ou intercaler une classe commune entre Moto et Kawasaki, entre Voiture et Renault

Vu que le multi heritage est impossible en java, je vois pas comment.

Citation:

Tu utilises une interface [...] Mais je suis pas sur car si tu veux la même dans toutes les classes, l'interface devient un peu inutile car tu dois redéfinir la méthode à chaque fois.

En effet, je ne souhaite pas redefinir la methode a chaque fois
09/01/2007, 13h51
thibaut

Oui c'est vrai désolé.

Mais à part utiliser une interface comme je t'ai dit, pas d'autre solution.

Mais rien ne t'empêche dans la définition concréte de ton interface, de faire appel à une méthode d'une classe déléguée qui contient ta grande méthode. Ou quelquechose comme un pattern visiteur.

A+

Code:

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
30
31
public class Vehicule {
...
}
 
public class Bateau extends Vehicule {
...
}
 
public class Voiture extends Vehicule {
...
}
 
public class Moto extends Vehicule {
...
}
 
public class Honda extends Moto {
...
}
 
public class Yamaha extends Moto {
...
}
 
public class Peugeot extends Voiture {
...
}
 
public class Riamar extends Bateau {
...
}

Code:

1
2
3
4
public interface Visited {
    public void visit(Visitor visitor);
    public void visit();
}

Code:

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
30
31
32
33
34
35
public class Renault extends Voiture implements Visited {
 
    public void visit(Visitor visitor) {
        visitor.execute(this);
    }
 
    public void visit() {
        Visitor visitor = new Visitor();
        visitor.execute(this);
    }
}
 
public class Mariner extends Bateau implements Visited {
 
    public void visit(Visitor visitor) {
        visitor.execute(this);
    }
 
    public void visit() {
        Visitor visitor = new Visitor();
        visitor.execute(this);
     }
}
 
public class Kawasaki extends Moto implements Visited {
 
    public void visit(Visitor visitor) {
        visitor.execute(this);
    }
 
    public void visit() {
        Visitor visitor = new Visitor();
        visitor.execute(this);
     }
}

Code:

1
2
3
4
5
6
7
8
9
public class Visitor {
    public void execute(Visited visited) {
        // Tu codes ta grosse méthode ici
 
        // "visited" est passé en paramètre si jamais
        // tu as besoin dans cette méthode des données
        // de la classe appelante.
    }
}

cela me semble tout a fait correct, merci beaucoup thibaut.

Du coup je pense qu'on peut faire encore plus simple, vu que l'algorithme est identique pour toute les classes concernées
Code:

1 2 3 4 5 6 7 8 9 10 11 12 class Kawazaki extends Moto implement VehiculeSpe{...} class Renault extends Voiture implement VehiculeSpe{...} interface VehiculeSpe{}//cette interface peut à la limite être vide ou bien alors contenir des champs si besoin class A { public static void traitement(VehiculeSpe vs){ //gros algorithme } }
si j'ai besoin d'effectuer l'algo j'appelle A.traitement(this).

L'avantage est que je n'ai besoin d'ecrire aucune methode en plus dans Kawazaki et Renault.
Qu'est ce que tu en penses?

09/01/2007, 15h01
thibaut

Ouais ça me semble pas mal.

Mais ou vas tu appeler : A.traitement(this). ? Tu es obligé(e) de l'appler dans Kawasaki, Renault ?
09/01/2007, 15h16
Malo

oui c vrai. C'est plutot A.traitement(machin), avec machin une instance de Kawazaki ou Renault.
09/01/2007, 15h29
Amine_sas

Citation:

Si ce sont toutes des classes, pour ne pas avoir à définir ta méthode tu dois la définir au niveau de vehicule et la rédéfinir dans les autres cas où elle autres cas où elle ne doit pas être appelée.

Citation:

Envoyé par Malo

Je ne pense pas que cela soit satisfaisant, car cela revient a redefinir la methode pour toute les classes qui n'utilise pas la methode, ce qui est tout aussi contraignant.

Pourquoi?. Rien ne vous oblige de redéfinir une methode d'une super classe héritée meme si vous n'appelez pas cette methode dans les sous classe.
09/01/2007, 15h37
thibaut

Citation:

Envoyé par Amine_sas

Pourquoi?. Rien ne vous oblige de redéfinir une methode d'une super classe héritée meme si vous n'appelez pas cette methode dans les sous classe.

Certes mais je suppose que c'est piour éviter tout appel inopprotun de la méthode sur un autre objet que ceux désirés.
10/01/2007, 09h52
moritan

Plutot que d'utiliser une interface, j'utiliserais une abstract classe qui permet donc de définir la méthode.
10/01/2007, 11h46
natha

Citation:

Envoyé par moritan

Plutot que d'utiliser une interface, j'utiliserais une abstract classe qui permet donc de définir la méthode.

T'as pas dû tout bien lire...
On n'a pas d'héritage multiple en java et ces classes héritent déjà d'autres classes.