Pierwszy "projekt"-kilka pytań

0

Jest to mini projekt z porzuconego c++ i kursu Zielenta, kalkulator i mini gierka zgadnij liczbę
wrzucam kod:

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


using namespace std;
int wybor; float x , y; int losowa; int wybor2;
int przedzial;
int przedzial2;
char opcja;
int main()
{
    for (;;)
    {


     cout << "MENU"<<endl;
     cout << "****"<<endl;
     cout << "1.KALKULATOR"<<endl;
     cout << "2.ZGADNIJ JAKA TO LICZBA"<<endl;
     cout << "3.WYJSCIE"<<endl;
     cin >> wybor;
     system ("cls");

          if(wybor==1)
          {
          cout << "podaj x:"<<endl;
          cin >> x;
          cout << "podaj y:"<<endl;
          cin >> y;

          cout<< "1.dodawanie"<<endl;
          cout<< "2.odejmowanie"<<endl;
          cout<< "3.mnozenie"<<endl;
          cout<< "4.dzielenie"<<endl;
          cout << "wybierz:"<<endl;
          opcja = getch();


          switch(opcja)
          {
          case '1':
            cout << "suma = "<<x+y;
            break;
          case '2':
            cout << "roznica = "<<x-y;
            break;
          case '3':
            cout << "iloczyn = "<<x*y;
            break;
          case '4':
            cout << "iloraz = "<<x/y;
            break;

          }
          }
           else if(wybor==2)

           {
        cout << "wybierz przedzial losowania liczb:"<<endl;
        cout << "od:";
        cin >> przedzial;
        cout << "do:";
        cin >> przedzial2;


        cout << "zgadnij liczbe od:"<< przedzial <<" do:"<< przedzial2 <<endl;
           losowa = rand ()%przedzial2 + przedzial;
            srand(time(NULL));
            while(wybor2!=losowa)
           {
            cin >> wybor2;
            if(wybor2==losowa)
                cout << "brawo wygrales!!!"<<endl;
            else if(wybor2<losowa)
                cout << "za malo!!!"<<endl;
            else if(wybor2>losowa)
                cout << "za duzo!!!"<<endl;
            }
           }
else if(wybor==3)
{
    exit(0);
}
getchar(); getchar();
system ("cls");

     }

return 0;
     }

1.Czy jest to dobrze napisane program?
2.Czy mogę to pokazać komuś kto uczy się cpp?
3.Czemu przy wyborze zakresu losowania(np.12-19) liczba szukana wychodzi poza ten zakres? (pytam z ciekawości)

1

Nie nadaje się z dwóch powodów: zmienne globalne i brzydkie formatowanie :P
A teraz na poważnie, wychodzi poza zakres bo masz losowanie z zakresu <koniec, koniec+początek>,
żeby losować z przedziału <początek, koniec> powinieneś raczej napisać: rand()%(koniec - początek) + początek.

1 użytkowników online, w tym zalogowanych: 0, gości: 1