Discussion :

[somnix]

Bonjour,

je suis en train de développer une app dans laquel j'ai les deux méthodes suivantes mais en fait j'en ai une dizaine identiques:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
	private boolean hasSegm(List<Reportparameters> repParams) {
		boolean hasIt = false;
		int i = 0;
		while (repParams!=null && !hasIt && repParams.size() > 0  && i < repParams.size()){
			Reportparameters param = repParams.get(i);
			if (param.getHassegm())
				hasIt = true;
			i++;
		}
		return hasIt;
	}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
	private boolean hasIndustries(List<Reportparameters> repParams) {
		boolean hasIt = false;
		int i = 0;
		while (repParams!=null && !hasIt && repParams.size() > 0  && i < repParams.size()){
			Reportparameters param = repParams.get(i);
			if (param.getHasindustry())
				hasIt = true;
			i++;
		}
		return hasIt;
	}

comme on peut le remarquer la seule chose qui varie mis à part le nom de la méthode, c'est le nom de de la méthode qui est appelée à l'intérieur
à savoir param.getHassegm() dans la premiere et param.getHasindustry() dans la deuxième.


je me demande alors si il exite un moyen de faire une fonction générique qui prendrait un param supplémentaire qui serait le nom de la fonction qui sert de test

un truc du genre

Code :

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

1
2
3
4
5
6
7
8
9
10
11
	private boolean hasIt(List<Reportparameters> repParams, <?> maFunctionDeTest) {
		boolean hasIt = false;
		int i = 0;
		while (repParams!=null && !hasIt && repParams.size() > 0  && i < repParams.size()){
			Reportparameters param = repParams.get(i);
			if ( eval (maFunctionDeTest) on param )
				hasIt = true;
			i++;
		}
		return hasIt;
	}

merci pour votre aide

[thelvin]

Hello, il y a des moyens de faire ça en Java, oui. Grosso-modo deux :

# 1 - L'introspection : invoquer la méthode d'une classe par son nom

ça peut se faire par exemple comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
private boolean hasIt(List<Reportparameters> repParams, String predicateMethodName) throws ReflectiveOperationException {
 
  Method method = Reportparameters.class.getMethod(predicateMethodName);
 
  Reportparameters param = repParams.get(0);
 
  if((Boolean)method.invoke(param)) {
    return true;
  }
 
  return false;
}

qui s'appellerait comme ça :

Code :

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

hasIt(repParams, "getHassegm");

Avantage :
- c'est exactement ce que tu essayais de faire dans ta tête

Inconvénients :
- ce n'est pas très lisible
- le compilateur ne vérifie plus si les méthodes qu'on cherche à appeler existent et sont correctement appelées. Les erreurs ne se verront qu'en essayant.

# 2 - Le prédicat : la manière objet.

Code :

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

1
2
3
4
5
6
7
8
9
10
private boolean hasIt(List<Reportparameters> repParams, Predicate<Reportparameters> predicate) {
 
  Reportparameters param = repParams.get(0);
 
  if(predicate.test(param)) {
    return true;
  }
 
  return false;
}

qu'on appellerait comme ceci :

Code :

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

hasIt(params, p -> p.getHassegm());

ou comme ceci, c'est pareil :

Code :

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

hasIt(params, Reportparameters::getHassegm);

- le compilateur vérifie tout
- on se traîne pas des casts et des exceptions à rallonge
- par contre il faut s'habituer à penser comme ça

[tchize_]

je me demande alors si il exite un moyen de faire une fonction générique qui prendrait un param supplémentaire qui serait le nom de la fonction qui sert de test

Ca s'appelle un predicat, comme thelvin l'a dit.

C'est assez court pour toi ça? On peut diffciliment faire plus court

Code :

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

1
2
List<Reportparameters> reportParameters = .....;
boolean anyIndutryPresent = reportParameters.stream().anyMatch(Reportparameters::getHasindustry);

Parfois je regrette d'être coincé avec java 6 au boulot

[somnix]

merci beaucoup pour vos réponses !!!