Различное поведение с вызывающими функциями внутри и снаружи объектов

Question

С моим кодом на C ++ происходит очень странное поведение.У меня есть функция backward () в моем классе MultiplyOperation, которую я вызываю из другого объекта класса FloatTensor, который имеет экземпляр этого класса MultiplyOperation.

Результат вывода будет другим, если я вызову three.backOperation->backward(1); и three.backOperation->backward(1);, тогда как он должен быть одинаковым.Пожалуйста, помогите.

Это мой код:

#include<iostream>
using namespace std; 

class FloatTensor;

class MultiplyOperation{
     public:
        FloatTensor *t1, *t2; 
        float grad = 10;

        MultiplyOperation(FloatTensor* t1, FloatTensor* t2);

        FloatTensor compute();

        void backward(float gradient);
};

class FloatTensor {
    public:
    float val; 
    float grad; 
    MultiplyOperation* backOperation = NULL, *frontOperation = NULL;

    FloatTensor() {
        // default
    }

    FloatTensor(float value) {
        this->val = value;
        this->backOperation = NULL;
    }

    FloatTensor(float value, MultiplyOperation* backOp) {
        this->val = value;
        this->backOperation = backOp;
    }

    void backward(float gradient) {
        this->backOperation->backward(gradient);
    }


    FloatTensor operator * (FloatTensor two) { 
        MultiplyOperation ope(this,&two);
        this->frontOperation = &ope;
        return this->frontOperation->compute();
    }
};


MultiplyOperation::MultiplyOperation(FloatTensor* te1, FloatTensor* te2) {
    this->t1 = te1;
    this->t2 = te2;
}

FloatTensor MultiplyOperation::compute() {
    return FloatTensor(this->t1->val*this->t2->val, this);
}

void MultiplyOperation::backward(float gradient) {

    cout<<this->t2->val<<endl;
}


int main() {

    FloatTensor one(2);
    FloatTensor two(4);

    FloatTensor three = one*two;

    three.backOperation->backward(1); // should be same as output of next line and is 4. (which is correct)
    three.backward(1); // should be same as output of above line but is garbage value -4.12131 

}

Answer 1

Ваша программа имеет неопределенное поведение из-за следующей функции:

FloatTensor operator * (FloatTensor two) { 
   MultiplyOperation ope(this,&two);
   this->frontOperation = &ope;
   return this->frontOperation->compute();
}

Вы храните указатель на объект ope, который больше не действителен после возврата из функции.

Та же проблема существует с использованием two при создании ope.

Вы можете изменить функцию для хранения указателя на динамически размещенный объект для решения этой проблемы.

FloatTensor operator * (FloatTensor two) { 
   this->frontOperation = new MultiplyOperation(this, new FloatTensor(two));
   return this->frontOperation->compute();
}

или используйте

FloatTensor operator * (FloatTensor& two) { 
   this->frontOperation = new MultiplyOperation(this, &two);
   return this->frontOperation->compute();
}

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