迭代器和重载<<操作者

时间:2011-08-03 04:32:40

标签: c++ stl iterator operator-overloading

我已经截断了这篇文章。最初的帖子已经消失,以便于阅读。相关的部分和问题仍然存在。


更新

我被要求发布的错误是:

[mehoggan@desktop qsort]$ g++ -o qsort -Wall main.cpp
/tmp/ccuAUzlh.o: In function `Sorters::QuickSorter<float>::test_and_sort(float*, int)':
main.cpp:(.text._ZN7Sorters11QuickSorterIfE13test_and_sortEPfi[Sorters::QuickSorter<float>::test_and_sort(float*, int)]+0x61): undefined reference to `std::basic_ostream<char, std::char_traits<char> >& Sorters::operator<< <float>(std::basic_ostream<char, std::char_traits<char> >&, Sorters::Sorter<float>&)'
collect2: ld returned 1 exit status
[mehoggan@desktop qsort]$ 

即使用以下代码:

#ifndef SORTERS_H_
#define SORTERS_H_
#include <iostream>
#include <vector>
#include <algorithm>
#include <iterator>
#include <cmath>
#include <ctime>
#include <assert.h>

using std::vector;
using std::cin;
using std::cout;
using std::endl;
using std::ostream_iterator;
using std::istream_iterator;
using std::next_permutation;
using std::back_inserter;
using std::ostream;

namespace Sorters {
    template<typename T>
    class Sorter {
        public:
            Sorter( ) { };
            virtual ~Sorter( ) { };
            virtual void test_and_sort( T *data, int size )=0;
        protected:
            virtual void sort( typename vector<T>::iterator left, typename vector<T>::iterator right )=0;
            vector<T> m_data;
    };

    template<typename T>
    class QuickSorter : public Sorter<T> {
        public:
            QuickSorter( ) { };
            virtual ~QuickSorter( ) { };
            virtual void test_and_sort( T *data, int size );
        private:
            virtual void sort( typename std::vector<T>::iterator left, typename std::vector<T>::iterator right );
            template<typename S> friend ostream& operator<< ( ostream &stream, Sorter<S> &sorter );
    };
}

template<typename T>
void Sorters::QuickSorter<T>::sort( typename vector<T>::iterator left, typename vector<T>::iterator right ) {

}

template<typename T>
void Sorters::QuickSorter<T>::test_and_sort( T *data, int size ) {
    for( int i=0;i<size;i++ ) {
        vector<T> perm( &data[0], &data[i+1] );
        do {
            cout << (*this) << endl;
            copy( perm.begin( ),perm.end( ),back_inserter( m_data ) );
            this->sort( m_data.begin( ), m_data.end( ) );
        } while( next_permutation( perm.begin( ), perm.end( ) ) );
        m_data.clear( );
    }     
}

template<typename S> ostream& operator<< ( ostream &stream, Sorters::Sorter<S> &sorter ) {
    copy( sorter->m_data.begin( ),sorter->m_data.end( ), ostream_iterator<S>( stream," " ) );
    return stream;
}
#endif

更新 我写了一个较小的例子,所以我知道我的概念是有效的,当我使用多态和朋友函数时,它只会被混淆。

#include <vector>
#include <iostream>
#include <algorithm>
#include <iterator>

using namespace std;

class Sample {
    public:
        Sample( ) { };
        Sample( float *data, int size ) {
            copy(&data[0],&data[size],back_inserter( m_data ) );
        };
        ~Sample( ) { };
    private:
        vector<float> m_data;
        friend ostream& operator<< ( ostream &stream, Sample &s ) {
            copy( s.m_data.begin( ), s.m_data.end( ), ostream_iterator<float>( stream, " " ) );  
            return stream;
        }
};

int main( int argc, char *argv[] ) {
    float data[ ] = {1,2,3,4,5};
    Sample s(data,5);
    cout << s;
}

Now to write the actual algorithm. I noticed though if I move m_data up to the parrent class I get compiler errors saying that m_data cannot be found. I guess that just means Insertion Sort, Radix Sort, Stooge Sort, ... will all have there own container.

#ifndef SORTERS_H_
#define SORTERS_H_
#include <iostream>
#include <vector>
#include <algorithm>
#include <iterator>
#include <cmath>
#include <ctime>
#include <assert.h>

using std::vector;
using std::cin;
using std::cout;
using std::endl;
using std::ostream_iterator;
using std::istream_iterator;
using std::next_permutation;
using std::back_inserter;
using std::ostream;

namespace Sorters {
    template<typename T>
    class Sorter {
        public:
            Sorter( ) { };
            virtual ~Sorter( ) { };
            virtual void test_and_sort( T *data, int size )=0;
        protected:
            virtual void sort( typename vector<T>::iterator left, typename vector<T>::iterator right )=0;
    };

    template<typename T>
    class QuickSorter : public Sorter<T> {
        public:
            QuickSorter( ) { };
            virtual ~QuickSorter( ) { };
            virtual void test_and_sort( T *data, int size );
        private:
            vector<T> m_data;
            virtual void sort( typename std::vector<T>::iterator left, typename std::vector<T>::iterator right );
            friend ostream& operator<< ( ostream &stream, const QuickSorter &sorter ) {
                copy( sorter.m_data.begin( ),sorter.m_data.end( ),ostream_iterator<T>( stream," " ) );
                return stream;
            }
    };
}

template<typename T>
void Sorters::QuickSorter<T>::sort( typename vector<T>::iterator left, typename vector<T>::iterator right ) {

}

template<typename T>
void Sorters::QuickSorter<T>::test_and_sort( T *data, int size ) {
    for( int i=0;i<size;i++ ) {
        vector<T> perm( &data[0], &data[i+1] );
        do {
            copy( perm.begin( ),perm.end( ),back_inserter( m_data ) );
            cout << (*this) << endl;
            this->sort( m_data.begin( ), m_data.end( ) );
            m_data.clear( );
        } while( next_permutation( perm.begin( ), perm.end( ) ) );
    }
}
#endif

3 个答案:

答案 0 :(得分:3)

首先要打印this打印地址。你需要打印为,

cout << *this << endl;

在您提到的评论中,您会收到错误消息。这是因为operator << ()中的以下行:

vector<S> copy = sorter.m_data;  // <--- where is 'm_data' ??

sorter的类型为class Sorter<S>,其中没有m_data

要解决此问题,请将m_dataQuickSorter移至Sorter,或将operator <<移至QuickSorter<S>&

答案 1 :(得分:2)

您在operator <<命名空间之外定义了Sorters重载。这是一个错误:你需要在与类相同的命名空间中定义它(首先,这是你使用friend声明声明它的地方;第二,这是与参数相关的循环的地方在以后使用它时正在寻找过载。

答案 2 :(得分:1)

为什么显示的代码按照它的方式执行,在注释中说明:cout << this打印一个指针,导致输出一个地址。您的operator <<重载需要引用对象而不是指针,因此不会使用它。正如您所说,解决方法是使用cout << *this

当我进行更改时,我最终出现了链接器错误。在模板和命名空间之间的某个地方,有些东西混淆了,我不知道是什么。所以我做了简单的解决方案:在类中定义函数。

而不是

template<typename T>
struct QuickSorter {
   template<typename T> friend ostream& operator <<(ostream&, QuickSorter<T>&);
}

template<typename T>
ostream& operator <<(ostream&, QuickSorter<T>&) { }

我做了

template<typename T>
struct QuickSorter {  
   friend ostream& operator <<(ostream&, QuickSorter&) { }
};

现在,您不必担心正确获取模板参数等等。模板化函数无论如何都必须以内联方式声明,所以你不妨这样做,并将其完成。请注意,您也没有将operator <<声明为模板本身......您只对将一个重载赋予友谊感兴趣,因为该类恰好正在使用该特定T

然后,您会被告知该函数访问m_data中不存在的成员Sorter<T>。这很简单:Sorter<T>没有该成员; QuickSorter<T>。因此,将第二个参数更改为函数QuickSorter<T>&,以便编译器可以找到该特定成员。这种变化也反映在上面。

我要做的最后一件事是让运算符重载接受const QuickSorter<T>&而不是非常量的{{1}}。它不会以任何方式修改对象,因此没有理由不这样做。