参数包扩展有问题

时间:2018-01-25 04:17:26

标签: c++ templates variadic-functions

我在一个类模板上工作,模板本身不是一个可变类型,但是类的构造函数是。我正在尝试获取参数包的所有值并将它们存储到向量中,但我似乎无法获得编译的代码,我不断收到编译器错误消息,必须在此上下文中扩展参数包,以及我不确定正确的语法是什么。这是我的班级:

template<class T>
class TableRow {
private:
    std::string id_;
    std::vector<T> values_;

public:
    template<class T, class... Params>
    TableRow( const std::string& id, T firstVal, Params... valuePack ) :
    id_(id) {
        values.push_back( firstVal );
        for ( auto t : (valuePack...) ) {  // Compiler error here
             values_.push_back( t ); 
        }
    }

    // ... other methods
};

除了让这个正确编译之外,我唯一关注的另一个问题是有没有办法确保所有Params都是T类型的?

另一个词是我在实例化这个类时想要实现的

TableRow<int> trInt1( "Row1", 1, 2, 3, 4, 5 );
TableRow<int> trInt2( "Row2", 6, 7, 8, 9, 10 );

TableRow<float> trFloat1( "Row1", 2.2f, 3.3f, 4.4f );
TableRow<float> trFloat2( "Row1", 4.5f, 7.9f );

// In the above examples the first two rows are of the same length
// meaning their vector sizes are equal in length therefor they are
// compatible to fit inside of single table. The table is
// a class template that takes type <T> the same type as the RowTable<T>.
// The Table<T>'s constructor accepts a TableRow<T>. 

// With the next two examples of the floats they are both indeed table rows
// but they can not fit into the same table since they are of different length

// The following is what I do not want:
TableRow<char>  row( "row1", 'a', 3, 5, 2.4f, someClass ); 
// This is not allowed, all data types from the 2nd parameter on
// must all be of the same type!

修改

我提出了liliscent建议,并将我的课程模板声明更改为:

template<class T>
class TableRow {
private:
    std::string id_;
    std::vector<T> values_;

public:
    template<class... Params>
    TableRow( std::string id, Params&&... valuePack ) :
        id_( id ),
        values_ { std::forward<Params>( valuePack )... }
    {}

    std::string getID() const {
        return id_;
    }

    std::vector<T> getValues() const {
        return values_;
    }

    std::size_t getSize() const {
        return values_.size();
    }
};

现在这已经编译了......感谢您的帮助,因为我现在可以继续编写本课程的其余部分。晚些时候;一旦我的两个班级一起工作,如果一切顺利,我会接受他们的回答。

2 个答案:

答案 0 :(得分:2)

您的编译失败只是语法错误,您不能以这种方式使用基于范围的for循环。请参阅http://en.cppreference.com/w/cpp/language/range-for

要解决此问题,您可以使用std::vector基于std::initializer_list的构造函数,以确保所有值都自动具有相同的类型。

template<class... Args>
TableRow( const std::string& id, Args&&... valuePack )
    : id_(id)
    , values_{ std::forward<Args>(valuePack)... }
{}

我认为如果您想进一步修复TableRow中的元素数量,则应使用std::array。然后你可以有另一个模板参数int N来检查编译时是否可以将某些行放入表中。

答案 1 :(得分:0)

修复了语法错误:

template<class T, class... Params>
TableRow( const std::string& id, T firstVal, Params... valuePack ) :
id_(id) {
    values.push_back( firstVal );
    for ( auto t : { valuePack...} ) { // or { static_cast<T>(valuePack) ...}
         values_.push_back( t ); 
    }
}

我建议直接使用vector

TableRow(const std::string& id, std::vector<T> values) :
id_(id), values(std::move(values)) {}

使用方法略有不同:

TableRow<int> trInt1( "Row1", { 1, 2, 3, 4, 5 });
TableRow<int> trInt2( "Row2", { 6, 7, 8, 9, 10 });

TableRow<float> trFloat1( "Row1", { 2.2f, 3.3f, 4.4f });
TableRow<float> trFloat2( "Row1", { 4.5f, 7.9f });