Невозможно преобразовать int * в int ** в присваивании в C ++

Question

Невозможно преобразовать int * в int ** в присваивании в C ++

0

Я начинаю с C++, поэтому начал немного попрактиковаться, написав свой собственный класс Vector. Он хранит количество строк и столбцов массива, а элементы динамически распределены.

template <class T> 
class Vector {
    private:
        unsigned int rows;
        unsigned int cols;
        T **elements;
    public:
        Vector(unsigned int, unsigned int);
        ~Vector();
};

template <class T>
Vector<T>::Vector(unsigned int rows, unsigned int cols) {
    this->rows = rows;
    this->cols = cols;

    this->elements = new T[this->rows];
    for (int i = 0; i < this->rows; i++) {
        this->elements[i] = new T[this->cols];
    }
}

template <class T>
Vector<T>::~Vector() {
};

Вот код. Когда я его скомпилирую (я создаю объект для его проверки: Vector<int> test;), я получаю сообщение об ошибке: "Cannot convert 'int* to 'int** in assignment".

Почему я получаю эту ошибку? Я видел многомерный пример динамического размещения в Интернете: http://www.cplusplus.com/forum/beginner/63/

Mihail Feraru 25 окт. 2014, в 18:18

Источник

0

new T[size] возвращает T* . elements T** .
juanchopanza 25 окт. 2014, в 16:19
4

Прочитайте ту ссылку, на которую вы указываете внимательно. Вы пропускаете * где-то в первом new выражении. (И называть эту вещь вектором - очень странный выбор. Векторы имеют одно измерение, вы строите матрицу.)
Mat 25 окт. 2014, в 16:19
0

vector<vector<int>>
Neil Kirk 25 окт. 2014, в 16:21
0

Спасибо, Мат! Я должен быть более осторожным. Я прошу прощения за этот идиотский вопрос.
Mihail Feraru 25 окт. 2014, в 16:22
0

@NeilKirk: если местность не имеет значения
Ed S. 25 окт. 2014, в 16:31
0

пожалуйста, отметьте проблемную строку (в следующий раз ...)
Karoly Horvath 25 окт. 2014, в 16:46
0

Также не следует оставлять деструктор таким, какой он есть, вам нужно освободить память, выделенную для elements
P0W 25 окт. 2014, в 16:49
0

@EdS. Не хуже Т **
Neil Kirk 25 окт. 2014, в 17:09
0

@NeilKirk: я бы не заменил его на T** , я бы заменил его на vector<T>(rows * columns) (при необходимости) и вычислил бы индексы вручную. В большинстве случаев это, вероятно, не имеет значения, конечно.
Ed S. 25 окт. 2014, в 17:20
0

@NeilKirk: но да, как прямая замена, вы на 100% правы, очевидно.
Ed S. 25 окт. 2014, в 17:48

Показать ещё 8 комментариев

Теги:

c++

class

arrays

templates

dynamic-memory-allocation

1 ответ

Ещё вопросы

new T[size] возвращает T* . elements T** .
Прочитайте ту ссылку, на которую вы указываете внимательно. Вы пропускаете * где-то в первом new выражении. (И называть эту вещь вектором - очень странный выбор. Векторы имеют одно измерение, вы строите матрицу.)
Спасибо, Мат! Я должен быть более осторожным. Я прошу прощения за этот идиотский вопрос.
@NeilKirk: если местность не имеет значения
пожалуйста, отметьте проблемную строку (в следующий раз ...)
Также не следует оставлять деструктор таким, какой он есть, вам нужно освободить память, выделенную для elements
@NeilKirk: я бы не заменил его на T** , я бы заменил его на vector<T>(rows * columns) (при необходимости) и вычислил бы индексы вручную. В большинстве случаев это, вероятно, не имеет значения, конечно.
@NeilKirk: но да, как прямая замена, вы на 100% правы, очевидно.

Marco A. · Answer 1 · 2014-10-25T14-40-00.000Z

Самая большая ошибка заключается в том, что вам не хватает " * " при распределении массива указателей:

this->elements = new T*[this->rows];

Помимо этого:

Почему вы определяете вектор как матрицу с строками и столбцами?
Свободная память после того, как вы выделили и использовали, когда она больше не нужна
Должны быть доступны аксессоры

в качестве отправной точки возьмите следующий код:

template <class T> 
class Vector {
    public:
        unsigned int rows;
        unsigned int cols;
        T** elements;
        Vector(unsigned int, unsigned int);
        ~Vector();
};

template <class T>
Vector<T>::Vector(unsigned int rows, unsigned int cols) {
    this->rows = rows;
    this->cols = cols;

    this->elements = new T*[this->rows];
    for (int i = 0; i < this->rows; i++) {
        this->elements[i] = new T[this->cols];
    }
}

template <class T>
Vector<T>::~Vector() {
    for(int i=0; i<this->rows;++i)
        delete[] this->elements[i];
    delete[] this->elements;
};


int main()
{
    Vector<int> obj(2,2);
    obj.elements[0][0] = 1;
    obj.elements[0][1] = 2;
    obj.elements[1][0] = 3;
    obj.elements[1][1] = 4;

    std::cout << obj.elements[0][0] << " " << obj.elements[0][1] << std::endl;
    std::cout << obj.elements[1][0] << " " << obj.elements[1][1] << std::endl;

    return 0;
}

Example