Problem z wysyłaniem obiektów przez referencje

Question

Siemanko. Robiąc sobie powtórzenie teorii przed Builderem, grzebiac w kodzie napotkałem mały błąd związany z przesyłaniem obiektów funkcji przez referencje. W komentarzach // w kodzie opisałem krok po kroku co robię.

 Mój kod :

main

#include <iostream>
#include "przyjaciele.h"

using namespace std;

int main()
{
    Punkt pkt1;
    pkt1.wczytaj();

    Prostokat p1;
    p1.wczytaj();
    p1.sedzia(&pkt1,&p1); // 3. wysylam oryginalne obiekty funkcji i ja wywoluje

    return 0;
}

przyjaciele.h

#include <iostream>

using namespace std;


class Punkt
{
    string nazwa;
    float x,y;

public:
    Punkt(string="A",float=0,float=0);       
    void wczytaj();               

    friend void sedzia(); 
};



class Prostokat
{
    string nazwa;
    float x,y,szerokosc,wysokosc;

public:
    Prostokat(string="brak",float=0,float=0,float=1,float=1);
    void wczytaj();
   
    void sedzia(Punkt pkt, Prostokat p); // 1. ustawiam parametry formalne bez &
};

void sedzia() w przyjaciele.cpp

void Prostokat::sedzia(Punkt pkt, Prostokat p) // 2. tutaj także bez & 
{
    if ( (pkt.x>=p.x)&&(pkt.x<=p.x+p.szerokosc)
        &&(pkt.y>=p.y)&&(pkt.y<=p.x+p.wysokosc) )

    cout<<endl<<"Punkt "<<pkt.nazwa<<" nalezy do prostokata: "<<p.nazwa;
    else
    cout<<endl<<"Punkt "<<pkt.nazwa<<" lezy POZA prostokatem: "<<p.nazwa;
}

Kompilator pokazuje błąd:

In function 'int main()':
 no matching function for call to 'Prostokat::sedzia(Punkt*, Prostokat*)'|
|13|note: candidate is:
|29|note: void Prostokat::sedzia(Punkt, Prostokat)|
|29|note:   no known conversion for argument 1 from 'Punkt*' to 'Punkt'|

Mam wrażenie , że nie do końca rozumiem te przesyłanie obiektów przez referencje. Proszę o jakieś rady

Answer 1

Witam,

w pliku przyjaciel.h zmień w klasie Punkt

//friend void sedzia();
// na
friend void sedzia(Punkt& p, Prostokat& pr);

Następnie popraw funkcje składową sędzia w klasie Prostokąt

void sedzia(Punkt& p, Prostokat& pr);

Dodatkowo w main.cpp usuń ampersand z funkcji sedzia.

p1.sedzia(&pkt1,&p1); // tutaj przesyłasz adresy do pkt i p1
p1.sedzia(pkt1,p1);// tutaj przesyłasz obiekty przez referencje

Może to pomoże.

Comments

Zmieniłem tak jak mówiłeś. Poprzednie błędy się nie pokazują , natomiast teraz doszedłem do problemu ,który mnie nurtuje od samego początku. Związany jest on jak widać z przyjaźnieniem się funkcji.

Kompilator twierdzi , że wszystkie atrybuty klasy Punkt (nazwa , x , y) są prywatne (nie wykonuje się funkcja na nich polegająca).

Mimo tego , że w klasie Punkt określiłem , że jest ona "przyjacielem domu" :D

    friend void sedzia(Punkt &p, Prostokat &pr);

oraz na wszelki wypadek NAD klasą Punkt dodałem informacje ,że będzie taka klasa jak Prostokat

class Prostokat; // info dla klasy Punkt ze bedzie taka klasa pozniej w kodzie

Jak to jest z tym przyjaznieniem sie funkcji? Dlaczego w tym przypadku nie działa?

---

friend void Prostokat::sedzia(Punkt &p, Prostokat &pr);

 

---

Nie pomogło. Po zmianie kompilator daje błędy:

|17|error: invalid use of incomplete type 'class Prostokat'|
|5|error: forward declaration of 'class Prostokat'|

---

Wczoraj był podobny temat

https://forum.pasja-informatyki.pl/324831/jak-pozwolic-metodzie-na-korzystanie-z-argumentow-innej-klasy

Ewentualnie możesz wywołać definicje funkcji składowej

void Punkt::sedzia(Punkt &p, Prostokat &pr);

i w niej odwoływać się do składników klasy.

---

Kompilator musi znać definicję klasy Prostokat przed deklaracją zaprzyjaźniającą metody  Prostokat::sedzia(). A najlepiej będzie jak zrobisz pola nazwa, x i y publiczne (ewentualnie zrób jakieś akcesory do nich).