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

Moj kod co jest zle?

VPS Starter Arubacloud
0 głosów
511 wizyt
pytanie zadane 12 lipca 2016 w C i C++ przez Albort Nowicjusz (140 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,240 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 (140 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,330 p.)
I zamiast instrukcji if mogłoby być switch case.
0 głosów
odpowiedź 12 lipca 2016 przez manjaro Nałogowiec (37,390 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,390 p.)
Na Windowsie Sleep z dużej.
1
komentarz 12 lipca 2016 przez manjaro Nałogowiec (37,390 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 425 wizyt
+1 głos
0 odpowiedzi 332 wizyt
pytanie zadane 20 czerwca 2023 w Offtop przez niezalogowany
0 głosów
1 odpowiedź 178 wizyt
pytanie zadane 16 września 2019 w C i C++ przez KonDZIKs Bywalec (2,770 p.)

92,417 zapytań

141,222 odpowiedzi

318,985 komentarzy

61,831 pasjonatów

Motyw:

Akcja Pajacyk

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

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...