Как освободить ресурсы от shared_ptr ResourceManager

Question

Я реализовал простой ResourceManager - все было в порядке, пока я не попытался реализовать его деструктор для (экстренной) очистки (например, в случае фатального исключения). Я не смог найти ни одного способа сделать это:

template<typename T> 
class ResourceManager 
{  
public: 
    std::unordered_map<std::string, std::weak_ptr<T> > resource_map; 
    std::shared_ptr<T> getResource(std::string name) 
    { 
     std::shared_ptr<T> res = resource_map[name].lock(); 
     if(!res) 
     { 
      res = std::shared_ptr<T>(new T(this, name)); 
      resource_map[name] = res; 
     } 
     return std::move(res); 
    } 

    void Release(std::string name) 
    { 
     resource_map.erase(name); 
    }; 
    ~ResourceManager(void) { /* ???? */} 
}; 

class Texture 
{ 
private: 
    ResourceManager<Texture> * manager_; 
    std::string name_; 

public: 
    Texture(ResourceManager<Texture> * manager, std::string& name) 
      : manager_(manager), name_(name) { } 
    ~Texture(void){ 
     manager_->Release(name_); 
    } 
}; 

Очевидно, я должен перебрать все активные ресурсы ... но как я могу освободить их, если ResourceManager не является технически (единоличный) владелец ресурсов? Это может быть недостатком дизайна, если это так, предложите альтернативу.

EDIT: в ответ на ответы, чтобы определить «диспетчер ресурсов». Я представляю себе авторитетный кеш-хранилище для ссылок на ресурсы, которые могут искать ресурсы (= нет дубликатов) и управляет их состоянием в памяти (in-memory, description -одно (= путь + тип) и освобождается), все вышеописанное максимально автоматизировано. (Там должны быть отдельные ResourceLoaders, но это не изменится на этот вопрос)

Answer 1

Итак, ваш код не делает много, чтобы осветить ваш общий дизайн, здесь, но ...

Как реализовано, ваш менеджер ресурсов, по-видимому, имеет слабые указатели на ресурсы. Это указывает на то, что он не несет ответственности за время жизни самих реальных ресурсов и поэтому не должен очищать эти ресурсы в своем деструкторе.

Если вы хотите, менеджер ресурсов будет авторитетным владельцем данных ресурса, вы захотите изменить его дизайн/реализацию. Например, вы можете хранить его shared_ptr s самим ресурсам и только передавать weak_ptr s клиентам. Или просто сохраните unique_ptr s и передайте голые указатели на код клиента.

Но, как написано, вам не нужно (и действительно не может) ничего сделать, чтобы очистить ресурсы в ~ResourceManager().

Answer 2

Чтобы предложить альтернативы, нам нужно знать, какую цель должен иметь ваш менеджер ресурсов.

1. Вы хотите, чтобы он функционировал как «коллекция ресурсов», т. Е. Сохраняя ресурсы до тех пор, пока они не будут явно выпущены?
2. Или вы хотите, чтобы это был «кеш ресурсов», сохраняя ресурсы до тех пор, пока он не решит, что некоторые должны быть выпущены для освобождения памяти?
3. Или вы действительно хотите, чтобы он не поддерживал какие-либо ресурсы, а просто сохранил список ресурсов, которые, возможно, были живы чем-то другим?

Помните, что shared_ptr s на C++ не работает, как работает GC. То есть если вы уничтожите/перезагрузите последний shared_ptr объекту, этот объект будет удален немедленно, даже если есть weak_ptr s ему.

Таким образом, подходы (1) и (2) могут иметь большой смысл. (3), однако, это то, что вы в настоящее время имеете, очень редко имеет смысл (если вообще).