Duplication de code

Marsh Posté le 17-03-2008 à 19:48:56

Bonjour,

On me demande de dupliquer une fonction pour avoir le comportement suivant :
- Sous certaines conditions, lever une exception
- Sous les même conditions, continuer le traitement
J'aurai donc une fonction f et une fonction fbis que j'appellerais et qui seraient strictement semblables concernant le traitement, à part que justement si j'appelle f elle doit s'arrêter dans certaines conditions, et si j'appelle fbis, dans les mêmes conditions elle doit continuer.

Je ne suis évidemment pas d'accord avec cette solution, car elle implique une maintenabilité beaucoup trop risquée (ce sont des fonctions pas piquées des hannetons en plus)
Le problème que j'ai est que je ne peux pas modifier les paramètres (les interfaces ne sont pas modifiables), donc pas rajouter un booléen dans ma fonction.
Je ne peux pas non plus "vider" f et la laisser de coté, car elle peut être appelée directement par du code sur lequel je n'ai aucun accès. De toute façon cette solution ne serait pas très élégante.

Je n'arrive pas à trouver le montage qui me permettrait d'avoir une solution élégante et sans risque.

Merci d'avance ...

Reply

Marsh Posté le 17-03-2008 à 19:48:56

Reply

Marsh Posté le 17-03-2008 à 20:52:54

Et si tu fais une sorte de filtre?

Code :

public interface Filter
{
/** Détermine si ton objet est accepté.
@param anObject l'objet à vérifier.
@return un booléen.
*/
boolean isAccepted(Object anObject);
}

et:

Code :

public class FunctionFilter implements Filter
{
public FunctionFilter() {
}
public boolean isAccepted(Object anObject)
{
// ta condition
}
}

Comme ça pas besoin de modifier ta fonction ou ton interface.

^^ Bon j'espère que ça t'aidera j'débute!

Reply

Marsh Posté le 04-04-2008 à 17:04:03

Ce que je ferais, serait de mettre quelque part, dans une classe statique ou un singleton, une variable statique de type boolean dans ce genre :

Code :

public final class RunType {
public static boolean stopOnError = true;
}

Ensuite, dans la fonction incriminée, sans changer son interface, tu fais quelque chose comme ça :

Code :

function toto(String titi) throws Exception {
try {
// Ton code ici
} catch (Exception e) {
if (RunType.stopOnError) {
throw e;
}
}
}

Quelque chose dans ce genre là devrait répondre à ton problème, non ?

---------------
Un ordinateur sert à résoudre des problèmes qui n'existaient pas avant son invention.

Reply

Duplication de code

Sujets relatifs:

Leave a Replay