为什么我的函数调用不匹配这个通用函数实现?

时间:2012-10-15 21:09:45

标签: c++ templates

似乎编译器非常接近于我想做的事情(因为它将我的函数称为候选者),但我不知道我做错了什么。

#include <stdio.h>
#include <stdlib.h>
#include <list>

using namespace std;

template <class U, template<class U> class T>
void AppendSorted( T<U>& l, U val )
{
  typename T<U>::reverse_iterator rt = l.rbegin();

  while( ((*rt) > val) && (rt != l.rend()) )
    rt++;

  l.insert( rt.base(), val );
}

int main( int argc, char* argv[] )
{
    list<int> foo;
    AppendSorted<int, list<int> >( foo, 5 );

    list<int>::iterator i;
    for( i = foo.begin(); i != foo.end(); i++ )
    {
        printf("%d\n",*i);
    }

    return 0;
}

我得到的错误是:

test.cpp: In function ‘int main(int, char**)’:
test.cpp:21:43: error: no matching function for call to ‘AppendSorted(std::list<int>&, int)’
test.cpp:21:43: note: candidate is:
test.cpp:8:6: note: template<class U, template<class U> class T> void AppendSorted(T<U>&, U)

3 个答案:

答案 0 :(得分:2)

此签名

template <class U, template<class U> class T>
void AppendSorted( T<U>& l, U val )

表示将有一个具体类型(class U)和一个模板(template<class U> class T)。

此调用

AppendSorted<int, list<int> >( foo, 5 );

提供了两种具体类型:intlist<int>list是一个模板,list<int>是具体类型,模板实例,但不是模板。

只需更改函数即可接受集合的具体类型:

template <class U, class T>
void AppendSorted( T& l, U val )
{
  typename T::reverse_iterator /* or auto */ rt = l.rbegin();

  while( (rt != l.rend()) && ((*rt) > val) )
    rt++;

  l.insert( rt.base(), val );
}

让编译器推断出类型参数,而不是指定它们。

AppendSorted( foo, 5 );

答案 1 :(得分:2)

std::list是两个参数的模板 - 不仅仅是一个参数。有第二个默认参数。

您需要这样的模板功能来匹配列表:

template <class U, template<class U,class A> class T>
void AppendSorted( T<U,std::allocator<T>>& l, U val );

std::set怎么办 - 它需要3个参数?

我不确定 - 也许可变参数模板会有所帮助...

但试试这个:

template <class U, class Container>
void AppendSorted(  Container& l, U val);

答案 2 :(得分:1)

std::list实际上有两个模板参数:

#include <stdio.h>
#include <stdlib.h>
#include <list>

using namespace std;

template <class U, template<class> class AL, template<class,class> class T> 
void AppendSorted( T<U,AL<U>>& l, U val ) {
        typename T<U,AL<U>>::reverse_iterator rt = l.rbegin();

        while( ((*rt) > val) && (rt != l.rend()) )
                rt++;

        l.insert( rt.base(), val );
}

int main( int argc, char* argv[] ) {
        list<int> foo;
        AppendSorted( foo, 5 ); 

        list<int>::iterator i; 
        for( i = foo.begin(); i != foo.end(); i++ ) {
                printf("%d\n",*i);
        }  

        return 0; 
}

现在它将编译,但你的代码中有逻辑错误 - 你有过去的迭代器。要解决此问题,请在循环播放时首先检查rend()

    while(rt != l.rend()  &&  *rt > val)