0

我有一个数据库类，它应该提供一个模板方法来返回表示表的存储库对象。每个存储库应仅存在一次。我找到了这个例子，但我想要一个内置的缓存。

标题database.h如下所示：

class Database
{
public:
  Database(const char *connectionString);
  template<typename T> shared_ptr<class T : RepositoryBase> GetRepository();

private:
  sqlite3* _connection;
  map<string, RepositoryBase> _repositoryCache;
};

该GetRepository方法的实现如下所示：

template<typename T> shared_ptr<class T : RepositoryBase> Database::GetRepository()
{
  string name = typeid(T).name();

  if(_repositoryCache.count(name) == 0)
  {
    _repositoryCache[name] = shared_ptr<T>(new T(_connection));
  }

  return _repositoryCache[name];
}

我想这样称呼它：

// SampleRepository is derived from RepositoryBase
shared_ptr<SampleRepository> sampleRepo = GetRepository<SampleRepository>();

当我编译我得到错误：

error : declaration of 'T' shadows template parameter

我需要在模板方法中更改什么以不隐藏任何模板参数？

更新

在 NathanOliver 的建议之后，模板方法如下所示：

template<typename T, typename enable_if<is_base_of<RepositoryBase, T>::value>::type* = nullptr> shared_ptr<T> Database::GetRepository()
{
  string name = typeid(T).name();

  if(_repositoryCache.count(name) == 0)
  {
    _repositoryCache[name] = shared_ptr<T>(new T(_connection));
  }

  return _repositoryCache[name]();
}

这次我得到错误：

error : template parameter redefines default argument

它指向nullptr模板签名中。

更新 2

根据@NathanOliver 给出的答案，我尝试了几种参数组合。我仍然得到错误：

1>C:\..\Database.cpp(65,98): error : template parameter redefines default argument
1>template<typename T, typename std::enable_if<std::is_base_of<RepositoryBase, T>::value>::type* = nullptr> std::shared_ptr<T> Database::GetRepository()
1>                                                                                                 ^
1>C:\..\Database.h(25,108) :  note: previous default template argument defined here
1>          template<typename T, typename std::enable_if<std::is_base_of<RepositoryBase, T>::value>::type* = nullptr> std::shared_ptr<T> GetRepository();
1>                                                                                                           ^

为了更好地理解，我还添加了存储库基类。

class RepositoryBase
{
public:
  RepositoryBase(sqlite3* connection);
  virtual string GetTableName() = 0;

  protected:
    sqlite3* _connection;
 };
4

1 回答 1

4

这里的问题是

template<typename T> shared_ptr<class T : RepositoryBase> GetRepository();

在 C++ 中不是有效的语法。如果您想限制T为 aRepositoryBase或派生自，RepositoryBase那么您需要在template<typename T>零件中执行此操作。我们所做的是添加类型检查，T如果检查失败，则编译器将不会生成将导致编译器错误的函数。那看起来像

template<typename T, typename std::enable_if<std::is_base_of<RepositoryBase, T>::value>::type* = nullptr> 
shared_ptr<T> GetRepository();
于 2017-08-25T15:26:32.573 回答