c ++ функция сортировки List

Question

Моя функция сортировки выглядит следующим образом:

template <typename Compare, typename T>    
void List<T>::sort(const Compare& comparer){   
...   
}    

Но я получил следующую ошибку:

определение шаблона не-шаблона void List<T>::sort(const Compare&)'<br> invalid use of undefined type список классов '

Что это значит?

Это полный код списка:

template <typename T> class Node;    
template <typename T> class Iterator;    
template <typename T>    
class List{     
private:    
    Node<T> * first;     
    Node<T> * last;    
    int size;    
    friend class Predicate ;    
    friend class Compare;    
public:    
    typedef Iterator<T> iterator;    

    List(){    
       first = NULL;    
       last = NULL;    
       size = 0;    
    }    
    List(const List<T> & l);    
    void pushBack(T element);    
    void insert( const T& element, iterator i);    
    iterator remove(iterator i);    
    iterator find(const Predicate& predicate);    
    void sort(const Compare& comparer);    
    int getSize() const;    
    iterator begin();    
    iterator end();    
    ~List();    
};    

template <class T>    
List<T>::List(const List & l) {    
    first = 0;    
    last = 0;    
    size = 0;    
    for (Node<T> * current = l.first; current != 0; current = current -> next){
        pushBack(current -> data);    
    }    
}    

template <typename T>    
void List<T>::pushBack(T element){    
   Node<T>* newnode = new Node<T>(element);    
   if (newnode->prev == NULL) {    
      first = newnode;    
      last = newnode;    
   }else{    
      newnode->prev = last;    
      last->next = newnode;    
      last = newnode;    
   }    
}    

template <typename T>    
void List<T>::insert( const T& element, iterator i){    
   if (i.position == NULL){    
      pushBack(element);    
      ++size;    
      return;    
   }    
   Node<T>* after = i.position;    
   Node<T>* before =  after->prev;    
   Node<T>* newnode = new Node<T>(element);    
   newnode->prev = before;    
   newnode->next = after;    
   after->prev = newnode;    
   if (before == NULL) {    
      first = newnode;    
   }    
   else{    
      before->next = newnode;    
   }    
   ++size;    
}    

template <typename T>    
typename List<T>::iterator List<T>::remove(iterator iter){    
      if(iter.position != NULL){    
          Node<T>* remove = iter.position;    
           Node<T>* before = remove->prev;    
           Node<T>* after = remove->next;    
           if (remove == first){    
               first = after;    
           } else{    
               before->next = after;    
           }    
           if (remove == last){    
               last = before;    
           }else{    
               after->prev = before;    
           }    
           iter.position = after;    
           --size;    
           delete remove;    
           return iter;    
      }else{    
       throw ElementNotFound();    
   }    
}    

template <typename T>    
typename List<T>::iterator List<T>::begin(){    
   //iterator iter;    
   //iter.position = first;    
   //iter.last = last;    
    return iterator(first);    
}    

template <typename T>    
typename List<T>::iterator List<T>::end(){    
     return iterator (last);    
}    


template <typename Predicate, typename T>    
List<T>::iterator List<T>::find(const Predicate& predicate){    
    iterator iter;    
    for( iter = begin(); iter != end(); iter = next()){    
        if( predicate(iter.getElement())){    
            return iter;    
        }    
    }    
    return end();    
}    

template <typename Compare, typename T>    
void List<T>::sort(const Compare& comparer){    
    Iterator<T> iter;    
    for( iter = begin(); iter != end(); iter = iter.next() ){    
        if( comparer( iter.getElement() , (iter+1).getElement()) != true){    

        }    
    }    
}    

template <typename T>    
int List<T>::getSize() const{    
    return size;    
}    

template <class T>    
List <T>::~List() {    
    Node <T> * firstNode = first;    
    while (firstNode != 0)     {    
        Node <T> * nextNode = firstNode->next;    
        delete firstNode;    
        firstNode = nextNode;    
    }    
}    
template <typename T> class Node {    
private:    
    T data;    
    Node* next;    
    Node* prev;    
    friend class List<T>;    
    friend class Iterator<T>;    
public:    
    Node(T element){    
        data = element;    
        prev = NULL;    
        next = NULL;    
    }    
    ~Node(){}    
};    

template <typename T> class Iterator{    
private:    
   Node<T>* position;    
   Node<T>* last;    
   friend class List<T>;    
public:     
   Iterator(){    
       position = NULL;    
       last = NULL;    
   }    
   Iterator(Node<T> * source): position(source) { }    
   T& getElement()const;    
   bool operator==(Iterator b) const;    
   bool operator!=(Iterator b) const;    
   T & operator*();    
   Iterator & operator++();    
   Iterator & operator++(int);    
   ~Iterator(){}    
};    

template <class T>    
T& Iterator<T>::getElement() const{    
   if(position != NULL){     
       return position->data;    
   }    
   else{    
       throw ElementNotFound();    
   }    
}    

template <typename T>    
bool Iterator<T>::operator==(Iterator b) const{    
   return position == b.position;    
}    

template <typename T>    
bool Iterator<T>::operator!=(Iterator b) const{    
    return position != b.position;    
}    

template <typename T>    
T & Iterator<T>::operator*() {    
    return position->data;    
}    

template <class T>    
Iterator<T> & Iterator<T>::operator++() {    
    position = position->next;    
    return *this;    
}    

template <class T>    
Iterator<T> & Iterator<T>::operator++(int){    
    position = position->next;    
    return *this;    
}    


#endif /* LISTGENERIC_H_ */    

Answer 1

Либо вы объявляете sort как шаблонную функцию с одним аргументом шаблона (Compare) внутри класса.Тогда ваше определение должно выглядеть следующим образом:

template <typename T>
template <typename Compare>
void List<T>::sort(const Compare& comparer) {
    …
}

И вам также необходимо удалить теперь избыточное объявление friend class Compare из вашего List класса.

или вы сохраняете Compare и sort как есть.В этом случае просто не используйте sort в качестве шаблона и пропустите аргумент шаблона Compare:

template <typename T>
void List<T>::sort(const Compare& comparer) {
    …
}

Конечно, это работает, только если вы определили (не просто объявил!) класс Compare перед этой функцией.

Но это второе решение было бы крайне неортодоксальным и довольно бесполезным, поскольку аргумент compare не имеет большого смысла: есть только один Compare класс, и пользователь не может изменить его.Обычно это должно быть аргументом шаблона (первое решение).