Tworzę wielofunkcyjny program połączony z wielu lecz nie mam pojęcia co muszę w nim jeszcze zawrzeć żeby zadziałał.

Question

Mam zamiar stworzyć program w którym połączę kalkulator zrobiony przy użyciu switcha oraz parę innych programów jak np. zgadywanie liczby, lecz nie wiem co tu nie gra i czego musiał bym użyć żeby program losowania liczby się restartował albowiem liczba prób

#include <iostream>
#include <cstdlib>
#include <conio.h>
#include <time.h>
#include <windows.h>
#include <stdio.h>
using namespace std;
float x,y;
char wybor,wybor2,wybor3;
int liczba,strzal,ile_prob;


int main()
{

for (;;)
{
    cout <<"MENU GLOWNE" << endl;
    cout << "----------------" << endl;
    cout << "1.Kalkulator" << endl;
    cout << "2.Zgadywanie liczby" << endl;
    wybor=getch();
    system("cls");



    switch (wybor)
    {
        case '1':
        cout<<"Witaj w kalkulatorze, wybierz operacje: "<<endl<<endl;
        cout << "MENU KALKULATORA" << endl;
        cout << "----------------" << endl;
        cout << "1.Dodawanie" << endl;
        cout << "2.Odejmowanie" << endl;
        cout << "3.Mnozenie" << endl;
        cout << "4.Dzielenie" << endl;
        cout<<"---------------"<<endl;
        cout<<"5. wyjscie"<<endl;
        cout << "Enter = Powrot do Menu Glownego" << endl;

         wybor2=getch();
         system ("cls");

            switch (wybor2)
            {
                case '1':
                cout<<"Dodawanie"<<endl<<endl;

                cout<<"Podaj 1 liczbe: ";
                cin>>x;

                cout<<"Podaj 2 liczbe: ";
                cin>>y;

                cout<< "Suma = "<<x+y<<endl;
                break;

                case '2':
                cout<<"Odejmowanie"<<endl<<endl;

                cout<<"Podaj 1 liczbe: ";
                cin>>x;

                cout<<"Podaj 2 liczbe: ";
                cin>>y;

                cout<< "Roznica = "<<x-y<<endl;
                break;

                case '3':
                cout<<"Mnozenie"<<endl<<endl;

                cout<<"Podaj 1 liczbe: ";
                cin>>x;

                cout<<"Podaj 2 liczbe: ";
                cin>>y;

                cout<< "Iloczyn = "<<x*y<<endl;
                break;

                case '4':
                cout<<"Dzielenie"<<endl<<endl;

                cout<<"Podaj 1 liczbe: ";
                cin>>x;

                cout<<"Podaj 2 liczbe: ";
                cin>>y;

                if (y==0) cout <<"Nie dzielimy przez zero!";
                else cout<< "Roznica = "<<x/y<<endl;
                break;

                case '5':
                cout<<"DO ZOBACZENIA (:"<<endl;
                exit(0);
            }

                wybor3=getch();
                system ("cls");
            case '2':
              cout<<"Witaj pomyslalem sobie liczbe od 1...100"<<endl;
              srand(time(NULL));
              liczba = rand()% 100+1;

              while(strzal!=liczba)
             {
            ile_prob++;
            cout<<"Zgadni jaka to liczba(to twoja "<<ile_prob<<" proba:";
            cin>>strzal;

            if(liczba==strzal)
            cout<<endl<<"udalo sie, wygrywasz w "<<ile_prob<<" probie"<<endl;

            else if(liczba>strzal)
            cout<<"za malo"<<endl;

            else if(liczba<strzal)
            cout<<"za duzo"<<endl;

              }



    }
}
    return 0;
}

ciągle się inkrementuje a w programie występuje błąd polegający na tym że po skończeniu obliczania w kalkulatorze nie ukazuje się menu tylko od razu 2 program.

Answer 1

W linii 102 brakuje break;. Miejsce przed case 2. 

Comments

wszystko dobrze chodzi. Tyle jeszcze nie mam pojęcia co bym mógł zrobić żeby zmienna   ile_ prób się restartowała a nie ciągle inkrementowała ?

Answer 2

Moim zdaniem niepotrzebnie użyłeś zmiennych globalnych. Najlepiej jakbyś deklarował je przed jakąś funkcja, w której będziesz je używać. np.

liczba = rand()% 100+1;
              int ile_prob = 0;  //o tutaj
              while(strzal!=liczba)
             {
            ile_prob++;