Metoda zwracająca enum w formie tekstowej

Question

Witam, mam problem z "enum". Mianowicie muszę napisać klasę, która reprezentuje numer telefonu i spełnić kilka wymagań: 
- nr telefonu ma być opisywany przez enumerację typu numeru (komórkowy, domowy, służbowy), numer kierunkowy kraju (dwie cyfry) oraz numer właściwy (od jednej do dziesięciu cyfr),
-klasa ma być wyposażona w konstruktor parametryczny umożliwiający ustawienie wszystkich parametrów numeru,
-klasa ma posiadać metodę zwracającą typ numeru telefonu w formie tekstowej,
-klasa ma posiadać metodę zwracającą liczbowo numer kierunkowy kraju,
-klasa ma posiadać metodę zwracającą liczbowo numer telefonu.

Problem polega na wymaganiu zaznaczonym czerwonym kolorem. Poniżej wstawiam swój kod, wydaje mi się, że wszystko jest okej oprócz zwrócenia enuma, czy ma ktoś jakiś pomysł ? 
Dodam, że dopiero poznaję obiektówkę. 

#include <iostream>
#include <stdio.h>
#include <conio.h>

enum typ_numeru
    {
        domowy = 1,
        komorkowy = 2,
        sluzbowy = 3
    };
class NumerTelefonu
{
private:
    enum typ_numeru jaki;
    int nr_kierunkowy,nr_numer;
public:
    NumerTelefonu();
    NumerTelefonu(enum typ_numeru jaki);
    NumerTelefonu(enum typ_numeru jaki, int nr_kierunkowy);
    NumerTelefonu( int nr_kierunkowy, int nr_numer );
    NumerTelefonu(enum typ_numeru jaki , int nr_kierunkowy, int nr_numer );
    typ_numeru ZwrocTyp() const;
    int ZwrocKierunkowy() const;
    int ZwrocNumer() const;
    void Wyswietl() const;
};
NumerTelefonu::NumerTelefonu()
{
jaki = komorkowy;
nr_kierunkowy=32;
nr_numer=123456789;
}
NumerTelefonu::NumerTelefonu(enum typ_numeru jaki )
    {
        this->jaki = jaki;
        printf("konstruktor\n");
    };
NumerTelefonu::NumerTelefonu(enum typ_numeru jaki  , int nr_kierunkowy  )
    {
        this->jaki = jaki;
        this->nr_kierunkowy = nr_kierunkowy;
        printf("konstruktor\n");
    };
    NumerTelefonu::NumerTelefonu( int nr_kierunkowy , int nr_numer )
    {

        this->nr_kierunkowy = nr_kierunkowy;
        this->nr_numer = nr_numer;
        printf("konstruktor\n");
    };
NumerTelefonu::NumerTelefonu(enum typ_numeru jaki  , int nr_kierunkowy , int nr_numer )
    {
        this->jaki = jaki;
        this->nr_kierunkowy = nr_kierunkowy;
        this->nr_numer = nr_numer;
        printf("konstruktor\n");
    };
 typ_numeru NumerTelefonu::ZwrocTyp() const
    {
            return jaki;
    };
int NumerTelefonu::ZwrocKierunkowy() const
{
    return nr_kierunkowy;
}
int NumerTelefonu::ZwrocNumer() const
{
    return nr_numer;
}
void NumerTelefonu::Wyswietl() const
{
    std::cout<<jaki<<"\n"<<nr_kierunkowy<<"\n"<<nr_numer;
}

int main()
{

NumerTelefonu pierwszy(sluzbowy,82,987654321);
pierwszy.Wyswietl();

    _getch();
    return 0;
}

 

Comments

    NumerTelefonu();
    NumerTelefonu(enum typ_numeru jaki);
    NumerTelefonu(enum typ_numeru jaki, int nr_kierunkowy);

Te konstruktory są bez sensu.

---

Przy podaniu np dwóch parametrów, a nie trzech był błąd więc pomyślałem, żeby tak to ominąć, czyli to nie jest poprawne ?

---

To nie kwestia poprawności, tylko sensowności. Istotą klasy NumerTelefonu jest przechowywanie numeru telefonu. Więc jaki ma sens tworzenie obiektu tej klasy, który nie przechowuje żadnego numeru lub tylko kierunkowy? Gdyby twoja klasa miała mutatory, to co innego, ale ma tylko akcesory, zatem sensu tych konstruktorów nie widzę (no może jeszcze domyślny bym zostawił)

---

Słuszna uwaga, dziękuję 

Answer 1

const char* ZwrocTyp() const
{
	switch(jaki)
	{
	case domowy: return "domowy"; 
	case komorkowy: return "komorkowy"; 
	case sluzbowy: return "sluzbowy"; 	
	};
}

 

Comments

Nadal wyświetla numer z enum, coś trzeba zmienić w metodzie Wyswietl() ?

---

Usuń swoją implementację ZwrocTyp i zostaw moją.

 

Skup się bardziej...

---

Tak zrobiłem...

#include <iostream>
#include <stdio.h>
#include <conio.h>

enum typ_numeru
    {
        domowy = 1,
        komorkowy = 2,
        sluzbowy = 3
    };
class NumerTelefonu
{
private:
    enum typ_numeru jaki;
    int nr_kierunkowy,nr_numer;
public:
    NumerTelefonu();
    NumerTelefonu(enum typ_numeru jaki);
    NumerTelefonu(enum typ_numeru jaki, int nr_kierunkowy);
    NumerTelefonu( int nr_kierunkowy, int nr_numer );
    NumerTelefonu(enum typ_numeru jaki , int nr_kierunkowy, int nr_numer );
    const char* ZwrocTyp() const
    {
        switch(jaki)
        {
            case domowy: return "domowy";
            case komorkowy: return "komorkowy";
            case sluzbowy: return "sluzbowy";
        };
    }
    int ZwrocKierunkowy() const;
    int ZwrocNumer() const;
    void Wyswietl() const;
};
NumerTelefonu::NumerTelefonu()
{
jaki = komorkowy;
nr_kierunkowy=32;
nr_numer=123456789;
}
NumerTelefonu::NumerTelefonu(enum typ_numeru jaki )
    {
        this->jaki = jaki;
        printf("konstruktor\n");
    };
NumerTelefonu::NumerTelefonu(enum typ_numeru jaki  , int nr_kierunkowy  )
    {
        this->jaki = jaki;
        this->nr_kierunkowy = nr_kierunkowy;
        printf("konstruktor\n");
    };
    NumerTelefonu::NumerTelefonu( int nr_kierunkowy , int nr_numer )
    {

        this->nr_kierunkowy = nr_kierunkowy;
        this->nr_numer = nr_numer;
        printf("konstruktor\n");
    };
NumerTelefonu::NumerTelefonu(enum typ_numeru jaki  , int nr_kierunkowy , int nr_numer )
    {
        this->jaki = jaki;
        this->nr_kierunkowy = nr_kierunkowy;
        this->nr_numer = nr_numer;
        printf("konstruktor\n");
    };

int NumerTelefonu::ZwrocKierunkowy() const
{
    return nr_kierunkowy;
}
int NumerTelefonu::ZwrocNumer() const
{
    return nr_numer;
}
void NumerTelefonu::Wyswietl() const
{

    std::cout<<jaki<<"\n"<<nr_kierunkowy<<"\n"<<nr_numer;
}

int main()
{

NumerTelefonu pierwszy(domowy,82,987654321);
pierwszy.Wyswietl();

    _getch();
    return 0;
}

 

---

Ręce opadają... Przyjrzyj się implementacji Wyswietl.

---

Tak jak wspomniałem, dopiero się uczę...
Przyglądam się... na pewno jest tu coś nie tak, stąd pytanie wyżej o metodę Wyświetl().
Mógłbyś mnie oświecić ?

---

W treści zadania nie ma nic na temat metody Wyswietl. Wywal ja i wyświetl (w mainie) zawartość obiektu z wykorzystaniem metod ZwrocTyp, ZwrocKierunkowy i ZwrocNumer - po to w końcu one są.

---

Słuszna uwaga, o tym nie pomyślałem. Tak jak z ZwrocKierunkowy() i ZwrocNumer() nie ma problemu, tak w ZwrocTyp() nadal jest, robię coś nie tak... Nadal wypisuje cyfrę, mógłbyś napisać jak powinno to poprawnie wyglądać ?

#include <iostream>
#include <stdio.h>
#include <conio.h>

enum typ_numeru
    {
        domowy = 1,
        komorkowy = 2,
        sluzbowy = 3
    };
class NumerTelefonu
{
private:
    enum typ_numeru jaki;
    int nr_kierunkowy,nr_numer;
public:
    NumerTelefonu();
    NumerTelefonu(enum typ_numeru jaki , int nr_kierunkowy, int nr_numer );
    const char* ZwrocTyp() const
    {

        switch(jaki)
        {
            case domowy: 
                std::cout<<domowy<<std::endl; 
                return "domowy";
            case komorkowy: 
                std::cout<<komorkowy<<std::endl; 
                return "komorkowy";
            case sluzbowy: 
                std::cout<<sluzbowy<<std::endl; 
                return "sluzbowy";
        };

    }
    int ZwrocKierunkowy() const;
    int ZwrocNumer() const;

};
NumerTelefonu::NumerTelefonu()
{
jaki = komorkowy;
nr_kierunkowy=32;
nr_numer=123456789;
}
NumerTelefonu::NumerTelefonu(enum typ_numeru jaki  , int nr_kierunkowy , int nr_numer )
    {
        this->jaki = jaki;
        this->nr_kierunkowy = nr_kierunkowy;
        this->nr_numer = nr_numer;

    };
int NumerTelefonu::ZwrocKierunkowy() const
{
    std::cout<<nr_kierunkowy<<std::endl;
    return nr_kierunkowy;
}
int NumerTelefonu::ZwrocNumer() const
{
    std::cout<<nr_numer<<std::endl;
    return nr_numer;
}

int main()
{

NumerTelefonu pierwszy(komorkowy,82,987654321);
pierwszy.ZwrocTyp();
pierwszy.ZwrocKierunkowy();
pierwszy.ZwrocNumer();


    _getch();
    return 0;
}

 W switch'u w cout zamiast domowy, komorkowy, sluzbowy wpisywałem też jaki lecz to też nic nie dało, nie wiem jak to ugryźć..

---

Co żeś się tak uparł na to, że klasa ma cokolwiek wyświetlać? Wywal te couty z metod i zrób tak:

std::cout << "Typ: " << pierwszy.ZwrocTyp() << '\n';
std::cout << "Kierunkowy:  " << pierwszy.ZwrocKierunkowy()  << '\n';
std::cout << "Numer: " << pierwszy.ZwrocNumer() << '\n';

 

---

Aaaaa  Tak to ugryźć... Teraz rozumiem. Uparłem się bo nie wiedziałem jak to inaczej zrobić .
Dzięki wielkie za cierpliwość, pomoc i czas !