Stała PI z math.h nie działa.

Question

Witam właśnie jestem po przerobieniu 5. odcinka kursu obiektowego C++ pana Mirosława, więc zasiadłem do zrobienia zadania domowego z tego filmu. Gdy chciąłem skompilować program napisany na tym filmie ujrzałem taki komunikat: 

Z racji że jakiś miesiąc temu zacząłem też czytać Opus Magnum C++11 myślę, że naturalna jest rzeczą, że wymieniłem domyślny kompilator Code::Blocks na obsługujący C++11 (TDM-GCC-32) i wiedząc to jedną z pierwszych rzeczy które zrobiłem było ręczne przywrócenie tego pierworodnego, gdy to nie pomogło, ponownie wgrałem ten poprzedni. Wiem, że prawdopodobnie najłatwiej byłoby preinstalować Code::Blocks'a, ale ja jednak wolałbym znac przyczynę problemu na przyszłość. Z góry zaznaczam też, że pozostałe "właściwości" biblioteki math.h działają poprawnie, typu potęgowanie, wartość bezwzględna itp.. 

Przepraszam też za ewentualne niedopowiedzenia, ale gdyby takie były proszę o upomnienie mnie. 

Z góry dziękuje :)

 

Edit:  A tutaj kod:

#include <iostream>
#include <math.h>

using namespace std;

class Punkt
{
   float x,y;
   string nazwa;

public:
   void wyswietl()
   {
      cout<<nazwa<<"("<<x<<","<<y<<")"<<endl;
   }
   Punkt(string n="S", float a=0, float b=0)
   {
      nazwa = n;
      x = a;
      y = b;
   }
};

class Kolo :public Punkt //klasa Kolo dziedziczy publicznie z klasy Punkt
{
   float r;
   string nazwa;

public:
   void wyswietl()
   {
      cout<<"Kolo o nazwie: "<<nazwa<<endl;
      cout<<"Srodek kola: "<<endl;
      Punkt::wyswietl();
      cout<<"Promien: "<<r<<endl;
      cout<<"Pole kola : "<<M_PI*r*r<<endl;
   }

   Kolo(string nk="Kolko", string np="S", float a=0, float b=0, float pr=1)
   :Punkt(np,a,b)
   {
      nazwa = nk;
      r = pr;
   }
};


int main()
{
    Kolo k1;
    k1.wyswietl();

    return 0;
}

 

Comments

Proszę wstawiać kody źródłowe w postaci tekstu (podczas zakładania pytania masz link do krótkiego poradnika, który znajduje się w FAQ). Dlaczego?

1. Ja mogę sobie skopiować Twój kod i skompilować u mnie
2. Jest to z pewnością czytelniejsze niż mały (bo silnik naszego forum coś zmniejsza rozmiary obrazków) zrzut ekranu.

---

Tak wiem, czytałem FAQ, ale nie wystarczy link przekierowywujący do filmu, który kolejno z opisu przekierowywuje do pliku zawierającego kod? Ale gdyby to było takie ważne zawsze mogę dokleić kod.

---

nie wystarczy link przekierowywujący do filmu, który kolejno z opisu przekierowywuje do pliku zawierającego kod?

Tak średnio to wygląda. Nie wpadłbym na to, że aby dostać kod muszę wejść w link do filmu. ;)

---

No to żeby nie było to już doklejam. 

---

No to teraz mogę Ci powiedzieć: "u mnie działa"

---

A u mnie właśnie nie działa i nie wiem co teraz robić, czy reinstalować, czy kombinować dalej, czy może to zignorować... Hmm

---

Jak koledzy wcześniej wspomnieli, M_PI nie jest częścią standardu C++, a więc nie musi być częścią kompilatora. Użyj jednego z rozwiązań, które wcześniej Ci przedstawiono. Stworzenie własnej stałej to chyba najlepsze rozwiązanie.

---

A czy mógłbyś mi powiedzieć jakiego używasz kompilatora?

---

GCC 7.2.1 na Linuksie

---

Ok dziękuje :)

Answer 1

Skoro piszesz kod C++, to warto byłoby używać cmath zamiast math.h.

Spróbuj 

#define _USE_MATH_DEFINES
#include <cmath>

(zachowując kolejność).

 

A jak to nie pomoże, to zawsze możesz ręcznie zdefiniować stałe...

#define M_PI 3.14159265358979323846
// lub
const double M_PI = 3.14159265358979323846;

Poza tym, o ile pamiętam, to C::B używa domyślnie kompilatora GCC, który obsługuje C++11.

Comments

Dziękuje, choć wolałbym bardziej jakiś sposób jak naprawić bibliotekę cmath/math.h, ale jak mawiają "Trza brać co dają" ;)

---

W swojej najnowszej wersji kompilator GCC obsługuje "nawet" C++17

---

Wątpię, żeby C::B używał najnowszego GCC, Eryku.

---

Code::Blocks to tylko IDE, nie musi być sztywno powiązane z kompilatorem w konkretnej jego wersji. Zawsze można zainstalować osobno najnowszą wersję MinGW 

---

Co też autor uczynił - doinstalował inną wersję. Arkadiusz Sikorski mówił przecież o tym, co jest out of the box.

Answer 2

Stała M_PI nie należy do standardu, ale niektóre kompilatory dodają ją jako rozszerzenie. Zdefiniuj ją sam:

constexpr double M_PI {3.14159265358979323846};

btw. używaj cmath zamiast math.h