Реализация класса «полином» с динамическим массивом?

Question

[Даниил Вершинин]

В массив k не записываются коэффициенты, собственно в этом и проблема. (Параллельный вопрос - так как я накодил - так вообще можно? то есть правильно?)

class polynom {
    int polynomsize;
    float *k;
    //float k[100];

    public:
    // Конструктор с параметрами, инициализирующий степень полинома и выделяющий память под массив.
    polynom(int S,float K[]) {
        polynomsize = S;
        float *k = new float[polynomsize];
        for(int i=0; i<polynomsize; ++i)
            k[i]=K[i];
    }
    string get() {
        string s="[Poly] ";
        for (int i=0; i<polynomsize; i++) {
            if (k[i]!=0)
                if ((polynomsize-i-1)!=0)
                    s+=to_string(k[i])+"x^"+to_string(polynomsize-i-1)+" + ";
                else s+=to_string(k[i]);
        }
        return s;
    }

    void div(float n) {
        for (int i=0; i<polynomsize; i++) {
            k[i]=k[i]/n;
        }
    }
    ~ polynom(){
        delete[] k;
        cout << "destructor worked";
    };
};

Answer 1

[Alexander Movchan]

float *k = new float[polynomsize];

Вы переопределили k и присваиваете локальному указателю, а не полю.

Comments

[Даниил Вершинин]

а как правильно это сделать? указать свойство класса, и при этом хранить в нем динамический массив?

[Василий Мельников]

Даниил Вершинин,

float *k = new float[polynomsize];

заменить на

k = new float[polynomsize];

А еще лучше использовать vector.
В классе описать vector k;
В конструкторе его инициализировать до нужного рамера.

[Даниил Вершинин]

Василий Мельников, Спасибо, заработало, вектор нельзя было по заданию)

[Василий Мельников]

Даниил Вершинин, Неплохо бы для такого класса реализовать или запретить конструкторы копирования и перемещения, а также соответствующие операторы.
Иначе код вида

polynom a(2, {1, 2});
polynom b(2, {3, 4});

a = b;

компилируется, но будет приводить к падению

[Даниил Вершинин]

Василий Мельников, спасибо за то, что подчеркнул это - учту в будущем. Но в рамках поставленной задачи данной реализации достаточно. Не подскажешь литературу, где описываются подобные тонкости?

[Василий Мельников]

Даниил Вершинин, Можно Sutter C++ Coding Standarts (101 правило и рекомендация). Хотя под конец там может оказаться тяжеловато для начинающего.