Proč se mi nevolá funkce? – C / C++ – Fórum – Programujte.com

Zdravím,

postoupila jsem k dalším úkolům, co mám naprogramovat. Uvedu pozadí toho programu, pokud by jste chtěli být v obraze

Vytvořte program, který si od uživatele vyžádá souřadnice a poloměr pro dvě kružnice. Potom v jednom okně vykreslí tyto dvě kružnice a navíc třetí kružnici, jejíž střed bude ležet na spojnici středů těchto dvou kružnic a velikost poloměru bude průměrem z poloměru dvou zadaných kružnic. Použijte stejnou třídu Circle jako v předchozí úloze. Navíc v ní implementujte metody getCentreX(), getCentreY() a getRadius() pro získání hodnot příslušných členských proměnných. Dále mplementujte metodu setAverageCircle(), která přijme jako parametr dvě kružnice a z nich spočítá hodnoty svého středu a poloměru, jak je uvedeno výše. První kružnice bude vždy zelená, druhá modrá a třetí zprůměrovaná kružnice bude červená.

Jsem zatím u vykreslení spojnice metodou drawLine(), která zatím přijímá předem zadané parametry. Tato funkce se ale nevykonává. Proč? 

#include <iostream>
#include <g2.h>
#include <g2_X11.h>

using namespace std;
class Circle
{
  public:
    Circle();
    void setCentre(int ax, int ay);
    void setRadius(int r);
    void setColor(int c);
    void printValues() const;
    void draw();
    void drawLine();
    void readValues();
    void readCentre();
    void readRadius();
    void readColor();
  

  private:
    int x, y;
    int radius;
    int color;
};
Circle::Circle()
{
    x=1;
    y=1;
    radius=1;
    color=1;
}

void Circle::readCentre()
{
      int x, y;
      cout<<"Zadejte souradnice stredu: ";
      cin>>x>>y;
      this->setCentre(x,y);
}
void Circle::readRadius()
{
     int r;
     cout<<"Zadejte polomer: ";
     cin>>r;
     this->setRadius(r);
}

void Circle::readColor()
  {
      int c;
      cout<<"Zadejte barvu: ";
      cin>>c;
      setColor(c);
  }
void Circle::setCentre(int ax, int ay)
{
    x=ax;
    y=ay;
}

void Circle::setRadius(int r)
{
    radius=r;
}
void Circle::setColor(int c)
{
    color=c;
}

void Circle::printValues() const
{
    cout<<"Souradnice: "<<x<<", "<<y<<endl;
    cout<<"Polomer: "<<radius<<endl;
    cout<<"Barva: "<<color<<endl;
}
void Circle::draw()
{
    g2_set_line_width(0, 5);    // Tloustka cary bude 5 pixelu
    g2_pen(0, color);
    g2_circle(0, x, y, radius);
}

void Circle::drawLine()
{
    g2_set_line_width(2, 5);   
    g2_pen(2, 7);
    g2_line(2, 300, 400, 200, 250);
}
void Circle::readValues()
{
  readCentre();
  readRadius();
  readColor();
}

void drawCircles(Circle circ1, Circle circ2)
{
  circ1.draw();
  circ2.draw();
}
int dev = 0;
int main()
{
    Circle circ1, circ2;
   
    circ1.readValues();
    circ2.readValues();

    circ1.printValues();
    circ2.printValues();

    dev = g2_open_X11(600, 600);
    drawLine();
    drawCircles(circ1, circ2);
   
    cin.get();
    cin.get();
    return 0;
}

jestli ono to neni tim, ze je to metoda a ty to volas jako nejakou funkci, ktera ke tride nepatri

#2 KIIV
jenže ona patří..nebo alespoň má patřit.. souřadnice kružnic budou využity jako souřadnice spojnice..

teď teda nevím, jestli nezaložit novou třídu.. ale to se mi zdá jako složitější řešení..

Z logiky věci vyplývá, že drawLine nemůže patřit třídě Circle neb ona musí pracovat s dvojicí souřadnic a když bude členská, tak bude mít jen jedny souřadnice. Udělej ji, stejně jako drawCircle nečlenskou a ty instance Circle jí předávej jako parametry. Tam pak využiješ ty metody getCentreX apod.

ale je receno ze se ma volat nad treti kruznici a ma ocekavat jako parametr dve dalsi kruznice

#4 liborb
Áha, tak to pak jo. Zkusím to nějak zpatlat. Co jsem ale teď zkusila - udělat z drawLine() nečlenskou funkci, tak se stejně ta fuknce nevolá..

Nevolá se nebo se nic nevykreslí? Zkus to odkrokovat v debuggeru.

hu

#7 hlucheucho
nic nevykreslí,... přitom kód na tu čáru jsem úspěšně už používala jinde..

no hlavne ze to vypisuje hlasky jako No such device: 2

aa problem vyresen ... chtelo by to asi predavat i to "zarizeni"

V jedné metodě používáš
 

g2_pen(0, color);

a v další
 

g2_pen(2, 7);

Jen věštím z křišťálové koule, nemám prototyp funkce.

hu

#9 KIIV a #10 hlucheucho
to jsem poopravila už předtím, než jsem napsala příspěvek č.#7

#10 hlucheucho
to g2_pen se může využívat vícekrát..

#12 delicacyy
jenze prvni parametr je "device" ... a ty mas jen jedno zarizeni (okno), melo by se pouzivat aspon to ...

Navíc se stejným parametrem voláš i ostatní funkce g2_.... v metodě. Jestli je to "device", tak to nefunguje proto, že device 2 nemáš.

hu

#14 hlucheucho
coz je prekvapive ta hlaska, co sem tu hodil pred par odpovedi nazpet

Pozdě jsem si všiml, že ti dělám ozvěnu :)

hu

chlapci, vy vůbec nečtete, co jsem napsala...  už jsem to dávno opravila, ...tedy esi toto není dobře..

 já jsem to bez problémů užívala v c..

void drawLine()
{
    g2_set_line_width(dev, 5);    
    g2_pen(dev, 7);
    g2_line(dev, 430, 235, 430, 205);

}

Je proměnná dev inicializovaná správně?

hu

no dev je definovany az pred main.. takze nic pred tim to nevidi... ale tezko rict jestli uz je to spravne.. novej kod tu neni

#18 hlucheucho
v c jsem ji vždy inicializovala nulou a šlo to bez problému..

#19 KIIV
to přeci nevadí, že je definováno před main jako globální proměnná ne? jako zkoušela jsem int dev = 0; švihnout dovnitř mainu i dovnitř drawLine(), ale je to furt stejné..

#include <iostream>
#include <g2.h>
#include <g2_X11.h>

using namespace std;

 class Circle
{
  public:
    Circle();
    void setCentre(int ax, int ay);
    void setRadius(int r);
    void setColor(int c);
    void printValues() const;
    void draw();
    
    void readValues();
    void readCentre();
    void readRadius();
    void readColor();
   

  private:
    int x, y;
    int radius;
    int color;
};

Circle::Circle()
{
    x=1;
    y=1;
    radius=1;
    color=1;
}

void Circle::readCentre()
 {
      int x, y;
      cout<<"Zadejte souradnice stredu: ";
      cin>>x>>y;
      this->setCentre(x,y);
 }

void Circle::readRadius()
{
     int r;
     cout<<"Zadejte polomer: ";
     cin>>r;
     this->setRadius(r);
 }

void Circle::readColor()
  {
      int c;
      cout<<"Zadejte barvu: ";
      cin>>c;
      setColor(c);
  }

void Circle::setCentre(int ax, int ay)
{
    x=ax;
    y=ay;
}

void Circle::setRadius(int r)
{
    radius=r;
}

void Circle::setColor(int c)
{
    color=c;
}

void Circle::printValues() const
{
    cout<<"Souradnice: "<<x<<", "<<y<<endl;
    cout<<"Polomer: "<<radius<<endl;
    cout<<"Barva: "<<color<<endl;
}

void Circle::draw()
{
    g2_set_line_width(dev, 5);    // Tloustka cary bude 5 pixelu
    g2_pen(dev, color);
    g2_circle(dev, x, y, radius);
}

void drawLine()
{
    g2_set_line_width(dev, 5);    
    g2_pen(dev, 7);
    g2_line(dev, 430, 235, 230, 205);

}

void Circle::readValues()
{
  readCentre();
  readRadius();
  readColor();
}

void drawCircles(Circle circ1, Circle circ2)
{
  circ1.draw();
  circ2.draw();
}

int dev = 0;
int main()
{
    Circle circ1, circ2;
    
    circ1.readValues();
    circ2.readValues();

    circ1.printValues();
    circ2.printValues();

    dev = g2_open_X11(600, 600);
    drawLine();
    drawCircles(circ1, circ2);
    
    cin.get();
    cin.get();
    return 0;
}

KIIV myslel dát  int dev = 0;  před implementaci metod třídy.

hu

hoši vy ze mě budete mít srandu, ale já celou dobu kompiluju circ1acirc2.cpp místo circ1acirc2_.cpp ... se vám velmi omlouvám..

#24 hlucheucho
ale to 'dát int dev = 0; před implementací metod třídy' by znamenalo dát to vyloženě dovnitř metod?

jenze se kompiluje odzhora dolu.. dat to za neco znamena ze to neco netusi, jakeho typu a co vubec pak dev je (pokud se pouzije pred deklaraci)

#26 KIIV
aha.. já měla představu, že se postupuje nejdřív od main a pak se ty všechny objekty propojí

#25 delicacyy
 

#include <iostream>
#include <g2.h>
#include <g2_X11.h>

using namespace std;

int dev = 0;

 class Circle
{
  public:
    Circle();
    void setCentre(int ax, int ay);
    void setRadius(int r);
    void setColor(int c);
    void printValues() const;
    void draw();
    
    void readValues();
    void readCentre();
    void readRadius();
    void readColor();
   

  private:
    int x, y;
    int radius;
    int color;
};

hu

Pro hodnoty, které se nemění, se používají makra. Překladač při rozvoji makra dosadí jeho hodnotu ve všech výskytech makra
 

#define DEVICE	0	//definuji makro, jeho hodnota je 0

g2_set_line_width(DEVICE, 5);  //prekladac misto DEVICE dosadi 0
g2_pen(DEVICE, 7);

Pokud později (např. při ladění programu) potřebuješ hodnotu konstanty změnit, změníš ji pouze v definici makra a při dalším překladu ji překladač vymění všude, kde se vyskytuje.

hu

#29 hlucheucho
ale device ciste teoreticky nemusi dostat jen 0 ... o to cislo se stara  g2_open_X11() ..  z praktickeho hlediska by bylo lepsi mit device jako parametr pro draw a podobne... (a nebo pridat rovnou do tridy ale to neni tak idealni)

#29 hlucheucho
díky, toto je velmi srozumitelné pro mě 

#30 KIIV
Buď seš tak dobrej, nebo někde máš ty hlavičkový soubory s prototypy funkcí.  

Vědět, co je makro a jak se používá, se jí bude někdy hodit, ikdyž to asi není tento případ

hu

#32 hlucheucho
samozrejme sem tak dobrej.. + mam i ty hlavickovy soubory + mam to i prelozeny :D

+ dedukce mi napovida ze:  dev = g2_open_X11(600, 600);  snad ma i nejakou teoretickou funkci :D

#33 KIIV
Máš pravdu. Jsem nejen hluchej, ale i slepej  

hu

Tak jsem s tím programem dále pohla, potřebovala bych nyní poradit, jak doladit funkci setAverageCircle().. hlásí mi to chybu 'lvalue required as left operand of assignment' ..můžete mi objasnit, proč se ta hláška objevuje a jak jinak by se to dalo ošetřit?

#include <iostream>
#include <g2.h>
#include <g2_X11.h>

using namespace std;

 class Circle
{
  public:
    Circle();
    void setCentre(int ax, int ay);
    void setRadius(int r);
    void setColor(int c);
    void printValues() const;
    void draw();
    
    void readValues();
    void readCentre();
    void readRadius();
    void readColor();
    
    int getCentreX();
    int getCentreY();
    int getRadius();
    //void setAverageCircle();

  private:
    int x, y;
    int radius;
    int color;
};

Circle::Circle()
{
    x=1;
    y=1;
    radius=1;
    color=1;
}

void Circle::readCentre()
 {
      int x, y;
      cout<<"Zadejte souradnice stredu: ";
      cin>>x>>y;
      this->setCentre(x,y);
 }

void Circle::readRadius()
{
     int r;
     cout<<"Zadejte polomer: ";
     cin>>r;
     this->setRadius(r);
 }

void Circle::readColor()
  {
      int c;
      cout<<"Zadejte barvu: ";
      cin>>c;
      setColor(c);
  }

int Circle::getCentreX()
{
 return x;
}

int Circle::getCentreY()
{
 return y;
}

int Circle::getRadius()
{
 return radius;
}

void Circle::setCentre(int ax, int ay)
{
    x=ax;
    y=ay;
}

void Circle::setRadius(int r)
{
    radius=r;
}

void Circle::setColor(int c)
{
    color=c;
}

void Circle::printValues() const
{
    cout<<"Souradnice: "<<x<<", "<<y<<endl;
    cout<<"Polomer: "<<radius<<endl;
    cout<<"Barva: "<<color<<endl;
}

void printValuesCircles(Circle circ1, Circle circ2)
{
 cout << "Prvni kruznice:" << endl;
 cout << circ1.getCentreX() <<endl;
 cout << circ1.getCentreY() <<endl;
 cout << circ1.getRadius() <<endl;
 cout << "Druha kruznice:" <<endl;
 cout << circ2.getCentreX() <<endl;
 cout << circ2.getCentreY() <<endl;
 cout << circ2.getRadius() <<endl;
 
}

void setAverageCircle(Circle circ1, Circle circ2, Circle circ3)
{
 circ3.getCentreX() = (circ1.getCentreX() + circ2.getCentreX())/2;
 circ3.getCentreY() = (circ1.getCentreY() + circ2.getCentreY())/2;
 circ3.getRadius() = (circ1.getRadius() + circ2.getRadius())/2;
}

void Circle::draw()
{
    int dev = 0;
    g2_set_line_width(dev, 5);    // Tloustka cary bude 5 pixelu
    g2_pen(dev, color);
    g2_circle(dev, x, y, radius);
}

void drawLine(Circle circ1, Circle circ2)
{
    int dev = 0;
    g2_set_line_width(dev, 5);    
    g2_pen(dev, 7);
    g2_line(dev, circ1.getCentreX(), circ1.getCentreY(), circ2.getCentreX(), circ2.getCentreY());

}

void Circle::readValues()
{
  readCentre();
  readRadius();
  readColor();
}

void drawCircles(Circle circ1, Circle circ2, Circle circ3)
{

  circ1.draw();
  circ2.draw();
  circ3.draw();
}


int main()
{
    int dev = 0;
    //cout << "ahoj"<<endl;
    Circle circ1, circ2, circ3;
    
    circ1.readValues();
    circ2.readValues();

    circ1.printValues();
    circ2.printValues();
    
    printValuesCircles(circ1, circ2);

    dev = g2_open_X11(600, 600);
    drawLine(circ1, circ2);
    drawCircles(circ1, circ2, circ3);
    
    cin.get();
    cin.get();
    return 0;
}

void printValuesCircles() funkce je přidaná jenom proto, abych si ověřila, že fungují get_ fukce, takže si jí klidně nemusíte všímat, get_ funkce fungují

circ3.getCentreX()   je metoda třídy, té nemůžeš přiřadit hodnotu

hu

#36 hlucheucho
jenže když zadám místo circ3.getCentre() circ3.x, tak na mne překladač řve, že x je private... nebo se tady dají nějak využít případné ukazatele? (ale to je asi blbost, že..)

Použij metody void setCentre(int ax, int ay); a  void setRadius(int r);, kde jako parametry použiješ výsledek výpočtu:
 

setRadius((circ1.getRadius() + circ2.getRadius())/2);

podobně i pro střed.

hu

co vyuzit setCentre?

#39 KIIV
že by ozvěna?  

hu

#35 delicacyy
Len tak zo zvedavosti, vieš čo je R_value a L_value, aký je medzi nimi rozdiel a hlavne čo znamenala tá chyba?

#41 vitamin
nevím..R_value navíc slyším prva..

S atributy private (soukromé) může manipulovat jen členská metoda třídy, z venku nejsou viditelné. Pokud jsou public (veřejné) je možné s nimi manipulovat i z venku třídy. Pokud by bylo: 

 class Circle
{
  public:
    Circle();
    void setCentre(int ax, int ay);
    void setRadius(int r);
    void setColor(int c);
    void printValues() const;
    void draw();
    
    void readValues();
    void readCentre();
    void readRadius();
    void readColor();
    
    int getCentreX();
    int getCentreY();
    int getRadius();
    //void setAverageCircle();

  //private:
    int x, y;
    int radius;
    int color;
};

pak můžeš udělat i:
 

circ3.radius = (circ1.getRadius() + circ2.getRadius())/2;

hu

#43 hlucheucho
proč se tedy vlastně používá private? Aby do těch proměnných nešlo zasahovat?

#44 delicacyy
kvůli bezpečnosti... pokud nechceš, aby hodnotu šlo změnit odkudkoli i z instance, tak tomu dáš modifikátor "private" a půjde změnit pouze uvnitř třídy

#45 ingiraxo
byls rychlejší

hu

#42 delicacyy
L(left) value je hodnota ktorá môže vystupovať na ľavej aj pravej strane výrazu.

R(right) value je hodnota ktorá môže vystupovať len na pravej strane výrazu.

Tá chyba ti vlastne povedala že si použila R value na ľavej strane výrazu.

R value sú konštanty a pomocné premenné a L value sú hlavne pomenované premenné, referencie a dereferencované pointre.

V tvojom prípade si zavolala metódu ktorá vracala int. Návratová hodnota sa zapísala do pomocnej premennej(r value) a potom si sa pokúšala do tejto premennej priradit inú hodnotu. Čiže pomocná premenná stála na ľavej strane výrazu a to nemôže. 

v OOP existuje pojem "zapouzdreni" - tj. objekt ma presne dane rozhrani jak se s nim da manipulovat (neco jako zasilat zpravy co ma delat) a pak ma vnitrni promenne, do kterych nicemu okolo vesmes nic neni

ty kdyz pak dodrzis to rozhrani, muzes menit samotne streva objektu prakticky jak se ti zlibi a navenek se bude tvarit stale stejne...

Na druhou stranu, pokud budes pristupovat primo k promennym tak nejen ze tam muzes vlozit nesmysly, ale taky nesmis zmenit vnitrek objektu protoze uz se pouziva i jinde ... zmenit treba radius na r kdyz ho pouzivas asi tak 100x okolo jinde je "lahudka"

a k ty LVALUE - nic nebrani udelat metodu:    int & Radius() { return radius; }  ... to se pak da pouzit klidne i takto: 

obj.Radius() += 10;

ale je to zase vicemene trosku dira do zapouzdreni ... leda by radius vracel zase ten samy objekt a bylo nad nim implementovano += a tak   (ikdyz blbe by se poznavalo, k cemu se chce pricitat)

tak to zatím vypadá takto:

#include <iostream>
#include <g2.h>
#include <g2_X11.h>

using namespace std;


 class Circle
{
  public:
    Circle();
    void setCentre(int ax, int ay);
    void setRadius(int r);
    void setColor(int c);
    void printValues() const;
    void draw();
    
    void readValues();
    void readCentre();
    void readRadius();
    void readColor();
    
    int getCentreX();
    int getCentreY();
    int getRadius();
    //void setAverageCircle();

  private:
    int x, y;
    int radius;
    int color;
};

Circle::Circle()
{
    x=1;
    y=1;
    radius=1;
    color=1;
}

void Circle::readCentre()
 {
      int x, y;
      cout<<"Zadejte souradnice stredu: ";
      cin>>x>>y;
      this->setCentre(x,y);
 }

void Circle::readRadius()
{
     int r;
     cout<<"Zadejte polomer: ";
     cin>>r;
     this->setRadius(r);
 }

void Circle::readColor()
  {
      int c;
      cout<<"Zadejte barvu: ";
      cin>>c;
      setColor(c);
  }

int Circle::getCentreX()
{
 return x;
}

int Circle::getCentreY()
{
 return y;
}

int Circle::getRadius()
{
 return radius;
}

void Circle::setCentre(int ax, int ay)
{
    x=ax;
    y=ay;
}

void Circle::setRadius(int r)
{
    radius=r;
}

void Circle::setColor(int c)
{
    color=c;
}

void Circle::printValues() const
{
    cout<<"Souradnice: "<<x<<", "<<y<<endl;
    cout<<"Polomer: "<<radius<<endl;
    cout<<"Barva: "<<color<<endl;
}

void printValuesCircles(Circle circ1, Circle circ2, Circle circ3)
{
 cout << "Prvni kruznice:" << endl;
 cout << circ1.getCentreX() <<endl;
 cout << circ1.getCentreY() <<endl;
 cout << circ1.getRadius() <<endl;
 cout << "Druha kruznice:" <<endl;
 cout << circ2.getCentreX() <<endl;
 cout << circ2.getCentreY() <<endl;
 cout << circ2.getRadius() <<endl;
 cout << "Treti kruznice:" <<endl;
 cout << circ3.getCentreX() <<endl;
 cout << circ3.getCentreY() <<endl;
 cout << circ3.getRadius() <<endl;
 
}

void setAverageCircle(Circle circ1, Circle circ2, Circle circ3)
{
 circ3.setCentre((circ1.getCentreX() + circ2.getCentreX())/2, (circ1.getCentreY() + circ2.getCentreY())/2);
 circ3.setRadius((circ1.getRadius() + circ2.getRadius())/2);
 circ3.setColor(25);
}

void Circle::draw()
{
    int dev = 0;
    g2_set_line_width(dev, 5);    // Tloustka cary bude 5 pixelu
    g2_pen(dev, color);
    g2_circle(dev, x, y, radius);
}

void drawLine(Circle circ1, Circle circ2)
{
    int dev = 0;
    g2_set_line_width(dev, 5);    
    g2_pen(dev, 7);
    g2_line(dev, circ1.getCentreX(), circ1.getCentreY(), circ2.getCentreX(), circ2.getCentreY());

}

void Circle::readValues()
{
  readCentre();
  readRadius();
  readColor();
}

void drawCircles(Circle circ1, Circle circ2, Circle circ3)
{

  circ1.draw();
  circ2.draw();
  circ3.draw();
}


int main()
{
    int dev = 0;
    cout << "ahoj"<<endl;
    Circle circ1, circ2, circ3;
    
    circ1.readValues();
    circ2.readValues();

    circ1.printValues();
    circ2.printValues();
    
    setAverageCircle(circ1, circ2, circ3);
    printValuesCircles(circ1, circ2, circ3);

    dev = g2_open_X11(600, 600);
    drawLine(circ1, circ2);
    drawCircles(circ1, circ2, circ3);
    
    cin.get();
    cin.get();
    return 0;
}

funkce printValuesCircles pro kontrolu, esi se správně načítají hodnoty však pro circ3 vrací jedničky  ..je to vidět i ve vykreslovacím okně, že se tam cosi malého v rohu krčí.. tak mě napadá, esi by nemomohlo místo int get_ funkcí zavést float get_funkce

ted je problem ze upravujes kopii circ3, ktera se udela tim volanim...

#51 KIIV
kterým voláním?

setAverageCircle

totiz vsechno co das jen tak jako parametr, se predava hodnotou (udela se kopie a s tou se pracuje...) u takhle malyho objektu to jeste neni problem ale u neceho vetsiho by bylo lepsi pouzit napriklad:

setAverageCircle( Circle const & c1, Circle const & c2, Circle & c3 ) ...  (akorat by muselo byt u vsech tech getteru jeste klicove slovo  const  (jako ze nemeni vnitrek objektu) jinak to s toudle deklaraci nepobezi

Nech zakáže kopírovací konštruktor a uvidí(prekladač ukáže :) ) kde všade sa jej kopírujú objekty.

class Circle{
    public:
        //.....
        Circle(const Circle&) = delete;
};

Ono pokavaď se upravuje jen ta jedna instance i pro přehlednost by bylo lepší upravit metodu setAverageCircle a volat ji nad tou instancí circ3. A pak v ní this.set....

circ3.setAverageCircle(Circle const & circ1, Circle const & circ2);

Zdravím, zkoušela jsem to tedy překopat dneska na:

#include <iostream>
#include <g2.h>
#include <g2_X11.h>

using namespace std;

 class Circle
{
  public:
    Circle();
    //Circle(const Circle&) = delete;
    void setCentre(int ax, int ay);
    void setRadius(int r);
    void setColor(int c);
    void printValues() const;
    void draw();
    
    void readValues();
    void readCentre();
    void readRadius();
    void readColor();
    
    int getCentreX();
    int getCentreY();
    int getRadius();
    void setAverageCircle(Circle circ1, Circle circ2, Circle circ3);

  private:
    int x, y;
    int radius;
    int color;
};

Circle::Circle()
{
    x=1;
    y=1;
    radius=1;
    color=1;
}

void Circle::readCentre()
 {
      int x, y;
      cout<<"Zadejte souradnice stredu: ";
      cin>>x>>y;
      this->setCentre(x,y);
 }

void Circle::readRadius()
{
     int r;
     cout<<"Zadejte polomer: ";
     cin>>r;
     this->setRadius(r);
 }

void Circle::readColor()
  {
      int c;
      cout<<"Zadejte barvu: ";
      cin>>c;
      setColor(c);
  }

int Circle::getCentreX()
{
 return x;
}

int Circle::getCentreY()
{
 return y;
}

int Circle::getRadius()
{
 return radius;
}

void Circle::setCentre(int ax, int ay)
{
    x=ax;
    y=ay;
}

void Circle::setRadius(int r)
{
    radius=r;
}

void Circle::setColor(int c)
{
    color=c;
}

void Circle::printValues() const
{
    cout<<"Souradnice: "<<x<<", "<<y<<endl;
    cout<<"Polomer: "<<radius<<endl;
    cout<<"Barva: "<<color<<endl;
}

void printValuesCircles(Circle circ1, Circle circ2, Circle circ3)
{
 cout << "Prvni kruznice:" << endl;
 cout << circ1.getCentreX() <<endl;
 cout << circ1.getCentreY() <<endl;
 cout << circ1.getRadius() <<endl;
 cout << "Druha kruznice:" <<endl;
 cout << circ2.getCentreX() <<endl;
 cout << circ2.getCentreY() <<endl;
 cout << circ2.getRadius() <<endl;
 cout << "Treti kruznice:" <<endl;
 cout << circ3.getCentreX() <<endl;
 cout << circ3.getCentreY() <<endl;
 cout << circ3.getRadius() <<endl;
 
}

void Circle::setAverageCircle(Circle circ1, Circle circ2, Circle circ3)
{
 circ3.setCentre((circ1.getCentreX() + circ2.getCentreX())/2, (circ1.getCentreY() + circ2.getCentreY())/2);
 circ3.setRadius((circ1.getRadius() + circ2.getRadius())/2);
 circ3.setColor(25);
}

void Circle::draw()
{
    int dev = 0;
    g2_set_line_width(dev, 5);    // Tloustka cary bude 5 pixelu
    g2_pen(dev, color);
    g2_circle(dev, x, y, radius);
}

void drawLine(Circle circ1, Circle circ2)
{
    int dev = 0;
    g2_set_line_width(dev, 5);    
    g2_pen(dev, 7);
    g2_line(dev, circ1.getCentreX(), circ1.getCentreY(), circ2.getCentreX(), circ2.getCentreY());

}

void Circle::readValues()
{
  readCentre();
  readRadius();
  readColor();
}

void drawCircles(Circle circ1, Circle circ2, Circle circ3)
{

  circ1.draw();
  circ2.draw();
  circ3.draw();
}


int main()
{
    int dev = 0;
    cout << "ahoj"<<endl;
    Circle circ1, circ2, circ3;
    
    circ1.readValues();
    circ2.readValues();

    circ1.printValues();
    circ2.printValues();
    
    circ3.setAverageCircle(Circle const & circ1, Circle const & circ2);

    //setAverageCircle(circ1, circ2, circ3);
    printValuesCircles(circ1, circ2, circ3);

    dev = g2_open_X11(600, 600);
    drawLine(circ1, circ2);
    drawCircles(circ1, circ2, circ3);
    
    cin.get();
    cin.get();
    return 0;
}

jenže nevím si rady s const & protože nevím, co přesně to má znamenat a jak to tedy ošetřit.. poradíte prosím?

void Circle::setAverageCircle(Circle const & circ1, Circle const & circ2)
{
	setCentre((circ1.getCentreX() + circ2.getCentreX())/2, (circ1.getCentreY() + circ2.getCentreY())/2);
	setRadius((circ1.getRadius() + circ2.getRadius())/2);
	setColor(25);
}

...

circ3.setAverageCircle(circ1, circ2);

#58 liborb
co ale napsat dovnitř do třídy?  void setAverageCircle(Circle circ1, Circle circ2); ani jiné obdoby, co jsem zkoušela i podle návrhu kompilátoru nejedou..

error: prototype for ‘void Circle::setAverageCircle(const Circle&, const Circle&)’ does not match any in class ‘Circle’
error: candidate is: void Circle::setAverageCircle(Circle, Circle)

#57 delicacyy
a pak uz jen  getCentreX, getCentreY a getRadius oznacit jako const (jako ze nemeni objekt) - stejne jako mas u printValues

#60 KIIV
jenom pro ujištění: takže to 'const' se objeví ve třídě za getCentreX, getCentreY a getRadius a též (teď nevím, jak se tomu říká, ale objeví se tam, kde definujeme funkce v těchto formách ..void Circle::getCentreX const {..}).. do setAverageCircles() už ne?

 'const' sa nikde samo od seba neobjaví, musíš ho tam napísť. 

typedef int T;
class C{

        const T * const funkcia(const T x, T const y)const;
//        1         2             3            4       5

};

Tu máš deklaráciu metódy s 5 const.

1.  const znamená že pointer ukazuje na premennú ktorá sa nemôže meniť

2. const znamená že vrátený pointer sa nemôže meniť.

3. a 4. const znamenajú že parameter x a y sa nemôže meniť.

5. const znamená že metóda nemôže meniť žiadne atribúty triedy C (this môže byť const premenná). 

#61 delicacyy
pomoci: int getCentreX() const; oznamujes prekladaci, ze se nic uvnitr objektu nebude menit touto metodou

metodu pak muzes pouzit i u konstantni instance tridy - coz se neda rici o metodach, ktere pomoci const oznaceny nejsou - muzou menit obsah a tudiz nemuzou byt volany nad konstantnim objektem

už mi vše jede, jak má :)

jenom bych se chtěla zeptat na takovou drobnost, při kompilaci se mi totiž vypíše warning, že proměnná dev je sice nastavena, ale není užívána, avšak pokud ji odstraním, kompilátor ohlásí error že dev není deklarována... není mi jasné, proč mi to ten warning hlásí..

#64 delicacyy
int dev = 0;   se melo dat jen jednou pred samotnou tridu  (napriklad po using namespace std;)

a vynechat vsude jinde