Pisanie obiektowo - pierwsze problemy

Question

Witam! Po przestudiowaniu drugiego odcinka kursu programowania obiektowego chciałem napisać coś swojego.

Zbytnio kreatywny nie jestem więc pomyślałem o kalkulatorze. Poniżej wklejam skutki mojej pracy i powiem tak:
niby wszystko działa jak normalny kalkulator, ale wydaję mi się, że mimo wszystko coś jest tu nie tak szczególnie w pliku main. Prosiłbym o jakieś wskazówki oraz ew. co i jak mógłbym pozmieniać. Z góry dziękuję!

PLIK main.cpp

#include <iostream>
#include "dzialanie.h"

using namespace std;

int main()
{
    for(;;)
    {
    Dzialania kalkulator;
    kalkulator.menu();
    kalkulator.wybor();
    kalkulator.odp_dzialanie();
    }
}

PLIK dzialanie.h

#include <iostream>

using namespace std;

class Dzialania
{
  public:

   float a, b;
   int co;
   float wynik;

   void menu();
   void wybor();
   void odp_dzialanie();
};

PLIK dzialanie.cpp

#include <iostream>
#include <iomanip>
#include "dzialanie.h"

using namespace std;

void Dzialania::menu()
{
    cout << "MENU GLOWNE\n";
    cout << "--------------\n";
    cout << "1. Dodawanie\n";
    cout << "2. Odejmowanie\n";
    cout << "3. Mnozenie\n";
    cout << "4. Dzielenie\n";
    cout << "5. Wyjscie\n";
}

void Dzialania::wybor()
{
    cout << "--------------\n";
    cout << "Co wybierasz? [1-5]: "; cin >> co;
}

void Dzialania::odp_dzialanie()
{
    if(co == 1)
    {
        system("cls");
        cout << "======[DODAWANIE]======\n\n";
        cout << "Podaj pierwsza liczbe: "; cin >> a;
        cout << "Podaj druga liczbe: "; cin >> b;
        cout << "Wynik dzialania: "<< a + b;

        cout << "\n\nNacisnij dowolny przycisk aby kontynuowac...";
        getchar(); getchar(); system("cls");

    }

    if(co == 2)
    {
        system("cls");
        cout << "======[ODEJMOWANIE]======\n\n";
        cout << "Podaj pierwsza liczbe: "; cin >> a;
        cout << "Podaj druga liczbe: "; cin >> b;
        cout << "Wynik dzialania: "<< a - b;

        cout << "\n\nNacisnij dowolny przycisk aby kontynuowac...";
        getchar(); getchar(); system("cls");

    }

    if(co == 3)
    {
        system("cls");
        cout << "======[MNOZENIE]======\n\n";
        cout << "Podaj pierwsza liczbe: "; cin >> a;
        cout << "Podaj druga liczbe: "; cin >> b;
        cout << "Wynik dzialania: "<< a * b;

        cout << "\n\nNacisnij dowolny przycisk aby kontynuowac...";
        getchar(); getchar(); system("cls");

    }

    if(co == 4)
    {
        system("cls");
        cout << "======[DZIELENIE]======\n\n";
        cout << "Podaj pierwsza liczbe: "; cin >> a;
        cout << "Podaj druga liczbe: "; cin >> b;
        cout << "Wynik dzialania: "<<setprecision(2)<< a / b;

        cout << "\n\nNacisnij dowolny przycisk aby kontynuowac...";
        getchar(); getchar(); system("cls");

    }

    if(co == 5) exit(0);
}

 

Answer 1

if-y walą po oczach i powtarzający się kod

1. czyścisz ekran wszędzie ja bym to wywalił poza if
2. pobierasz a i b tak samo poza if-a
3. użyj if lub switch do wypisania tekstu i wykonania obliczeń
4. wypisz wynik poza if-em

w skrócie co ci się powtarza wywal za if-a

 

Comments

dobra praktyka: Stworzyc funkcje czyszczaca ekran :)

---

@Miki, czy tak jest teraz lepiej?

#include <iostream>
#include <iomanip>
#include "dzialanie.h"

using namespace std;

void Dzialania::menu()
{
    cout << "MENU GLOWNE\n";
    cout << "--------------\n";
    cout << "1. Dodawanie\n";
    cout << "2. Odejmowanie\n";
    cout << "3. Mnozenie\n";
    cout << "4. Dzielenie\n";
    cout << "5. Wyjscie\n";
}

void Dzialania::wybor()
{
    cout << "--------------\n";
    cout << "Co wybierasz? [1-5]: "; cin >> co;
}

void Dzialania::naglowek_dzialania()
{
    system("cls");

    if(co == 1)
    {
        cout << "======[DODAWANIE]======\n\n";
    }

    if(co == 2)
    {
        cout << "======[ODEJMOWANIE]======\n\n";
    }

    if(co == 3)
    {
        cout << "======[MNOZENIE]======\n\n";
    }

    if(co == 4)
    {
        cout << "======[DZIELENIE]======\n\n";
    }

    if(co == 5) exit(0);
}

void Dzialania::wpisz_liczby()
{

        cout << "Podaj pierwsza liczbe: "; cin >> a;
        cout << "Podaj druga liczbe: "; cin >> b;

}

void Dzialania::wypisz_wynik()
{
    cout << "Wynik dzialania: ";

    if(co == 1)
    {
        cout << a + b;
    }

    if(co == 2)
    {
        cout << a - b;
    }

    if(co == 3)
    {
        cout << a * b;
    }

    if(co == 4)
    {
        cout <<setprecision(3)<< a / b;
    }

    if(co == 5) exit(0);

     cout << "\n\nNacisnij dowolny przycisk aby kontynuowac...";
    getchar(); getchar(); system("cls");
}

 

---

if(co == 1) cout << a + b;
if(co == 2) cout << a - b;
if(co == 3) cout << a * b;
if(co == 4) cout << setprecision(3) << a / b;

ja bym jeszcze posprzątał w if-ach, jak masz tylko wprowadzanie tekstu lub wypisywanie go do konsoli to ci nawiasy są niepotrzebne bo bez nich też będzie działać. Ale to już takie wizualne poprawki