Discussion :

[Agoudard]

Bonsoir,

J'ai un petit soucis dans la réalisation d'un gestionnaire de ressources.
J'ai crée un singleton CRessourcesManager qui charge des ressources depuis le disque avec une méthode :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
template<class T>
    QSharedPointer<T> load<T>(QString filename){
        QListIterator it(m_loaders);
        while(it.hasNext()){
            IRessourcesLoader loader = it.next();
            if(loader.isValid<T>()){
                return loader.load<T>(filename);
            }
        }
    }

IRessourcesLoader est une classe interface qui doit implémenter le chargement d'un type de données.
Donc pour chaque implémentation de IRessourcesLoader ma méthode doit vérifier si le loader est abilité à charger le type en question.
J'aimerai conserver la possibilité d'écrire n'importe qu'elle implémentation de IRessourcesLoader et de simplement avoir à l'ajouter à CRessourcesManager avant d'apeller la méthode load avec le type attendu.

Existe-t-il une méthode plus élégante ? Que d'utiliser :

Code :

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

1
2
3
4
 
if(loader.isValid<T>()){
        return loader.load<T>(filename);
}

Merci

[Goten]

isValid() fait quoi?

[Agoudard]

Cette méthode est sensé vérifier la compatibilité entre le type demandé et le type géré par l'implémentation de IRessourcesLoader.

Je crois avoir trouvé ce que je cherchais :
http://loulou.developpez.com/tutorie...eur3d/partie4/
Il est vraiment incroyable ce mec xD, je trouve son code presque mystique parfois...

Merci

[Joel F]

ca serait encore mieux sans Singleton

[3DArchi]

Salut,
Présentation des classes de Traits et de Politiques en C++ par Alp Mestan

Sur le singleton : Etes-vous atteint de Singletonite ?

[Emmanuel Deloget]

Je confirme : préfère les traits au code proposé par loulou. Le code de loulou est sympathique, mais il n'est pas extensible. Hors, tu vas vouloir rajouter des loaders, et peut-être permettre à tes utilisateurs de rajouter des loaders - et eux ne peuvent pas modifier ton code.

Les traits, du fait qu'ils sont spécialisable à volonté, permettent au code client d'étendre le code de la librairie sans avoir à le modifier.

[Goten]

ca serait encore mieux sans Singleton

Et bien tu vois, j'ai eu l'occasion d'y réfléchir assez intensivement par le passé, et dans ce cas précis (Ressource manager pour un jeux), c'est une des rares fois où j'ai pu justifier un singleton (et pourtant je les ai en horreur).

[Agoudard]

Super, merci les classes de politiques c'est exactement ce que je cherchais, mais je savais pas que ça existé !
Rassurez moi c'est quand même un concept relativement avancé du c++ non ? Je dois m'y reprendre à plusieurs fois avant de saisir les fonctionnements expliqués dans les liens...

Pour ce qui est du singleton du gestionnaire de ressources, je n'ai aucun intérêt à avoir plusieurs instances d'une classe qui vise à centraliser la gestion de la mémoire et plus précisément à limiter les chargements depuis le disque... Donc il me semble qu'un singleton est justifié dans ce cas.

[Flob90]

Le singleton ne sert pas qu'a assurer l'uncité d'une instance, il vise à fournir un point d'accés global à une instance, or (et il me semble que c'est expliqué dans l'article d'Emmanuel Deloget), il est souvent possible de centraliser dans une portion restreinte de code l'utilisation de ton manager (avec une bonne architecture), il n'y a donc plus besoin d'accés global, la seul chose qui reste c'est l'unicté, et pour ca il suffit de ne créer qu'une instance.