Problem z Pracą Domową z 4 Odcinku Programowania Obiektowego

pytanie zadane 30 stycznia 2017 w C i C++ przez Piotr Lis Obywatel (1,310 p.)

Witam serdecznie,

Mam problem z pracą domową z odcinka czwartego. Nie wiem co źle zrobiłem, ale w kompilatorze wyświetlają się 2 błędy w pliku main.cpp.

Treść błędów:

|13|error: expected primary-expression before 'pkt1'

|13|error: expected primary-expression before 'p1'

main.cpp

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

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

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

    p1.sedzia(Punkt pkt1, Prostokat p1); //TU SĄ OBA BŁĘDY


    return 0;
}

Plik .h

#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 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();
};

Plik z ciałami klas (.cpp 2)

#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;
   cout<<endl<<"Nazwa obiektu: "<<nazwa;*/
}
void Prostokat::sedzia(Punkt pkt, Prostokat p)
{
if((pkt.x>=p.xx)&&(pkt.x<=p.xx+p.s)&&(pkt.y>=p.yy)&&(pkt.y<=p.yy+p.w))

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

else

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

Za wszelką pomoc z góry dziękuje.

2 odpowiedzi

odpowiedź 30 stycznia 2017 przez niezalogowany
edycja 30 stycznia 2017

Najlepsza

W pliku .h w klasie prostokąt jest metoda: void sedzia(), a w pliku .cpp jest sedzia(Punkt pkt, Prostokat p). Powinny być te same atrybuty. NP: "sedzia(Punkt pkt)"
```
p1.sedzia(Punkt pkt1, Prostokat p1);
```
Po co tak w funkcji main? Przy wystarczyłoby p1.sedzia(pkt1, p1); Albo jeszcze lepiej p1.sedzia(pkt1) bo po co po co nie potrzebnie wysyłać tej funkcji prostokąt?
Trzeba przebudować w takim razie całą funkcję pod to:

void Prostokat::sedzia(Punkt pkt) 
{ // zamiast p.x uzywam x, ponieważ ta funkcja odnosi sie do prostokata
    if((pkt.x>=x)&&(pkt.x<=x+szerokosc)&&(pkt.y>=y)&&(pkt.y<=y+wysokosc))
// zamiast s,w  - szerokosc i wysokosc tak jak w deklaracji klasy
// zmienne s,w przyjmowane przez konstruktor sa dostepne tylko w nim
    cout<<endl<<"Punkt "<<pkt.nazwa<<" nalezy do prostokata: "<<nazwa;

    else

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