Re: using vector to encapulate a tree - non const copy constructors

news1@lyonstech.net (terry) wrote (abridged):


Inheritance is a very tightly coupled relationship. One danger is that all
the names of the base class are injected into the derived class. Another
is that the base class can call any functions in the derived class, even
private ones, if it happens to declare a virtual function with the same
signature itself.

The upshot is that we generally avoid using inheritance where it is not
necessary, and it's not necessary here. Your example class could be
written with a vector member and forwarding functions instead. Eg:

    template<typename T, typename Id>
    class my_vector {
        typedef std::vector<T> Base;
        Base base;
    public:
        typedef Base::iterator iterator;
        typedef Base::const_iterator const_iterator;

        bool empty() const { return base.empty(); }
        size_t size() const { return base.size(); }
        // Scores more forwarding functions.
    };

It's slightly more work, but since this template is generic we only need
to do it once so that isn't significant.

Incidently, I replaced the int parameter with a type so you don't have to
mess with allocating numbers to types. Instead of:


you'd have:
    struct row_tag {};
    typedef my_vector<double,row_tag> row_vector_double;

-- Dave Harris, Nottingham, UK.

--
      [ See http://www.gotw.ca/resources/clcm.htm for info about ]
      [ comp.lang.c++.moderated. First time posters: Do this! ]