Как я могу передать объект, содержащийся внутри вектора, оператору? - PullRequest
Новая
       8

Как я могу передать объект, содержащийся внутри вектора, оператору?

0 голосов
/ 26 марта 2012

Моя домашняя работа требует, чтобы я использовал двоичный оператор для управления комплексными числами, которые хранятся в классе.

Поскольку каждый класс создается (используя new) в цикле чтения из файла, чтобы иметь возможность ссылаться на него, я сохранял их в векторе (используя pushback).

Текущее определение функции: 

Complex Complex::operator+ (vector<Complex> &v) 
{
result= new Complex;

result.real=real+v[y].real;
result.imaginary=imaginary+v[y].imaginary;

return result;
}

Текущее определение в классе: 

Complex operator+ (vector<Complex> &v);

Вызывается в основной функции: Результат = v [х] + V [у];

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

Я понимаю, что все это очень неправильно, но я не знаю, как лучше всего это сделать.

Любая помощь будет отличной! :) Нх

1 Ответ

0 голосов
/ 26 марта 2012 
result.real=real+v[y].real;

Откуда вы получаете y от?

Вы определяете добавление Complex к std::vector<Complex>.Это немного странно.Обычно вы хотите определить перегрузку Complex + Complex и использовать ее (ТОЛЬКО ) как синтаксическое удобство, когда оно очевидно или определено математически и однозначно ) для переменных этого типа (например, дляиндекс / доступ из вашего массива / вектора).Поведение, которое вы в настоящее время определяете, должно быть не в перегрузке оператора, а в простой функции.

Использование new не нужно и приводит к утечке памяти, потому что нет соответствующего delete,и, следовательно, этот адрес памяти в куче не освобождается.Вы должны просто инициализировать его как обычную переменную стека: 

Complex result;

Предпочитайте выделение стека, когда это возможно;это не только позволяет избежать проблем с памятью, но и быстрее.

В любом случае эта переменная является временной, и в ней сделано две копии;один на возврате, а другой внутри вектора на push_back.

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

Вы можете использовать шаблоны для определениякласс агрегации как шаблонный класс, а затем создать его с вашими числовыми типами, например Complex (просто убедитесь, что у них определен operator+()), затем в некоторой функции в классе вы можете использовать перегрузку + дляшаблонные переменные: 

template <typename T>
class MyNumericAggregatorClass
//... in some function
// T a; T b;
T result = a + b;
//...

Альтернативой этому является использование полиморфизма для создания базового числового класса с virtual operator+() и подклассами (например, Complex), которые переопределяют этот оператор, тогда вы можете работатьна коллекцию указателей базового класса.

Чтобы дать дальнейшие указания, нужно показать больше вашего кода.Но если все, что вы пытаетесь сделать, это суммировать экземпляры числового типа, сохраненные в векторе, используя оператор +, тогда извлеките std::accumulate из STL, и ваша проблема решена.

...