Odwołanie się do obiektu będącego składową innego obiektu w funkcji

Question

Witam.Obiekt klasy okrąg składa się z obiektu klasy punkt oraz ze zmiennej typu double oznaczonej jako promień.Podczas wywołania funkcji przecieciev2 funkcja operuje na wartościach 0 zamiast na wartościach przekazanych przez parametry funkcji i nie wiem jak to mógłbym naprawić.Jestem nowy na tym forum i nie widzę możliwości dodania załącznika w postaci pliku cpp zatem wkleję go tutaj:

Z góry dziękuję za pomoc
 

#include<iostream>
#include<cmath>
using namespace std;
static int counter=0;
class okrag;
class punkt
{
    public:
    friend void przecieciev2(okrag* jeden,okrag* dwa);    
    double x,y;    
    double pobierzX(punkt* obiekt)
    {
        cout<<"Podaj wspolrzedna x:\t";cin>>obiekt->x;cout<<endl;
        return obiekt->x;
    }
    double pobierzY(punkt* obiekt)
    {
        cout<<"Podaj wspolrzedna y:\t";cin>>obiekt->y;cout<<endl;
        return obiekt->y;
    }
    void wypisz(punkt* obiekt)
    {
        cout<<"x="<<obiekt->x<<"\t\ty="<<obiekt->y<<endl;
    }
    punkt(double jakiesx=0.,double jakiesy=0.)
    {
        x=jakiesx;
        y=jakiesy;
    }
    ~punkt()    {}

};
class odcinek
{
    friend class okrag;
    friend void przecieciev2(okrag* jeden,okrag* dwa);
    public:
    double pocz,kon;
    double dl_odcinka(punkt* jeden,punkt* dwa)
    {
        cout<<"WYWOLANIE METODY DL_ODCINKA"<<endl;
        double dl;
        double dx=(jeden->x)-(dwa->x);
        double dy=(jeden->y)-(dwa->y);
        cout<<"dx="<<dx<<"\t\tdy="<<dy<<endl;
        dl=sqrt((dx*dx)+(dy*dy));
        cout<<dl<<endl;
        return dl;    
    }    
    odcinek(punkt* obiekt1,punkt* obiekt2)
    {
        cout<<"odcinek sklada sie z punktow:\t\t("<<obiekt1->x<<","<<obiekt1->y<<"),("<<obiekt2->x<<","<<obiekt2->y<<")"<<endl;
    }
    odcinek()    {}
    ~odcinek()    {}
    
};
class okrag:public punkt
{
    public:    
    friend class punkt;friend class odcinek;
//    friend void punkt::wypisz(punkt* obiekt);
    friend void przecieciev2(okrag* jeden,okrag* dwa);
    double promien;
    okrag(punkt* obiekt,double promien)
    {
        counter++;
        cout<<"uzyto konstruktora\t\tcounter="<<counter<<endl;
        cout<<"okrag o wsp srodka:("<<obiekt->x<<","<<obiekt->y<<") i o promieniu r="<<promien<<endl;
    }
    ~okrag()
    {
        counter--;
        cout<<"uzyto destruktora\t\tcounter="<<counter<<endl;        
    }
    void rown(punkt* jakisokrag)
    {
        cout<<"WYWOLANIE METODY ROWN"<<endl;
        cout<<"Dany jest okrag o srodku:("<<jakisokrag->x<<","<<jakisokrag->y<<")"<<" i o promieniu"<<this->promien<<endl;
    }
    
};

void przecieciev2(okrag* jeden,okrag* dwa)
{
    cout<<"WYWOLANIE PRZECIECIE"<<endl;
    punkt* ob1=new punkt(jeden->x,jeden->y);
    punkt* ob2=new punkt(dwa->x,dwa->y);
    
    
    cout<<"TEST"<<endl<<endl;
    cout<<jeden->x<<jeden->y<<dwa->x<<dwa->y<<endl;
    
    
    odcinek sr;
    double odl_sr=sr.dl_odcinka(ob1,ob2);
    double sumapromieni=jeden->promien+dwa->promien;
    cout<<"dlugosc odcinka stworzonego ze srodkow okregow:"<<odl_sr<<endl;
    if(odl_sr==(jeden->promien+dwa->promien))
    {cout<<"okregi styczne zewnetrznie!"<<endl;}
    else if(odl_sr==(fabs(jeden->promien-dwa->promien)))
    {cout<<"okregi styczne wewnetrznie!"<<endl;}
    else if(odl_sr<(fabs(jeden->promien-dwa->promien)))
    {cout<<"okregi rozlaczne wewnetrznie!"<<endl;}
    else if(odl_sr>(jeden->promien+dwa->promien))
    {cout<<"okregi rozlaczne zewnetrznie!"<<endl;}
    else if(((fabs(jeden->promien-dwa->promien))<odl_sr)&&(odl_sr<(jeden->promien+dwa->promien)))
    {cout<<"okregi przecinajace sie!"<<endl;}
    else if(odl_sr==0)
    {cout<<"okregi wspolsrodkowe!"<<endl;}
}

int main()
{
punkt *p1=new punkt;
p1->pobierzX(p1);
p1->pobierzY(p1);
p1->wypisz(p1);    

punkt *p2=new punkt;
p2->pobierzX(p2);
p2->pobierzY(p2);
p2->wypisz(p2);

odcinek od(p1,p2);
od.dl_odcinka(p1,p2);

cout<<p1->x;
okrag* o1=new okrag(p1,5);
okrag* o2=new okrag(p2,5);
okrag o3(p2,5);
cout<<"TEST"<<endl<<endl<<o3.x<<o3.y<<endl<<endl;
o1->rown(o1);

cout<<"TEST"<<endl<<endl<<p1->x<<p1->y<<p2->x<<p2->y<<endl;
przecieciev2(o1,o2);
cout<<"TEST"<<endl<<endl<<p1->x<<p1->y<<p2->x<<p2->y<<endl;

delete p1;delete p2;
delete o1;delete o2;
return 0;    
}

 

Comments

Prosiłbym wstawiać kod źródłowy w bloczek i prawidłowo nazywać tagi. Uprości to innym korzystanie z forum. Takie niuanse opisane są w naszym FAQ. Dzięki! 

Answer 1

Obiekt klasy okrąg składa się z obiektu klasy punkt oraz ze zmiennej typu double oznaczonej jako promień

Nie, u Ciebie okrąg *jest* punktem i ma dodatkowo promień. Czy na pewno tak to powinno wyglądać?

    okrag(punkt* obiekt,double promien)
    {
        counter++;
        cout<<"uzyto konstruktora\t\tcounter="<<counter<<endl;
        cout<<"okrag o wsp srodka:("<<obiekt->x<<","<<obiekt->y<<") i o promieniu r="<<promien<<endl;
    }

Konstruktor obiektu nic nie robi z argumentami, ten okrąg nijak się nie inicjalizuje.

punkt* ob1=new punkt(jeden->x,jeden->y);

Skoro okrąg jest punktem (choć być może nie powinien, patrz wyżej), to możesz po prostu używać `jeden` zamiast `ob1`.

p1->pobierzX(p1);

Czemu metoda obiektu bierze argument na ten sam obiekt? To w ogóle się nie zachowuje jak metoda.

	odcinek od(p1,p2);
	od.dl_odcinka(p1,p2);

To samo. Odcinek ma pola `pocz`, `kon`, a nic z nimi nie robisz, tylko przekazujesz rzeczy jako argumenty do metod.

Czemu wszędzie jest tyle `friend`ów?

Na razie silnie proponuję podszkolenie się z podstawami użycia klas, zanim zaczniesz kombinować z dziedziczeniem.