Przeciążenie operatorów c++

Question

Witam

Potrzebuje zrobić mnożenie dwóch wektorów. Nie mam pojęcia jak zwrócić nowy wektor. Prosiłbym o pomoc. Mam problem w miejscu:

Wektor Wektor::operator* (Wektor const& a)
{
	return Wektor(this->wektor * a.wektor);
}

Oczywiscie musze zrobic takze mnozenie wektora razy macierz ale najpierw chce zajac sie wektorami.

Z gory dzieki za pomoc

 

 Z góry dzięki.

 

plik.cpp

#include <iostream>
#include <string>
#include <cstdlib>
#include "macierze_wektory.h"

using namespace std;

void Wektor::Wczytaj_Wektor()
{
	fstream plik;
	plik.open("wektor.txt");
	if (!plik.good())
	{
		cout << "Nie udalo sie otworzyc pliku";
		system("pause");
		exit(0);
	}

	for (int i = 0; i < 4; i++)
		plik >> wektor[i];

	plik.close();
}

void Macierz::Wczytaj_Macierz()
{
	fstream plik;
	plik.open("macierz.txt");
	if (!plik.good())
	{
		cout << "Nie udalo sie otworzyc pliku";
		system("pause");
		exit(0);
	}

	for (int j = 0; j < 4; j++)
	for (int i = 0; i < 4; i++)
		plik >> macierz[i][j];

	plik.close();
}

Wektor Wektor::operator* (Wektor const& a)
{
	return Wektor(this->wektor * a.wektor);
}

plik.h

#include <iostream>
#include <string>
#include <fstream>
#include <cstdlib>

using namespace std;

class Macierz;

class Wektor
{
private:
	double wektor[4];

public:
	Wektor()
	{
		for (int i = 0; i < 4; i++)
			this->wektor[i] = 0;
	}
	void Wczytaj_Wektor();
	Wektor operator*(const Wektor &a);
};

class Macierz
{
private:
	double macierz[4][4];

public:
	Macierz()
	{
		for (int j = 0; j < 4; j++)
			for (int i = 0; i < 4; i++)
				this->macierz[i][j] = 0;
	}
	void Wczytaj_Macierz();
};



main.cpp

#include <iostream>
#include <string>
#include <cstdlib>
#include "macierze_wektory.h"

using namespace std;

int main()
{
	Wektor w1;
	w1.Wczytaj_Wektor();
	Wektor w2;
	w2.Wczytaj_Wektor();
	Wektor w3 = w1*w2;

	system("pause");
	return 0;
}

 

Answer 1

Po pierwsze, aby tego dokonać powinieneś zmienić zmienną double wektor[4] na public lub stworzyć metodę, która będzie zwracała jej wartość, by można było wykorzystać ją w innych klasach czy funkcjach. Pod drugie, tablic nie można mnożyć tak po prostu przez siebie, tylko należy mnożyć pojedyncze elementy przez inne. Spróbuj z takim kodem.

Wektor Wektor::operator* (Wektor const& a) {

Wektor v;

for (int i = 0; i < 4; i++)
    v.wektor[i] = wektor[i] * a.wektor[i];

return v;
}

Jak nie zadziała to pisz. Pozdrawiam.

PS Polecam nazywać zmienne i klasy po angielsku, to dobra praktyka :)

Answer 2

Zwrócić.. Zwracasz dobrze. Ale próbujesz mnożyć wskaźnik przez wskaźnik czyli w sumie nie wiem co zrobić i przekazać wynik mnożenia do konstruktora Wektor. Z tym że Wektor nie ma zdefiniowanego konstruktora, który przyjmował by jakiekolwiek argumenty.

Przemnóż po kolei każdy z elementów tablicy obu Wektorow (bo pewnie o to ci chodzilo), a potem w jakis sposób stwórz vector z przemnożonymi wartościami. Jak to zrobisz już zależy od ciebie.

Answer 3

Dzieki za pomoc. Już działa .

Pozdrawiam !!!!