Błąd w metodach lub wskaźnikach

Question

Wywala błąd w 29 i 31 linijce: undefined reference to 'Dimensions::calculate();'

#include <iostream>
using namespace std;
class Dimensions {
 protected:
 float PI = '3,14';
 float radius;
 public:
 void setRadius(int p_r){
     radius=p_r;
 }
 void calculate();
};
class CircleArea: public Dimensions {
 public:
 virtual void calculate(){
     cout<< "Pole wynosi: "<< 2*PI*radius*radius << endl;
 }
};
class CircumferenceCircle: public Dimensions {
 public:
 virtual void calculate(){
     cout<< "Obwod wynosi: "<< 2*PI*radius << endl;
 }
};
int main_test() {
    Dimensions *circle_one = new CircleArea;
    Dimensions *circle_two = new CircumferenceCircle;
    circle_one->setRadius(4);
    circle_one->calculate();// tutaj
    circle_two->setRadius(4);
    circle_two->calculate();// i tutaj

 return 0;
}

 

Comments

A to:

 float PI = '3,14';

się w ogóle kompiluje?

Metody Dimensions::calculate() też nie masz - to co jest to tylko nagłówek.

Jeśli ona ma być virtual to musi taka być od klasy bazowej. Jeśli klasa bazowa może być abstrakcyjna to po nagłówku = 0;

 

Answer 1

Błąd to jest w zrozumieniu. Już w piątej linijce masz źle, bo Twoją intencją jest, by PI = 3.14, a zamiast tego masz jakieś tam konwersje, ponieważ poprawnie powinno być:
 

float PI = 3.14;

a raczej const float, bo jak mniemam PI jest liczbą stałą. 

Druga sprawa - źle metody. Virtual leci do klasy rodzica, nie dziecka, stąd można zrobić tak:
 

virtual void calculate() {};

a następnie poprawić obie metody w pozostałych klasach. 

Ponadto, pierwszy błąd (z float) pociąga za sobą inne rzeczy, na przykład źle będzie wyliczane pole oraz obwód. W dodatku warto zauważyć, że od strony matematycznej wzór na pole powinien być inny. W tej formie dostajesz dwukrotność pola. 

Ogółem polecam się zapoznać z materiałem pod tym linkiem. Może i nie jest najprostsze, ale to jak cały C++.