Kurs obiektowy c++ odc 4 zadanie domowe

Question

Witam mam problem. Nie wiem jak zrobić zadanie domowe jeśli rozdziele je na pliki punkt.h punkt.cpp prostokat.h prostokat.cpp nwm wgl jak wtedy zrobic funkcjie sedzia w pliku prostokat zeby widzial klase Punkt z innego pliku

main

#include <iostream>
#include "punkt.h"
#include "prostokat.h"


using namespace std;

int main()
{
    Punkt pkt;
    //pkt.wczytaj();
    Prostokat p;
  //  p.wczytaj();
    return 0;
}

punkt.h

#include <iostream>
using namespace std;
class Punkt
{
    public:
    string name;
    float x,y;
public:
    Punkt(string="A",float=0,float=0);
    void wczytaj();
};

punkt.cpp

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

using namespace std;

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

void Punkt::wczytaj()
{
    cout<<"Podaj nazwe punktu: ";cin>>name;
    cout<<"Podaj x: ";cin>>x;
    cout<<"Podaj y: ";cin>>y;

}

prostokat.h

#include <iostream>
using namespace std;
class Prostokat
{
string name;
float x,y,szerokosc,wysokosc;
public:
Prostokat(string="A",float=0,float=0,float=1,float=1);
void wczytaj();
};

prostokat.cpp

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

Prostokat::Prostokat(string n,float xx,float yy,float s,float w)
{
    name=n;
    x=xx;
    y=yy;
    szerokosc=s;
    wysokosc=w;
}
void Prostokat::wczytaj()
{
    cout<<"Podaj nazwe prostokata: ";cin>>name;
    cout<<"Podaj x: ";cin>>x;
    cout<<"Podaj y: ";cin>>y;
    cout<<"Podaj szerokosc: ";cin>>szerokosc;
    cout<<"Podaj wysokosc: ";cin>>wysokosc;
}

 

Answer 1

Po prostu #include "punkt.h" w prostokat.h. Ale jeśli funkcja ta funkcja będzie tylko zadeklarowana w prostokat.h a zdefiniowana w prostokat.cpp i taka funkcja będzie przyjmować referencje lub wskaźnik do Punkt, to wystarczy, że tylko zadeklarujesz sobie Punkt (`class Punkt;`) w prostokat.h, a #include "punkt.h" dasz w prostokat.cpp.

Comments

Kiedy probuje dodać  #include "punkt.h" w prostokat.h to wyskakuje mi błąd w pliku punkt.h

||=== Build: Debug in hitbox (compiler: GNU GCC Compiler) ===|
C:\Users\Maks\Desktop\CodeBlock\hitbox\punkt.h|4|error: redefinition of 'class Punkt'|
C:\Users\Maks\Desktop\CodeBlock\hitbox\punkt.h|4|error: redefinition of 'class Punkt'|
C:\Users\Maks\Desktop\CodeBlock\hitbox\punkt.h|4|error: previous definition of 'class Punkt'|
||=== Build failed: 3 error(s), 0 warning(s) (0 minute(s), 0 second(s)) ===|
C:\Users\Maks\Desktop\CodeBlock\hitbox\punkt.h|4|error: previous definition of 'class Punkt'|
||=== Build failed: 4 error(s), 0 warning(s) (0 minute(s), 0 second(s)) ===|

---

Poczytaj nt. include guards

tldr: powinno starczyć na początku każdego pliku *.h dodać

#pragma once

---

dodalem #pragma once w pierwsej linijce w prostokat.h i punkt.h ale dalej twierdzi ze "Prostokat" nie jest zadeklarowany w punkt.h i odwrotna sytuacjia w prostokat.h brak "Punkt"

przez to nie moge zaprzyjaznic funkcji sedzia

---

No jeśli te dwa pliki nawzajem sie includują, to nic dziwnego. Pliki nie mogą się nawzajem includować, bo w grafie zależności powstaje cykl. W takim wypadku, możesz zrobić tak, żebyś mógł

Ale jeśli funkcja ta funkcja będzie tylko zadeklarowana w prostokat.h a zdefiniowana w prostokat.cpp i taka funkcja będzie przyjmować referencje lub wskaźnik do Punkt, to wystarczy, że tylko zadeklarujesz sobie Punkt (`class Punkt;`) w prostokat.h, a #include "punkt.h" dasz w prostokat.cpp.

jak opisalem wyżej 

---

Tego właśnie nie rozumiem jak mam to zrobić

---

Najlepiej jakbyś pokazał swój aktualny cały kod z podzialem na pliki

---

main

#include <iostream>
#include "punkt.h"
#include "prostokat.h"


using namespace std;

int main()
{
    Punkt pkt1;
    //pkt.wczytaj();
    Prostokat p1;
  //  p.wczytaj();


    return 0;
}

punkt.h

#include <iostream>

using namespace std;


class Punkt
{

    string name;
    float x,y;
public:
    Punkt(string="A",float=0,float=0);
    void wczytaj();
};

punkt.cpp

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

using namespace std;

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

void Punkt::wczytaj()
{
    cout<<"Podaj nazwe punktu: ";cin>>name;
    cout<<"Podaj x: ";cin>>x;
    cout<<"Podaj y: ";cin>>y;

}

prostokat.h

#include <iostream>

using namespace std;

class Prostokat
{

string name;
float x,y,szerokosc,wysokosc;
public:
Prostokat(string="A",float=0,float=0,float=1,float=1);
void wczytaj();
void sedzia(Punkt pkt, Prostokat p);
};

prostokat.cpp

#include <iostream>
#include "prostokat.h"
#include "punkt.h"
using namespace std;
Prostokat::Prostokat(string n,float xx,float yy,float s,float w)
{
    name=n;
    x=xx;
    y=yy;
    szerokosc=s;
    wysokosc=w;
}
void Prostokat::wczytaj()
{
    cout<<"Podaj nazwe prostokata: ";cin>>name;
    cout<<"Podaj x: ";cin>>x;
    cout<<"Podaj y: ";cin>>y;
    cout<<"Podaj szerokosc: ";cin>>szerokosc;
    cout<<"Podaj wysokosc: ";cin>>wysokosc;
}
void Prostokat::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<<"tafiles"<<endl;
    else
        cout<<"Nie trafiles"<<endl;
}

 

---

I chcesz, Prostokat::sedzia byla friendem klasy Punkt?

Do prostokat.h dodaj nad definicją klasy Prostokat deklaracje klasy Punkt i wyrzuć #include "punkt.h". Tj.:

class Punkt;

Natomiast sygnature Prostokat sedzia zamien na

void sedzia(const Punkt& pkt, const Prostokat& p)

W taki sposób przyjmujesz obiekt pkt (oraz p ale to tutaj mniej istotne) przez reference i nie potrzebujesz znać definicji Punkt w momencie deklaracji funkcji. Dodatkowo zapobiegasz kopiowaniu obiektów bez wyraźnego powodu - kod powinien wyglądać w ten sposób nawet bez obecnego problemu.
(swoją drogą po co ten prametr p skoro sedzia juz jest metoda klasy Prostokat?)

Natomiast #include "punkt.h" umieść w prostokat.cpp bo tam już jest ci potrzebna definicja Punkt

---

czyli zamiast pisac p.x moge pisac poprostu x i wtedy moge usunac const Prostokat&p? 

void sedzia(const Punkt& pkt)

 

---

Tak, wtedy sedzia bedzie sprawdzal czy punkt lezy w obiekcie prostokata na ktorym zostal wywolany sedzia. np. x.sedzia(y) - sedzia sprawdza czy y leży wewnatrz x

---

Ale dalej tweirdzi mi ze rzeczy z class Punkt sa prywatne

dodalem do tamtej klasy taka linijke:

    friend void sedzia(const Punkt &pkt);

 

---

A chciales dodać Prostokat::sedzia. Globalna sedzia nawet nie istnieje

---

friend void Prostokat:: sedzia(const Punkt &pkt);

jezeli w takim sensie to wtedy wyskakuje mi bład ze Prostokat nie zostal zadeklarowany 

---

Daj #include "prostokat.h" w punkt.h, myslalem ze to juz masz

---

i jak mam teraz wywolac tego sedzia w mainie?

---

Stary, zrób coś sam, serio XD

---

Dzieki za wytlumaczenie jak dla idioty:P Pytalem sie o tego sedziego bo jak glupi pisalem w mainie

 Prostokat.sedzia(pkt1);

I nie rozumialem o co chodzi dopiero gdy doszlo do mnie co zrobilem to juz teraz to rozumiem. A bylem juz sfrustrowany, że ciagle nie rozumialem jakies rzeczy.

---

Trzeba było pokazać swój aktualny kod :P Po prostu nie chciałem za ciebie pisać wszystkiego