Orzel lub reszka

Question

Witam napisalem taki program do losowania orla lub reszki prosze o uwagi dotyczace kodu i jakies propozycje. Jestem poczatkujacy i przerobilem dopiero 4 tematy od pana Miroslawa ,zadanie pochodzi z ksiazki Alexa Allana rozdzial 8.

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

using namespace std;

int moneta,rzut;


int main()
{
   cout << "    Orzel czy reszka" << endl;

   srand(time(NULL));


   while(moneta<=2 )
   {
       rzut=rand()%2+1;
       cout << "|----------------------|\n";
       cout << "|Rzuc moneta wcisnij |1|\n";
       cout << "|----------------------|\n";
       cout << "|Zakoncz wcisnij     |2|\n";
       cout << "|----------------------|\n";
       cout << "Wcisnij klawisz "; cin >> moneta;

       if (moneta==2)
       {
           cout << "KONIEC\n\n";
           break;
       }

        cout << endl;

        if(rzut==2)

        cout<<"ORZEL\n\n";

        else if (rzut==1)

        cout <<"RESZKA\n\n";



   }

    return 0;
}

 

Answer 1

Kilka uwag:

...
#include <time.h> piszesz w C, czy C++? Użyj nagłówka <ctime>

using namespace std; // powiedzmy ze jest jeszcze to akcpetowalne przy malym kodzie, ale lepiej jest uzywac std::cout ...
 
int moneta,rzut; // a to (zmienne globalne) nie ma uzasadnienia w tym kodzie

..

    while(moneta<=2 ) // rob odstepy pomiedzy operatorami while(moneta <= 2)
    ...
        cout << "Wcisnij klawisz "; cin >> moneta; // to powinny byc 2 osobne linie, 1 linia = 1 instrukcja
...

Comments

W C++ pisze.

---

Widzę, moje pytanie było czysto retoryczne. Jeżeli piszesz w C++ to stosuj się do konwencji C++

---

Ok, i dzieki za ocene :)  ja działam dalej piszac kolejny kod na jednorekigo bandyte :)

Answer 2

Kod jest w porządku, ale uważam, że zbyt szybko próbujesz być doskonały. Mając 4 poradniki na karku i tak być mógł nie zrozumieć porad, jak np. sławne zaprzestanie używania using namespace std, bądź zmiennych globalnych. Na początku nauki programowania, jeśli się kompiluje i działa poprawnie to już bardzo dobrze ;)