if/lotto .

Question

Napisałem cuś takiego i chciałbym aby na końcu pisało ile razy trafiłem ale pisze ostatniego cin'a(to co wprowadziłęm)

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

int liczba ,los[5],trafione=0;

int main()

{
      fstream plik;
    plik.open("wyniki.txt" ,ios::out | ios::app);
        std::ofstream stream("wyniki.txt");
    if(!stream.good())
        std::cerr<<"Cannot open file\n";
        stream<<"Moje strzaly";

       cout<<"Wybierz swoje szczesliwe liczby od 0 do 49 !"<<endl;
    cin>>los[0];
    cin>>los[1];
    cin>>los[2];
    cin>>los[3];
    cin>>los[4];
    cin>>los[5];
if(los[0]>49,los[1]>49,los[2]>49,los[3]>49,los[4]>49,los[5]>49)
{
    system("cls");
    cout<<"liczby od 0 do 49!";
    Sleep (2000);
    exit(9);}
if ((los[0]==los[1])||(los[0]==los[2])||(los[0]==los[3])||(los[0]==los[4])||(los[0]==los[5])||(los[1]==los[2])||(los[1]==los[3])||(los[1]==los[4])||(los[1]==los[2])||(los[2]==los[3])||(los[2]==los[4])||(los[2]==los[5])||(los[3]==los[4])||(los[3]==los[5])||(los[0]==los[1]))
{
    system("cls");
    cout<<"liczby sie nie moga powtarzac!";
    Sleep (2000);
    exit(9);
}
      plik<<los[0]<<endl;
   plik<<los[1]<<endl;
    plik<<los[2]<<endl;
     plik<<los[3]<<endl;
      plik<<los[4]<<endl;
       plik<<los[5]<<endl;


    cout<<"Za 3 sekundy nastapi zwolnienie blokady";
    Sleep(3000);
    cout<<endl;

string los = "losy";
     plik<<los<<endl;


    srand(time(NULL));
    for (int i=1; i<=6; i++)
    {
        liczba = rand()%49+1;
        Sleep(1000);
if((los[0]==liczba)||(los[1]==liczba)||(los[2]==liczba)||(los[3]==liczba)||(los[4]==liczba)||(los[5]==liczba))
{
    trafione++;
}
        cout<<liczba<<"\a"<<endl;
         plik<<liczba<<endl;
    }



cout<<"tarfiles: "; cout<<trafione; cout<<" razy";

plik<<endl;
        plik.close();
        getchar();getchar();
    return 0;
}

dokładnie źle jest to:

   srand(time(NULL));
    for (int i=1; i<=6; i++)
    {
        liczba = rand()%49+1;
        Sleep(1000);
if((los[0]==liczba)||(los[1]==liczba)||(los[2]==liczba)||(los[3]==liczba)||(los[4]==liczba)||(los[5]==liczba))
{
    trafione++;
}
        cout<<liczba<<"\a"<<endl;
         plik<<liczba<<endl;
    }



cout<<"tarfiles: "; cout<<trafione; cout<<" razy";

 

Comments

Czemu nie używasz pętli... ?

---

Sory nie zbyt ogarniam jak to zrobić mógłbyś mi pokazać jak to zrobić? I dlaczego pokazuje ostatnią wpisaną przez użytkownika liczbę?

---

Pętli używasz, kiedy musisz powtórzyć jakiś kawałek kodu. np. to:

    cin>>los[0];
    cin>>los[1];
    cin>>los[2];
    cin>>los[3];
    cin>>los[4];
    cin>>los[5]

Zamiast tego mogłoby być:

for(unsigned i = 0; i<6; ++i){  
    cin>>los[i];
}

 

---

Ok dzięki a pomożesz mi rozwiązać problem? Zapętliłem już wcześniej i żeby się w int liczby zapisywały tu masz kod:

int trafione=0;
    srand(time(NULL));
    for (int i=1; i<=6; i++)
    {

        liczba = rand()%49+1;
        Sleep(1000);
        if(los[0]==liczba)
trafione++;  //ma się dodawać za każde trafienie czyli los[0] los[1] itd...
        cout<<liczba<<"\a"<<endl;
         plik<<liczba<<endl;
    }

 

---

Zamiast

if(los[0] == liczba)

daj

if(los[i] == liczba)

 

---

Nie zależnie ile razy się trafi zawsze pokazuje 0

float traf=0;
srand(time(NULL));
    for (int i=1; i<=6; i++)
    {
        liczba = rand()%49+1;
        Sleep(1000);
        if (los[i]==liczba)
            { traf++; }

        cout<<liczba<<"\a"<<endl;
         plik<<liczba<<endl;
    }



plik<<endl;
cout<<"trafiles: "<<traf<<" razy";

 

Answer 1

Zadeklarowales tablice los 

los[5]

z pięcioma elementami a pozniej probujesz odwołać sie do nieistniejacego elementu.

cin>>los[5];

Dodatkowo pozniej nadpisujesz tablice stringiem.

string los = "losy"