(c ++) stl векторная реализация

Question

Я реализовал простую векторную структуру. Она хорошо работает, если я использую vector<int> или vector<char>, но когда я использую <vector<vector<int>>, это вызывает ошибку. Есть ли хороший код реализации для вектора stl или проблема в моем коде?

вот мой код

class _vector {
private:
    int _size;
    int _capacity;
    T* vc;

public:
    _vector(int size = 1) {
        _size = 0;
        _capacity = size;
        vc = new T[size];
    }
    ~_vector() {
        delete[] vc;
    }
    int size() { return _size; }
    bool empty() { return !_size; }
    void resize(int size) {
        _capacity = size;
        T* tmp = new T[size];
        for (int i = 0; i < _size; i++) tmp[i] = vc[i];
        delete[] vc;
        vc = tmp;
    }
    void clear() {
        delete[] vc;
        _capacity = 1;
        _size = 0;
        vc = new T[_capacity];
    }
    void push_back(T val) {
        if (_size == _capacity) resize(2 * _capacity);
        vc[_size++] = val;
    }
    void pop_back() {
        if (_size == 0) return;
        vc[--_size] = 0;
    }
    T& operator[](int i) const { return vc[i]; }
    _vector<T>& operator=(_vector<T> &tmp) {
        _capacity = tmp._capacity;
        _size = tmp._size;
        delete[] vc;
        vc = new T[_capacity];
        for (int i = 0; i < _size; i++) vc[i] = tmp[i];
        return *this;
    }

Answer 1

Ваша реализация не следует правилу 3 , так как в нем отсутствует конструктор копирования и правильный оператор присваивания копии (который может быть реализован с использованием конструктора копирования).А в C ++ 11 и более поздних, Правило 5, добавляя конструктор перемещения и оператор присваивания перемещения.

Кроме того, ваша реализация не работает правильно с нетривиальными типами, для которых определены конструкторы / деструкторынапример, когда T - это другой тип _vector, или любой другой тип, в котором есть указатели / ресурсы, выделенные внутри него.Итак, вашему классу необходимо создавать новые объекты при добавлении элементов в массив, используя placement-new, и уничтожать объекты при удалении элементов из массива, напрямую вызывая их деструкторы.

Вместо этого попробуйте что-то вроде этого:

template <typename T>
class _vector {
public:
    typedef unsigned int size_type;
    typedef T value_type;

private:
    size_type _size;
    size_type _capacity;
    value_type* vc;

public:
    _vector(size_type initalcap = 0) : _size(0), _capacity(0), vc(0) {
        reserve(initialcap);
    }

    _vector(const _vector<T> &src) : _size(0), _capacity(0), vc(0) {
        reserve(src._capacity);
        for(size_type i = 0; i < src._size; ++i) {
            new(vc[i]) value_type(src.vc[i]);
        }
        _size = src._size;
    }

    // C++11 and later only...
    _vector(_vector<T> &&src) : _size(src._size), _capacity(src._capacity), vc(src._vc) {
        src._size = 0;
        src._capacity = 0;
        src.vc = 0;
    }

    ~_vector() {
        clear();
        delete[] reinterpret_cast<char*>(vc);
    }

    size_type size() const { return _size; }
    size_type capacity() const { return _capacity; }
    bool empty() const { return !_size; }

    void reserve(size_type newcap) {
        if (newcap <= _capacity) return;
        value_type* tmp = reinterpret_cast<value_type*>(new char[sizeof(value_type) * newcap]);
        for (size_type i = 0; i < _size; ++i) {
            new(tmp[i]) value_type(vc[i]);
        }
        delete[] reinterpret_cast<char*>(vc);
        vc = tmp;
        _capacity = newcap;
    }

    void resize(size_type newsize) {
        if (newsize < _size) {
            for(size_type i = _size; i-- > newsize; ) {
                vc[i].~value_type();
            }
            _size = newsize;
        }
        else if (newsize > _size) {
            reserve(newsize);
            for (size_type i = _size; i < newsize; ++i) {
                 new(vc[i]) value_type();
            }
            _size = newsize;
        }
    }

    void clear() {
        resize(0);
    }

    void push_back(const T &val) {
        if (_size == _capacity) reserve(2 * _capacity);
        new(vc[_size]) value_type(val);
        ++_size;
    }

    void pop_back() {
        if (_size) {
            vc[--_size].~value_type();
        }
    }

    value_type& operator[](size_type i) { return vc[i]; }
    const value_type& operator[](size_type i) const { return vc[i]; }

    _vector<T>& operator=(const _vector<T> &rhs) {
        if (&rhs != this) {
            _vector<T> tmp(rhs);
            std::swap(tmp.vc, vc);
            std::swap(tmp._size, _size);
            std::swap(tmp._capacity, _capacity);
        }
        return *this;
    }

    // C++11 and later only...
    _vector<T>& operator=(_vector<T> &&rhs) {
        _vector<T> tmp(std::move(rhs));
        std::swap(tmp.vc, vc);
        std::swap(tmp._size, _size);
        std::swap(tmp._capacity, _capacity);
        return *this;
    }
};