• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Pisanie obiektowo - pierwsze problemy

VPS Starter Arubacloud
0 głosów
199 wizyt
pytanie zadane 21 października 2020 w C i C++ przez _Kuba_ Początkujący (250 p.)

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);
}

 

1 odpowiedź

0 głosów
odpowiedź 21 października 2020 przez Miki Bywalec (2,480 p.)

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

 

komentarz 22 października 2020 przez Eriss69 Gaduła (4,470 p.)
dobra praktyka: Stworzyc funkcje czyszczaca ekran :)
komentarz 23 października 2020 przez _Kuba_ Początkujący (250 p.)

@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");
}


 

komentarz 4 listopada 2020 przez Miki Bywalec (2,480 p.)
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

Podobne pytania

0 głosów
0 odpowiedzi 284 wizyt
pytanie zadane 18 kwietnia 2020 w C i C++ przez Bartosz Paterek Początkujący (410 p.)
+2 głosów
2 odpowiedzi 766 wizyt
pytanie zadane 4 kwietnia 2016 w C i C++ przez caramelleandamour Użytkownik (550 p.)
0 głosów
2 odpowiedzi 276 wizyt

92,452 zapytań

141,262 odpowiedzi

319,085 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...