Dlaczego nie działa? Problem switch case funkcje

Question

#include<iostream>
using namespace std;

void WybierzMenu(double &a, double &b, char wybor);
double ObliczKwadrat(double a);
double ObliczProstokat(double a, double b);
double ObliczKolo(double a);

int main()
{
    char wybor;
    double a=0, b=0;
    cout << "Wybierz figure ktorej chcesz policzyc obwod i pole" << endl;
    cout << "K - kwadrat, P - prostokat, O - okrag. "; cin >> wybor;
    WybierzMenu(a, b, wybor);

    return 0;
}

void WybierzMenu(double &a, double &b, char wybor)
{
    switch (wybor)
    {
    case 'K' : ObliczKwadrat(a);
    case 'P' : ObliczProstokat(a,b);
    case 'O' : ObliczKolo(a);
    }
}
double ObliczKwadrat(double &a)
{
    cout << "Podaj bok kwadratu: "; cin >> a;
    cout << "Obwod kwadratu wynosi = " << a*4;
    cout << "Pole kwadratu wynosi = " << a*a;
}

double ObliczProstokat(double &a, double &b)
{
    cout << "Podaj 1 bok prostokata: "; cin >> a;
    cout << "Podaj 2 bok prostokata: "; cin >> b;
    cout << "Obwod prostokata wynosi = " << a*2+b*2;
    cout << "Pole prostokata wynosi = " << a*b;
}
double ObliczKolo(double &a)
{
    cout << "Podaj promien kola: "; cin >> a;
    cout << "Obwod kola wynosi = " << 2*a*3.14;
    cout << "Pole prostokata wynosi = " << 3.14*a*a;
}

 

Comments

Zrobiłam tak jak mówiliście ale i tak się nie kompiluje

#include<iostream>
using namespace std;

class Obliczenia
{
public:

void WybierzMenu(void &a, void &b, char wybor)
{
    switch (wybor)
    {
    case 'K' : ObliczKwadrat(a); break;
    case 'P' : ObliczProstokat(a,b); break;
    case 'O' : ObliczKolo(a);
    }
}
double ObliczKwadrat(void &a)
{
    cout << "Podaj bok kwadratu: "; cin >> a;
    cout << "Obwod kwadratu wynosi = " << a*4<<endl;
    cout << "Pole kwadratu wynosi = " << a*a<<endl;
}

double ObliczProstokat(void &a, void &b)
{
    cout << "Podaj 1 bok prostokata: "; cin >> a;
    cout << "Podaj 2 bok prostokata: "; cin >> b;
    cout << "Obwod prostokata wynosi = " << a*2+b*2<<endl;
    cout << "Pole prostokata wynosi = " << a*b<<endl;
}
double ObliczKolo(void &a)
{
    cout << "Podaj promien kola: "; cin >> a;
    cout << "Obwod kola wynosi = " << 2*a*3.14<<endl;
    cout << "Pole prostokata wynosi = " << 3.14*a*a<<endl;
}
};

int main()
{
    char wybor;
    double a=0, b=0;
    cout << "Wybierz figure ktorej chcesz policzyc obwod i pole" << endl;
    cout << "K - kwadrat, P - prostokat, O - okrag. "; cin >> wybor;
    WybierzMenu(a, b, wybor);

    return 0;
}

Pisze tak:

error: cannot declare reference to 'void'

Jak jest double to pisze

że "WybierzMenu" was not declared in this scope

---

Działa!

#include<iostream>
using namespace std;

void ObliczKwadrat(double &a);
void ObliczProstokat(double &a, double &b);
void ObliczKolo(double &a);

void WybierzMenu(double &a, double &b, char wybor)
{
    switch (wybor)
    {
    case 'K' : ObliczKwadrat(a); break;
    case 'P' : ObliczProstokat(a,b); break;
    case 'O' : ObliczKolo(a);
    }
}
void ObliczKwadrat(double &a)
{
    cout << "Podaj bok kwadratu: "; cin >> a;
    cout << "Obwod kwadratu wynosi = " << a*4<<endl;
    cout << "Pole kwadratu wynosi = " << a*a<<endl;
}

void ObliczProstokat(double &a, double &b)
{
    cout << "Podaj 1 bok prostokata: "; cin >> a;
    cout << "Podaj 2 bok prostokata: "; cin >> b;
    cout << "Obwod prostokata wynosi = " << a*2+b*2<<endl;
    cout << "Pole prostokata wynosi = " << a*b<<endl;
}
void ObliczKolo(double &a)
{
    cout << "Podaj promien kola: "; cin >> a;
    cout << "Obwod kola wynosi = " << 2*a*3.14<<endl;
    cout << "Pole prostokata wynosi = " << 3.14*a*a<<endl;
}

int main()
{
    char wybor;
    double a=0, b=0;
    cout << "Wybierz figure ktorej chcesz policzyc obwod i pole" << endl;
    cout << "K - kwadrat, P - prostokat, O - okrag. "; cin >> wybor;
    WybierzMenu(a, b, wybor);

    return 0;
}

Dziękuję serdecznie za pomoc!!! 

---

Cieszę się, że działa, to z klasa też by zadziałało, ale nie utworzyłaś obiektu a funkcje w klasie powinny być napisane tak: void ObliczKolo(double &a). Typ funkcji void, parametry typu double. Zobacz swój kod z klasami i mój, który podałem niżej, może kiedyś Ci się przydać :D

Answer 1

Musisz przerwać wykonywanie kodu z wewnątrz switcha instrukcją break.

void WybierzMenu(double &a, double &b, char wybor)
{
    switch (wybor)
    {
    case 'K' : 
        ObliczKwadrat(a); 
        break;
    case 'P' : 
        ObliczProstokat(a,b); 
        break;
    case 'O' : 
        ObliczKolo(a);
        // ostatnia opcja nie potrzebuje breaka.
    }
}

Switch działa tak, że wyszukuje twój wybór a następnie przetwarza resztę kodu linia po lini.

Comments

Niestety nie dziala. Pisze tak:

---

Nic nie widać na tym obrazku.
Wklej tekst.

Answer 2

Nie jestem jakimś ekspertem, ale wydaje mi się, że lepiej zrobić do tego klasę.

#include<iostream>
using namespace std;

class Obliczenia
{
public:

void WybierzMenu(double &a, double &b, char wybor)
{
    switch (wybor)
    {
    case 'K' : ObliczKwadrat(a); break;
    case 'P' : ObliczProstokat(a,b); break;
    case 'O' : ObliczKolo(a);
    }
}
double ObliczKwadrat(double &a)
{
    cout << "Podaj bok kwadratu: "; cin >> a;
    cout << "Obwod kwadratu wynosi = " << a*4<<endl;
    cout << "Pole kwadratu wynosi = " << a*a<<endl;
}

double ObliczProstokat(double &a, double &b)
{
    cout << "Podaj 1 bok prostokata: "; cin >> a;
    cout << "Podaj 2 bok prostokata: "; cin >> b;
    cout << "Obwod prostokata wynosi = " << a*2+b*2<<endl;
    cout << "Pole prostokata wynosi = " << a*b<<endl;
}
double ObliczKolo(double &a)
{
    cout << "Podaj promien kola: "; cin >> a;
    cout << "Obwod kola wynosi = " << 2*a*3.14<<endl;
    cout << "Pole prostokata wynosi = " << 3.14*a*a<<endl;
}

};
int main()
{
    Obliczenia o;
    char wybor;
    double a=0, b=0;
    cout << "Wybierz figure ktorej chcesz policzyc obwod i pole" << endl;
    cout << "K - kwadrat, P - prostokat, O - okrag. "; cin >> wybor;
    o.WybierzMenu(a, b, wybor);

    return 0;
}

Po skompilowaniu działa ;)

//EDIT

W codeblocks przechodzi, ale w visual studio wywala błędy, bo nie zwracasz wartości w funkcji double, zamień ją na void i po sprawie.

Answer 3

Wydaje mi się, że funkcje obliczające pole powierzchni powinny być typu "void", a nie "double", ponieważ nie zwracają wartości, a jedynie wypisują dane.