• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Moj kod co jest zle?

0 głosów
172 wizyt
pytanie zadane 12 lipca 2016 w C i C++ przez Albort Nowicjusz (150 p.)
#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.

2 odpowiedzi

+3 głosów
odpowiedź 12 lipca 2016 przez Boshi VIP (100,720 p.)
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.
komentarz 12 lipca 2016 przez Albort Nowicjusz (150 p.)
#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

komentarz 12 lipca 2016 przez MichałGNU Gaduła (4,420 p.)
I zamiast instrukcji if mogłoby być switch case.
0 głosów
odpowiedź 12 lipca 2016 przez manjaro Nałogowiec (28,970 p.)
edycja 12 lipca 2016 przez manjaro

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ł.

komentarz 12 lipca 2016 przez mrcnsct Nałogowiec (36,790 p.)
Na Windowsie Sleep z dużej.
1
komentarz 12 lipca 2016 przez manjaro Nałogowiec (28,970 p.)

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

Podobne pytania

0 głosów
2 odpowiedzi 224 wizyt
0 głosów
3 odpowiedzi 130 wizyt
pytanie zadane 7 lutego w C i C++ przez KonDZIKs Obywatel (1,250 p.)
+1 głos
2 odpowiedzi 462 wizyt
pytanie zadane 7 stycznia 2016 w C i C++ przez Tony Początkujący (360 p.)
Porady nie od parady
Forum posiada swój własny chat IRC, dzięki któremu będziesz mógł po prostu pogadać z innymi Pasjonatami lub zapytać o jakiś problem. Podstrona z chatem znajduje się w menu pod ikoną człowieka w dymku.IRC

64,981 zapytań

111,459 odpowiedzi

234,639 komentarzy

46,802 pasjonatów

Przeglądających: 208
Pasjonatów: 7 Gości: 201

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto dwie polecane książki warte uwagi. Pełną listę znajdziesz tutaj.

...