Discussion :

[david87]

bonjour,

j'ai une classe requete dans laquelle j'ai plusieur fonctions:

Code :

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

1
2
3
public string GetListeProprio(String no_commune, String nom_prop){...}
public string  GetListeRue(String no_commune, String nom_rue){...}
...

et dans une seconde classe j'ai les memes fonctions qui appel celle de requete

Code :

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

1
2
3
4
5
6
7
8
9
public string GetListeProprio(String no_commune, String nom_prop)
{
    return MaMethode(Requete.GetListeProprio(no_commune, nom_prop)));
}

public string GetListeRue(String no_commune, String nom_rue)
{
    return MaMethode(Requete.GetListeRue(no_commune, nom_rue)));
}

comment puis-je eviter cette répétion de code semblable ?
(j'ai une vingtaine de fonction faisant exactement ceci)

merci d'avance de votre aide ;-)

David

[Zartan]

On voudrait une méthode générique

Code :

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

1
2
3
4
5
public string GetListe<T>(String param1, String param2)
{
    return MaMethode(Requete.GetListe<T>(param1, param2)));
}

ou bien

Code :

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

1
2
3
4
5
public string GetListe(T type, String param1, String param2)
{
    return MaMethode(Requete.GetListe(type, param1, param2)));
}

Cela nous indique que Proprio et Rue devraient hériter d'une interface commune qui implémenterait GetListe().

[david87]

je n'ai pas bien cerné,

quel sera le lien entre l'interface et ma classe Requete ?
et ma seconde classe (nomée Matrice) implementera cette interace ?

[hegros]

euh à vrai dire cela ne devrait pas vraiment être un problème c'est une conception courante et non erronée d'ailleurs cela s'appelle en terme de pattern la stratégie ou encore la délégation.

[Zartan]

Désolé, j'étais pressé et donc je n'ai pas eu le temps de développer. On a plusieurs façons de faire, en voici une qui finalement n'utilise pas les interfaces (en java mais c'est faisable en d'autres langages également) :

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
package generique;

public class SecondeClasse {

	/**
	 * @param args
	 */
	public Requete requete ;
	public String MaMethode(String s)
	{
		return s ;
	}
	public <T extends Colonne> String GetListe(T t, String param1, String param2)
	{
		return MaMethode(requete.GetListe(t, param1, param2)) ;
	}
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		SecondeClasse sec = new SecondeClasse() ;
		sec.requete = new Requete() ;
		Proprio proprio = new Proprio() ;
		
		System.out.println(sec.<Proprio>GetListe(proprio, "no_commune", "nom_prop")) ;

	}

}

Code :

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

1
2
3
4
5
6
7
8
9
package generique;

public class Requete {
	public <T extends Colonne>String GetListe(T t,String param1, String param2)
	{
		return t.GetListe(param1, param2) ;
	}
}

Code :

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

1
2
3
4
5
6
7
package generique;

public abstract class Colonne {
	public abstract String GetListe(String param1, String param2) ; 

}

Code :

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

1
2
3
4
5
6
7
8
9
10
package generique;

public class Proprio extends Colonne
{
	public String GetListe(String param1, String param2) {
		return "(stub)GetListe<Proprio>"+ ", p1=" + param1 + "p2=" + param2 ;
	}
	
}

Effectivement, passer par les delegates semble meilleur.

[david87]

ok super!
merci du coup de pouce