Pierwszy program Sklep - prośba o sprawdzenie

Question

Witam , właśnie skończyłem pisanie mojego pierwszego programu ,opanowałem do tej pory pierwsze 6 odcinków kursu programowania,jak widać jest to mały symulator sklepu , byłbym bardzo wdzięczny gdyby ktoś sprawdził kod programu , wskazał mi błędy i podpowiedział jak je poprawić :)

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

using namespace std;
char decyzja;
int opcja;


int main()
{
    float suma = 0;
    float money;
    cout <<"Ile masz pieniedzy ?"<<endl ;
      if(!(cin >> money))
      {
        cerr << "To nie jest liczba ! ";
        exit(0);
      }
    for(;;)
    {
    if(decyzja != 'n' )
    {
    cout <<endl <<  "Witaj w Sklepie !" << endl;
     cout <<  "---------------------" << endl;
      cout <<  "1)Kup Banany 5.50 zl" << endl;
      cout <<  "2)Kup Maliny 2.20 zl" << endl;
      cout <<  "3)Kup Marchewki 7.99 zl" << endl;
      cout <<  "4)Kup Pomidory 4.00 zl" << endl;
      cout << "5)Pokaz rachunek do zaplaty ! "<<endl;
      cout <<  "6)Chce wyjsc , nic nie kupuje !" << endl;
      cout << "7)Chce zaplacic rachunek ."<<endl ;

      cout << "Co chcesz zrobic : " ;
      if(!(cin >> opcja))
      {
        cerr << "To nie jest liczba ! ";
        exit(0);
      }
switch (opcja)
{
case 1 :
    {
        cout << "Banany zostaly dodane do rachunku ! "<<endl ;
        suma += 5.50 ;
        if (money-suma<0)
            {
            cout << "Na zakupy zabraknie ci : "<<(money-suma)*(-1) << " zl "<<endl;
            }
            else
            {
                cout << "Po zakupach zostanie ci : "<<money-suma << " zl "<<endl;
            }
    }
    break ;
    case 2 :
    {
        cout << "Maliny zostaly dodane do rachunku ! " <<endl ;
        suma += 2.20 ;
        if (money-suma<0)
            {
            cout << "Na zakupy zabraknie ci : "<<(money-suma)*(-1) << " zl "<<endl;
            }
            else
            {
                cout << "Po zakupach zostanie ci : "<<money-suma << " zl "<<endl;
            }
    }
    break ;
    case 3 :
    {
        cout << "Marchewki zostaly dodane do rachunku ! " <<endl ;
        suma += 7.99 ;
        if (money-suma<0)
            {
            cout << "Na zakupy zabraknie ci : "<<(money-suma)*(-1) << " zl "<<endl;
            }
            else
            {
                cout << "Po zakupach zostanie ci : "<<money-suma << " zl "<<endl;
            }
    }
    break ;
    case 4 :
    {
        cout << "Pomidory zostaly dodane do rachunku ! " <<endl ;
        suma += 4 ;
        if (money-suma<0)
            {
            cout << "Na zakupy zabraknie ci : "<<(money-suma)*(-1) << " zl "<<endl;
            }
            else
            {
                cout << "Po zakupach zostanie ci : "<<money-suma << " zl "<<endl;
            }
    }
    break ;
    case 5 :
        {
            cout << "Twoj rachunek wynosi aktualnie : " << suma << " zl" <<endl;
            cout << "Masz przy sobie : "<< money <<" zl " <<endl ;
            if (money-suma<0)
            {
            cout << "Na zakupy zabraknie ci : "<<(money-suma)*(-1) << " zl "<<endl;
            }
            else
                cout << "Po zakupach zostanie ci : "<<money-suma << " zl "<<endl;
        }
        break ;
    case 6 :
        {
            exit(0);
        }
        break ;
    case 7 :
        {
            if (suma<=money)
    {
    cout << "*******RACHUNEK********"<<endl;
    cout <<endl ;
    cout << "    Zaplac " << suma << " zl ! "<<endl;
    cout <<endl ;
    cout << "***********************"<<endl;
    getchar();getchar();
    exit(0);
    }
    else
    {
        cout << "*******RACHUNEK********"<<endl;
    cout <<endl ;
    cout << "    Zaplac " << suma << " zl ! "<<endl;
    cout << "    Brakuje ci jeszcze : " << suma - money << "zl" <<endl;
    cout <<endl ;
    cout << "***********************"<<endl;
    getchar();getchar();
    exit(0);
    }
        }
default :
    {
        cout <<endl <<"Nie ma takiej opcji ! " ;
    }
    break ;

}
cout << "kontynuowac zakupy ?" << endl << "t/n  " <<endl;
cin >> decyzja ;
system("cls");
}
else
{
    if (suma<=money)
    {
    cout << "*******RACHUNEK********"<<endl;
    cout <<endl ;
    cout << "    Zaplac " << suma << " zl ! "<<endl;
    cout <<endl ;
    cout << "***********************"<<endl;
    getchar();getchar();
    exit(0);
    }
    else
    {
    cout << "*******RACHUNEK********"<<endl;
    cout <<endl ;
    cout << "    Zaplac " << suma << " zl ! "<<endl;
    cout << "    Brakuje ci jeszcze : " << suma - money << "zl" <<endl;
    cout <<endl ;
    cout << "***********************"<<endl;
    getchar();getchar();
    exit(0);
    }
}
    }
    return 0;
}

 

Comments

Jaka składnia?

Answer 1

No więc z takich na początek....

1. Przerzuć się na język angielski w kodzie. Dla ludzi którzy trochę już piszą polski w kodzie działa jak płachta na byka.
2. Unikaj zmiennych globalnych.
3. Unikaj using namespace. Im więcej std tym szybciej zajrzysz co tam jeszcze jest dobrego.
4. Tabulacja w niektórych miejscach mogłaby być lepsza.

A tak poza tym nie ma co oceniać  mały programik więc leć dalej do obiektowości i po drodze naucz się gita. 

Comments

[mokrowski:]

Ogólnie +1, ale:

Przypomnę jeszcze że using można użyć bardzo sensownie:

using std::vector;
(...)

Wyżej była mowa raczej o dyrektywie przestrzeni nazw (using directive), niż o deklaracji (using declaration). :) I w takim wypadku odradzanie ma jakiś tam sens. Jest oczywistym, że wszystko ma jakąś domenę zastosowań, nawet taka dyrektywa. :)

---

 @mokrowski

No, trochę przesadziłem :P

---

Nie wiedziałem że wywołam tym prostym pytaniem taką ciotodramę .. Nadal nie rozumiem o co chodzi, ale chyba nie chcę już wiedzieć ..

---

Nie wiedziałem że wywołam tym prostym pytaniem taką ciotodramę ..

Nadal nie rozumiem o co chodzi, ale chyba nie chcę już wiedzieć ..

Postaraj się zrozumieć, to nie takie trudne a wiedza (w miarę) przydatna. :)

---

No i dobrze :-) Będzie Ci łatwiej w życiu... "Wystarczy rzyć" :-)