[C++] Iloczyn wektorowy

[Kaburagi 0]

Witam.

Mam problem ze zwracaniem wartości z funkcji:

 

VECTOR3 IloczynWektorowy(VECTOR3 vWektor1, VECTOR3 vWektor2)
{
VECTOR3 vWynik;

vWynik.x = vWektor1.y * vWektor2.z - vWektor2.y * vWektor1.z;
vWynik.y = vWektor2.x * vWektor1.z - vWektor1.x * vWektor1.z;
vWynik.z = vWektor1.x * vWektor2.y - vWektor2.x * vWektor1.y;

return vWynik;
}

 

Przy czym tak jest wywoływana:

 

cout << "Iloczyn wektorowy dla podanych wartosci = " << IloczynWektorowy(Wektor1, Wektor2).x << " " << IloczynWektorowy(Wektor1, Wektor2).y << " " << IloczynWektorowy(Wektor1, Wektor2).z;

 

 

Program kompiluje się, odpala, wszystko liczy.. Ale taki zapis pozostawia u mnie pewien niesmak. Po co robić coś 3 razy, skoro można raz.. Można to zrobić za jednym razem? Ewentualnie jak.

 

 

Poniżej cały kod programu.

 

 

#include <cmath>
#include <conio.h>
#include <iostream>

using namespace std;

struct VECTOR3
{
// współrzędne wektora
float x, y, z;

// konstruktory
VECTOR3()			        { x = y = z = 0.0; }		
VECTOR3(float fX, float fY, float fZ)	{ x = fX; y = fY; z = fZ; }

// -------------------------------------------------------

// metody

float Dlugosc() const { return sqrt(x * x + y * y + z * z); }
void Normalizuj()
{
	float fDlugosc = Dlugosc();

	// dzielimy każdą współrzędną przez długość
	x /= fDlugosc; y /= fDlugosc; z /= fDlugosc;
}
};

float IloczynSkalarny(VECTOR3 vWektor1, VECTOR3 vWektor2)
{
float fWynik = (vWektor1.x * vWektor2.x) + (vWektor1.y * vWektor2.y) + (vWektor1.z * vWektor2.z);

return fWynik;
}

VECTOR3 IloczynWektorowy(VECTOR3 vWektor1, VECTOR3 vWektor2)
{
VECTOR3 vWynik;

vWynik.x = vWektor1.y * vWektor2.z - vWektor2.y * vWektor1.z;
vWynik.y = vWektor2.x * vWektor1.z - vWektor1.x * vWektor1.z;
vWynik.z = vWektor1.x * vWektor2.y - vWektor2.x * vWektor1.y;

return vWynik;
}

void main()
{
VECTOR3 Wektor1(1.0, 2.0, 3.0);
VECTOR3 Wektor2(4.0, 5.0, 6.0);
//Wektor1.Normalizuj();
//Wektor2.Normalizuj();

cout << "Iloczyn skalarny dla podanych wartosci  = " << IloczynSkalarny(Wektor1, Wektor2) << endl;
cout << "Iloczyn wektorowy dla podanych wartosci = " << IloczynWektorowy(Wektor1, Wektor2).x << " " << IloczynWektorowy(Wektor1, Wektor2).y << " " << IloczynWektorowy(Wektor1, Wektor2).z;

getch();
}

[wesolyzbigniew 0]

Nie da się po prostu przypisać wyniku do jakiejś struktury? Nie da się stworzyć jakiegoś wskaźnika do wyniku?

Od bardzo dawna nie programowałem w C++ (i nigdy tego języka porządnie nie opanowałem), ale pytanie wydaje się dziwne.

[Kaburagi 0]

To jest przypisane do struktury właśnie.

[wesolyzbigniew 0]

Wygrzebałem u siebie jakieś IDE do C++ (tak, DevC+...).

 

int main()
{
       VECTOR3 Wektor1(1.0, 2.0, 3.0);
       VECTOR3 Wektor2(4.0, 5.0, 6.0);
       //Wektor1.Normalizuj();
       //Wektor2.Normalizuj();
       //VECTOR3 ilw = IloczynWektorowy(Wektor1, Wektor2);
       VECTOR3 *ilw = &IloczynWektorowy(Wektor1, Wektor2);
       VECTOR3 ilwn = IloczynWektorowy(Wektor1, Wektor2);
       cout << "Iloczyn skalarny dla podanych wartosci  = " << IloczynSkalarny(Wektor1, Wektor2) << endl;
       cout << "Iloczyn wektorowy dla podanych wartosci = " << ilw->x<< " " << ilw->y << " " << ilw->z;
       cout << "Iloczyn wektorowy dla podanych wartosci = " << ilwn.x<< " " << ilwn.y << " " << ilwn.z;

       getch();
}

Na dwa sposoby. Teraz wiesz o co mi chodziło?

[Kaburagi 0]

No tak, ale to w istocie niczego nie zmienia.

[qwerty5712 0]

Raczej oczywistym się wydaje, że jak masz

X.{x,y,z}, to musisz napisać X.x, X.y, X.z. No mógłbyś to jakoś uproscić gdybyć np napisał metodę toString i raz byś miał, albo jakąś łądną abstrakcje. Tylko powoli, tu wchodzisz w programowanie obiektowe.

EDIT napisane bardzo ideowo.

struct Vector2{
float x,y;
string to String(){
 std::ostringstream stringStream;

 stringStream<<"x = "<<x<<" y= "<<y
 std::string copyOfStr = stringStream.str();
 return copyOfStr;
}
}
struct Vector3:Vector2{
float z;
strint toString(){
 std::ostringstream stringStream;

 stringStream<<Vector2::toString() << " z = "<<z;
 std::string copyOfStr = stringStream.str();
 return copyOfStr;
}
}

 

 

EDIT Druga rzecz, którą widać w kodzie wyżej, ale mi się nie chciało pisać i w twoim też. Zwracasz struktury przez wartość, co prawda kompilator, to optymalizuje ale nie jest to zbyt bezpieczne. Możesz, to rozwiązać na kilka sposobów, albo zwracać wskaźnik do zaalackowanego Vectora(lub string jak wyzej), albo podać refencje, wskaźnik w funkcji do wyniku. To będzie działać co piszesz nawet dla małych struktór powinno się tak robić ze względu na szybkość tego rozwiązania, ale stos jest mały i kiedyś możesz przez to cierpieć.

 

Lepiej, żebyś tego nie robił, bo kiedyś napiszesz coś takiego. To jest złe, bo tworzy się referencję do zmiennej, która zaraz zginie i ma niezdefiniowane zachowanie.

string & name(){
string nam= "foo";
return nam;
}

[wesolyzbigniew 0]

Jak to nie zmienia? Funkcja jest wielokrotnie wywoływana.

[Kaburagi 0]

Raczej oczywistym się wydaje, że jak masz

X.{x,y,z}, to musisz napisać X.x, X.y, X.z. No mógłbyś to jakoś uproscić gdybyć np napisał metodę toString i raz byś miał, albo jakąś łądną abstrakcje. Tylko powoli, tu wchodzisz w programowanie obiektowe.

 

Tak, oczywistym jest, że muszę "dostać się" do x,y,z. Rzecz w tym czy nie da się tego jakoś "oprawić", uprościć.

[BlackBishop 1]

przecież wyżej masz rozwiązanie - przypisujesz do zmiennej i z niej wyciągasz x,y i z

[hot_stuff 0]

Już nie pamiętam jak się to robiło i nie do końca jestem pewny czy o to autorowi chodzi, ale.. przecież można zaprzyjaźnić operator >> z klasą VECTOR3 (działa to jak toString() w Javie o którym już ktoś powyżej napisał). Wtedy wyświetlenie VECTOR3 odbywa się zgodnie z oczekiwaniami<?> czyli:

VECTOR3 test;
//...
cout << test;

 

Na szybko znalezione --> http://pl.wikibooks.org/wiki/C++/Przeci%C4%85%C5%BCanie_operator%C3%B3w#Przyk.C5.82ad_zastosowania

[qwerty5712 0]

Tak, oczywistym jest, że muszę "dostać się" do x,y,z. Rzecz w tym czy nie da się tego jakoś "oprawić", uprościć.

#include<iostream>
using namespace std;
struct v3{
   int &x;
   int &y;
   int &z;

};
void do_sth(v3 & r){
   r.x=1;
   r.y=2;
   r.z=3;
}
int main(){
   int x[3];
   v3 y={x[0],x[1],x[2]};
   do_sth(y);
   cout<<x[0]<<x[1]<<x[2]<<endl;
   return 0;
}

O takie coś ci chodzi? Nie musisz wtedy robić. y.x,y.y,y.z. Możesz napisać pętle for. Nie jest jednak to eleganckie moim zdaniem.

[Kaburagi 0]

@Qwerty5712 - myślałem o takim czymś, ale moim zdaniem też nie jest to "eleganckie" rozwiązanie.

 

Pobawię się jeszcze przeciążeniami.

 

Dzięki za podpowiedzi.