Il est parti !!! Youhou ! De qui parle-t-il me direz vous ? De l’auteur de pleins de bêtises dans l’application (pour rester poli). Il n’est resté que quelques semaines et c’est heureusement car il a fait de gros dégâts.
La crasse d’aujourd’hui est une problématique de mise en cache d’objets. Évidement, comme dans beaucoup d’applications, on a préféré réinventer la roue plutôt que d’utiliser un framework de cache tout fait. Le cache a grossi et il a fallu mettre un place un timeout pour libérer les objets inutilisés.
Notre bel (ex)architecte, a eu la bonne idée de mettre des SoftReference pour que :
Utilisation d’une référence douce qui serait facilement supprimée par le Garbage Collector en cas de besoin de mémoire.
Jusque là, je dis super, les SoftReference sont très bien pour la mise en cache d’objets susceptibles d’être réutilisés. Mais voyons un peu plus en détail :
-
private static Map<KeyMapRepository, List<? extends IBaseVO>> mapRepositoryVO =
-
new HashMap<KeyMapRepository, List<? extends IBaseVO>>();
Tiens, étrange, la déclaration de la Map ne fait pas apparaitre les SoftReference…
-
/* Utilisation d'une référence douce qui serait facilement
-
supprimée par le Garbage Collector en cas de besoin de mémoire .*/
-
SoftReference<List<E>> softResultList = null;
-
-
List<E> mapValue = (ValueMapRepository<E>) mapRepositoryVO.get(keyRepository);
-
// on va chercher dans la map
-
if (mapValue != null) {
-
// l'element existe deja dans le cache
-
return mapValue;
-
} else {
-
// pas encore mis en cache, on va chercher la donnée
-
List<E> results = new ArrayList<E>();
-
…
-
-
mapValue = Collections.unmodifiableList(results);
-
-
softResultList = new SoftReference<List<E>>(mapValue);
-
-
/* Ajout du resultat dans la map pour une prochaine utilisation
-
(on alimente le cache par les SoftReferences
-
* qui sont facilement éligibles pour la garbage
-
collection en cas de problème de mémoitre) */
-
mapRepositoryVO.put(keyRepository, softResultList.get());
-
-
return mapValue;
-
}
Vous aurez surement remarquer la belle ligne mapRepositoryVO.put(keyRepository, softResultList.get()); qui met dans le cache une strong référence vers notre objet et non la softréférence qu’il a construit ?
Autre subtilité, il a mis en place un champ pour pourvoir nettoyer son cache si les objets n’ont pas été utilisés depuis longtemps. Mais hélas, il a mis l’information sur les éléments contenus dans la liste elle même !
De plus, les listes stockées dans le cache sont des unmodifiable list (Ceci pour que les utilisateurs de notre cache ne puissent faire des add ou des remove). Je vous laisse apprecier le commentaire et la fonction :
-
/**
-
* Néttoyage du cache via la politique d'éviction pour éviter l'accumulation pouvant entrainer
-
* des memory leak.(Méthode à revoir car les listes sont non modifiables)
-
*/
-
private void cleanCache() {
-
int tailleMap = mapRepositoryVO.size();
-
if (tailleMap > 0) {
-
List resultList = null;
-
-
for (final Iterator iter = mapRepositoryVO.values().iterator(); iter.hasNext();) {
-
resultList = (List) iter.next();
-
if (resultList != null && resultList.size() > 0) {
-
for (final Iterator iterList = resultList.iterator(); iterList.hasNext();) {
-
final IBaseVO oldIBaseVO = (IBaseVO) iterList.next();
-
if (oldIBaseVO != null) {
-
if (oldIBaseVO.isExpired(timeout)) {
-
log.debug("–Taille de la liste avant éviction du cache — : "
-
+ resultList.size());
-
iterList.remove();
-
log.debug("–Taille de la liste après éviction du cache — : "
-
+ resultList.size());
-
}
-
}
-
}
-
}
-
}
-
}
-
}