Odcinek 4 obiektowy c++. Błędy związane z brakiem deklaracji. Funkcja jako metoda w klasie prostokąt

Question

Mianowicie mam problem z kodem. Komunikat otrzymuje o braku deklaracji, nie mam pojęcia dlaczego. Czy ktoś mógłby pomóc?

#include <iostream>
#include "przyjaciele.h"
using namespace std;

/* void sedzia(Punkt pkt, Prostokat p)
{
   if((pkt.x >= p.x)&&(pkt.x <= p.x + p.szerokosc)&&(pkt.y >= p.y)&&(pkt.y <= p.y + p.wysokosc))
        cout << endl << "Punkt " << pkt.nazwa << " nalezy do prostokata: " << p.nazwa;
   else
        cout << endl << "Punkt " << pkt.nazwa << " lezy poza prostokatem: " << p.nazwa;
} */

int main()
{
    Punkt pkt1("A", 3, 17);
    Prostokat p1("Prostokat", 0, 0, 6, 4);
    sedzia(pkt1, p1);
    return 0;
}

#include <iostream>

using namespace std;

class Prostokat;

class Punkt
{
    string nazwa;
    float x, y;
public:
    Punkt(string = "A", float = 0, float = 0);
    void wczytaj();
    // friend void sedzia(Punkt pkt, Prostokat p);
    friend class Prostokat;
};

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();
    friend class Punkt;

};

#include <iostream>
#include "przyjaciele.h"
using namespace std;

Punkt::Punkt(string n, float xx, float yy)
{
    nazwa = n;
    x = xx;
    y = yy;
}

void Punkt::wczytaj()
{
    cout << "Podaj x: "; cin >> x;
    cout << "Podaj y: "; cin >> y;
    cout << "Nazwa punktu: "; cin >> nazwa;
}

Prostokat::Prostokat(string n, float xx, float yy, float s, float w)
{
    nazwa = n;
    x = xx;
    y = yy;
    szerokosc = s;
    wysokosc = w;
}

void Prostokat::wczytaj()
{
    cout << "Podaj x lewego dolnego naroznika: "; cin >> x;
    cout << "Podaj y lewego dolnego naroznika: "; cin >> y;
    cout << "Podaj szerokosc: "; cin >> szerokosc;
    cout << "Podaj wysokosc: "; cin >> wysokosc;
    cout << "Podaj nazwe prostokata: "; cin >> nazwa;
}

void Prostokat::sedzia(Punkt pp, Prostokat prro)
{
       if((pkt.x >= p.x)&&(pkt.x <= p.x + p.szerokosc)&&(pkt.y >= p.y)&&(pkt.y <= p.y + p.wysokosc))
        cout << endl << "Punkt " << pkt.nazwa << " nalezy do prostokata: " << p.nazwa;
   else
        cout << endl << "Punkt " << pkt.nazwa << " lezy poza prostokatem: " << p.nazwa;
}

 

Comments

Poczytać kurs C++ na temat funkcji.

---

Chcę to zmienić, żeby ta funkcja była metodą klasy prostokąt, a nie funkcją z maina. Stąd mam problem i oczekuje nakierowania a nie rzucenia hasłem.

---

Chcę to zmienić, żeby ta funkcja była metodą klasy prostokąt, a nie funkcją z maina. Stąd mam problem i oczekuje nakierowania a nie rzucenia hasłem.

Dopiero teraz o tym piszesz. W pytaniu jest tylko, że nie działa. 

To zrób tą metode w klasie prostokąt, gdzie argumentem będzie zmienna typu Prostokat.  Wtedy pobierasz w metodzie dane z klasy oraz z argumentu i dokonujesz obliczen.

Ba, nawet widzę, że coś jest, ale obie funkcje się nie pokrywają w klasie, jedna nie ma wgl parametrów, a druga ma. Strasznie to namieszane :P 

---

Tak, nie pokrywaja bo kolega zle spisal kod z odcinka.... Prosze popatrz na 21:50.... Tam widac deklaracje funkcji przyjazni....

 

W klasie dajesz
 

friend void sedzia(Punkt ptk, Prostokat p);

A ty probujesz zadeklarowac klase jako przyjaciela, a nie funkcje jako przyjaciela obu klas.

---

A ja jestem zdania, że jak jest friend to znak, że nie umiemy do końca obiektówki.

Tutaj wg mnie lepsze rozwiązanie. 
this. oznacza składowe w klasie Prostokąt. 

void Prostokat::sprawdzPunkt(Punkt  &punkt)
{
       if((punkt.x >= this.x)&&(punkt.x <= this.x + this.szerokosc)&&(punkt.y >= this.y)&&(punkt.y <= this.y + this.wysokosc))
        cout << endl << "Punkt " << punkt.nazwa << " nalezy do prostokata: " << p.nazwa;
   else
        cout << endl << "Punkt " << punkt.nazwa << " lezy poza prostokatem: " << p.nazwa;
}

 

---

@efik ja juz nie wiem w ilu ty jezykach siedzisz :D nie do konca sie moge zgodzic, bo funkcja friend jest jednak malym udogdnieniem, np. gdy piszesz walke z mobami

---

Dziękuje za odpowiedzi. Mam jednak problem ciągle z tym, że dostaję komunikat w mainie, że class:punkt has no member named sprawdzPunkt

#include <iostream>
#include "przyjaciele.h"
using namespace std;

void sedzia(Punkt &pkt,Prostokat &p)
{
   p.nazwa="PODMIANA";

if((pkt.x>=p.x)&&(pkt.x<=p.x+p.szerokosc)&&(pkt.y>=p.y)&&(pkt.y<=p.y+p.wysokosc))

cout<<endl<<"Punkt "<<pkt.nazwa<<" nalezy do prostokata: "<<p.nazwa;

else

cout<<endl<<"Punkt "<<pkt.nazwa<<" lezy POZA prostokatem: "<<p.nazwa;


}

int main()
{
    Punkt pkt1("A",3,17);
    //pkt1.wczytaj();

    Prostokat p1("Prostokat",0,0,6,4);
    //p1.wczytaj();

    sedzia(pkt1,p1);
    pkt1.sprawdzPunkt();
    p1.wczytaj();

    return 0;
}

 

---

No jak kopiujesz mój kod bez znajomości OOP w C++ to aż dziw, że nie działa.

Naprawdę pomoże poczytanie o klasach w C++ metodach ich składowych..

---

Nie jestem ekspertem w c++, ale na moje friend to obejście prawidłowej architektury systemu obiektowego. Nie da się podpiąć pod prywatne metody? jak to nie da, dajmy friend!

Answer 1

1. W mainie źle wywołujesz funkcję. Skoro sprawdzPunkt jest w klasie prostokąt to wywołujemy go na obiekcie klasy prostokąt i podsyłamy obiekt punkt:

p1.sprawdzPunkt(pkt1);

2. W pliku przyjaciele.h w klasie prostokąt brakuje w funkcji sprawdzPunkt jest za to jakaś funkcja "void sedzia()" zamień ją na:

void sprawdzPunkt(Punkt  &punkt);

3. W pliku przyjaciele.cpp usuń funkcję sprawdz i wklej tą:

void Prostokat::sprawdzPunkt(Punkt  &punkt)
{
       if((punkt.x >= x)&&(punkt.x <= x + szerokosc)&&(punkt.y >= y)&&(punkt.y <= y + wysokosc))
        cout << endl << "Punkt " << punkt.nazwa << " nalezy do prostokata: " << nazwa;
   else
        cout << endl << "Punkt " << punkt.nazwa << " lezy poza prostokatem: " << nazwa;
}

Comments

Dziękuję za odpowiedź z wytłumaczeniem. Teraz wiem, że źle uzywałam wywoływania funkcji.

---

Oj wiem jak to jest jak zaczyna się obiektówkę - potrzeba dużo czasu by wszystko zrozumieć :)
Pozdrawiam