Moj kod co jest zle?

Question

#include <iostream>
#include <windows.h>
#include <time.h>
using namespace std;

int liczba, strzal;


int main()
{
cout<<"Wiatamy w dzisiejszym duzym lotto. Prosze wpisac 6 liczb w zakresie 1-50"<<endl;

cout<<"Wybierz liczbe #1 ------>";
cin>>strzal;

cout<<"Wybierz liczbe #2 ------>";
cin>>strzal;

cout<<"Wybierz liczbe #3 ------>";
cin>>strzal;

cout<<"Wybierz liczbe #4 ------>";
cin>>strzal;

cout<<"Wybierz liczbe #5 ------>";
cin>>strzal;

cout<<"Wybierz liczbe #6 ------>";
cin>>strzal;





    srand(time(NULL));

    cout<<"Pierwsza nasza liczba to:"<<endl;
    liczba = rand()%49+1;
    Sleep(300);
    cout<<"$----";
    cout<<liczba;
    cout<<"----$"<<endl;
    for(int i=1; i<=5; i++)
    {
    liczba = rand()%49+1;
    Sleep(300);
    cout<<"nastepna liczba to"<<endl;
    cout<<"$----";
    cout<<liczba;
    cout<<"----$"<<endl;
    }


if(liczba==strzal)
{
    cout<<"Udalo ci sie!"<<strzal<<" sie zgadza!";
}

    return 0;
}

 

 

Zrobilem takie cos i siedze nad tym i szukam bledu i jestem lekko zirytowany :(

Chcialem zeby jak wylosuje liczby takie jak my wpisalismy wyswietlio komunikat 

"Udalo ci sie!"<<strzal<<" sie zgadza!"

Jednak to sie nie dzieje mimo ze liczby sie zgadzają :( 

Przepraszam za zanizanie poziomu forum, dopiero zaczynam.

Answer 1

Po pierwsze wczytujesz dane do jedenj zmiennej i ją nadpisujesz cały czas, czyli liczy się tylko liczba nr 6, po drugie if powinien być w pętli.

Zamiast tylu zmiennych po prostu użyj tablicy 6 elementowej a potem w niej sprawdzaj.

Comments

#include <iostream>
#include <windows.h>
#include <time.h>
using namespace std;

int liczba, strzal1, strzal2, strzal3, strzal4, strzal5, strzal6;


int main()
{
  cout<<"Wiatamy w dzisiejszym duzym lotto. Prosze wpisac 6 liczb w zakresie 1-50"<<endl;

cout<<"Wybierz liczbe #1 ------>";
cin>>strzal1;

cout<<"Wybierz liczbe #2 ------>";
cin>>strzal2;

cout<<"Wybierz liczbe #3 ------>";
cin>>strzal3;

cout<<"Wybierz liczbe #4 ------>";
cin>>strzal4;

cout<<"Wybierz liczbe #5 ------>";
cin>>strzal5;

cout<<"Wybierz liczbe #6 ------>";
cin>>strzal6;




    cout<<endl;
    srand(time(NULL));

    cout<<"Pierwsza nasza liczba to:"<<endl;
    liczba = rand()%49+1;
    Sleep(300);
    cout<<"$----";
    cout<<liczba;
    cout<<"----$"<<endl;
    for(int i=1; i<=5; i++)
    {
    liczba = rand()%49+1;
    Sleep(300);
    cout<<"nastepna liczba to"<<endl;
    cout<<"$----";
    cout<<liczba;
    cout<<"----$"<<endl;
    }


if(liczba==strzal1)
{
    cout<<"Udalo ci sie!"<<strzal1<<" sie zgadza!";
}

if(liczba==strzal2)
{
    cout<<"Udalo ci sie!"<<strzal2<<" sie zgadza!";
}

if(liczba==strzal3)
{
    cout<<"Udalo ci sie!"<<strzal3<<" sie zgadza!";
}

if(liczba==strzal4)
{
    cout<<"Udalo ci sie!"<<strzal4<<" sie zgadza!";
}

if(liczba==strzal5)
{
    cout<<"Udalo ci sie!"<<strzal5<<" sie zgadza!";
}

if(liczba==strzal6)
{
    cout<<"Udalo ci sie!"<<strzal6<<" sie zgadza!";
}


    return 0;
}

Średnio zrozumialem :\ Ten zapis mialem jeszcze wczesniej ale uznalem ze jest zly. Jest w czyms lepszy od poprzedniego? Dopiero zaczynam sie uczyc i teblice poznalem tylko je troche

---

I zamiast instrukcji if mogłoby być switch case.

Answer 2

Nie przejmuj się każdy kiedyś zaczynał.

Ten poprzedni kod z 6 zmiennymi będzie działał jak tylko przestawisz 1 znaczek. Dokładnie klamrę zamykają pętlę for (wiersz 51) przeniesiesz na sam dół i dasz ją po wszystkich ifach a przed returnem (wiersz 83).

Poza tym funkcja sleep z małej litery powinna być.

Oczywiście z punktu widzenia programisty ten kod jest fatalny ale będzie działał.

Comments

Na Windowsie Sleep z dużej.

---

Dzięki @mrcnsct nie wiedziałem o tym, nie używam windowsów już od 10 lat.